soc: ti: k3-socinfo: Avoid overriding return value
authorNeha Malcom Francis <n-francis@ti.com>
Mon, 16 Oct 2023 10:16:07 +0000 (15:46 +0530)
committerNishanth Menon <nm@ti.com>
Mon, 16 Oct 2023 16:38:17 +0000 (11:38 -0500)
Avoid overriding the return value and make sure the right error code
is reflected. If the part is none of the identified list present in
k3_soc_ids[], return -ENODEV.

Signed-off-by: Neha Malcom Francis <n-francis@ti.com>
Link: https://lore.kernel.org/r/20231016101608.993921-3-n-francis@ti.com
Signed-off-by: Nishanth Menon <nm@ti.com>
drivers/soc/ti/k3-socinfo.c

index d45f5cb955a66814e4c545c6b5c56db84d696c70..7fc3548e084cd4ee7d430a296e0c6d83088d098e 100644 (file)
@@ -60,7 +60,7 @@ k3_chipinfo_partno_to_names(unsigned int partno,
                        return 0;
                }
 
-       return -EINVAL;
+       return -ENODEV;
 }
 
 static int k3_chipinfo_probe(struct platform_device *pdev)
@@ -111,8 +111,7 @@ static int k3_chipinfo_probe(struct platform_device *pdev)
 
        ret = k3_chipinfo_partno_to_names(partno_id, soc_dev_attr);
        if (ret) {
-               dev_err(dev, "Unknown SoC JTAGID[0x%08X]\n", jtag_id);
-               ret = -ENODEV;
+               dev_err(dev, "Unknown SoC JTAGID[0x%08X]: %d\n", jtag_id, ret);
                goto err_free_rev;
        }