netdev - Re: [net-next,v3,05/11] net: phy: Create SFP phy

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 for Android: free password hash cracker in your pocket

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

Message-ID: <849815af-9f8f-4703-bab3-167b65cd7b69@bootlin.com>
Date: Tue, 3 Feb 2026 19:12:20 +0100
From: Maxime Chevallier <maxime.chevallier@...tlin.com>
To: Simon Horman <horms@...nel.org>
Cc: davem@...emloft.net, linux-kernel@...r.kernel.org, edumazet@...gle.com,
 o.rempel@...gutronix.de, andrew@...n.ch, christophe.leroy@...roup.eu,
 kuba@...nel.org, pabeni@...hat.com, thomas.petazzoni@...tlin.com,
 nicveronese@...il.com, kory.maincent@...tlin.com, mwojtas@...omium.org,
 daniel@...rotopia.org, dimitri.fedrau@...bherr.com,
 herve.codina@...tlin.com, linux@...linux.org.uk, vladimir.oltean@....com,
 kabel@...nel.org, romain.gantois@...tlin.com, f.fainelli@...il.com,
 hkallweit1@...il.com, netdev@...r.kernel.org
Subject: Re: [net-next,v3,05/11] net: phy: Create SFP phy_port before
 registering usptream

Hi Simon,

On 03/02/2026 19:03, Simon Horman wrote:
> This is an AI-generated review of your patch. The human sending this
> email has considered the AI review valid, or at least plausible.
> 
> For local reproduction steps see https://netdev-ai.bots.linux.dev/ai-local.html
> ---
> net: phy: Create SFP phy_port before registering usptream
> 
> This commit moves the creation of the SFP phy_port to occur before
> registering the SFP upstream ops. This reordering allows passing
> the phy_port along with the upstream information to the SFP bus.
> 
>> net: phy: Create SFP phy_port before registering usptream
> 
> This isn't a bug, but there's a typo in the subject line: "usptream"
> should be "upstream".

That's aleady addressed in V4 :)

Maxime