diff options
author | Neha Malcom Francis <n-francis@ti.com> | 2023-10-16 13:16:07 +0300 |
---|---|---|
committer | Nishanth Menon <nm@ti.com> | 2023-10-16 19:38:17 +0300 |
commit | 3aeb0d3694e16b5066db82aa1152884f2e6aace0 (patch) | |
tree | f9564cd6673fa836d5cd0a7ef7d370ef34efd8b9 /drivers/soc | |
parent | 8dec342ead710dace27dc82096144bf7a1011827 (diff) | |
download | linux-3aeb0d3694e16b5066db82aa1152884f2e6aace0.tar.xz |
soc: ti: k3-socinfo: Avoid overriding return value
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>
Diffstat (limited to 'drivers/soc')
-rw-r--r-- | drivers/soc/ti/k3-socinfo.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/soc/ti/k3-socinfo.c b/drivers/soc/ti/k3-socinfo.c index d45f5cb955a6..7fc3548e084c 100644 --- a/drivers/soc/ti/k3-socinfo.c +++ b/drivers/soc/ti/k3-socinfo.c @@ -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; } |