Re: [PATCH] net: uclass: Save ethernet MAC address when generated

From: Tom Rini <trini@konsulko.com>
To: Wolfgang Denk <wd@denx.de>
Cc: Ramon Fried <rfried.dev@gmail.com>,
	Michael Walle <michael@walle.cc>,
	Michal Simek <michal.simek@xilinx.com>,
	Grygorii Strashko <grygorii.strashko@ti.com>,
	git <git@xilinx.com>, Joe Hershberger <joe.hershberger@ni.com>,
	Simon Glass <sjg@chromium.org>,
	U-Boot Mailing List <u-boot@lists.denx.de>
Subject: Re: [PATCH] net: uclass: Save ethernet MAC address when generated
Date: Thu, 18 Nov 2021 09:51:55 -0500	[thread overview]
Message-ID: <20211118145155.GF24579@bill-the-cat> (raw)
In-Reply-To: <1901784.1637228788@gemini.denx.de>

[-- Attachment #1: Type: text/plain, Size: 1221 bytes --]

On Thu, Nov 18, 2021 at 10:46:28AM +0100, Wolfgang Denk wrote:
> Dear Tom,
> 
> In message <1889944.1637219294@gemini.denx.de> I wrote:
> >
> > > We're about to
> > > introduce the 3rd variant.  I'd feel a whole lot better about taking in
> > > a v2 of this patch that correct the help (and maybe updates
> > > doc/README.enetaddr!) if someone could report back on what's going on
> > > with the layerscape, imx* and socfpga platforms.  There's in fact a
> > > number of platforms enabling it that I'm pretty sure DENX has or had the
> > > hardware on, so can we get some spot checking done there?
> >
> > I will check and provide an update later, but from the best of my
> > knowledge none of the boards we ported actually need or use this
> > feature.  This is just a copy&paste mess.
> 
> We could not find even a single board maintained or upstreamed
> by DENX where NET_RANDOM_ETHADDR was even mentioned in the
> specifications, and certainly not a mandatory feature.
> 
> That's all copy & paste garbage.

OK, can we please get some patches going then to remove them, starting
with the platforms you have, and then the reference platforms it was
copypastad over from?  Thanks!

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]