diff mbox series

[02/14] soc: ti: k3-socinfo: Fix SOC JTAG entry order

Message ID 20240529132448.459330-3-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 JTAG_ID_PARTNO_* in alphabetical order.

Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
---
 drivers/soc/soc_ti_k3.c | 28 ++++++++++++++--------------
 1 file changed, 14 insertions(+), 14 deletions(-)

Comments

Neha Malcom Francis May 30, 2024, 2:21 a.m. UTC | #1
On 29/05/24 18:54, Jayesh Choudhary wrote:
> Add JTAG_ID_PARTNO_* in alphabetical order.
> 
> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
> ---
>   drivers/soc/soc_ti_k3.c | 28 ++++++++++++++--------------
>   1 file changed, 14 insertions(+), 14 deletions(-)
> 
> diff --git a/drivers/soc/soc_ti_k3.c b/drivers/soc/soc_ti_k3.c
> index b585e47d46..0838808515 100644
> --- a/drivers/soc/soc_ti_k3.c
> +++ b/drivers/soc/soc_ti_k3.c
> @@ -23,33 +23,33 @@ static const char *get_family_string(u32 idreg)
>   	soc = (idreg & JTAG_ID_PARTNO_MASK) >> JTAG_ID_PARTNO_SHIFT;
>   
>   	switch (soc) {
> +	case JTAG_ID_PARTNO_AM62X:
> +		family = "AM62X";
> +		break;
> +	case JTAG_ID_PARTNO_AM62AX:
> +		family = "AM62AX";
> +		break;
> +	case JTAG_ID_PARTNO_AM62PX:
> +		family = "AM62PX";
> +		break;
> +	case JTAG_ID_PARTNO_AM64X:
> +		family = "AM64X";
> +		break;
>   	case JTAG_ID_PARTNO_AM65X:
>   		family = "AM65X";
>   		break;
> -	case JTAG_ID_PARTNO_J721E:
> -		family = "J721E";
> -		break;
>   	case JTAG_ID_PARTNO_J7200:
>   		family = "J7200";
>   		break;
> -	case JTAG_ID_PARTNO_AM64X:
> -		family = "AM64X";
> +	case JTAG_ID_PARTNO_J721E:
> +		family = "J721E";
>   		break;
>   	case JTAG_ID_PARTNO_J721S2:
>   		family = "J721S2";
>   		break;
> -	case JTAG_ID_PARTNO_AM62X:
> -		family = "AM62X";
> -		break;
> -	case JTAG_ID_PARTNO_AM62AX:
> -		family = "AM62AX";
> -		break;
>   	case JTAG_ID_PARTNO_J784S4:
>   		family = "J784S4";
>   		break;
> -	case JTAG_ID_PARTNO_AM62PX:
> -		family = "AM62PX";
> -		break;
>   	default:
>   		family = "Unknown Silicon";
>   	};

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

Patch

diff --git a/drivers/soc/soc_ti_k3.c b/drivers/soc/soc_ti_k3.c
index b585e47d46..0838808515 100644
--- a/drivers/soc/soc_ti_k3.c
+++ b/drivers/soc/soc_ti_k3.c
@@ -23,33 +23,33 @@  static const char *get_family_string(u32 idreg)
 	soc = (idreg & JTAG_ID_PARTNO_MASK) >> JTAG_ID_PARTNO_SHIFT;
 
 	switch (soc) {
+	case JTAG_ID_PARTNO_AM62X:
+		family = "AM62X";
+		break;
+	case JTAG_ID_PARTNO_AM62AX:
+		family = "AM62AX";
+		break;
+	case JTAG_ID_PARTNO_AM62PX:
+		family = "AM62PX";
+		break;
+	case JTAG_ID_PARTNO_AM64X:
+		family = "AM64X";
+		break;
 	case JTAG_ID_PARTNO_AM65X:
 		family = "AM65X";
 		break;
-	case JTAG_ID_PARTNO_J721E:
-		family = "J721E";
-		break;
 	case JTAG_ID_PARTNO_J7200:
 		family = "J7200";
 		break;
-	case JTAG_ID_PARTNO_AM64X:
-		family = "AM64X";
+	case JTAG_ID_PARTNO_J721E:
+		family = "J721E";
 		break;
 	case JTAG_ID_PARTNO_J721S2:
 		family = "J721S2";
 		break;
-	case JTAG_ID_PARTNO_AM62X:
-		family = "AM62X";
-		break;
-	case JTAG_ID_PARTNO_AM62AX:
-		family = "AM62AX";
-		break;
 	case JTAG_ID_PARTNO_J784S4:
 		family = "J784S4";
 		break;
-	case JTAG_ID_PARTNO_AM62PX:
-		family = "AM62PX";
-		break;
 	default:
 		family = "Unknown Silicon";
 	};