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
| ||
|
Message-ID: <c50b91b7-4dfc-45d7-9fd2-f1afb4480acc@lunn.ch> Date: Wed, 29 Mar 2023 00:35:00 +0200 From: Andrew Lunn <andrew@...n.ch> To: Daniel Golle <daniel@...rotopia.org> Cc: netdev@...r.kernel.org, linux-mediatek@...ts.infradead.org, linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org, Florian Fainelli <f.fainelli@...il.com>, Vladimir Oltean <olteanv@...il.com>, "David S. Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>, Matthias Brugger <matthias.bgg@...il.com>, AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>, Sean Wang <sean.wang@...iatek.com>, Landen Chao <Landen.Chao@...iatek.com>, DENG Qingfang <dqfext@...il.com>, Philipp Zabel <p.zabel@...gutronix.de>, Sam Shih <Sam.Shih@...iatek.com>, Lorenzo Bianconi <lorenzo@...nel.org>, John Crispin <john@...ozen.org>, Felix Fietkau <nbd@....name> Subject: Re: [RFC PATCH net-next v2 1/2] net: dsa: mt7530: split-off MDIO driver On Tue, Mar 28, 2023 at 11:16:51PM +0100, Daniel Golle wrote: > In order to support the built-in switch of some MediaTek SoCs we need > to use MMIO instead of MDIO to access the switch. > Prepare this be splitting-off the part of the driver registering an > MDIO driver, so we can add another module acting as MMIO/platform driver. > > Signed-off-by: Daniel Golle <daniel@...rotopia.org> Hi Daniel This is getting better. Please try to split this patch up. Ideally you want lots of small patches which are obviously correct. So maybe refactor the creation of the PCS as a patch. Move p5_intf_modes() as a patch, etc. Also, look at mt7530_probe() and mt7988_probe() side by side and see what is common and can be put into one shared function in the core. Same for remove. It could be that creating that helper is a patch done before adding mt7530-mdio.c. Andrew
Powered by blists - more mailing lists