diff mbox series

[U-Boot,v2,5/6] net: phy: mv88e61xx: register phy_driver struct for 88E6071

Message ID 20190709232251.31746-6-agust@denx.de
State Changes Requested
Delegated to: Joe Hershberger
Headers show
Series Extend mv88e61xx driver to support 88E6071 | expand

Commit Message

Anatolij Gustschin July 9, 2019, 11:22 p.m. UTC
Support probing and init for 88E6071 switch.

Signed-off-by: Anatolij Gustschin <agust@denx.de>
---
 drivers/net/phy/mv88e61xx.c | 12 ++++++++++++
 1 file changed, 12 insertions(+)

Comments

Joe Hershberger July 23, 2019, 4:29 a.m. UTC | #1
On Tue, Jul 9, 2019 at 6:25 PM Anatolij Gustschin <agust@denx.de> wrote:
>
> Support probing and init for 88E6071 switch.
>
> Signed-off-by: Anatolij Gustschin <agust@denx.de>
> ---
>  drivers/net/phy/mv88e61xx.c | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
>
> diff --git a/drivers/net/phy/mv88e61xx.c b/drivers/net/phy/mv88e61xx.c
> index 165bc261e4..d827ecc1bc 100644
> --- a/drivers/net/phy/mv88e61xx.c
> +++ b/drivers/net/phy/mv88e61xx.c
> @@ -1116,10 +1116,22 @@ static struct phy_driver mv88e609x_driver = {
>         .shutdown = &genphy_shutdown,
>  };
>
> +static struct phy_driver mv88e6071_driver = {
> +       .name = "Marvell MV88E6071",

Is it really just the 6071? What about all the other ones mentioned in
the Kconfig?

> +       .uid = 0x1410db0,
> +       .mask = 0xfffffff0,
> +       .features = PHY_BASIC_FEATURES | SUPPORTED_MII,
> +       .probe = mv88e61xx_probe,
> +       .config = mv88e61xx_phy_config,
> +       .startup = mv88e61xx_phy_startup,
> +       .shutdown = &genphy_shutdown,
> +};
> +
>  int phy_mv88e61xx_init(void)
>  {
>         phy_register(&mv88e61xx_driver);
>         phy_register(&mv88e609x_driver);
> +       phy_register(&mv88e6071_driver);
>
>         return 0;
>  }
> --
> 2.17.1
>
> _______________________________________________
> U-Boot mailing list
> U-Boot@lists.denx.de
> https://lists.denx.de/listinfo/u-boot
Anatolij Gustschin July 25, 2019, 9:41 p.m. UTC | #2
On Tue, 23 Jul 2019 04:29:05 +0000
Joe Hershberger joe.hershberger@ni.com wrote:
...
> > +static struct phy_driver mv88e6071_driver = {
> > +       .name = "Marvell MV88E6071",  
> 
> Is it really just the 6071? What about all the other ones mentioned in
> the Kconfig?

Yes, only 6071 for now. I do not have other compatible devices to
get the IDs for adding them in the driver.

--
Anatolij
diff mbox series

Patch

diff --git a/drivers/net/phy/mv88e61xx.c b/drivers/net/phy/mv88e61xx.c
index 165bc261e4..d827ecc1bc 100644
--- a/drivers/net/phy/mv88e61xx.c
+++ b/drivers/net/phy/mv88e61xx.c
@@ -1116,10 +1116,22 @@  static struct phy_driver mv88e609x_driver = {
 	.shutdown = &genphy_shutdown,
 };
 
+static struct phy_driver mv88e6071_driver = {
+	.name = "Marvell MV88E6071",
+	.uid = 0x1410db0,
+	.mask = 0xfffffff0,
+	.features = PHY_BASIC_FEATURES | SUPPORTED_MII,
+	.probe = mv88e61xx_probe,
+	.config = mv88e61xx_phy_config,
+	.startup = mv88e61xx_phy_startup,
+	.shutdown = &genphy_shutdown,
+};
+
 int phy_mv88e61xx_init(void)
 {
 	phy_register(&mv88e61xx_driver);
 	phy_register(&mv88e609x_driver);
+	phy_register(&mv88e6071_driver);
 
 	return 0;
 }