On Friday 11 November 2011 07:33:00 Wolfgang Grandegger wrote:
> On 11/11/2011 12:04 PM, Marek Vasut wrote:
> >> Commit 79ad54400932d6484178a372fb3b659e3437473b broke the MAC
> >> address programming. Fix this by using the method from Linux'
> >> smsc95xx_set_mac_address().
> >>
> >> --- a/drivers/usb/eth/smsc95xx.c
> >> +++ b/drivers/usb/eth/smsc95xx.c
> >>
> >> - u32 addr_lo, addr_hi;
> >> + u32 addr_lo = eth->enetaddr[0] | eth->enetaddr[1] << 8 |
> >> + eth->enetaddr[2] << 16 | eth->enetaddr[3] << 24;
> >> + u32 addr_hi = eth->enetaddr[4] | eth->enetaddr[5] << 8;
> >>
> >> int ret;
> >>
> >> /* set hardware address */
> >> debug("** %s()\n", __func__);
> >>
> >> - addr_lo = cpu_to_le32(*eth->enetaddr);
> >> - addr_hi = cpu_to_le16(*((u16 *)(eth->enetaddr + 4)));
> >> +
> >>
> >
> > didn't Mike Frysinger send similar patch yesterday? Also, do you have the
>
> Ah, I obviously missed that.
>
> > hardware? If so, that's good, we know you can provide tested fix.
>
> Well, yes, I know somebody who has smsc95xx hardware ;-).
>
> > Mike, are you good with using this fix instead or can you two negotiate?
>
> Let's await Mike's answer (now on CC as well). I need to re-send the
> series anyway due to a stupid typo.mine might take a little longer due to wrangling with wolfgang. i'd suggest we go with your patch (although i have feedback on it). -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
