From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Hershberger Date: Tue, 29 Nov 2016 15:24:43 -0600 Subject: [U-Boot] [PATCH 01/11] net: dw: Add read_rom_hwaddr net_op hook In-Reply-To: References: <20161108155437.1085-1-oliver@schinagl.nl> <20161108155437.1085-2-oliver@schinagl.nl> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Simon, On Thu, Nov 17, 2016 at 7:13 PM, Simon Glass wrote: > Hi Oliver, > > On 8 November 2016 at 08:54, Olliver Schinagl wrote: >> Add the read_rom_hwaddr net_op hook so that it can be called from boards >> to read the mac from a ROM chip. >> >> Signed-off-by: Olliver Schinagl >> --- >> drivers/net/designware.c | 18 ++++++++++++++++++ >> 1 file changed, 18 insertions(+) >> >> diff --git a/drivers/net/designware.c b/drivers/net/designware.c >> index 9e6d726..aa87f30 100644 >> --- a/drivers/net/designware.c >> +++ b/drivers/net/designware.c >> @@ -230,6 +230,23 @@ static int _dw_write_hwaddr(struct dw_eth_dev *priv, u8 *mac_id) >> return 0; >> } >> >> +__weak int dw_board_read_rom_hwaddr(unsigned char *enetaddr) >> +{ >> + return -ENOSYS; >> +} > > Instead of a weak function I think this should use driver model, with > a driver supplied by the board to read this value. It should be > possible to supply the 'hardware-address reading' device to any > Ethernet driver, not just dwmmc. How do we reconcile something like that with the concern of using the device tree for boards using only Linux bindings, and sharing the device tree with Linux? Linux probably doesn't care about this and so won't have a binding for defining this relationship. This is a fairly generic question. Where have we landed on this? Thanks, -Joe