From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alban Bedel Date: Tue, 21 Oct 2014 14:51:01 +0200 Subject: [U-Boot] [PATCH v3 2/3] net: Add a command to access the EEPROM from ethernet devices In-Reply-To: References: <1413303972-26143-1-git-send-email-alban.bedel@avionic-design.de> <1413303972-26143-2-git-send-email-alban.bedel@avionic-design.de> <20141015114242.37f7f502@avionic-0020> Message-ID: <20141021145101.642c27a9@avionic-0020> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Fri, 17 Oct 2014 14:12:18 -0600 Simon Glass 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. Alban -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: