Thread (11 messages) 11 messages, 3 authors, 2019-03-02

Re: [PATCH v3] net: e1000e: add MAC address kernel cmd line parameter

From: Andrew Lunn <andrew@lunn.ch>
Date: 2019-02-28 19:30:27
Also in: intel-wired-lan, lkml

On Thu, Feb 28, 2019 at 05:13:27PM +0000, Flavio Suligoi wrote:
quoted
quoted
Hi Andrew,

we produce a lot of boards and we have to change the MAC address,
from u-boot, for every board.  So I must save in the u-boot
environment (SPI NOR flash) the MAC address for every board.
Hi Flavio

u-boot should be able to write the MAC address in the correct part of
device tree. Boards have been doing this a long time.

Module parameters are considered bad. You should only do it if you
have no other option. Here you do have another options, so it is going
to be a hard sell getting David to access your patch.

You will have more success by adding a call to
eth_platform_get_mac_address() to the e1000e driver.
You have right, and thanks for your suggestions, 
but with a kernel parameter I can use the same method
for any board where the NVM is missed, independently of any architecture
(with or without the device tree presence - ARM or x86 or others).
Hi Flavio

Well, lets wait for David to say what he thinks about the module
parameter.

	Andrew
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help