On Sat, Apr 05, 2025 at 10:45:03AM +0800, [email protected] wrote: > From: sunliming <[email protected]> > > Fix below smatch warnings: > drivers/gpu/drm/vc4/vc4_gem.c:604 vc4_lock_bo_reservations() error: > uninitialized symbol 'ret'. > > Reported-by: kernel test robot <[email protected]> > Reported-by: Dan Carpenter <[email protected]> > Closes: https://lore.kernel.org/r/[email protected]/ > Signed-off-by: sunliming <[email protected]> > --- > drivers/gpu/drm/vc4/vc4_gem.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/vc4/vc4_gem.c b/drivers/gpu/drm/vc4/vc4_gem.c > index 8125f87edc60..04ea1696fc5d 100644 > --- a/drivers/gpu/drm/vc4/vc4_gem.c > +++ b/drivers/gpu/drm/vc4/vc4_gem.c > @@ -582,7 +582,7 @@ static int > vc4_lock_bo_reservations(struct vc4_exec_info *exec, > struct drm_exec *exec_ctx) > { > - int ret; > + int ret = 0; > > /* Reserve space for our shared (read-only) fence references, > * before we commit the CL to the hardware.
I don't see a code path where ret is used without being initialized. Can you clarify that in the commit log? Maxime
signature.asc
Description: PGP signature
