On Saturday, August 20, 2011 09:28:44 Marek Vasut wrote:
> -     ulong iobuf[ATA_SECTORWORDS];
>       unsigned char c;
> -     hd_driveid_t *iop = (hd_driveid_t *)iobuf;
> +     hd_driveid_t iop __attribute__((aligned(4)));

gcc takes care of making sure the hd_driveid_t struct has proper alignment on 
the stack.  i see it has at least one unsigned int in there, so that gets you 
4byte alignment.  so drop the attribute.

also, same comment as the other patch ... please change your log/summary to 
say "fix strict aliasing violations".
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

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

Reply via email to