All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Russell King (Oracle)" <linux@armlinux.org.uk>
To: Janpieter Sollie <janpieter.sollie@kabelmail.de>
Cc: Andrew Lunn <andrew@lunn.ch>,
	netdev@vger.kernel.org, Heiner Kallweit <hkallweit1@gmail.com>,
	"David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>
Subject: Re: [RFC] increase MDIO i2c poll timeout gradually (including patch)
Date: Sat, 20 Sep 2025 12:18:27 +0100	[thread overview]
Message-ID: <aM6Ng7tnEYdWmI1F@shell.armlinux.org.uk> (raw)
In-Reply-To: <f86737b0-a0fe-49a6-aeca-9e51fbdf0f0d@kabelmail.de>

On Sat, Sep 20, 2025 at 12:00:50PM +0200, Janpieter Sollie wrote:
> Op 19/09/2025 om 19:04 schreef Andrew Lunn:
> > On Fri, Sep 19, 2025 at 03:52:55PM +0200, Janpieter Sollie wrote:
> > > Hello everyone,
> > Please ensure you Cc: the correct Maintainers.
> > 
> > ./scripts/get_maintainer.pl drivers/net/phy/sfp.c
> > Russell King <linux@armlinux.org.uk> (maintainer:SFF/SFP/SFP+ MODULE SUPPORT)
> > Andrew Lunn <andrew@lunn.ch> (maintainer:ETHERNET PHY LIBRARY)
> > Heiner Kallweit <hkallweit1@gmail.com> (maintainer:ETHERNET PHY LIBRARY)
> > "David S. Miller" <davem@davemloft.net> (maintainer:NETWORKING DRIVERS)
> > Eric Dumazet <edumazet@google.com> (maintainer:NETWORKING DRIVERS)
> > Jakub Kicinski <kuba@kernel.org> (maintainer:NETWORKING DRIVERS)
> > Paolo Abeni <pabeni@redhat.com> (maintainer:NETWORKING DRIVERS)
> > netdev@vger.kernel.org (open list:SFF/SFP/SFP+ MODULE SUPPORT)
> > linux-kernel@vger.kernel.org (open list)
> Done, sorry, this is my first post here
> > 
> > > I tested a SFP module where the i2c bus is "unstable" at best.
> > Please tell us more about the hardware.
> > 
> > Also, what speed do you have the I2C bus running at? Have you tried
> > different clock-frequency values to slow down the I2C bus? Have you
> > checked the pull-up resistors? I2C problems are sometimes due to too
> > strong pull-ups.
> The hardware is a bananapi R4 2xSFP using a MT7988a SoC.
> The SFP+ module is a RJ45 rollball module using a AQR113C phy, but needs a
> quirk in sfp.c (added below)
> I'm not a i2c expert at all,
> but about the i2c bus speed, the SFP cage seems to be behind a muxer, not a i2c root.
> I could not find anything about i2c bus speed in /proc or /sys, maybe it's impossible to tell?
> 
> The dtsi or dtso files do not mention anything about bus speeds, so I honestly do not know.

As you have not include the author of the SFP support (me) in your
initial email, and have not provided a repeat of the description,
I'm afraid I have no idea what the issue is that you're encountering.

Thanks.

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!

  reply	other threads:[~2025-09-20 11:18 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-19 13:52 [RFC] increase MDIO i2c poll timeout gradually (including patch) Janpieter Sollie
2025-09-19 17:04 ` Andrew Lunn
2025-09-20 10:00   ` Janpieter Sollie
2025-09-20 11:18     ` Russell King (Oracle) [this message]
2025-09-20 12:34       ` Janpieter Sollie
     [not found]       ` <6d444507-1c97-4904-8edb-e8cc1aa4399e@kabelmail.de>
2025-09-20 13:53         ` Russell King (Oracle)
2025-09-22  8:04           ` Janpieter Sollie
2025-09-22 10:54             ` Janpieter Sollie
2025-09-22 12:38               ` Russell King (Oracle)
2025-09-22 13:16                 ` Russell King (Oracle)
2025-09-22 13:50                   ` Russell King (Oracle)
2025-09-22 14:30                 ` Janpieter Sollie
2025-09-22 15:25                   ` Russell King (Oracle)
2025-09-23  7:02                     ` Janpieter Sollie
2025-09-22 14:46                 ` Janpieter Sollie

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aM6Ng7tnEYdWmI1F@shell.armlinux.org.uk \
    --to=linux@armlinux.org.uk \
    --cc=andrew@lunn.ch \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=hkallweit1@gmail.com \
    --cc=janpieter.sollie@kabelmail.de \
    --cc=kuba@kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.