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

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