diff options
| author | Tom Rini <[email protected]> | 2025-08-19 11:26:03 -0600 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2025-08-19 11:26:03 -0600 |
| commit | cf11b98f139c996bf60c0abe20d343060c4fef26 (patch) | |
| tree | 74411a3c01bfbd87369bd7a7736c664924930198 /drivers | |
| parent | 66ff673a8e9f56d2c08a32d12867094c77ea5fd3 (diff) | |
| parent | c9cd480b5c5496b809d954424ce3554673c278d3 (diff) | |
Merge patch series "remoteproc: k3: Fix two Smatch issue reports"
Andrew Goodbody <[email protected]> says:
Smatch reported two issues, firstly attempting to compare a u8 to a 16
bit macro and secondly a potentially uninitialised variable.
Link: https://lore.kernel.org/r/[email protected]
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/remoteproc/ti_k3_arm64_rproc.c | 2 | ||||
| -rw-r--r-- | drivers/remoteproc/ti_k3_r5f_rproc.c | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/drivers/remoteproc/ti_k3_arm64_rproc.c b/drivers/remoteproc/ti_k3_arm64_rproc.c index d3eb957b2e4..403c6bed2e8 100644 --- a/drivers/remoteproc/ti_k3_arm64_rproc.c +++ b/drivers/remoteproc/ti_k3_arm64_rproc.c @@ -19,7 +19,7 @@ #include <linux/soc/ti/ti_sci_protocol.h> #include "ti_sci_proc.h" -#define INVALID_ID 0xffff +#define INVALID_ID 0xff #define GTC_CNTCR_REG 0x0 #define GTC_CNTFID0_REG 0x20 diff --git a/drivers/remoteproc/ti_k3_r5f_rproc.c b/drivers/remoteproc/ti_k3_r5f_rproc.c index f4bab6868ee..3a25ef6bf87 100644 --- a/drivers/remoteproc/ti_k3_r5f_rproc.c +++ b/drivers/remoteproc/ti_k3_r5f_rproc.c @@ -441,7 +441,7 @@ proc_release: static int k3_r5f_split_reset(struct k3_r5f_core *core) { - int ret; + int ret = 0; dev_dbg(core->dev, "%s\n", __func__); @@ -476,7 +476,7 @@ static int k3_r5f_unprepare(struct udevice *dev) { struct k3_r5f_core *core = dev_get_priv(dev); struct k3_r5f_cluster *cluster = core->cluster; - int ret; + int ret = 0; dev_dbg(dev, "%s\n", __func__); |
