netdev - Re: [RESEND][PATCH] net: cpsw: fix obtaining mac address for am3517

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <ad0ec6ac-2760-4a03-8cee-0d933aea98eb@lunn.ch>
Date: Sat, 24 Jun 2023 16:17:33 +0200
From: Andrew Lunn <andrew@...n.ch>
To: Mans Rullgard <mans@...sr.com>
Cc: Grygorii Strashko <grygorii.strashko@...com>,
	"David S. Miller" <davem@...emloft.net>,
	Eric Dumazet <edumazet@...gle.com>,
	Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
	linux-omap@...r.kernel.org, netdev@...r.kernel.org,
	linux-kernel@...r.kernel.org,
	Jeroen Hofstee <jhofstee@...tronenergy.com>,
	Tony Lindgren <tony@...mide.com>
Subject: Re: [RESEND][PATCH] net: cpsw: fix obtaining mac address for am3517

On Sat, Jun 24, 2023 at 01:10:59PM +0100, Mans Rullgard wrote:
> From: Jeroen Hofstee <jhofstee@...tronenergy.com>
> 
> Commit b6745f6e4e63 ("drivers: net: cpsw: davinci_emac: move reading mac
> id to common file") did not only move the code for an am3517, it also
> added the slave parameter, resulting in an invalid (all zero) mac address
> being returned for an am3517, since it only has a single emac

Hi Mans

If there is only a single emac, why is the function being called with
slave=1? Given the description, it seems like you are fixing the wrong
problem.

	Andrew