karplus / PteroDAQ

Other
3 stars 2 forks source link

Clock speed not being used correctly on Teensy 3.2 #53

Closed karplus closed 7 years ago

karplus commented 7 years ago

Original report by kevin_karplus (Bitbucket: kevin_karplus, ).


I've just replicated the behavior I was experiencing while collecting data for the aliasing lab. The time measurements that were collected for the graphs included in my lab report are absolutely incorrect - and I was aware of that when I submitted the paper. In my confusion I did not realize that the problem originated from a compiler setting used to compile the PteroDAQ firmware.

Here is the protocol to replicate my results:

(Tools used: Teensyduino 1.34, Arduino 1.6.12, Python 3.4.4, and a Teensy 3.2)

I was able to fix the problem by configuring the 'CPU Speed' option to '72 MHz' and reprogramming the Teensy.

I hope this helps, Christian Riegels

karplus commented 7 years ago

Original comment by kevin_karplus (Bitbucket: kevin_karplus, ).


The bug is confirmed. Currently the Teensy boards do not report their clock speeds in the model command. Changes will not be trivial, as the model command for ATMega boards reports clock speed as 2 bytes in kHz, and the 96MHz clock for Teensy 3.1/3.2 overclocked wouldn't fit in 2 bytes.

Probably need to change the model command and model.c to use more bytes for the clock speed, then change boards.py to understand the new model command and use it on the various boards.

karplus commented 7 years ago

Original comment by kevin_karplus (Bitbucket: kevin_karplus, ).


Should be fixed by the changes in the pull request made by Juan Huerta.

karplus commented 7 years ago

Original comment by kevin_karplus (Bitbucket: kevin_karplus, ).


I believe that Juan's changes fix this. I should probably confirm independently.