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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 28 Mar 2022 09:53:21 +0200 From: Clément Léger <clement.leger@...tlin.com> To: Andrew Lunn <andrew@...n.ch> Cc: Heiner Kallweit <hkallweit1@...il.com>, Russell King <linux@...linux.org.uk>, "David S . Miller" <davem@...emloft.net>, Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>, Horatiu Vultur <horatiu.vultur@...rochip.com>, Thomas Petazzoni <thomas.petazzoni@...tlin.com>, Alexandre Belloni <alexandre.belloni@...tlin.com>, Allan Nielsen <allan.nielsen@...rochip.com>, netdev@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [net-next 2/5] net: mdio: of: use fwnode_mdiobus_* functions Le Fri, 25 Mar 2022 19:32:45 +0100, Andrew Lunn <andrew@...n.ch> a écrit : > On Fri, Mar 25, 2022 at 06:22:31PM +0100, Clément Léger wrote: > > Now that fwnode support has been added and implements the same behavior > > expected by device-tree parsing > > The problem is, we cannot actually see that. There is no side by side > comparison which makes it clear it has the same behaviour. > > Please see if something like this will work: > > 1/4: copy drivers/net/mdio/of_mdio.c to drivers/net/mdio/fwnode_mdio.c > 2/4: Delete from fwnode_mdio.c the bits you don't need, like the whitelist > 3/4: modify what is left of fwnode_mdio.c to actually use fwnode. > 4/4: Rework of_mdio.c to use the code in fwnode_mdio.c > > The 3/4 should make it clear it has the same behaviour, because we can > see what you have actually changed. Indeed, that would be more clear to provide this as separate patches that shows clearly the conversion. Will do that. > > FYI: net-next is closed at the moment, so you need to post RFC > patches. Ok, I refered to http://vger.kernel.org/~davem/net-next.html which seems wrong actually > > Andrew Thanks -- Clément Léger, Embedded Linux and Kernel engineer at Bootlin https://bootlin.com
Powered by blists - more mailing lists