[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAFcVECKp0N-82WX-mtT_J_jZ0u8C221bwLRrrSELZ1jgMKs-wA@mail.gmail.com>
Date: Tue, 6 Aug 2019 11:17:11 +0530
From: Harini Katakam <harinik@...inx.com>
To: Andrew Lunn <andrew@...n.ch>
Cc: Harini Katakam <harini.katakam@...inx.com>,
Nicolas Ferre <nicolas.ferre@...rochip.com>,
David Miller <davem@...emloft.net>,
Claudiu Beznea <claudiu.beznea@...rochip.com>,
Rob Herring <robh+dt@...nel.org>,
Mark Rutland <mark.rutland@....com>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Michal Simek <michal.simek@...inx.com>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>
Subject: Re: [RFC PATCH 1/2] dt-bindings: net: macb: Add new property for PS
SGMII only
Hi Andrew,
On Mon, Aug 5, 2019 at 10:47 PM Andrew Lunn <andrew@...n.ch> wrote:
>
> > Even with the use of this interrupt, the link status actions (link print and
> > netif ops) will still be required. And also the need for macb_open to
> > proceed without phydev. Could you please let me know if that is acceptable
> > to patch or if there's a cleaner way to
> > report this link status?
>
> It sounds like you need to convert to phylink, so you get full sfp
> support. phylib does not handle hotplug of PHYs.
>
> Please look at the comments Russell gave the last time this was
> attempted.
Yes, I looked at the comments from Russell and wasn't sure if this
case qualified for phylink.
Regards,
Harini
Powered by blists - more mailing lists