[U-Boot] [PATCH 01/20] dm: net: Support usbethaddr environment variable

Joe Hershberger joe.hershberger at gmail.com
Wed Jul 8 22:43:53 CEST 2015


Hi Simon,

On Wed, Jul 8, 2015 at 3:31 PM, Simon Glass <sjg at chromium.org> wrote:
> +Hans
>
> Hi Joe,
>
> On 7 July 2015 at 22:04, Joe Hershberger <joe.hershberger at gmail.com> wrote:
>> Hi Simon,
>>
>> On Tue, Jul 7, 2015 at 9:53 PM, Simon Glass <sjg at chromium.org> wrote:
>>> For USB Ethernet devices we need to use the usbethaddr environment variable
>>> (instead of ethaddr) the Ethernet hardware address. Add this to the uclass
>>> so that it happens automatically.
>>
>> I have always thought that this approach of having a separate prefix
>> for usbethaddr was a smelly hack. Are we sure we want to propagate it
>> here? Can we not just use dev->seq? It should already be unique in the
>> DM, right? I was really looking forward to that all going away.
>
> Ah, OK, sorry. Do you think we need a way of specifying the eth
> interface # as we do with aliases at present? We did have one but Han
> has just removed it :-)

Can you reference where this happened. A quick search didn't turn it up for me.

> Otherwise we'll just end up counting up from the last 'fixed' ethernet
> interface. Maybe that is good enough.

That's probably good enough, but some may prefer more explicit control.

Thanks,
-Joe


More information about the U-Boot mailing list