<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Oct 7, 2011, at 5:11 , <a href="mailto:rtems-vc@rtems.org">rtems-vc@rtems.org</a> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><span class="Apple-style-span" style="border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><pre> uint32_t R;
struct {
uint32_t:5;
<font color="#880000">- uint32_t SIZE:3;
- uint32_t:1;
</font><font color="#000088">+ uint32_t SIZE:4;
</font> uint32_t LAS:3;
uint32_t:3;
uint32_t MAS:1;
</pre></span><br class="Apple-interchange-newline"></blockquote><br></div><div>I think that's definitely wrong. The MPC55xx structure starts out:</div><div><br></div><div><div><font class="Apple-style-span" face="Monaco"> struct FLASH_tag {</font></div><div><font class="Apple-style-span" face="Monaco"> union { /* Module Configuration Register */</font></div><div><font class="Apple-style-span" face="Monaco"> uint32_t R;</font></div><div><font class="Apple-style-span" face="Monaco"> struct {</font></div><div><font class="Apple-style-span" face="Monaco"> uint32_t:4;</font></div><div><font class="Apple-style-span" face="Monaco"> uint32_t SIZE:4;</font></div><div><font class="Apple-style-span" face="Monaco"> uint32_t:1;</font></div><div><font class="Apple-style-span" face="Monaco"> uint32_t LAS:3;</font></div><div><font class="Apple-style-span" face="Monaco"> uint32_t:3;</font></div><div><font class="Apple-style-span" face="Monaco"> uint32_t MAS:1;</font></div><div><br></div><div>so if it is compatible in any way then the reserved field of :5 needs be cut down to :4 instead of shifting the size bit to the right.</div><div><br></div><div>As I said when I reopened the bug, I think it's best to disable this file for the 567x and let someone with the chip reenable it when they can test it, and when they fix it they should conditionalize the switch statement to keep that warning intact for other systems.</div></div><br><div>
<div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><font class="Apple-style-span" size="3"><span class="Apple-style-span" style="font-size: 12px; ">Peter<br>-----------------<br>Peter Dufault<br>HD Associates, Inc. Software and System Engineering</span></font></div>
</div>
<br></body></html>