netdev - Re: [PATCH v3 net-next] net: ethernet: ti: davinci

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: <YwE8pob0SOiHZ7K7@lunn.ch>
Date:   Sat, 20 Aug 2022 21:57:26 +0200
From:   Andrew Lunn <andrew@...n.ch>
To:     Ravi Gunasekaran <r-gunasekaran@...com>
Cc:     davem@...emloft.net, edumazet@...gle.com, kuba@...nel.org,
        pabeni@...hat.com, linux-omap@...r.kernel.org,
        netdev@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        kishon@...com, vigneshr@...com, lkp@...el.com
Subject: Re: [PATCH v3 net-next] net: ethernet: ti: davinci_mdio: Add
 workaround for errata i2329

On Wed, Aug 17, 2022 at 03:14:06PM +0530, Ravi Gunasekaran wrote:
> On the CPSW and ICSS peripherals, there is a possibility that the MDIO
> interface returns corrupt data on MDIO reads or writes incorrect data
> on MDIO writes. There is also a possibility for the MDIO interface to
> become unavailable until the next peripheral reset.
> 
> The workaround is to configure the MDIO in manual mode and disable the
> MDIO state machine and emulate the MDIO protocol by reading and writing
> appropriate fields in MDIO_MANUAL_IF_REG register of the MDIO controller
> to manipulate the MDIO clock and data pins.
> 
> More details about the errata i2329 and the workaround is available in:
> https://www.ti.com/lit/er/sprz487a/sprz487a.pdf
> 
> Add implementation to disable MDIO state machine, configure MDIO in manual
> mode and achieve MDIO read and writes via MDIO Bitbanging
> 
> Signed-off-by: Ravi Gunasekaran <r-gunasekaran@...com>
> Reported-by: kernel test robot <lkp@...el.com>

Reviewed-by: Andrew Lunn <andrew@...n.ch>

    Andrew