On Friday, February 02, 2018 04:00:48 PM Arnd Bergmann wrote: > Building with LTO enabled reveals some functions whose prototypes > in the header are different from the definition: > > drivers/video/fbdev/sis/sis_main.h:765:0: error: type of 'SiS_SetCH70xxANDOR' does not match original declaration [-Werror=lto-type-mismatch] > extern void SiS_SetCH70xxANDOR(struct SiS_Private *SiS_Pr, unsigned short reg, > > drivers/video/fbdev/sis/init301.c:8937:0: note: type mismatch in parameter 4 > SiS_SetCH70xxANDOR(struct SiS_Private *SiS_Pr, unsigned short reg, > > drivers/video/fbdev/sis/init301.c:8937:0: note: type 'short unsigned int' should match type 'unsigned char' > drivers/video/fbdev/sis/init301.c:8937:0: note: 'SiS_SetCH70xxANDOR' was previously declared here > > The root cause appears to be the way that header files are used in this > driver, where they contain both static variable and declarations for > symbols in other files. > > To clean that up, I'm changing all mixed headers to only contain > declarations the way we normally do in C, or contain only static > variables, and move the rest to a more appropriate place. Once that > is done, the headers can be included in the other files as well, and > guarantee that the prototypes match. > > There are a few headers that now only contain static variables, and > I'm leaving those alone here as the patch is already too big. These > could be trivially moved into the respective .c files. > > Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx> Patch queued for 4.17, thanks. Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
