Hello Uwe, Thanks for your patch. On 4/25/22 21:23, Uwe Kleine-König wrote: > This function returns zero unconditionally, so there isn't any benefit > of returning a value. Make it return void to be able to see at a glance > that the return value of ssd130x_i2c_remove() is always zero. > > This patch is a preparation for making i2c remove callbacks return void. > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> > --- Acked-by: Javier Martinez Canillas <javierm@xxxxxxxxxx> and I'll push to drm-misc (drm-misc-next) now. -- Best regards, Javier Martinez Canillas Linux Engineering Red Hat