[U-Boot] [PATCH v3 2/3] net: Add a command to access the EEPROM from ethernet devices

Simon Glass sjg at chromium.org
Tue Oct 21 18:19:57 CEST 2014


Hi Alban,

On 21 October 2014 06:51, Alban Bedel <alban.bedel at avionic-design.de> wrote:
> On Fri, 17 Oct 2014 14:12:18 -0600
> Simon Glass <sjg at chromium.org> wrote:
>
>> >
>> > I'm unsure about the 'defaults' commands, I would tend to see it at
>> > the eeprom level as it might be needed in various situations. However
>> > it must be provided by the device, not the eeprom driver itself.
>> > It might also make sense to drop the callback in favour of a simple
>> > data array as it would allow setting it from device tree.
>>
>> I'm not sure what this refers to sorry.
>
> It refer to the 'eth eeprom defaults' command from this patch. The USB
> device I wrote that for need a lot more in the EEPROM than just the MAC
> address. This command allow writing the "factory defaults" in the
> EEPROM. Somehow I have a feeling that such a command would make sense
> for a few other use-case than just USB ethernet devices.

Quite possibly, although one option is to cross that bridge when you come to it.

Regards,
Simon


More information about the U-Boot mailing list