Re: [PATCH 5/9] rtc: cros-ec: Switch to SPDX identifier.

From: Alexandre Belloni
Date: Mon Jun 25 2018 - 03:35:17 EST


On 25/06/2018 09:58:32+0800, Greg KH wrote:
> On Tue, Jun 05, 2018 at 08:59:52PM +0200, Alexandre Belloni wrote:
> > On 05/06/2018 20:51:06+0200, Enric Balletbo i Serra wrote:
> > > Hi Alexandre,
> > >
> > > On 05/06/18 11:46, Alexandre Belloni wrote:
> > > > On 05/06/2018 11:22:05+0200, Enric Balletbo i Serra wrote:
> > > >> Adopt the SPDX license identifier headers to ease license compliance
> > > >> management.
> > > >>
> > > >> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@xxxxxxxxxxxxx>
> > > >> ---
> > > >>
> > > >> drivers/rtc/rtc-cros-ec.c | 21 +++++----------------
> > > >> 1 file changed, 5 insertions(+), 16 deletions(-)
> > > >>
> > > > Applied, thanks.
> > > >
> > >
> > > Looks like all the ChromeOS drivers should be licensed as GPL v2 only. There was
> > > a mismatch between what the header says and the MODULE_LICENSE and I wrongly
> > > assumed that the MODULE_LICENSE is the one I should follow. So could you not
> > > apply this patch for now and I'll send another version with the correct license?
> > >
> >
> > Hum, ok, this seemed to be the correct way to solve the mismatch to me.
>
> No, the license text trumps the MODULE_LICENSE() string. Please revert
> this, or fix it up to be correct.
>

Isn't it correct in Linus' tree?


--
Alexandre Belloni, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com