On Sun, Mar 19, 2023 at 08:20:11PM +0100, Borislav Petkov wrote: > On Sun, Jan 29, 2023 at 05:50:54PM +0100, Jonathan Neuschäfer wrote: [...] > Wanna fix the other two in the EDAC tree: > > drivers/edac/e752x_edac.c:1465:MODULE_AUTHOR("Linux Networx (http://lnxi.com) Tom Zimmerman\n"); > drivers/edac/e7xxx_edac.c:599:MODULE_AUTHOR("Linux Networx (http://lnxi.com) Thayne Harbaugh et al\n" > > ? > > That macro takes: > > /* > * Author(s), use "Name " or just "Name", for multiple > * authors use multiple MODULE_AUTHOR() statements/lines. > */ > #define MODULE_AUTHOR(_author) MODULE_INFO(author, _author) > > and not prose. > > You can move the prose to the top comment in the respective file, if it > doesn't contain it yet. > > Thx. Sorry for the late reply. I'm replying now because I saw that you applied the patch. I might fix the above if/when I get around to it, but there's currently no guarantee of that. Best regards, Jonathan