diff options
| author | Andrew Goodbody <[email protected]> | 2025-08-12 11:13:50 +0100 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2025-08-19 11:26:07 -0600 |
| commit | 13ca68b104ea75d54e86816ba6d86ef5e70a2668 (patch) | |
| tree | 4d34e68d01b2875f148e33a32e4eae9f721dbe08 /drivers | |
| parent | b90927bd0d81096239ff4a48a71c3d9e87ef7209 (diff) | |
soc: ti: k3-navss-ringacc: Do not use uninitialised variable
In k3_nav_ringacc_probe_dt there can be no error code returned from
dev_read_u32_default so ret is not assigned to and should not be used.
Remove the use of ret from the dev_err call as it is unitialised.
This issue was found by Smatch.
Signed-off-by: Andrew Goodbody <[email protected]>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/soc/ti/k3-navss-ringacc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/soc/ti/k3-navss-ringacc.c b/drivers/soc/ti/k3-navss-ringacc.c index ac29158d489..d37c454143b 100644 --- a/drivers/soc/ti/k3-navss-ringacc.c +++ b/drivers/soc/ti/k3-navss-ringacc.c @@ -932,7 +932,7 @@ static int k3_nav_ringacc_probe_dt(struct k3_nav_ringacc *ringacc) ringacc->num_rings = dev_read_u32_default(dev, "ti,num-rings", 0); if (!ringacc->num_rings) { - dev_err(dev, "ti,num-rings read failure %d\n", ret); + dev_err(dev, "ti,num-rings read failure\n"); return -EINVAL; } |