On 02/15/2019 09:25 AM, Geert Uytterhoeven wrote: > On Fri, Feb 15, 2019 at 3:19 AM Finn Thain <fth...@telegraphics.com.au> wrote: >> The value of info->var.bits_per_pixel get checked in macfb_setcolreg(). >> Remove additional checks as they are redundant. >> >> macfb_defined.activate gets initialized to FB_ACTIVATE_NOW by the struct >> initializer. Remove redundant assignments. >> >> macfb_defined.bits_per_pixel, .width and .height all get assigned >> unconditionally. Remove redundant initializers. >> >> Cc: linux-m...@lists.linux-m68k.org >> Tested-by: Stan Johnson <user...@yahoo.com> >> Signed-off-by: Finn Thain <fth...@telegraphics.com.au> > > Reviewed-by: Geert Uytterhoeven <ge...@linux-m68k.org>
Patch queued for v5.2, thanks. Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics