diff mbox series

[05/14] power: domain: ti: Fix the order for platform data entries

Message ID 20240529132448.459330-6-j-choudhary@ti.com
State Superseded
Delegated to: Tom Rini
Headers show
Series Add basic U-Boot Support for J722S-EVM | expand

Commit Message

Jayesh Choudhary May 29, 2024, 1:24 p.m. UTC
Add the power domain platform data entries in alphabetical order.

Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
---
 drivers/power/domain/ti-power-domain.c | 36 +++++++++++++-------------
 1 file changed, 18 insertions(+), 18 deletions(-)

Comments

Neha Malcom Francis May 30, 2024, 2:23 a.m. UTC | #1
On 29/05/24 18:54, Jayesh Choudhary wrote:
> Add the power domain platform data entries in alphabetical order.
> 
> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
> ---
>   drivers/power/domain/ti-power-domain.c | 36 +++++++++++++-------------
>   1 file changed, 18 insertions(+), 18 deletions(-)
> 
> diff --git a/drivers/power/domain/ti-power-domain.c b/drivers/power/domain/ti-power-domain.c
> index b059dd3737..362fae86a2 100644
> --- a/drivers/power/domain/ti-power-domain.c
> +++ b/drivers/power/domain/ti-power-domain.c
> @@ -71,6 +71,24 @@ static void lpsc_write(u32 val, struct ti_lpsc *lpsc, u32 reg)
>   }
>   
>   static const struct soc_attr ti_k3_soc_pd_data[] = {
> +#if IS_ENABLED(CONFIG_SOC_K3_AM625)
> +	{
> +		.family = "AM62X",
> +		.data = &am62x_pd_platdata,
> +	},
> +#endif
> +#if IS_ENABLED(CONFIG_SOC_K3_AM62A7)
> +	{
> +		.family = "AM62AX",
> +		.data = &am62ax_pd_platdata,
> +	},
> +#endif
> +#if IS_ENABLED(CONFIG_SOC_K3_AM62P5)
> +	{
> +		.family = "AM62PX",
> +		.data = &am62px_pd_platdata,
> +	},
> +#endif
>   #if IS_ENABLED(CONFIG_SOC_K3_J721E)
>   	{
>   		.family = "J721E",
> @@ -87,29 +105,11 @@ static const struct soc_attr ti_k3_soc_pd_data[] = {
>   		.data = &j721s2_pd_platdata,
>   	},
>   #endif
> -#if IS_ENABLED(CONFIG_SOC_K3_AM625)
> -	{
> -		.family = "AM62X",
> -		.data = &am62x_pd_platdata,
> -	},
> -#endif
> -#if IS_ENABLED(CONFIG_SOC_K3_AM62A7)
> -	{
> -		.family = "AM62AX",
> -		.data = &am62ax_pd_platdata,
> -	},
> -#endif
>   #if IS_ENABLED(CONFIG_SOC_K3_J784S4)
>   	{
>   		.family = "J784S4",
>   		.data = &j784s4_pd_platdata,
>   	},
> -#endif
> -#if IS_ENABLED(CONFIG_SOC_K3_AM62P5)
> -	{
> -		.family = "AM62PX",
> -		.data = &am62px_pd_platdata,
> -	},
>   #endif
>   	{ /* sentinel */ }
>   };

Reviewed-by: Neha Malcom Francis <n-francis@ti.com>
diff mbox series

Patch

diff --git a/drivers/power/domain/ti-power-domain.c b/drivers/power/domain/ti-power-domain.c
index b059dd3737..362fae86a2 100644
--- a/drivers/power/domain/ti-power-domain.c
+++ b/drivers/power/domain/ti-power-domain.c
@@ -71,6 +71,24 @@  static void lpsc_write(u32 val, struct ti_lpsc *lpsc, u32 reg)
 }
 
 static const struct soc_attr ti_k3_soc_pd_data[] = {
+#if IS_ENABLED(CONFIG_SOC_K3_AM625)
+	{
+		.family = "AM62X",
+		.data = &am62x_pd_platdata,
+	},
+#endif
+#if IS_ENABLED(CONFIG_SOC_K3_AM62A7)
+	{
+		.family = "AM62AX",
+		.data = &am62ax_pd_platdata,
+	},
+#endif
+#if IS_ENABLED(CONFIG_SOC_K3_AM62P5)
+	{
+		.family = "AM62PX",
+		.data = &am62px_pd_platdata,
+	},
+#endif
 #if IS_ENABLED(CONFIG_SOC_K3_J721E)
 	{
 		.family = "J721E",
@@ -87,29 +105,11 @@  static const struct soc_attr ti_k3_soc_pd_data[] = {
 		.data = &j721s2_pd_platdata,
 	},
 #endif
-#if IS_ENABLED(CONFIG_SOC_K3_AM625)
-	{
-		.family = "AM62X",
-		.data = &am62x_pd_platdata,
-	},
-#endif
-#if IS_ENABLED(CONFIG_SOC_K3_AM62A7)
-	{
-		.family = "AM62AX",
-		.data = &am62ax_pd_platdata,
-	},
-#endif
 #if IS_ENABLED(CONFIG_SOC_K3_J784S4)
 	{
 		.family = "J784S4",
 		.data = &j784s4_pd_platdata,
 	},
-#endif
-#if IS_ENABLED(CONFIG_SOC_K3_AM62P5)
-	{
-		.family = "AM62PX",
-		.data = &am62px_pd_platdata,
-	},
 #endif
 	{ /* sentinel */ }
 };