* [PATCH net-next 0/5] dwmac-sun8i cleanup and shutdown hook
@ 2021-01-03 11:25 ` Samuel Holland
0 siblings, 0 replies; 22+ messages in thread
From: Samuel Holland @ 2021-01-03 11:25 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Chen-Yu Tsai,
Jernej Skrabec, Corentin Labbe
Cc: Ondrej Jirman, netdev, linux-arm-kernel, linux-kernel,
linux-sunxi, Samuel Holland
These patches clean up some things I noticed while fixing suspend/resume
behavior. The first four are minor code improvements. The last one adds
a shutdown hook to minimize power consumption on boards without a PMIC.
Samuel Holland (5):
net: stmmac: dwmac-sun8i: Return void from PHY unpower
net: stmmac: dwmac-sun8i: Remove unnecessary PHY power check
net: stmmac: dwmac-sun8i: Use reset_control_reset
net: stmmac: dwmac-sun8i: Minor probe function cleanup
net: stmmac: dwmac-sun8i: Add a shutdown callback
.../net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 31 ++++++++++++-------
1 file changed, 19 insertions(+), 12 deletions(-)
--
2.26.2
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH net-next 0/5] dwmac-sun8i cleanup and shutdown hook
@ 2021-01-03 11:25 ` Samuel Holland
0 siblings, 0 replies; 22+ messages in thread
From: Samuel Holland @ 2021-01-03 11:25 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Chen-Yu Tsai,
Jernej Skrabec, Corentin Labbe
Cc: Ondrej Jirman, Samuel Holland, netdev, linux-kernel, linux-sunxi,
linux-arm-kernel
These patches clean up some things I noticed while fixing suspend/resume
behavior. The first four are minor code improvements. The last one adds
a shutdown hook to minimize power consumption on boards without a PMIC.
Samuel Holland (5):
net: stmmac: dwmac-sun8i: Return void from PHY unpower
net: stmmac: dwmac-sun8i: Remove unnecessary PHY power check
net: stmmac: dwmac-sun8i: Use reset_control_reset
net: stmmac: dwmac-sun8i: Minor probe function cleanup
net: stmmac: dwmac-sun8i: Add a shutdown callback
.../net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 31 ++++++++++++-------
1 file changed, 19 insertions(+), 12 deletions(-)
--
2.26.2
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH net-next 1/5] net: stmmac: dwmac-sun8i: Return void from PHY unpower
2021-01-03 11:25 ` Samuel Holland
@ 2021-01-03 11:25 ` Samuel Holland
-1 siblings, 0 replies; 22+ messages in thread
From: Samuel Holland @ 2021-01-03 11:25 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Chen-Yu Tsai,
Jernej Skrabec, Corentin Labbe
Cc: Ondrej Jirman, netdev, linux-arm-kernel, linux-kernel,
linux-sunxi, Samuel Holland
This is a deinitialization function that always returned zero, and that
return value was always ignored. Have it return void instead.
Signed-off-by: Samuel Holland <samuel@sholland.org>
---
drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
index a5e0eff4a387..8e505019adf8 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
@@ -820,15 +820,14 @@ static int sun8i_dwmac_power_internal_phy(struct stmmac_priv *priv)
return 0;
}
-static int sun8i_dwmac_unpower_internal_phy(struct sunxi_priv_data *gmac)
+static void sun8i_dwmac_unpower_internal_phy(struct sunxi_priv_data *gmac)
{
if (!gmac->internal_phy_powered)
- return 0;
+ return;
clk_disable_unprepare(gmac->ephy_clk);
reset_control_assert(gmac->rst_ephy);
gmac->internal_phy_powered = false;
- return 0;
}
/* MDIO multiplexing switch function
--
2.26.2
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH net-next 1/5] net: stmmac: dwmac-sun8i: Return void from PHY unpower
@ 2021-01-03 11:25 ` Samuel Holland
0 siblings, 0 replies; 22+ messages in thread
From: Samuel Holland @ 2021-01-03 11:25 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Chen-Yu Tsai,
Jernej Skrabec, Corentin Labbe
Cc: Ondrej Jirman, Samuel Holland, netdev, linux-kernel, linux-sunxi,
linux-arm-kernel
This is a deinitialization function that always returned zero, and that
return value was always ignored. Have it return void instead.
Signed-off-by: Samuel Holland <samuel@sholland.org>
---
drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
index a5e0eff4a387..8e505019adf8 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
@@ -820,15 +820,14 @@ static int sun8i_dwmac_power_internal_phy(struct stmmac_priv *priv)
return 0;
}
-static int sun8i_dwmac_unpower_internal_phy(struct sunxi_priv_data *gmac)
+static void sun8i_dwmac_unpower_internal_phy(struct sunxi_priv_data *gmac)
{
if (!gmac->internal_phy_powered)
- return 0;
+ return;
clk_disable_unprepare(gmac->ephy_clk);
reset_control_assert(gmac->rst_ephy);
gmac->internal_phy_powered = false;
- return 0;
}
/* MDIO multiplexing switch function
--
2.26.2
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH net-next 2/5] net: stmmac: dwmac-sun8i: Remove unnecessary PHY power check
2021-01-03 11:25 ` Samuel Holland
@ 2021-01-03 11:25 ` Samuel Holland
-1 siblings, 0 replies; 22+ messages in thread
From: Samuel Holland @ 2021-01-03 11:25 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Chen-Yu Tsai,
Jernej Skrabec, Corentin Labbe
Cc: Ondrej Jirman, netdev, linux-arm-kernel, linux-kernel,
linux-sunxi, Samuel Holland
sun8i_dwmac_unpower_internal_phy already checks if the PHY is powered,
so there is no need to do it again here.
Signed-off-by: Samuel Holland <samuel@sholland.org>
---
drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
index 8e505019adf8..3c3d0b99d3e8 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
@@ -1018,10 +1018,8 @@ static void sun8i_dwmac_exit(struct platform_device *pdev, void *priv)
{
struct sunxi_priv_data *gmac = priv;
- if (gmac->variant->soc_has_internal_phy) {
- if (gmac->internal_phy_powered)
- sun8i_dwmac_unpower_internal_phy(gmac);
- }
+ if (gmac->variant->soc_has_internal_phy)
+ sun8i_dwmac_unpower_internal_phy(gmac);
clk_disable_unprepare(gmac->tx_clk);
--
2.26.2
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH net-next 2/5] net: stmmac: dwmac-sun8i: Remove unnecessary PHY power check
@ 2021-01-03 11:25 ` Samuel Holland
0 siblings, 0 replies; 22+ messages in thread
From: Samuel Holland @ 2021-01-03 11:25 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Chen-Yu Tsai,
Jernej Skrabec, Corentin Labbe
Cc: Ondrej Jirman, Samuel Holland, netdev, linux-kernel, linux-sunxi,
linux-arm-kernel
sun8i_dwmac_unpower_internal_phy already checks if the PHY is powered,
so there is no need to do it again here.
Signed-off-by: Samuel Holland <samuel@sholland.org>
---
drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
index 8e505019adf8..3c3d0b99d3e8 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
@@ -1018,10 +1018,8 @@ static void sun8i_dwmac_exit(struct platform_device *pdev, void *priv)
{
struct sunxi_priv_data *gmac = priv;
- if (gmac->variant->soc_has_internal_phy) {
- if (gmac->internal_phy_powered)
- sun8i_dwmac_unpower_internal_phy(gmac);
- }
+ if (gmac->variant->soc_has_internal_phy)
+ sun8i_dwmac_unpower_internal_phy(gmac);
clk_disable_unprepare(gmac->tx_clk);
--
2.26.2
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH net-next 3/5] net: stmmac: dwmac-sun8i: Use reset_control_reset
2021-01-03 11:25 ` Samuel Holland
@ 2021-01-03 11:25 ` Samuel Holland
-1 siblings, 0 replies; 22+ messages in thread
From: Samuel Holland @ 2021-01-03 11:25 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Chen-Yu Tsai,
Jernej Skrabec, Corentin Labbe
Cc: Ondrej Jirman, netdev, linux-arm-kernel, linux-kernel,
linux-sunxi, Samuel Holland
Use the appropriate function instead of reimplementing it,
and update the error message to match the code.
Signed-off-by: Samuel Holland <samuel@sholland.org>
---
drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
index 3c3d0b99d3e8..0e8d88417251 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
@@ -806,11 +806,9 @@ static int sun8i_dwmac_power_internal_phy(struct stmmac_priv *priv)
/* Make sure the EPHY is properly reseted, as U-Boot may leave
* it at deasserted state, and thus it may fail to reset EMAC.
*/
- reset_control_assert(gmac->rst_ephy);
-
- ret = reset_control_deassert(gmac->rst_ephy);
+ ret = reset_control_reset(gmac->rst_ephy);
if (ret) {
- dev_err(priv->device, "Cannot deassert internal phy\n");
+ dev_err(priv->device, "Cannot reset internal PHY\n");
clk_disable_unprepare(gmac->ephy_clk);
return ret;
}
--
2.26.2
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH net-next 3/5] net: stmmac: dwmac-sun8i: Use reset_control_reset
@ 2021-01-03 11:25 ` Samuel Holland
0 siblings, 0 replies; 22+ messages in thread
From: Samuel Holland @ 2021-01-03 11:25 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Chen-Yu Tsai,
Jernej Skrabec, Corentin Labbe
Cc: Ondrej Jirman, Samuel Holland, netdev, linux-kernel, linux-sunxi,
linux-arm-kernel
Use the appropriate function instead of reimplementing it,
and update the error message to match the code.
Signed-off-by: Samuel Holland <samuel@sholland.org>
---
drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
index 3c3d0b99d3e8..0e8d88417251 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
@@ -806,11 +806,9 @@ static int sun8i_dwmac_power_internal_phy(struct stmmac_priv *priv)
/* Make sure the EPHY is properly reseted, as U-Boot may leave
* it at deasserted state, and thus it may fail to reset EMAC.
*/
- reset_control_assert(gmac->rst_ephy);
-
- ret = reset_control_deassert(gmac->rst_ephy);
+ ret = reset_control_reset(gmac->rst_ephy);
if (ret) {
- dev_err(priv->device, "Cannot deassert internal phy\n");
+ dev_err(priv->device, "Cannot reset internal PHY\n");
clk_disable_unprepare(gmac->ephy_clk);
return ret;
}
--
2.26.2
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH net-next 4/5] net: stmmac: dwmac-sun8i: Minor probe function cleanup
2021-01-03 11:25 ` Samuel Holland
@ 2021-01-03 11:25 ` Samuel Holland
-1 siblings, 0 replies; 22+ messages in thread
From: Samuel Holland @ 2021-01-03 11:25 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Chen-Yu Tsai,
Jernej Skrabec, Corentin Labbe
Cc: Ondrej Jirman, netdev, linux-arm-kernel, linux-kernel,
linux-sunxi, Samuel Holland
Adjust the spacing and use an explicit "return 0" in the success path
to make the function easier to parse.
Signed-off-by: Samuel Holland <samuel@sholland.org>
---
drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
index 0e8d88417251..4638d4203af5 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
@@ -1227,6 +1227,7 @@ static int sun8i_dwmac_probe(struct platform_device *pdev)
ndev = dev_get_drvdata(&pdev->dev);
priv = netdev_priv(ndev);
+
/* The mux must be registered after parent MDIO
* so after stmmac_dvr_probe()
*/
@@ -1245,7 +1246,8 @@ static int sun8i_dwmac_probe(struct platform_device *pdev)
goto dwmac_remove;
}
- return ret;
+ return 0;
+
dwmac_mux:
reset_control_put(gmac->rst_ephy);
clk_put(gmac->ephy_clk);
--
2.26.2
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH net-next 4/5] net: stmmac: dwmac-sun8i: Minor probe function cleanup
@ 2021-01-03 11:25 ` Samuel Holland
0 siblings, 0 replies; 22+ messages in thread
From: Samuel Holland @ 2021-01-03 11:25 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Chen-Yu Tsai,
Jernej Skrabec, Corentin Labbe
Cc: Ondrej Jirman, Samuel Holland, netdev, linux-kernel, linux-sunxi,
linux-arm-kernel
Adjust the spacing and use an explicit "return 0" in the success path
to make the function easier to parse.
Signed-off-by: Samuel Holland <samuel@sholland.org>
---
drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
index 0e8d88417251..4638d4203af5 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
@@ -1227,6 +1227,7 @@ static int sun8i_dwmac_probe(struct platform_device *pdev)
ndev = dev_get_drvdata(&pdev->dev);
priv = netdev_priv(ndev);
+
/* The mux must be registered after parent MDIO
* so after stmmac_dvr_probe()
*/
@@ -1245,7 +1246,8 @@ static int sun8i_dwmac_probe(struct platform_device *pdev)
goto dwmac_remove;
}
- return ret;
+ return 0;
+
dwmac_mux:
reset_control_put(gmac->rst_ephy);
clk_put(gmac->ephy_clk);
--
2.26.2
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH net-next 5/5] net: stmmac: dwmac-sun8i: Add a shutdown callback
2021-01-03 11:25 ` Samuel Holland
@ 2021-01-03 11:25 ` Samuel Holland
-1 siblings, 0 replies; 22+ messages in thread
From: Samuel Holland @ 2021-01-03 11:25 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Chen-Yu Tsai,
Jernej Skrabec, Corentin Labbe
Cc: Ondrej Jirman, netdev, linux-arm-kernel, linux-kernel,
linux-sunxi, Samuel Holland
The Ethernet MAC and PHY are usually major consumers of power on boards
which may not be able to fully power off (that have no PMIC). Powering
down the MAC and internal PHY saves power while these boards are "off".
Signed-off-by: Samuel Holland <samuel@sholland.org>
---
drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
index 4638d4203af5..926e8d5e8963 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
@@ -1282,6 +1282,15 @@ static int sun8i_dwmac_remove(struct platform_device *pdev)
return 0;
}
+static void sun8i_dwmac_shutdown(struct platform_device *pdev)
+{
+ struct net_device *ndev = platform_get_drvdata(pdev);
+ struct stmmac_priv *priv = netdev_priv(ndev);
+ struct sunxi_priv_data *gmac = priv->plat->bsp_priv;
+
+ sun8i_dwmac_exit(pdev, gmac);
+}
+
static const struct of_device_id sun8i_dwmac_match[] = {
{ .compatible = "allwinner,sun8i-h3-emac",
.data = &emac_variant_h3 },
@@ -1302,6 +1311,7 @@ MODULE_DEVICE_TABLE(of, sun8i_dwmac_match);
static struct platform_driver sun8i_dwmac_driver = {
.probe = sun8i_dwmac_probe,
.remove = sun8i_dwmac_remove,
+ .shutdown = sun8i_dwmac_shutdown,
.driver = {
.name = "dwmac-sun8i",
.pm = &stmmac_pltfr_pm_ops,
--
2.26.2
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH net-next 5/5] net: stmmac: dwmac-sun8i: Add a shutdown callback
@ 2021-01-03 11:25 ` Samuel Holland
0 siblings, 0 replies; 22+ messages in thread
From: Samuel Holland @ 2021-01-03 11:25 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Chen-Yu Tsai,
Jernej Skrabec, Corentin Labbe
Cc: Ondrej Jirman, Samuel Holland, netdev, linux-kernel, linux-sunxi,
linux-arm-kernel
The Ethernet MAC and PHY are usually major consumers of power on boards
which may not be able to fully power off (that have no PMIC). Powering
down the MAC and internal PHY saves power while these boards are "off".
Signed-off-by: Samuel Holland <samuel@sholland.org>
---
drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
index 4638d4203af5..926e8d5e8963 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
@@ -1282,6 +1282,15 @@ static int sun8i_dwmac_remove(struct platform_device *pdev)
return 0;
}
+static void sun8i_dwmac_shutdown(struct platform_device *pdev)
+{
+ struct net_device *ndev = platform_get_drvdata(pdev);
+ struct stmmac_priv *priv = netdev_priv(ndev);
+ struct sunxi_priv_data *gmac = priv->plat->bsp_priv;
+
+ sun8i_dwmac_exit(pdev, gmac);
+}
+
static const struct of_device_id sun8i_dwmac_match[] = {
{ .compatible = "allwinner,sun8i-h3-emac",
.data = &emac_variant_h3 },
@@ -1302,6 +1311,7 @@ MODULE_DEVICE_TABLE(of, sun8i_dwmac_match);
static struct platform_driver sun8i_dwmac_driver = {
.probe = sun8i_dwmac_probe,
.remove = sun8i_dwmac_remove,
+ .shutdown = sun8i_dwmac_shutdown,
.driver = {
.name = "dwmac-sun8i",
.pm = &stmmac_pltfr_pm_ops,
--
2.26.2
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH net-next 1/5] net: stmmac: dwmac-sun8i: Return void from PHY unpower
2021-01-03 11:25 ` Samuel Holland
@ 2021-01-06 10:00 ` Chen-Yu Tsai
-1 siblings, 0 replies; 22+ messages in thread
From: Chen-Yu Tsai @ 2021-01-06 10:00 UTC (permalink / raw)
To: Samuel Holland
Cc: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Jernej Skrabec,
Corentin Labbe, Ondrej Jirman, netdev, linux-arm-kernel,
linux-kernel, linux-sunxi
On Sun, Jan 3, 2021 at 7:25 PM Samuel Holland <samuel@sholland.org> wrote:
>
> This is a deinitialization function that always returned zero, and that
> return value was always ignored. Have it return void instead.
>
> Signed-off-by: Samuel Holland <samuel@sholland.org>
Reviewed-by: Chen-Yu Tsai <wens@csie.org>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH net-next 1/5] net: stmmac: dwmac-sun8i: Return void from PHY unpower
@ 2021-01-06 10:00 ` Chen-Yu Tsai
0 siblings, 0 replies; 22+ messages in thread
From: Chen-Yu Tsai @ 2021-01-06 10:00 UTC (permalink / raw)
To: Samuel Holland
Cc: Ondrej Jirman, Jernej Skrabec, Alexandre Torgue, netdev,
linux-kernel, Maxime Ripard, linux-sunxi, Jose Abreu,
Corentin Labbe, Jakub Kicinski, Giuseppe Cavallaro,
David S. Miller, linux-arm-kernel
On Sun, Jan 3, 2021 at 7:25 PM Samuel Holland <samuel@sholland.org> wrote:
>
> This is a deinitialization function that always returned zero, and that
> return value was always ignored. Have it return void instead.
>
> Signed-off-by: Samuel Holland <samuel@sholland.org>
Reviewed-by: Chen-Yu Tsai <wens@csie.org>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH net-next 2/5] net: stmmac: dwmac-sun8i: Remove unnecessary PHY power check
2021-01-03 11:25 ` Samuel Holland
@ 2021-01-06 10:00 ` Chen-Yu Tsai
-1 siblings, 0 replies; 22+ messages in thread
From: Chen-Yu Tsai @ 2021-01-06 10:00 UTC (permalink / raw)
To: Samuel Holland
Cc: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Jernej Skrabec,
Corentin Labbe, Ondrej Jirman, netdev, linux-arm-kernel,
linux-kernel, linux-sunxi
On Sun, Jan 3, 2021 at 7:25 PM Samuel Holland <samuel@sholland.org> wrote:
>
> sun8i_dwmac_unpower_internal_phy already checks if the PHY is powered,
> so there is no need to do it again here.
>
> Signed-off-by: Samuel Holland <samuel@sholland.org>
Reviewed-by: Chen-Yu Tsai <wens@csie.org>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH net-next 2/5] net: stmmac: dwmac-sun8i: Remove unnecessary PHY power check
@ 2021-01-06 10:00 ` Chen-Yu Tsai
0 siblings, 0 replies; 22+ messages in thread
From: Chen-Yu Tsai @ 2021-01-06 10:00 UTC (permalink / raw)
To: Samuel Holland
Cc: Ondrej Jirman, Jernej Skrabec, Alexandre Torgue, netdev,
linux-kernel, Maxime Ripard, linux-sunxi, Jose Abreu,
Corentin Labbe, Jakub Kicinski, Giuseppe Cavallaro,
David S. Miller, linux-arm-kernel
On Sun, Jan 3, 2021 at 7:25 PM Samuel Holland <samuel@sholland.org> wrote:
>
> sun8i_dwmac_unpower_internal_phy already checks if the PHY is powered,
> so there is no need to do it again here.
>
> Signed-off-by: Samuel Holland <samuel@sholland.org>
Reviewed-by: Chen-Yu Tsai <wens@csie.org>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH net-next 3/5] net: stmmac: dwmac-sun8i: Use reset_control_reset
2021-01-03 11:25 ` Samuel Holland
@ 2021-01-06 10:00 ` Chen-Yu Tsai
-1 siblings, 0 replies; 22+ messages in thread
From: Chen-Yu Tsai @ 2021-01-06 10:00 UTC (permalink / raw)
To: Samuel Holland
Cc: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Jernej Skrabec,
Corentin Labbe, Ondrej Jirman, netdev, linux-arm-kernel,
linux-kernel, linux-sunxi
On Sun, Jan 3, 2021 at 7:25 PM Samuel Holland <samuel@sholland.org> wrote:
>
> Use the appropriate function instead of reimplementing it,
> and update the error message to match the code.
>
> Signed-off-by: Samuel Holland <samuel@sholland.org>
Reviewed-by: Chen-Yu Tsai <wens@csie.org>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH net-next 3/5] net: stmmac: dwmac-sun8i: Use reset_control_reset
@ 2021-01-06 10:00 ` Chen-Yu Tsai
0 siblings, 0 replies; 22+ messages in thread
From: Chen-Yu Tsai @ 2021-01-06 10:00 UTC (permalink / raw)
To: Samuel Holland
Cc: Ondrej Jirman, Jernej Skrabec, Alexandre Torgue, netdev,
linux-kernel, Maxime Ripard, linux-sunxi, Jose Abreu,
Corentin Labbe, Jakub Kicinski, Giuseppe Cavallaro,
David S. Miller, linux-arm-kernel
On Sun, Jan 3, 2021 at 7:25 PM Samuel Holland <samuel@sholland.org> wrote:
>
> Use the appropriate function instead of reimplementing it,
> and update the error message to match the code.
>
> Signed-off-by: Samuel Holland <samuel@sholland.org>
Reviewed-by: Chen-Yu Tsai <wens@csie.org>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH net-next 4/5] net: stmmac: dwmac-sun8i: Minor probe function cleanup
2021-01-03 11:25 ` Samuel Holland
@ 2021-01-06 10:01 ` Chen-Yu Tsai
-1 siblings, 0 replies; 22+ messages in thread
From: Chen-Yu Tsai @ 2021-01-06 10:01 UTC (permalink / raw)
To: Samuel Holland
Cc: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Jernej Skrabec,
Corentin Labbe, Ondrej Jirman, netdev, linux-arm-kernel,
linux-kernel, linux-sunxi
On Sun, Jan 3, 2021 at 7:25 PM Samuel Holland <samuel@sholland.org> wrote:
>
> Adjust the spacing and use an explicit "return 0" in the success path
> to make the function easier to parse.
>
> Signed-off-by: Samuel Holland <samuel@sholland.org>
Reviewed-by: Chen-Yu Tsai <wens@csie.org>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH net-next 4/5] net: stmmac: dwmac-sun8i: Minor probe function cleanup
@ 2021-01-06 10:01 ` Chen-Yu Tsai
0 siblings, 0 replies; 22+ messages in thread
From: Chen-Yu Tsai @ 2021-01-06 10:01 UTC (permalink / raw)
To: Samuel Holland
Cc: Ondrej Jirman, Jernej Skrabec, Alexandre Torgue, netdev,
linux-kernel, Maxime Ripard, linux-sunxi, Jose Abreu,
Corentin Labbe, Jakub Kicinski, Giuseppe Cavallaro,
David S. Miller, linux-arm-kernel
On Sun, Jan 3, 2021 at 7:25 PM Samuel Holland <samuel@sholland.org> wrote:
>
> Adjust the spacing and use an explicit "return 0" in the success path
> to make the function easier to parse.
>
> Signed-off-by: Samuel Holland <samuel@sholland.org>
Reviewed-by: Chen-Yu Tsai <wens@csie.org>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH net-next 5/5] net: stmmac: dwmac-sun8i: Add a shutdown callback
2021-01-03 11:25 ` Samuel Holland
@ 2021-01-06 10:07 ` Chen-Yu Tsai
-1 siblings, 0 replies; 22+ messages in thread
From: Chen-Yu Tsai @ 2021-01-06 10:07 UTC (permalink / raw)
To: Samuel Holland
Cc: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu,
David S. Miller, Jakub Kicinski, Maxime Ripard, Jernej Skrabec,
Corentin Labbe, Ondrej Jirman, netdev, linux-arm-kernel,
linux-kernel, linux-sunxi
On Sun, Jan 3, 2021 at 7:25 PM Samuel Holland <samuel@sholland.org> wrote:
>
> The Ethernet MAC and PHY are usually major consumers of power on boards
> which may not be able to fully power off (that have no PMIC). Powering
> down the MAC and internal PHY saves power while these boards are "off".
>
> Signed-off-by: Samuel Holland <samuel@sholland.org>
Reviewed-by: Chen-Yu Tsai <wens@csie.org>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH net-next 5/5] net: stmmac: dwmac-sun8i: Add a shutdown callback
@ 2021-01-06 10:07 ` Chen-Yu Tsai
0 siblings, 0 replies; 22+ messages in thread
From: Chen-Yu Tsai @ 2021-01-06 10:07 UTC (permalink / raw)
To: Samuel Holland
Cc: Ondrej Jirman, Jernej Skrabec, Alexandre Torgue, netdev,
linux-kernel, Maxime Ripard, linux-sunxi, Jose Abreu,
Corentin Labbe, Jakub Kicinski, Giuseppe Cavallaro,
David S. Miller, linux-arm-kernel
On Sun, Jan 3, 2021 at 7:25 PM Samuel Holland <samuel@sholland.org> wrote:
>
> The Ethernet MAC and PHY are usually major consumers of power on boards
> which may not be able to fully power off (that have no PMIC). Powering
> down the MAC and internal PHY saves power while these boards are "off".
>
> Signed-off-by: Samuel Holland <samuel@sholland.org>
Reviewed-by: Chen-Yu Tsai <wens@csie.org>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2021-01-06 10:09 UTC | newest]
Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-03 11:25 [PATCH net-next 0/5] dwmac-sun8i cleanup and shutdown hook Samuel Holland
2021-01-03 11:25 ` Samuel Holland
2021-01-03 11:25 ` [PATCH net-next 1/5] net: stmmac: dwmac-sun8i: Return void from PHY unpower Samuel Holland
2021-01-03 11:25 ` Samuel Holland
2021-01-06 10:00 ` Chen-Yu Tsai
2021-01-06 10:00 ` Chen-Yu Tsai
2021-01-03 11:25 ` [PATCH net-next 2/5] net: stmmac: dwmac-sun8i: Remove unnecessary PHY power check Samuel Holland
2021-01-03 11:25 ` Samuel Holland
2021-01-06 10:00 ` Chen-Yu Tsai
2021-01-06 10:00 ` Chen-Yu Tsai
2021-01-03 11:25 ` [PATCH net-next 3/5] net: stmmac: dwmac-sun8i: Use reset_control_reset Samuel Holland
2021-01-03 11:25 ` Samuel Holland
2021-01-06 10:00 ` Chen-Yu Tsai
2021-01-06 10:00 ` Chen-Yu Tsai
2021-01-03 11:25 ` [PATCH net-next 4/5] net: stmmac: dwmac-sun8i: Minor probe function cleanup Samuel Holland
2021-01-03 11:25 ` Samuel Holland
2021-01-06 10:01 ` Chen-Yu Tsai
2021-01-06 10:01 ` Chen-Yu Tsai
2021-01-03 11:25 ` [PATCH net-next 5/5] net: stmmac: dwmac-sun8i: Add a shutdown callback Samuel Holland
2021-01-03 11:25 ` Samuel Holland
2021-01-06 10:07 ` Chen-Yu Tsai
2021-01-06 10:07 ` Chen-Yu Tsai
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.