From: "Marek Behún" <kabel@kernel.org>
To: Eric Woudstra <ericwouds@gmail.com>
Cc: netdev@vger.kernel.org, Russell King <rmk+kernel@armlinux.org.uk>,
Raju Lakkaraju <Raju.Lakkaraju@microchip.com>,
Frank Wunderlich <frank-w@public-files.de>,
Simon Horman <simon.horman@corigine.com>,
Andrew Lunn <andrew@lunn.ch>,
Heiner Kallweit <hkallweit1@gmail.com>
Subject: Re: [PATCH net-next 2/2] net: sfp: enhance quirk for Fibrestore 2.5G copper SFP module
Date: Tue, 23 Apr 2024 12:17:25 +0200 [thread overview]
Message-ID: <20240423121725.6539c023@dellmb> (raw)
In-Reply-To: <919416c4-334a-42da-8140-3ee85e71c15a@gmail.com>
On Tue, 23 Apr 2024 10:51:09 +0200
Eric Woudstra <ericwouds@gmail.com> wrote:
> On 4/23/24 10:40, Marek Behún wrote:
>
> >>
> >> Also any code I found for the yt8821 is C22 only. And even more, even
> >> though we are facing with an almost similar MCU, RollBall protocol does
> >> not work. I think it is almost the same mcu, as it responds to the same
> >> eeprom password, and also the rollball password does something, but not
> >> give the expected result.
> >>
> >
> > What about I2C address 0x56?
> >
> > I noticed that the Fibrestore FS SFP-2.5G-T with the realtek chip
> > also exposes clause 22 registers, but the current mdio-i2c driver wont
> > work, because the module implements the access in an incompatible way
> > (we would need to extend mdio-i2c).
> >
> > Eric, can you check whether the motorcomm module exposes something on
> > address 0x56? With i2cdump, i.e. on omnia:
> > i2cdump -y 5 0x56
> > (you will need to change 5 to the i2c controller of the sfp cage).
> > What does it print?
> >
> > Marek
>
> The device at 0x56 is not up at the time the eeprom is checked and read.
> So when it is time to decide whether to use RollBall protocol or not,
> the data at 0x56 cannot be used.
The fixup function for those modules could enable the module and wait
for the 0x56 i2c address...
> We have more info on i2cdumps on the BPI forum (rtl8221b topic and
> yt8821 topic).
next prev parent reply other threads:[~2024-04-23 10:17 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-22 9:44 [PATCH net-next 1/2] net: sfp: update comment for FS SFP-10G-T quirk Marek Behún
2024-04-22 9:44 ` [PATCH net-next 2/2] net: sfp: enhance quirk for Fibrestore 2.5G copper SFP module Marek Behún
2024-04-22 13:12 ` Andrew Lunn
2024-04-22 13:43 ` Marek Behún
2024-04-23 5:54 ` Eric Woudstra
2024-04-23 8:40 ` Marek Behún
2024-04-23 8:51 ` Eric Woudstra
2024-04-23 10:17 ` Marek Behún [this message]
2024-04-22 13:09 ` [PATCH net-next 1/2] net: sfp: update comment for FS SFP-10G-T quirk Andrew Lunn
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=20240423121725.6539c023@dellmb \
--to=kabel@kernel.org \
--cc=Raju.Lakkaraju@microchip.com \
--cc=andrew@lunn.ch \
--cc=ericwouds@gmail.com \
--cc=frank-w@public-files.de \
--cc=hkallweit1@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=rmk+kernel@armlinux.org.uk \
--cc=simon.horman@corigine.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.