mbox series

[V2,0/2] phy: xilinx: phy-zynqmp: dynamic clock support

Message ID 20230613140250.3018947-1-piyush.mehta@amd.com
Headers show
Series phy: xilinx: phy-zynqmp: dynamic clock support | expand

Message

Mehta, Piyush June 13, 2023, 2:02 p.m. UTC 
This patch of the series adds the following supports:
- Runtime PM support:
  Added runtime power management support to power saving during the
  suspend and resume calls.

- Dynamic clock support:
  Current zynqmp-phy driver, by default all the clock enabled for all
  the lanes, that consumes power even PHY is active or idle.
  So, the dynamic clock feature will be enabled clock only for active
  PHY in the phy_init and disabled in phy_exit. Clock enabling is not
  required at multiple times. Activation of PHY depends on the peripheral
  DT node status (status = "okay";).

---
Changes in V2:
- Addressed Vinod Koul review comments:
 - Added runtime PM support - "DEFINE_RUNTIME_DEV_PM_OPS" MACRO used.
 - Updated commit message for dynamic clock support.

Link: https://lore.kernel.org/all/MN2PR12MB43339B1089E1CF552152DA2D881A9@MN2PR12MB4333.namprd12.prod.outlook.com/#t
---

Piyush Mehta (2):
  phy: xilinx: add runtime PM support
  phy: xilinx: phy-zynqmp: dynamic clock support for power-save

 drivers/phy/xilinx/phy-zynqmp.c | 90 +++++++++++++++------------------
 1 file changed, 40 insertions(+), 50 deletions(-)

Comments

Vinod Koul July 11, 2023, 7:44 a.m. UTC | #1 
On Tue, 13 Jun 2023 19:32:48 +0530, Piyush Mehta wrote:
> This patch of the series adds the following supports:
> - Runtime PM support:
>   Added runtime power management support to power saving during the
>   suspend and resume calls.
> 
> - Dynamic clock support:
>   Current zynqmp-phy driver, by default all the clock enabled for all
>   the lanes, that consumes power even PHY is active or idle.
>   So, the dynamic clock feature will be enabled clock only for active
>   PHY in the phy_init and disabled in phy_exit. Clock enabling is not
>   required at multiple times. Activation of PHY depends on the peripheral
>   DT node status (status = "okay";).
> 
> [...]

Applied, thanks!

[1/2] phy: xilinx: add runtime PM support
      commit: 1414920a5fd570e67bb7d367eda08f8851b2e378
[2/2] phy: xilinx: phy-zynqmp: dynamic clock support for power-save
      commit: e4f01e75cdcf003f07ca5f4ba61823b687fd941c

Best regards,