9 Feb
2009
9 Feb
'09
3:27 p.m.
Dear Ilya,
In message 49902403.2010406@emcraft.com you wrote:
...
- u32 TX_JUMBO_CNT;
- u32 TX_BYTE_CNT;
+} __attribute__((packed));
Why is this packed? All fields are 32-bit aligned already.
Generally, I prefer declaring such structures with ((packed)) regardless of if it's really do something. Just to show that it should be packed.
Can you please verify that it works as well without the "packed", and if it does (as expected) resubmit without it?
Thanks.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de
Drawing on my fine command of language, I said nothing.