* [PATCH net-next V2 0/3] Add option to enable PHY WOL with PMT enabled
@ 2021-06-29 3:08 ` Ling Pei Lee
0 siblings, 0 replies; 10+ messages in thread
From: Ling Pei Lee @ 2021-06-29 3:08 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu, davem,
Jakub Kicinski, Maxime Coquelin, Russell King, weifeng.voon,
boon.leong.ong, vee.khee.wong, vee.khee.wong, tee.min.tan,
michael.wei.hong.sit, netdev, linux-stm32, linux-arm-kernel,
linux-kernel
Cc: pei.lee.ling
This patchset main objective is to provide an option to enable PHY WoL even the PMT is enabled by default in the HW features.
The current stmmac driver WOL implementation will enable MAC WOL if MAC HW PMT feature is on. Else, the driver will check for PHY WOL support.
Intel EHL mgbe are designed to wake up through PHY WOL although the HW PMT is enabled.Hence, introduced use_phy_wol platform data to provide this PHY WOL option. Set use_phy_wol will disable the plat->pmt which currently used to determine the system to wake up by MAC WOL or PHY WOL.
This WOL patchset includes of setting the device power state to D3hot.
This is because the EHL PSE will need to PSE mgbe to be in D3 state in order for the PSE to goes into suspend mode.
Change Log:
V2: Drop Patch #3 net: stmmac: Reconfigure the PHY WOL settings in stmmac_resume().
Ling Pei Lee (2):
net: stmmac: option to enable PHY WOL with PMT enabled
stmmac: intel: Enable PHY WOL option in EHL
Voon Weifeng (1):
stmmac: intel: set PCI_D3hot in suspend
drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c | 2 ++
drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 3 ++-
include/linux/stmmac.h | 1 +
3 files changed, 5 insertions(+), 1 deletion(-)
--
2.25.1
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH net-next V2 0/3] Add option to enable PHY WOL with PMT enabled
@ 2021-06-29 3:08 ` Ling Pei Lee
0 siblings, 0 replies; 10+ messages in thread
From: Ling Pei Lee @ 2021-06-29 3:08 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu, davem,
Jakub Kicinski, Maxime Coquelin, Russell King, weifeng.voon,
boon.leong.ong, vee.khee.wong, vee.khee.wong, tee.min.tan,
michael.wei.hong.sit, netdev, linux-stm32, linux-arm-kernel,
linux-kernel
Cc: pei.lee.ling
This patchset main objective is to provide an option to enable PHY WoL even the PMT is enabled by default in the HW features.
The current stmmac driver WOL implementation will enable MAC WOL if MAC HW PMT feature is on. Else, the driver will check for PHY WOL support.
Intel EHL mgbe are designed to wake up through PHY WOL although the HW PMT is enabled.Hence, introduced use_phy_wol platform data to provide this PHY WOL option. Set use_phy_wol will disable the plat->pmt which currently used to determine the system to wake up by MAC WOL or PHY WOL.
This WOL patchset includes of setting the device power state to D3hot.
This is because the EHL PSE will need to PSE mgbe to be in D3 state in order for the PSE to goes into suspend mode.
Change Log:
V2: Drop Patch #3 net: stmmac: Reconfigure the PHY WOL settings in stmmac_resume().
Ling Pei Lee (2):
net: stmmac: option to enable PHY WOL with PMT enabled
stmmac: intel: Enable PHY WOL option in EHL
Voon Weifeng (1):
stmmac: intel: set PCI_D3hot in suspend
drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c | 2 ++
drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 3 ++-
include/linux/stmmac.h | 1 +
3 files changed, 5 insertions(+), 1 deletion(-)
--
2.25.1
_______________________________________________
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] 10+ messages in thread
* [PATCH net-next V2 1/3] net: stmmac: option to enable PHY WOL with PMT enabled
2021-06-29 3:08 ` Ling Pei Lee
@ 2021-06-29 3:08 ` Ling Pei Lee
-1 siblings, 0 replies; 10+ messages in thread
From: Ling Pei Lee @ 2021-06-29 3:08 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu, davem,
Jakub Kicinski, Maxime Coquelin, Russell King, weifeng.voon,
boon.leong.ong, vee.khee.wong, vee.khee.wong, tee.min.tan,
michael.wei.hong.sit, netdev, linux-stm32, linux-arm-kernel,
linux-kernel
Cc: pei.lee.ling
The current stmmac driver WOL implementation will enable MAC WOL
if MAC HW PMT feature is on. Else, the driver will check for
PHY WOL support. There is another case where MAC HW PMT is
enabled but the platform still goes for the PHY WOL option.
E.g, Intel platform are designed for PHY WOL but not MAC WOL
although HW MAC PMT features are enabled.
Introduce use_phy_wol platform data to select PHY WOL
instead of depending on HW PMT features. Set use_phy_wol
will disable the plat->pmt which currently used to
determine the system to wake up by MAC WOL or PHY WOL.
Signed-off-by: Ling Pei Lee <pei.lee.ling@intel.com>
---
drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 3 ++-
include/linux/stmmac.h | 1 +
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
index 0a266fa0af7e..a3b79ddcf08e 100644
--- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
+++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
@@ -6533,7 +6533,8 @@ static int stmmac_hw_init(struct stmmac_priv *priv)
* register (if supported).
*/
priv->plat->enh_desc = priv->dma_cap.enh_desc;
- priv->plat->pmt = priv->dma_cap.pmt_remote_wake_up;
+ priv->plat->pmt = priv->dma_cap.pmt_remote_wake_up &&
+ !priv->plat->use_phy_wol;
priv->hw->pmt = priv->plat->pmt;
if (priv->dma_cap.hash_tb_sz) {
priv->hw->multicast_filter_bins =
diff --git a/include/linux/stmmac.h b/include/linux/stmmac.h
index e55a4807e3ea..9496e6c9ee82 100644
--- a/include/linux/stmmac.h
+++ b/include/linux/stmmac.h
@@ -263,5 +263,6 @@ struct plat_stmmacenet_data {
int msi_sfty_ue_vec;
int msi_rx_base_vec;
int msi_tx_base_vec;
+ bool use_phy_wol;
};
#endif
--
2.25.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH net-next V2 1/3] net: stmmac: option to enable PHY WOL with PMT enabled
@ 2021-06-29 3:08 ` Ling Pei Lee
0 siblings, 0 replies; 10+ messages in thread
From: Ling Pei Lee @ 2021-06-29 3:08 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu, davem,
Jakub Kicinski, Maxime Coquelin, Russell King, weifeng.voon,
boon.leong.ong, vee.khee.wong, vee.khee.wong, tee.min.tan,
michael.wei.hong.sit, netdev, linux-stm32, linux-arm-kernel,
linux-kernel
Cc: pei.lee.ling
The current stmmac driver WOL implementation will enable MAC WOL
if MAC HW PMT feature is on. Else, the driver will check for
PHY WOL support. There is another case where MAC HW PMT is
enabled but the platform still goes for the PHY WOL option.
E.g, Intel platform are designed for PHY WOL but not MAC WOL
although HW MAC PMT features are enabled.
Introduce use_phy_wol platform data to select PHY WOL
instead of depending on HW PMT features. Set use_phy_wol
will disable the plat->pmt which currently used to
determine the system to wake up by MAC WOL or PHY WOL.
Signed-off-by: Ling Pei Lee <pei.lee.ling@intel.com>
---
drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 3 ++-
include/linux/stmmac.h | 1 +
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
index 0a266fa0af7e..a3b79ddcf08e 100644
--- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
+++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
@@ -6533,7 +6533,8 @@ static int stmmac_hw_init(struct stmmac_priv *priv)
* register (if supported).
*/
priv->plat->enh_desc = priv->dma_cap.enh_desc;
- priv->plat->pmt = priv->dma_cap.pmt_remote_wake_up;
+ priv->plat->pmt = priv->dma_cap.pmt_remote_wake_up &&
+ !priv->plat->use_phy_wol;
priv->hw->pmt = priv->plat->pmt;
if (priv->dma_cap.hash_tb_sz) {
priv->hw->multicast_filter_bins =
diff --git a/include/linux/stmmac.h b/include/linux/stmmac.h
index e55a4807e3ea..9496e6c9ee82 100644
--- a/include/linux/stmmac.h
+++ b/include/linux/stmmac.h
@@ -263,5 +263,6 @@ struct plat_stmmacenet_data {
int msi_sfty_ue_vec;
int msi_rx_base_vec;
int msi_tx_base_vec;
+ bool use_phy_wol;
};
#endif
--
2.25.1
_______________________________________________
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] 10+ messages in thread
* [PATCH net-next V2 2/3] stmmac: intel: Enable PHY WOL option in EHL
2021-06-29 3:08 ` Ling Pei Lee
@ 2021-06-29 3:08 ` Ling Pei Lee
-1 siblings, 0 replies; 10+ messages in thread
From: Ling Pei Lee @ 2021-06-29 3:08 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu, davem,
Jakub Kicinski, Maxime Coquelin, Russell King, weifeng.voon,
boon.leong.ong, vee.khee.wong, vee.khee.wong, tee.min.tan,
michael.wei.hong.sit, netdev, linux-stm32, linux-arm-kernel,
linux-kernel
Cc: pei.lee.ling
Enable PHY Wake On LAN in Intel EHL Intel platform.
PHY Wake on LAN option is enabled due to
Intel EHL Intel platform is designed for
PHY Wake On LAN but not MAC Wake On LAN.
Signed-off-by: Ling Pei Lee <pei.lee.ling@intel.com>
---
drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c
index 2ecf93c84b9d..73be34a10a4c 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c
@@ -567,6 +567,7 @@ static int ehl_common_data(struct pci_dev *pdev,
plat->rx_queues_to_use = 8;
plat->tx_queues_to_use = 8;
plat->clk_ptp_rate = 200000000;
+ plat->use_phy_wol = 1;
plat->safety_feat_cfg->tsoee = 1;
plat->safety_feat_cfg->mrxpee = 1;
--
2.25.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH net-next V2 2/3] stmmac: intel: Enable PHY WOL option in EHL
@ 2021-06-29 3:08 ` Ling Pei Lee
0 siblings, 0 replies; 10+ messages in thread
From: Ling Pei Lee @ 2021-06-29 3:08 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu, davem,
Jakub Kicinski, Maxime Coquelin, Russell King, weifeng.voon,
boon.leong.ong, vee.khee.wong, vee.khee.wong, tee.min.tan,
michael.wei.hong.sit, netdev, linux-stm32, linux-arm-kernel,
linux-kernel
Cc: pei.lee.ling
Enable PHY Wake On LAN in Intel EHL Intel platform.
PHY Wake on LAN option is enabled due to
Intel EHL Intel platform is designed for
PHY Wake On LAN but not MAC Wake On LAN.
Signed-off-by: Ling Pei Lee <pei.lee.ling@intel.com>
---
drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c
index 2ecf93c84b9d..73be34a10a4c 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c
@@ -567,6 +567,7 @@ static int ehl_common_data(struct pci_dev *pdev,
plat->rx_queues_to_use = 8;
plat->tx_queues_to_use = 8;
plat->clk_ptp_rate = 200000000;
+ plat->use_phy_wol = 1;
plat->safety_feat_cfg->tsoee = 1;
plat->safety_feat_cfg->mrxpee = 1;
--
2.25.1
_______________________________________________
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] 10+ messages in thread
* [PATCH net-next V2 3/3] stmmac: intel: set PCI_D3hot in suspend
2021-06-29 3:08 ` Ling Pei Lee
@ 2021-06-29 3:08 ` Ling Pei Lee
-1 siblings, 0 replies; 10+ messages in thread
From: Ling Pei Lee @ 2021-06-29 3:08 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu, davem,
Jakub Kicinski, Maxime Coquelin, Russell King, weifeng.voon,
boon.leong.ong, vee.khee.wong, vee.khee.wong, tee.min.tan,
michael.wei.hong.sit, netdev, linux-stm32, linux-arm-kernel,
linux-kernel
Cc: pei.lee.ling
From: Voon Weifeng <weifeng.voon@intel.com>
During suspend, set the Intel mgbe to D3hot state
to save power consumption.
Signed-off-by: Voon Weifeng <weifeng.voon@intel.com>
Signed-off-by: Ling Pei Lee <pei.lee.ling@intel.com>
---
drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c
index 73be34a10a4c..69a725b661c2 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c
@@ -1088,6 +1088,7 @@ static int __maybe_unused intel_eth_pci_suspend(struct device *dev)
return ret;
pci_wake_from_d3(pdev, true);
+ pci_set_power_state(pdev, PCI_D3hot);
return 0;
}
--
2.25.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH net-next V2 3/3] stmmac: intel: set PCI_D3hot in suspend
@ 2021-06-29 3:08 ` Ling Pei Lee
0 siblings, 0 replies; 10+ messages in thread
From: Ling Pei Lee @ 2021-06-29 3:08 UTC (permalink / raw)
To: Giuseppe Cavallaro, Alexandre Torgue, Jose Abreu, davem,
Jakub Kicinski, Maxime Coquelin, Russell King, weifeng.voon,
boon.leong.ong, vee.khee.wong, vee.khee.wong, tee.min.tan,
michael.wei.hong.sit, netdev, linux-stm32, linux-arm-kernel,
linux-kernel
Cc: pei.lee.ling
From: Voon Weifeng <weifeng.voon@intel.com>
During suspend, set the Intel mgbe to D3hot state
to save power consumption.
Signed-off-by: Voon Weifeng <weifeng.voon@intel.com>
Signed-off-by: Ling Pei Lee <pei.lee.ling@intel.com>
---
drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c
index 73be34a10a4c..69a725b661c2 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c
@@ -1088,6 +1088,7 @@ static int __maybe_unused intel_eth_pci_suspend(struct device *dev)
return ret;
pci_wake_from_d3(pdev, true);
+ pci_set_power_state(pdev, PCI_D3hot);
return 0;
}
--
2.25.1
_______________________________________________
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] 10+ messages in thread
* Re: [PATCH net-next V2 0/3] Add option to enable PHY WOL with PMT enabled
2021-06-29 3:08 ` Ling Pei Lee
@ 2021-06-29 18:40 ` patchwork-bot+netdevbpf
-1 siblings, 0 replies; 10+ messages in thread
From: patchwork-bot+netdevbpf @ 2021-06-29 18:40 UTC (permalink / raw)
To: Ling Pei Lee
Cc: peppe.cavallaro, alexandre.torgue, joabreu, davem, kuba,
mcoquelin.stm32, linux, weifeng.voon, boon.leong.ong,
vee.khee.wong, vee.khee.wong, tee.min.tan, michael.wei.hong.sit,
netdev, linux-stm32, linux-arm-kernel, linux-kernel
Hello:
This series was applied to netdev/net-next.git (refs/heads/master):
On Tue, 29 Jun 2021 11:08:56 +0800 you wrote:
> This patchset main objective is to provide an option to enable PHY WoL even the PMT is enabled by default in the HW features.
>
> The current stmmac driver WOL implementation will enable MAC WOL if MAC HW PMT feature is on. Else, the driver will check for PHY WOL support.
> Intel EHL mgbe are designed to wake up through PHY WOL although the HW PMT is enabled.Hence, introduced use_phy_wol platform data to provide this PHY WOL option. Set use_phy_wol will disable the plat->pmt which currently used to determine the system to wake up by MAC WOL or PHY WOL.
>
> This WOL patchset includes of setting the device power state to D3hot.
> This is because the EHL PSE will need to PSE mgbe to be in D3 state in order for the PSE to goes into suspend mode.
>
> [...]
Here is the summary with links:
- [net-next,V2,1/3] net: stmmac: option to enable PHY WOL with PMT enabled
https://git.kernel.org/netdev/net-next/c/5a9b876e9d76
- [net-next,V2,2/3] stmmac: intel: Enable PHY WOL option in EHL
https://git.kernel.org/netdev/net-next/c/945beb755633
- [net-next,V2,3/3] stmmac: intel: set PCI_D3hot in suspend
https://git.kernel.org/netdev/net-next/c/1dd53a61488d
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH net-next V2 0/3] Add option to enable PHY WOL with PMT enabled
@ 2021-06-29 18:40 ` patchwork-bot+netdevbpf
0 siblings, 0 replies; 10+ messages in thread
From: patchwork-bot+netdevbpf @ 2021-06-29 18:40 UTC (permalink / raw)
To: Ling Pei Lee
Cc: peppe.cavallaro, alexandre.torgue, joabreu, davem, kuba,
mcoquelin.stm32, linux, weifeng.voon, boon.leong.ong,
vee.khee.wong, vee.khee.wong, tee.min.tan, michael.wei.hong.sit,
netdev, linux-stm32, linux-arm-kernel, linux-kernel
Hello:
This series was applied to netdev/net-next.git (refs/heads/master):
On Tue, 29 Jun 2021 11:08:56 +0800 you wrote:
> This patchset main objective is to provide an option to enable PHY WoL even the PMT is enabled by default in the HW features.
>
> The current stmmac driver WOL implementation will enable MAC WOL if MAC HW PMT feature is on. Else, the driver will check for PHY WOL support.
> Intel EHL mgbe are designed to wake up through PHY WOL although the HW PMT is enabled.Hence, introduced use_phy_wol platform data to provide this PHY WOL option. Set use_phy_wol will disable the plat->pmt which currently used to determine the system to wake up by MAC WOL or PHY WOL.
>
> This WOL patchset includes of setting the device power state to D3hot.
> This is because the EHL PSE will need to PSE mgbe to be in D3 state in order for the PSE to goes into suspend mode.
>
> [...]
Here is the summary with links:
- [net-next,V2,1/3] net: stmmac: option to enable PHY WOL with PMT enabled
https://git.kernel.org/netdev/net-next/c/5a9b876e9d76
- [net-next,V2,2/3] stmmac: intel: Enable PHY WOL option in EHL
https://git.kernel.org/netdev/net-next/c/945beb755633
- [net-next,V2,3/3] stmmac: intel: set PCI_D3hot in suspend
https://git.kernel.org/netdev/net-next/c/1dd53a61488d
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
_______________________________________________
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] 10+ messages in thread
end of thread, other threads:[~2021-06-29 18:41 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-29 3:08 [PATCH net-next V2 0/3] Add option to enable PHY WOL with PMT enabled Ling Pei Lee
2021-06-29 3:08 ` Ling Pei Lee
2021-06-29 3:08 ` [PATCH net-next V2 1/3] net: stmmac: " Ling Pei Lee
2021-06-29 3:08 ` Ling Pei Lee
2021-06-29 3:08 ` [PATCH net-next V2 2/3] stmmac: intel: Enable PHY WOL option in EHL Ling Pei Lee
2021-06-29 3:08 ` Ling Pei Lee
2021-06-29 3:08 ` [PATCH net-next V2 3/3] stmmac: intel: set PCI_D3hot in suspend Ling Pei Lee
2021-06-29 3:08 ` Ling Pei Lee
2021-06-29 18:40 ` [PATCH net-next V2 0/3] Add option to enable PHY WOL with PMT enabled patchwork-bot+netdevbpf
2021-06-29 18:40 ` patchwork-bot+netdevbpf
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.