[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <a1231b3a-cd4d-4e74-9266-95350f880449@csgroup.eu>
Date: Fri, 16 Aug 2024 19:02:20 +0200
From: Christophe Leroy <christophe.leroy@...roup.eu>
To: Jakub Kicinski <kuba@...nel.org>, Russell King <linux@...linux.org.uk>
Cc: davem@...emloft.net, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org, thomas.petazzoni@...tlin.com,
Andrew Lunn <andrew@...n.ch>, Eric Dumazet <edumazet@...gle.com>,
Paolo Abeni <pabeni@...hat.com>, linux-arm-kernel@...ts.infradead.org,
Herve Codina <herve.codina@...tlin.com>,
Florian Fainelli <f.fainelli@...il.com>,
Heiner Kallweit <hkallweit1@...il.com>,
Vladimir Oltean <vladimir.oltean@....com>,
Köry Maincent <kory.maincent@...tlin.com>,
Jesse Brandeburg <jesse.brandeburg@...el.com>, Marek Behún
<kabel@...nel.org>, Piergiorgio Beruto <piergiorgio.beruto@...il.com>,
Oleksij Rempel <o.rempel@...gutronix.de>,
Nicolò Veronese <nicveronese@...il.com>,
Simon Horman <horms@...nel.org>, mwojtas@...omium.org,
Nathan Chancellor <nathan@...nel.org>, Antoine Tenart <atenart@...nel.org>,
Marc Kleine-Budde <mkl@...gutronix.de>,
Dan Carpenter <dan.carpenter@...aro.org>,
Romain Gantois <romain.gantois@...tlin.com>,
Maxime Chevallier <maxime.chevallier@...tlin.com>
Subject: Re: [PATCH net-next v17 00/14] Introduce PHY listing and
link_topology tracking
Hi Jakub, Russell
Le 17/07/2024 à 17:26, Jakub Kicinski a écrit :
> On Tue, 16 Jul 2024 10:16:26 +0200 Maxime Chevallier wrote:
>>> I lack the confidence to take this during the merge window, without
>>> Russell's acks. So Deferred, sorry :(
>>
>> Understood. Is there anything I can make next time to make that series
>> more digestable and easy to review ? I didn't want to split the netlink
>> part from the core part, as just the phy_link_topology alone doesn't
>> make much sense for now, but it that makes the lives of reviewers
>> easier I could submit these separately.
>
> TBH I can only review this from coding and netlink perspective, and
> it looks solid. Folk who actually know PHYs and SFPs may have more
> meaningful feedback :(
How can we progress on this ?
Russell, have you been able to have a look at that latest version of the
series ? I know you reviewed earlier versions already but I understand
Jakub is willing some feedback from you.
Jakub, as you say it looks solid. I can add to that that I have been
using this series widely through the double Ethernet attachment on
several boards and it works well, it is stable and more performant than
the dirty home-made solution we had on v4.14.
So it would be great if the series could be merged for v6.12, and I
guess the earliest it is merged into net-next the more time it spends in
linux-next before the merge window. Any chance to get it merged anytime
soon even without a formal feedback from Russell ? We are really looking
forward to getting that series merged and step forward with all the work
that depends on it and is awaiting.
Thanks
Christophe
Powered by blists - more mailing lists