netdev - Re: [PATCH net-next v2 1/1] net: phy: dp83867: retrigger SGMII AN when link change

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: <20220527014709.GA26992@linux.intel.com>
Date:   Fri, 27 May 2022 09:47:09 +0800
From:   Tan Tee Min <tee.min.tan@...ux.intel.com>
To:     Andrew Lunn <andrew@...n.ch>
Cc:     Heiner Kallweit <hkallweit1@...il.com>,
        Russell King <linux@...linux.org.uk>,
        "David S . Miller" <davem@...emloft.net>,
        Eric Dumazet <edumazet@...gle.com>,
        Jakub Kicinski <kuba@...nel.org>,
        Paolo Abeni <pabeni@...hat.com>, Dan Murphy <dmurphy@...com>,
        netdev@...r.kernel.org, linux-kernel@...r.kernel.org,
        stable@...r.kernel.org, Voon Wei Feng <weifeng.voon@...el.com>,
        Sit Michael Wei Hong <michael.wei.hong.sit@...el.com>,
        Ling Pei Lee <pei.lee.ling@...el.com>,
        Looi Hong Aun <hong.aun.looi@...el.com>,
        Wong Vee Khee <vee.khee.wong@...el.com>,
        Tan Tee Min <tee.min.tan@...el.com>
Subject: Re: [PATCH net-next v2 1/1] net: phy: dp83867: retrigger SGMII AN
 when link change

On Thu, May 26, 2022 at 02:32:14PM +0200, Andrew Lunn wrote:
> On Thu, May 26, 2022 at 05:03:47PM +0800, Tan Tee Min wrote:
> > This could cause an issue during power up, when PHY is up prior to MAC.
> > At this condition, once MAC side SGMII is up, MAC side SGMII wouldn`t
> > receive new in-band message from TI PHY with correct link status, speed
> > and duplex info.
> > 
> > As suggested by TI, implemented a SW solution here to retrigger SGMII
> > Auto-Neg whenever there is a link change.
> 
> Is there a bit in the PHY which reports host side link? There is no
> point triggering an AN if there is already link.
> 
>       Andrew

Thanks for your comment.

There is no register bit in TI PHY which reports the SGMII AN link status.
But, there is a bit that only reports the SGMII AN completion status.

In this case, the PHY side SGMII AN has been already completed prior to MAC is up.
So, once MAC side SGMII is up, MAC side SGMII wouldn`t receive any new
in-band message from TI PHY.

Thanks,
Tee Min