On Fri, Nov 17, 2017 at 03:50:25PM +0100, Philippe Ombredanne wrote: > On Fri, Nov 17, 2017 at 3:16 PM, Greg Kroah-Hartman > <[email protected]> wrote: > > There is no need to #define the license of the driver, just put it in > > the MODULE_LICENSE() line directly as a text string. > > > > This allows tools that check that the module license matches the source > > code license to work properly, as there is no need to unwind the > > unneeded dereference, especially when the string is defined in a .h file > > far away from the .c file it is used in. > > > > Cc: Patrik Jakobsson <[email protected]> > > Cc: David Airlie <[email protected]> > > Reported-by: Philippe Ombredanne <[email protected]> > > Signed-off-by: Greg Kroah-Hartman <[email protected]> > > Reviewed-by: Philippe Ombredanne <[email protected]>
Queued for 4.16, it'll pop up in linux-next as soon as -rc1 is tagged. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch
