On Tuesday 21 August 2012 09:23:13 Lucas Stach wrote: > + struct ueth_data *dev = (struct ueth_data *)eth->priv; > + struct asix_private *priv = (struct asix_private *)dev->dev_priv;
not that it's harmful, but priv is void*, so the casts aren't needed > + ALLOC_CACHE_ALIGN_BUFFER(unsigned char, ee_buf, 2); > + ALLOC_CACHE_ALIGN_BUFFER(unsigned char, buf, ETH_ALEN); since you don't use both buffers at the same time, just declare one that is big enough to work for either code path. so looks like you could throw away ee_buf here. -mike
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
