Hi Wolfgang,

Wolfgang Denk 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?
>   

Ok. It works as expected. I'll post the updated patch soon.

Regards, Ilya.


_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to