[Bug 1928] mpc5674fevb: powerpc/mpc55xx/misc/flash_support.c:183:9: warning: case label value exceeds maximum value for type

bugzilla-daemon at rtems.org bugzilla-daemon at rtems.org
Fri Oct 7 10:30:57 UTC 2011


https://www.rtems.org/bugzilla/show_bug.cgi?id=1928

dufault at hda.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |

--- Comment #5 from dufault at hda.com 2011-10-07 05:30:55 CDT ---
I just checked the MPC5674 reference manual.  The size field is 3 bits, and the
only valid size is 6, which isn't even in the current code.
The code will kick it out as an unknown chip, but I think it's better to
disable this file in the build for an MPC5674 until someone with that chip
fixes it.

Note that this failure is for a MPC56XX, not the MPC55XX that flash code is
written for.  There is no warning in the MPC55XX case.

If the flash code algorithm applies to the MPC56XX I now think the right fix is
to conditionalize the sizes in the switch statement on CPU type and not
eliminate the switch on the register bit field (by copying it into an int,
avoiding the warning).  That switch has now shown its worth by detecting an
attempt to build the code on an unsupported target.

-- 
Configure bugmail: https://www.rtems.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.



More information about the bugs mailing list