From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934601AbbHLJPW (ORCPT ); Wed, 12 Aug 2015 05:15:22 -0400 Received: from eu-smtp-delivery-143.mimecast.com ([207.82.80.143]:64543 "EHLO eu-smtp-delivery-143.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932808AbbHLJPQ convert rfc822-to-8bit (ORCPT ); Wed, 12 Aug 2015 05:15:16 -0400 Date: Wed, 12 Aug 2015 10:15:13 +0100 From: Liviu Dudau To: Sergei Shtylyov Cc: Stephen Hemminger , Florian Fainelli , Mirko Lindner , "ryan.harkin@linaro.org" , netdev , lkml Subject: Re: [PATCH v2 net-next] sky2: use random address if EEPROM is bad Message-ID: <20150812091513.GU20890@e106497-lin.cambridge.arm.com> References: <55C2AB51.1040409@gmail.com> <1439303756-30333-1-git-send-email-Liviu.Dudau@arm.com> <55CA4546.40904@cogentembedded.com> MIME-Version: 1.0 In-Reply-To: <55CA4546.40904@cogentembedded.com> User-Agent: Mutt/1.5.22 (2013-10-16) X-OriginalArrivalTime: 12 Aug 2015 09:15:13.0743 (UTC) FILETIME=[656185F0:01D0D4DF] X-MC-Unique: cm7ZixJlRlyxy6GFtseIPg-1 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Aug 11, 2015 at 07:56:06PM +0100, Sergei Shtylyov wrote: > Hello. > > On 08/11/2015 05:35 PM, Liviu Dudau wrote: > > > On some embedded systems the EEPROM does not contain a valid MAC address. > > In that case it is better to fallback to a generated mac address and > > let init scripts fix the value later. > > > Reported-by: Liviu Dudau > > Signed-off-by: Stephen Hemminger > > [Changed handcoded setup to use eth_hw_addr_random() instead] > > Signed-off-by: Liviu Dudau > > --- > > I have tested this on my Juno platform and I can successfully do an nfsroot boot. > > > Best regards, > > Liviu > > > drivers/net/ethernet/marvell/sky2.c | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > diff --git a/drivers/net/ethernet/marvell/sky2.c b/drivers/net/ethernet/marvell/sky2.c > > index d9f4498..c309879 100644 > > --- a/drivers/net/ethernet/marvell/sky2.c > > +++ b/drivers/net/ethernet/marvell/sky2.c > > @@ -4819,6 +4819,13 @@ static struct net_device *sky2_init_netdev(struct sky2_hw *hw, unsigned port, > > memcpy_fromio(dev->dev_addr, hw->regs + B2_MAC_1 + port * 8, > > ETH_ALEN); > > > > + /* if the address is invalid, use a random value */ > > + if (!is_valid_ether_addr(dev->dev_addr)) { > > + netdev_warn(dev, > > + "Invalid MAC address, defaulting to random\n"); > > Please start the continuation line right under 'dev' on the borken up line. OK, I will make the changes for v3. > > > + eth_hw_addr_random(dev); > > + } > > + > > return dev; > > } > > MBR, Sergei > Thanks for reviewing this! Best regards, Liviu -- ==================== | I would like to | | fix the world, | | but they're not | | giving me the | \ source code! / --------------- ¯\_(ツ)_/¯