>>>>> "Stefan" == Stefan Weil <[email protected]> writes:

Stefan> cppcheck detected a condition which was always false.
Stefan> According to the MCIMX31 Reference Manual, the PRCS bits have
Stefan> to be 01 to select the Frequency Pre-Multiplier (FPM). PRCS
Stefan> uses bits 1 and 2, so we have to test for 2.

Good catch.  I hate hard coded constants, they're so easy to get wrong!

Stefan> Signed-off-by: Stefan Weil <[email protected]> 

Signed-off-by: Peter Chubb <[email protected]>

--
Dr Peter Chubb                                  peter.chubb AT nicta.com.au
http://www.ssrg.nicta.com.au          Software Systems Research Group/NICTA

Reply via email to