linux-kernel - Re: [PATCH net-next v4] net: sfp: add SMBus I2C block support

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: <d5c11fec-1e75-46cf-aeae-593fb6a4af09@gmail.com>
Date: Fri, 16 Jan 2026 12:16:11 +0100
From: Jonas Jelonek <jelonek.jonas@...il.com>
To: Maxime Chevallier <maxime.chevallier@...tlin.com>
Cc: netdev@...r.kernel.org, linux-kernel@...r.kernel.org,
 Bjørn Mork <bjorn@...k.no>, Paolo Abeni
 <pabeni@...hat.com>, Jakub Kicinski <kuba@...nel.org>,
 Eric Dumazet <edumazet@...gle.com>, "David S . Miller"
 <davem@...emloft.net>, Russell King <linux@...linux.org.uk>,
 Andrew Lunn <andrew@...n.ch>, Heiner Kallweit <hkallweit1@...il.com>
Subject: Re: [PATCH net-next v4] net: sfp: add SMBus I2C block support

Hi Maxime,

On 09.01.26 18:03, Maxime Chevallier wrote:
> ACK, I'll gladly help with testing. This should actually be easily
> achievable with a board that has a real i2c interface connected to the
> SFP cage, as there's a i2c smbus emulation layer. The SMBus helpers will
> work with a true I2C adapter, you may have to tweak the code though.
>
> This is relevant for modules that have a built-in PHY that you can
> access, if you don't have any I can run some tests here, I have more
> than enough modules...
>
> If you don't have time at all for that, I may give this a shot at some
> point, but my time is a bit scarce right now :'(
>

I'd postpone this part if that's ok. Quite busy at the moment :(

When I come to trying to work on that, should that all be kept in
mdio-i2c.c? I'm asking because we have a downstream implementation
moving that SMbus stuff to mdio-smbus.c. This covers quite a lot right
now, C22/C45 and Rollball, but just with byte access [1]. Because that
isn't my work, I'll need to check with the original authors and adapt this
for an upstream patch, trying to add word + block access.

Kind regards,
Jonas

[1] https://github.com/openwrt/openwrt/blob/66b6791abe6f08dec2924b5d9e9e7dac93f37bc4/target/linux/realtek/patches-6.12/712-net-phy-add-an-MDIO-SMBus-library.patch