> From: Joe Perches [[email protected]] > ... >On Thu, 2017-11-02 at 17:21 +0100, Aleksandar Markovic wrote: > ... > > > Signed-off-by: Miodrag Dinic <[email protected]> > > Signed-off-by: Goran Ferenc <[email protected]> > > Signed-off-by: Aleksandar Markovic <[email protected]> > > wq > > vi much? >
:) > > +/* > > + * Driver for MIPS Goldfish Programmable Interrupt Controller. > > + * > > + * Author: Miodrag Dinic <[email protected]> > > + * > > + * This program is free software; you can redistribute it and/or > > modify it > > + * under the terms of the GNU General Public License as published > > by the > > + * Free Software Foundation; either version 2 of the License, or (at your > > + * option) any later version. > > + */ > > Odd mix of spaces and tabs > Will be fixed in v8. > It'd be good to add a #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt > before any #include so the pr_<level> output is prefixed appropriately. Error and info messages will be completely revised in v8. Thanks for the review! Aleksandar

