Re: [PATCH v2 05/15] net: phy: adin: add {write,read}_mmd hooks

[Ardelean, Alexandru]

On Thu, 2019-08-08 at 17:35 +0200, Andrew Lunn wrote:
> [External]
> 
> On Thu, Aug 08, 2019 at 03:30:16PM +0300, Alexandru Ardelean wrote:
> > Both ADIN1200 & ADIN1300 support Clause 45 access.
> > The Extended Management Interface (EMI) registers are accessible via both
> > Clause 45 (at register MDIO_MMD_VEND1) and using Clause 22.
> > 
> > However, the Clause 22 MMD access operations differ from the implementation
> > in the kernel, in the sense that it uses registers ExtRegPtr (0x10) &
> > ExtRegData (0x11) to access Clause 45 & EMI registers.
> 
> It is not that they differ from what the kernel supports. Its that
> they differ from what the Standard says they should use. These
> registers are defined in 802.3, part of C22, and this hardware
> implements the standard incorrectly.

Will update comment.
I did not find a document describing this as a standard.
But, I admit I am terrible when finding some docs.

I only found this presentation from a while back:
http://www.ieee802.org/3/efm/public/nov02/oam/pannell_oam_1_1102.pdf

It seemed more like a proposal.

Thanks
Alex

> 
> 	   Andrew