Hi Shuah and Advait, On 05/10/24 12:41 am, Shuah Khan wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know > the content is safe > > On 10/4/24 09:08, Advait Dhamorikar wrote: >> atmel_hlcdc_plane_update_buffers: may use an uninitialized >> sr variable when the if condition remains unsatisfied >> > > As mentioned in my response to another one of your patches, > include how you found the problem in the change log. > This is smatch warning reported by Dan and Kernel Test Robot. Kindly add the tags mentioned in the below link in the next version. https://lore.kernel.org/oe-kbuild/202409240320.MZPgi3Up-lkp@xxxxxxxxx/ >> Signed-off-by: Advait Dhamorikar <advaitdhamorikar@xxxxxxxxx> >> --- >> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c >> b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c >> index 4a7ba0918eca..4150c4d0b4f2 100644 >> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c >> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c >> @@ -559,7 +559,7 @@ static void >> atmel_hlcdc_plane_update_buffers(struct atmel_hlcdc_plane *plane, >> const struct atmel_hlcdc_layer_desc *desc = plane->layer.desc; >> struct atmel_hlcdc_dc *dc = plane->base.dev->dev_private; >> struct drm_framebuffer *fb = state->base.fb; >> - u32 sr; >> + u32 sr = 0; > >> int i; >> >> if (!dc->desc->is_xlcdc) > > sr could be used uninitialized in the for loop, !dc->desc->is_xlcdc > case. However, is 0 the right initialization for this value? > initializing sr to 0 is the right way Thank you for taking care of the same > thanks, > -- Shuah -- Thanks and Regards, Manikandan M.
