linux-kernel - Re: [PATCH net-next] net: phy: at803x: add support for qca 8327 internal phy

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: <YUCVo4+wS3Q1Tg6Q@Ansuel-xps.localdomain>
Date:   Tue, 14 Sep 2021 14:29:23 +0200
From:   Ansuel Smith <ansuelsmth@...il.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>,
        Jakub Kicinski <kuba@...nel.org>, netdev@...r.kernel.org,
        linux-kernel@...r.kernel.org, Rosen Penev <rosenp@...il.com>
Subject: Re: [PATCH net-next] net: phy: at803x: add support for qca 8327
 internal phy

On Tue, Sep 14, 2021 at 02:24:10PM +0200, Andrew Lunn wrote:
> On Tue, Sep 14, 2021 at 09:11:41AM +0200, Ansuel Smith wrote:
> > Add support for qca8327 internal phy needed for correct init of the
> > switch port. It does use the same qca8337 function and reg just with a
> > different id.
> 
> Hi Ansuel
> 
> Please also add it to the atheros_tbl array. It looks like the 8337 is
> also missing as well.
>

Sure will send v2.

> Have you tried the cable test code on this PHY?

Yes I tried, the documentation is very confusionary and with a simple
implementation it looks like it doesn't work at all... In one
documentation version the reg for cable test are described but by
actually implementing and setting the correct regs nothing happen and
the random results are reported. I honestly thing it doesn't support
cable test at all...

> 
>      Andrew