Quoting Abhinav Kumar (2021-03-05 11:17:19) > Currently the error checking logic in the dp_debug module could > pass zero to PTR_ERR and it causes the below kbot warnings: > > drivers/gpu/drm/msm/dp/dp_debug.c:378 dp_debug_init() > warn: passing zero to 'PTR_ERR' > drivers/gpu/drm/msm/dp/dp_debug.c:387 dp_debug_init() > warn: passing zero to 'PTR_ERR' > drivers/gpu/drm/msm/dp/dp_debug.c:396 dp_debug_init() > warn: passing zero to 'PTR_ERR' > drivers/gpu/drm/msm/dp/dp_debug.c:405 dp_debug_init() > warn: passing zero to 'PTR_ERR' > > Debugfs functions are not supposed to be checked in the normal > case so delete this code. Also it silences the above Smatch > warnings that we're checking for NULL when these functions only > return error pointers. > > Reported-by: kernel test robot <[email protected]> > Reported-by: Dan Carpenter <[email protected]> > Signed-off-by: Abhinav Kumar <[email protected]> > ---
Reviewed-by: Stephen Boyd <[email protected]> Nice cleanup! _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
