[PATCH v3 2/2] phy: mv88e61xx: add support for MV88E6171

[Chris Packham]

On Tue, 5 Jan 2021, 10:28 AM Pawel Dembicki, <paweldembicki at gmail.com>
wrote:

> This patch add MV88E6171 id to driver data.
>
> Tested on Checkpoint L-50 board.
>
> Cc: Chris Packham <judge.packham at gmail.com>
> Cc: Joe Hershberger <joe.hershberger at ni.com>
> Cc: Anatolij Gustschin <agust at denx.de>
> Cc: Tim Harvey <tharvey at gateworks.com>
> Signed-off-by: Pawel Dembicki <paweldembicki at gmail.com>
>

Looks good to me.

Reviewed-by: Chris Packham <judge.packham at gmail.com>

---
> Changes in v3,v2:
> - resend only
>  drivers/net/phy/mv88e61xx.c | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
>
> diff --git a/drivers/net/phy/mv88e61xx.c b/drivers/net/phy/mv88e61xx.c
> index 3bcd2ba242..82f1587ff5 100644
> --- a/drivers/net/phy/mv88e61xx.c
> +++ b/drivers/net/phy/mv88e61xx.c
> @@ -183,6 +183,7 @@
>  #define PORT_SWITCH_ID_6071            0x0710
>  #define PORT_SWITCH_ID_6096            0x0980
>  #define PORT_SWITCH_ID_6097            0x0990
> +#define PORT_SWITCH_ID_6171            0x1710
>  #define PORT_SWITCH_ID_6172            0x1720
>  #define PORT_SWITCH_ID_6176            0x1760
>  #define PORT_SWITCH_ID_6220            0x2200
> @@ -1055,6 +1056,7 @@ static int mv88e61xx_probe(struct phy_device *phydev)
>         switch (priv->id) {
>         case PORT_SWITCH_ID_6096:
>         case PORT_SWITCH_ID_6097:
> +       case PORT_SWITCH_ID_6171:
>         case PORT_SWITCH_ID_6172:
>         case PORT_SWITCH_ID_6176:
>         case PORT_SWITCH_ID_6240:
> @@ -1210,6 +1212,17 @@ static struct phy_driver mv88e61xx_driver = {
>         .shutdown = &genphy_shutdown,
>  };
>
> +static struct phy_driver mv88e617x_driver = {
> +       .name = "Marvell MV88E617x",
> +       .uid = 0x01410e70,
> +       .mask = 0xfffffff0,
> +       .features = PHY_GBIT_FEATURES,
> +       .probe = mv88e61xx_probe,
> +       .config = mv88e61xx_phy_config,
> +       .startup = mv88e61xx_phy_startup,
> +       .shutdown = &genphy_shutdown,
> +};
> +
>  static struct phy_driver mv88e609x_driver = {
>         .name = "Marvell MV88E609x",
>         .uid = 0x1410c89,
> @@ -1235,6 +1248,7 @@ static struct phy_driver mv88e6071_driver = {
>  int phy_mv88e61xx_init(void)
>  {
>         phy_register(&mv88e61xx_driver);
> +       phy_register(&mv88e617x_driver);
>         phy_register(&mv88e609x_driver);
>         phy_register(&mv88e6071_driver);
>
> --
> 2.25.1
>
>