[PATCH 2/3] drm/tests: helpers: Add module infos

Jani Nikula jani.nikula at linux.intel.com
Wed Nov 16 11:32:51 UTC 2022


On Wed, 16 Nov 2022, Maíra Canal <mairacanal at riseup.net> wrote:
> Hi Maxime,
>
> On 11/16/22 06:17, Maxime Ripard wrote:
>> The MODULE_LICENSE macro is missing from the kunit helpers file, thus
>> leading to a build error.
>> 
>> Let's introduce it along with MODULE_AUTHOR.
>> 
>> Fixes: 44a3928324e9 ("drm/tests: Add Kunit Helpers")
>> Reported-by: Stephen Rothwell <sfr at canb.auug.org.au>
>> Signed-off-by: Maxime Ripard <maxime at cerno.tech>
>
> It would be nice to add the SPDX-License-Identifier tag in the source
> file as well. Besides that,

It's not just nice, it's basically mandatory to add license boilerplate.

Checkpatch would've warned about this. And actually about a lot of stuff
in the series.

(And our CI checkpatch did too, although we don't send the replies to
the world, just intel-gfx [1].)

BR,
Jani.


[1] https://lore.kernel.org/r/166846421165.32750.1193593124785451784@emeril.freedesktop.org



-- 
Jani Nikula, Intel Open Source Graphics Center


More information about the dri-devel mailing list