Netdev Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Paul Barker <pbarker@konsulko.com>
To: Andrew Lunn <andrew@lunn.ch>
Cc: Woojung Huh <woojung.huh@microchip.com>,
Microchip Linux Driver Support <UNGLinuxDriver@microchip.com>,
Vivien Didelot <vivien.didelot@gmail.com>,
Florian Fainelli <f.fainelli@gmail.com>,
"David S . Miller" <davem@davemloft.net>,
netdev@vger.kernel.org
Subject: Re: [PATCH 3/4] net: dsa: microchip: Disable RGMII in-band status on KSZ9893
Date: Sat, 5 Sep 2020 16:53:20 +0100 [thread overview]
Message-ID: <CAM9ZRVs8e7hcS4T=Nt6M4iyDWA8uT42m=iRnYzQFg0ajL6rwTw@mail.gmail.com> (raw)
In-Reply-To: <20200905153238.GE3164319@lunn.ch>
On Sat, 5 Sep 2020 at 16:32, Andrew Lunn <andrew@lunn.ch> wrote:
>
> On Sat, Sep 05, 2020 at 03:03:24PM +0100, Paul Barker wrote:
> > We can't assume that the link partner supports the in-band status
> > reporting which is enabled by default on the KSZ9893 when using RGMII
> > for the upstream port.
>
> What do you mean by RGMII inband status reporting? SGMII/1000BaseX has
> in band signalling, but RGMII?
>
> Andrew
I'm referencing page 56 of the KSZ9893 datasheet
(http://ww1.microchip.com/downloads/en/DeviceDoc/KSZ9893R-Data-Sheet-DS00002420D.pdf).
The datasheet says "The RGMII port will not function properly if IBS
is enabled in the switch, but it is not receiving in-band status from
a connected PHY." Since we can't guarantee all possible link partners
will support this it should be disabled. In particular, the IMX6 SoC
we're using with this switch doesn't support this on its Ethernet
port.
I don't really know much about how this is implemented or how widely
it's supported.
Thanks,
--
Paul Barker
Konsulko Group
next prev parent reply other threads:[~2020-09-05 15:53 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-05 14:03 [PATCH 0/4] ksz9477 dsa switch driver improvements Paul Barker
2020-09-05 14:03 ` [PATCH 1/4] net: dsa: microchip: Make switch detection more informative Paul Barker
2020-09-05 15:28 ` Andrew Lunn
2020-09-05 14:03 ` [PATCH 2/4] net: dsa: microchip: Add debug print for XMII port mode Paul Barker
2020-09-05 15:31 ` Andrew Lunn
2020-09-05 14:03 ` [PATCH 3/4] net: dsa: microchip: Disable RGMII in-band status on KSZ9893 Paul Barker
2020-09-05 15:32 ` Andrew Lunn
2020-09-05 15:53 ` Paul Barker [this message]
2020-09-05 16:04 ` Florian Fainelli
2020-09-05 16:06 ` Andrew Lunn
2020-09-05 14:03 ` [PATCH 4/4] net: dsa: microchip: Implement recommended reset timing Paul Barker
2020-09-05 15:34 ` [PATCH 0/4] ksz9477 dsa switch driver improvements Andrew Lunn
2020-09-05 15:54 ` Paul Barker
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='CAM9ZRVs8e7hcS4T=Nt6M4iyDWA8uT42m=iRnYzQFg0ajL6rwTw@mail.gmail.com' \
--to=pbarker@konsulko.com \
--cc=UNGLinuxDriver@microchip.com \
--cc=andrew@lunn.ch \
--cc=davem@davemloft.net \
--cc=f.fainelli@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=vivien.didelot@gmail.com \
--cc=woojung.huh@microchip.com \
--subject='Re: [PATCH 3/4] net: dsa: microchip: Disable RGMII in-band status on KSZ9893' \
/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
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).