* [PATCH v3 0/2] net: phy: at803x: add ar9331 support
@ 2019-10-03 8:21 Oleksij Rempel
2019-10-03 8:21 ` [PATCH v3 1/2] " Oleksij Rempel
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Oleksij Rempel @ 2019-10-03 8:21 UTC (permalink / raw)
To: Andrew Lunn, Florian Fainelli, Heiner Kallweit
Cc: Oleksij Rempel, Pengutronix Kernel Team, David S. Miller, netdev,
linux-kernel
changes v3:
- use PHY_ID_MATCH_EXACT only for ATH9331 PHY
changes v2:
- use PHY_ID_MATCH_EXACT instead of leaky masking
- remove probe and struct at803x_priv
Oleksij Rempel (2):
net: phy: at803x: add ar9331 support
net: phy: at803x: remove probe and struct at803x_priv
drivers/net/phy/at803x.c | 33 ++++++++++++---------------------
1 file changed, 12 insertions(+), 21 deletions(-)
--
2.23.0
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v3 1/2] net: phy: at803x: add ar9331 support
2019-10-03 8:21 [PATCH v3 0/2] net: phy: at803x: add ar9331 support Oleksij Rempel
@ 2019-10-03 8:21 ` Oleksij Rempel
2019-10-03 8:21 ` [PATCH v3 2/2] net: phy: at803x: remove probe and struct at803x_priv Oleksij Rempel
2019-10-03 19:27 ` [PATCH v3 0/2] net: phy: at803x: add ar9331 support David Miller
2 siblings, 0 replies; 4+ messages in thread
From: Oleksij Rempel @ 2019-10-03 8:21 UTC (permalink / raw)
To: Andrew Lunn, Florian Fainelli, Heiner Kallweit
Cc: Oleksij Rempel, Pengutronix Kernel Team, David S. Miller, netdev,
linux-kernel
Mostly this hardware can work with generic PHY driver, but this change
is needed to provided interrupt handling support.
Tested with dsa ar9331-switch driver.
Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
Reviewed-by: Heiner Kallweit <hkallweit1@gmail.com>
---
drivers/net/phy/at803x.c | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/drivers/net/phy/at803x.c b/drivers/net/phy/at803x.c
index 6ad8b1c63c34..8fb7abdf10d0 100644
--- a/drivers/net/phy/at803x.c
+++ b/drivers/net/phy/at803x.c
@@ -53,6 +53,7 @@
#define AT803X_DEBUG_REG_5 0x05
#define AT803X_DEBUG_TX_CLK_DLY_EN BIT(8)
+#define ATH9331_PHY_ID 0x004dd041
#define ATH8030_PHY_ID 0x004dd076
#define ATH8031_PHY_ID 0x004dd074
#define ATH8035_PHY_ID 0x004dd072
@@ -406,6 +407,16 @@ static struct phy_driver at803x_driver[] = {
.aneg_done = at803x_aneg_done,
.ack_interrupt = &at803x_ack_interrupt,
.config_intr = &at803x_config_intr,
+}, {
+ /* ATHEROS AR9331 */
+ PHY_ID_MATCH_EXACT(ATH9331_PHY_ID),
+ .name = "Atheros AR9331 built-in PHY",
+ .config_init = at803x_config_init,
+ .suspend = at803x_suspend,
+ .resume = at803x_resume,
+ /* PHY_BASIC_FEATURES */
+ .ack_interrupt = &at803x_ack_interrupt,
+ .config_intr = &at803x_config_intr,
} };
module_phy_driver(at803x_driver);
@@ -414,6 +425,7 @@ static struct mdio_device_id __maybe_unused atheros_tbl[] = {
{ ATH8030_PHY_ID, AT803X_PHY_ID_MASK },
{ ATH8031_PHY_ID, AT803X_PHY_ID_MASK },
{ ATH8035_PHY_ID, AT803X_PHY_ID_MASK },
+ { PHY_ID_MATCH_EXACT(ATH9331_PHY_ID) },
{ }
};
--
2.23.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH v3 2/2] net: phy: at803x: remove probe and struct at803x_priv
2019-10-03 8:21 [PATCH v3 0/2] net: phy: at803x: add ar9331 support Oleksij Rempel
2019-10-03 8:21 ` [PATCH v3 1/2] " Oleksij Rempel
@ 2019-10-03 8:21 ` Oleksij Rempel
2019-10-03 19:27 ` [PATCH v3 0/2] net: phy: at803x: add ar9331 support David Miller
2 siblings, 0 replies; 4+ messages in thread
From: Oleksij Rempel @ 2019-10-03 8:21 UTC (permalink / raw)
To: Andrew Lunn, Florian Fainelli, Heiner Kallweit
Cc: Oleksij Rempel, Pengutronix Kernel Team, David S. Miller, netdev,
linux-kernel
struct at803x_priv is never used in this driver. So remove it
and the probe function allocating it.
Suggested-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
Reviewed-by: Heiner Kallweit <hkallweit1@gmail.com>
---
drivers/net/phy/at803x.c | 21 ---------------------
1 file changed, 21 deletions(-)
diff --git a/drivers/net/phy/at803x.c b/drivers/net/phy/at803x.c
index 8fb7abdf10d0..463fc681dead 100644
--- a/drivers/net/phy/at803x.c
+++ b/drivers/net/phy/at803x.c
@@ -63,10 +63,6 @@ MODULE_DESCRIPTION("Atheros 803x PHY driver");
MODULE_AUTHOR("Matus Ujhelyi");
MODULE_LICENSE("GPL");
-struct at803x_priv {
- bool phy_reset:1;
-};
-
struct at803x_context {
u16 bmcr;
u16 advertise;
@@ -232,20 +228,6 @@ static int at803x_resume(struct phy_device *phydev)
return phy_modify(phydev, MII_BMCR, BMCR_PDOWN | BMCR_ISOLATE, 0);
}
-static int at803x_probe(struct phy_device *phydev)
-{
- struct device *dev = &phydev->mdio.dev;
- struct at803x_priv *priv;
-
- priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
- if (!priv)
- return -ENOMEM;
-
- phydev->priv = priv;
-
- return 0;
-}
-
static int at803x_config_init(struct phy_device *phydev)
{
int ret;
@@ -368,7 +350,6 @@ static struct phy_driver at803x_driver[] = {
.phy_id = ATH8035_PHY_ID,
.name = "Atheros 8035 ethernet",
.phy_id_mask = AT803X_PHY_ID_MASK,
- .probe = at803x_probe,
.config_init = at803x_config_init,
.set_wol = at803x_set_wol,
.get_wol = at803x_get_wol,
@@ -382,7 +363,6 @@ static struct phy_driver at803x_driver[] = {
.phy_id = ATH8030_PHY_ID,
.name = "Atheros 8030 ethernet",
.phy_id_mask = AT803X_PHY_ID_MASK,
- .probe = at803x_probe,
.config_init = at803x_config_init,
.link_change_notify = at803x_link_change_notify,
.set_wol = at803x_set_wol,
@@ -397,7 +377,6 @@ static struct phy_driver at803x_driver[] = {
.phy_id = ATH8031_PHY_ID,
.name = "Atheros 8031 ethernet",
.phy_id_mask = AT803X_PHY_ID_MASK,
- .probe = at803x_probe,
.config_init = at803x_config_init,
.set_wol = at803x_set_wol,
.get_wol = at803x_get_wol,
--
2.23.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v3 0/2] net: phy: at803x: add ar9331 support
2019-10-03 8:21 [PATCH v3 0/2] net: phy: at803x: add ar9331 support Oleksij Rempel
2019-10-03 8:21 ` [PATCH v3 1/2] " Oleksij Rempel
2019-10-03 8:21 ` [PATCH v3 2/2] net: phy: at803x: remove probe and struct at803x_priv Oleksij Rempel
@ 2019-10-03 19:27 ` David Miller
2 siblings, 0 replies; 4+ messages in thread
From: David Miller @ 2019-10-03 19:27 UTC (permalink / raw)
To: o.rempel; +Cc: andrew, f.fainelli, hkallweit1, kernel, netdev, linux-kernel
From: Oleksij Rempel <o.rempel@pengutronix.de>
Date: Thu, 3 Oct 2019 10:21:11 +0200
> changes v3:
> - use PHY_ID_MATCH_EXACT only for ATH9331 PHY
>
> changes v2:
> - use PHY_ID_MATCH_EXACT instead of leaky masking
> - remove probe and struct at803x_priv
Looks good and all the feedback has been addressed.
Applied to net-next.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-10-03 19:27 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-03 8:21 [PATCH v3 0/2] net: phy: at803x: add ar9331 support Oleksij Rempel
2019-10-03 8:21 ` [PATCH v3 1/2] " Oleksij Rempel
2019-10-03 8:21 ` [PATCH v3 2/2] net: phy: at803x: remove probe and struct at803x_priv Oleksij Rempel
2019-10-03 19:27 ` [PATCH v3 0/2] net: phy: at803x: add ar9331 support David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).