Re: [PATCH v2] net: phy: add Maxlinear GPY115/21x/24x driver

On Fri, Jun 04, 2021 at 12:52:02PM +0000, Liang Xu wrote:

On 4/6/2021 8:15 pm, Andrew Lunn wrote:

quoted

This email was sent from outside of MaxLinear.

quoted

+config MXL_GPHY
+     tristate "Maxlinear PHYs"
+     help
+       Support for the Maxlinear GPY115, GPY211, GPY212, GPY215,
+       GPY241, GPY245 PHYs.

Do these PHYs have unique IDs in register 2 and 3? What is the format
of these IDs?

The OUI is fixed. But often the rest is split into two. The higher
part indicates the product, and the lower part is the revision. We
then have a struct phy_driver for each product, and the mask is used
to match on all the revisions of the product.

      Andrew

Register 2, Register 3 bit 10~15 - OUI

Register 3 bit 4~9 - product number

Register 3 bit 0~3 - revision number

These PHYs have same ID in register 2 and 3.

O.K, that is pretty normal. Please add a phy_device for each
individual PHY. There are macros to help you do this.

	Andrew