[3/6] dt-bindings: clock: imx93: drop TPM1/3 LPIT1/2 entry

Message ID 20221028095211.2598312-4-peng.fan@oss.nxp.com
State New
Headers
Series clk: imx93: fix and update |

Commit Message

Peng Fan (OSS) Oct. 28, 2022, 9:52 a.m. UTC
  From: Peng Fan <peng.fan@nxp.com>

Per updated Reference Mannual, the TPM[1,3] LPIT[1,2] root clock entries
are reserved, it is because writing the CCM registers does nothing
because the TPM[1,3] and LPIT[1,2] IPs source from bus clk, not from the
TPM[1,3] LPIT[1,2] entries. And because there is no SW entity is using the
entries since adding them, let's drop them.

Reviewed-by: Jacky Bai <ping.bai@nxp.com>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
 include/dt-bindings/clock/imx93-clock.h | 4 ----
 1 file changed, 4 deletions(-)
  

Comments

Krzysztof Kozlowski Oct. 28, 2022, 10:07 p.m. UTC | #1
On 28/10/2022 05:52, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
> 
> Per updated Reference Mannual, the TPM[1,3] LPIT[1,2] root clock entries
> are reserved, it is because writing the CCM registers does nothing
> because the TPM[1,3] and LPIT[1,2] IPs source from bus clk, not from the
> TPM[1,3] LPIT[1,2] entries. And because there is no SW entity is using the
> entries since adding them, let's drop them.
> 
> Reviewed-by: Jacky Bai <ping.bai@nxp.com>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>


Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof
  
Abel Vesa Nov. 4, 2022, 9:08 p.m. UTC | #2
On 22-10-28 17:52:08, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
> 
> Per updated Reference Mannual, the TPM[1,3] LPIT[1,2] root clock entries
> are reserved, it is because writing the CCM registers does nothing
> because the TPM[1,3] and LPIT[1,2] IPs source from bus clk, not from the
> TPM[1,3] LPIT[1,2] entries. And because there is no SW entity is using the
> entries since adding them, let's drop them.
> 
> Reviewed-by: Jacky Bai <ping.bai@nxp.com>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>

Reviewed-by: Abel Vesa <abel.vesa@linaro.org>

> ---
>  include/dt-bindings/clock/imx93-clock.h | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/include/dt-bindings/clock/imx93-clock.h b/include/dt-bindings/clock/imx93-clock.h
> index 19bc32788d81..8e02859d8ce2 100644
> --- a/include/dt-bindings/clock/imx93-clock.h
> +++ b/include/dt-bindings/clock/imx93-clock.h
> @@ -28,13 +28,9 @@
>  #define IMX93_CLK_M33_SYSTICK		19
>  #define IMX93_CLK_FLEXIO1		20
>  #define IMX93_CLK_FLEXIO2		21
> -#define IMX93_CLK_LPIT1			22
> -#define IMX93_CLK_LPIT2			23
>  #define IMX93_CLK_LPTMR1		24
>  #define IMX93_CLK_LPTMR2		25
> -#define IMX93_CLK_TPM1			26
>  #define IMX93_CLK_TPM2			27
> -#define IMX93_CLK_TPM3			28
>  #define IMX93_CLK_TPM4			29
>  #define IMX93_CLK_TPM5			30
>  #define IMX93_CLK_TPM6			31
> -- 
> 2.37.1
>
  

Patch

diff --git a/include/dt-bindings/clock/imx93-clock.h b/include/dt-bindings/clock/imx93-clock.h
index 19bc32788d81..8e02859d8ce2 100644
--- a/include/dt-bindings/clock/imx93-clock.h
+++ b/include/dt-bindings/clock/imx93-clock.h
@@ -28,13 +28,9 @@ 
 #define IMX93_CLK_M33_SYSTICK		19
 #define IMX93_CLK_FLEXIO1		20
 #define IMX93_CLK_FLEXIO2		21
-#define IMX93_CLK_LPIT1			22
-#define IMX93_CLK_LPIT2			23
 #define IMX93_CLK_LPTMR1		24
 #define IMX93_CLK_LPTMR2		25
-#define IMX93_CLK_TPM1			26
 #define IMX93_CLK_TPM2			27
-#define IMX93_CLK_TPM3			28
 #define IMX93_CLK_TPM4			29
 #define IMX93_CLK_TPM5			30
 #define IMX93_CLK_TPM6			31