Resolved Block comments use * on subsequent lines checkpatch warnings.
Issue found by checkpatch.
Signed-off-by: Dhaval Shah <[email protected]>
---
drivers/platform/x86/dell-rbtn.c | 26 +++++++++++++-------------
drivers/platform/x86/dell-rbtn.h | 26 +++++++++++++-------------
2 files changed, 26 insertions(+), 26 deletions(-)
diff --git a/drivers/platform/x86/dell-rbtn.c b/drivers/platform/x86/dell-rbtn.c
index f3afe778001e..cab93c6dcd34 100644
--- a/drivers/platform/x86/dell-rbtn.c
+++ b/drivers/platform/x86/dell-rbtn.c
@@ -1,17 +1,17 @@
/*
- Dell Airplane Mode Switch driver
- Copyright (C) 2014-2015 Pali Rohár <[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.
-
- This program is distributed in the hope that it will be useful,
- but WITHOUT ANY WARRANTY; without even the implied warranty of
- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- GNU General Public License for more details.
-*/
+ * Dell Airplane Mode Switch driver
+ * Copyright (C) 2014-2015 Pali Rohár <[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.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ */
#include <linux/module.h>
#include <linux/acpi.h>
diff --git a/drivers/platform/x86/dell-rbtn.h b/drivers/platform/x86/dell-rbtn.h
index c59cc6b8ec2b..46337b22f00d 100644
--- a/drivers/platform/x86/dell-rbtn.h
+++ b/drivers/platform/x86/dell-rbtn.h
@@ -1,17 +1,17 @@
/*
- Dell Airplane Mode Switch driver
- Copyright (C) 2014-2015 Pali Rohár <[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.
-
- This program is distributed in the hope that it will be useful,
- but WITHOUT ANY WARRANTY; without even the implied warranty of
- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- GNU General Public License for more details.
-*/
+ * Dell Airplane Mode Switch driver
+ * Copyright (C) 2014-2015 Pali Rohár <[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.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ */
#ifndef _DELL_RBTN_H_
#define _DELL_RBTN_H_
--
2.11.0
Dhaval,
On Wed, Dec 13, 2017 at 9:23 AM, Dhaval Shah
<[email protected]> wrote:
> Resolved Block comments use * on subsequent lines checkpatch warnings.
> Issue found by checkpatch.
>
> Signed-off-by: Dhaval Shah <[email protected]>
> ---
> drivers/platform/x86/dell-rbtn.c | 26 +++++++++++++-------------
> drivers/platform/x86/dell-rbtn.h | 26 +++++++++++++-------------
> 2 files changed, 26 insertions(+), 26 deletions(-)
>
> diff --git a/drivers/platform/x86/dell-rbtn.c b/drivers/platform/x86/dell-rbtn.c
> index f3afe778001e..cab93c6dcd34 100644
> --- a/drivers/platform/x86/dell-rbtn.c
> +++ b/drivers/platform/x86/dell-rbtn.c
> @@ -1,17 +1,17 @@
> /*
> - Dell Airplane Mode Switch driver
> - Copyright (C) 2014-2015 Pali Rohár <[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.
> -
> - This program is distributed in the hope that it will be useful,
> - but WITHOUT ANY WARRANTY; without even the implied warranty of
> - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> - GNU General Public License for more details.
> -*/
> + * Dell Airplane Mode Switch driver
> + * Copyright (C) 2014-2015 Pali Rohár <[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.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + */
Do you mind using the new SPDX ids instead of this long legalese?
You can check the doc patches from Thomas for details [1]
[1] https://lkml.org/lkml/2017/12/4/934
--
Cordially
Philippe Ombredanne
HI
Thanks a lot for the review.
________________________________________
> From: Philippe Ombredanne <[email protected]>
> Sent: Wednesday, December 13, 2017 2:54 PM
> To: Dhaval Shah
> Cc: [email protected]; Darren Hart; Andy Shevchenko; [email protected]; LKML
> Subject: Re: [PATCH] platform/x86: dell-rbtn: Block comments use * on subsequent lines
>
> Dhaval,
>
> On Wed, Dec 13, 2017 at 9:23 AM, Dhaval Shah
> <[email protected]> wrote:
> > Resolved Block comments use * on subsequent lines checkpatch warnings.
> > Issue found by checkpatch.
> >
> > Signed-off-by: Dhaval Shah <[email protected]>
> > ---
> > drivers/platform/x86/dell-rbtn.c | 26 +++++++++++++-------------
> > drivers/platform/x86/dell-rbtn.h | 26 +++++++++++++-------------
> > 2 files changed, 26 insertions(+), 26 deletions(-)
> >
> > diff --git a/drivers/platform/x86/dell-rbtn.c b/drivers/platform/x86/dell-rbtn.c
> > index f3afe778001e..cab93c6dcd34 100644
> > --- a/drivers/platform/x86/dell-rbtn.c
> > +++ b/drivers/platform/x86/dell-rbtn.c
> > @@ -1,17 +1,17 @@
> > /*
> > - Dell Airplane Mode Switch driver
> > - Copyright (C) 2014-2015 Pali Roh?r <[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.
> > -
> > - This program is distributed in the hope that it will be useful,
> > - but WITHOUT ANY WARRANTY; without even the implied warranty of
> > - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > - GNU General Public License for more details.
> > -*/
> > + * Dell Airplane Mode Switch driver
> > + * Copyright (C) 2014-2015 Pali Roh?r <[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.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License for more details.
> > + */
>
> Do you mind using the new SPDX ids instead of this long legalese?
> You can check the doc patches from Thomas for details [1]
>
>
> [1] https://lkml.org/lkml/2017/12/4/934
This is different issue. If we need to fix the issue of this if we update SPDX licensing as well.
I am agree with you on the use of the new SPDX IDs instead of the long legalese.
For this, we have to add the text "// SPDX-License-Identifier: GPL-2.0" at the top of the file and remove the
whole license part and keep on copyright and Author name related info. After change, it looks like
// SPDX-License-Identifier: GPL-2.0
/*
* Dell Airplane Mode Switch driver
* Copyright (C) 2014-2015 Pali Roh?r <[email protected]>
*/
> --
> Cordially
> Philippe Ombredanne
Dhaval,
On Wed, Dec 13, 2017 at 10:52 AM, Dhaval Shah
<[email protected]> wrote:
> HI
>
> Thanks a lot for the review.
> ________________________________________
>> From: Philippe Ombredanne <[email protected]>
>> Sent: Wednesday, December 13, 2017 2:54 PM
>> To: Dhaval Shah
>> Cc: [email protected]; Darren Hart; Andy Shevchenko; [email protected]; LKML
>> Subject: Re: [PATCH] platform/x86: dell-rbtn: Block comments use * on subsequent lines
>>
>> Dhaval,
>>
>> On Wed, Dec 13, 2017 at 9:23 AM, Dhaval Shah
>> <[email protected]> wrote:
>> > Resolved Block comments use * on subsequent lines checkpatch warnings.
>> > Issue found by checkpatch.
>> >
>> > Signed-off-by: Dhaval Shah <[email protected]>
>> > ---
>> > drivers/platform/x86/dell-rbtn.c | 26 +++++++++++++-------------
>> > drivers/platform/x86/dell-rbtn.h | 26 +++++++++++++-------------
>> > 2 files changed, 26 insertions(+), 26 deletions(-)
>> >
>> > diff --git a/drivers/platform/x86/dell-rbtn.c b/drivers/platform/x86/dell-rbtn.c
>> > index f3afe778001e..cab93c6dcd34 100644
>> > --- a/drivers/platform/x86/dell-rbtn.c
>> > +++ b/drivers/platform/x86/dell-rbtn.c
>> > @@ -1,17 +1,17 @@
>> > /*
>> > - Dell Airplane Mode Switch driver
>> > - Copyright (C) 2014-2015 Pali Rohár <[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.
>> > -
>> > - This program is distributed in the hope that it will be useful,
>> > - but WITHOUT ANY WARRANTY; without even the implied warranty of
>> > - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>> > - GNU General Public License for more details.
>> > -*/
>> > + * Dell Airplane Mode Switch driver
>> > + * Copyright (C) 2014-2015 Pali Rohár <[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.
>> > + *
>> > + * This program is distributed in the hope that it will be useful,
>> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>> > + * GNU General Public License for more details.
>> > + */
>>
>> Do you mind using the new SPDX ids instead of this long legalese?
>> You can check the doc patches from Thomas for details [1]
>>
>>
>> [1] https://lkml.org/lkml/2017/12/4/934
> This is different issue. If we need to fix the issue of this if we update SPDX licensing as well.
> I am agree with you on the use of the new SPDX IDs instead of the long legalese.
> For this, we have to add the text "// SPDX-License-Identifier: GPL-2.0" at the top of the file and remove the
> whole license part and keep on copyright and Author name related info. After change, it looks like
> // SPDX-License-Identifier: GPL-2.0
> /*
> * Dell Airplane Mode Switch driver
> * Copyright (C) 2014-2015 Pali Rohár <[email protected]>
> */
And for a short comment block like this, Linus preference is to avoid
mixing comment style and go all the way C++ style as in:
> // SPDX-License-Identifier: GPL-2.0
> // Dell Airplane Mode Switch driver
> // Copyright (C) 2014-2015 Pali Rohár <[email protected]>
... for even less boilerplate ;)
See Linus comments for a rationale [1] [2] [3] [4] [5]
[1] https://lkml.org/lkml/2017/11/25/133
[2] https://lkml.org/lkml/2017/11/25/125
[3] https://lkml.org/lkml/2017/11/2/715
[4] https://lkml.org/lkml/2017/11/2/805
--
Cordially
Philippe Ombredanne
Hi Philippe,
> From: Philippe Ombredanne <[email protected]>
> Sent: Wednesday, December 13, 2017 3:31 PM
> To: Dhaval Shah
> Cc: [email protected]; Darren Hart; Andy Shevchenko; [email protected]; LKML
> Subject: Re: [PATCH] platform/x86: dell-rbtn: Block comments use * on subsequent lines
> ?
> Dhaval,
>
> On Wed, Dec 13, 2017 at 10:52 AM, Dhaval Shah
> <[email protected]> wrote:
> > HI
> >
> > Thanks a lot for the review.
> > ________________________________________
> > > From: Philippe Ombredanne <[email protected]>
> > > Sent: Wednesday, December 13, 2017 2:54 PM
> > > To: Dhaval Shah
> > > Cc: [email protected]; Darren Hart; Andy Shevchenko; [email protected]; LKML
> > > Subject: Re: [PATCH] platform/x86: dell-rbtn: Block comments use * on subsequent lines
> > >
> > > Dhaval,
> > >
> > > On Wed, Dec 13, 2017 at 9:23 AM, Dhaval Shah
> > > <[email protected]> wrote:
> > > > Resolved Block comments use * on subsequent lines checkpatch warnings.
> > > > Issue found by checkpatch.
> > > >
> > > > Signed-off-by: Dhaval Shah <[email protected]>
> > > > ---
> > > >? drivers/platform/x86/dell-rbtn.c | 26 +++++++++++++-------------
> > > >? drivers/platform/x86/dell-rbtn.h | 26 +++++++++++++-------------
> > > >? 2 files changed, 26 insertions(+), 26 deletions(-)
> > > >
> > > > diff --git a/drivers/platform/x86/dell-rbtn.c b/drivers/platform/x86/dell-rbtn.c
> > > > index f3afe778001e..cab93c6dcd34 100644
> > > > --- a/drivers/platform/x86/dell-rbtn.c
> > > > +++ b/drivers/platform/x86/dell-rbtn.c
> > > > @@ -1,17 +1,17 @@
> > > >? /*
> > > > -??? Dell Airplane Mode Switch driver
> > > > -??? Copyright (C) 2014-2015? Pali Roh?r <[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.
> > > > -
> > > > -??? This program is distributed in the hope that it will be useful,
> > > > -??? but WITHOUT ANY WARRANTY; without even the implied warranty of
> > > > -??? MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.? See the
> > > > -??? GNU General Public License for more details.
> > > > -*/
> > > > + *?? Dell Airplane Mode Switch driver
> > > > + *?? Copyright (C) 2014-2015? Pali Roh?r <[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.
> > > > + *
> > > > + *?? This program is distributed in the hope that it will be useful,
> > > > + *?? but WITHOUT ANY WARRANTY; without even the implied warranty of
> > > > + *?? MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.? See the
> > > > + *?? GNU General Public License for more details.
> > > > + */
> > >
> > > Do you mind using the new SPDX ids instead of this long legalese?
> > > You can check the doc patches from Thomas for details [1]
> > >
> > >
> > > [1] https://lkml.org/lkml/2017/12/4/934
> > This is different issue. If we need to fix the issue of this if we update SPDX licensing as well.
> > I am agree with you on the use of the new SPDX IDs instead of the long legalese.
> > For this, we have to add the text "// SPDX-License-Identifier: GPL-2.0" at the top of the file and remove the
> > whole license part and keep on copyright and Author name related info. After change, it looks like
> > // SPDX-License-Identifier: GPL-2.0
> > /*
> >? *? Dell Airplane Mode Switch driver
> >? *? Copyright (C) 2014-2015? Pali Roh?r <[email protected]>
> >? */
>
> And for a short comment block like this, Linus preference is to avoid
> mixing comment style and go all the way C++ style as in:
>
> > // SPDX-License-Identifier: GPL-2.0
> > // Dell Airplane Mode Switch driver
> > // Copyright (C) 2014-2015? Pali Roh?r <[email protected]>
> >
> > ... for even less boilerplate ;)
Thanks a lot for the link.
Do you want to me to update along with SPDX in this patchset with V2 or we will update in the next patchset once this patch merge?
> >
> > See Linus comments for a rationale [1] [2] [3] [4] [5]
> >
> > [1] https://lkml.org/lkml/2017/11/25/133
> > [2] https://lkml.org/lkml/2017/11/25/125
> > [3] https://lkml.org/lkml/2017/11/2/715
> > [4] https://lkml.org/lkml/2017/11/2/805
>
> --
> Cordially
> Philippe Ombredanne
On Wed, 2017-12-13 at 10:53 +0000, Dhaval Shah wrote:
> > > This is different issue. If we need to fix the issue of this if we update SPDX licensing as well.
> > > I am agree with you on the use of the new SPDX IDs instead of the long legalese.
> > > For this, we have to add the text "// SPDX-License-Identifier: GPL-2.0" at the top of the file and remove the
> > > whole license part and keep on copyright and Author name related info. After change, it looks like
> > > // SPDX-License-Identifier: GPL-2.0
> > > /*
> > > * Dell Airplane Mode Switch driver
> > > * Copyright (C) 2014-2015 Pali Roh?r <[email protected]>
> > > */
> >
> > And for a short comment block like this, Linus preference is to avoid
> > mixing comment style and go all the way C++ style as in:
> >
> > > // SPDX-License-Identifier: GPL-2.0
> > > // Dell Airplane Mode Switch driver
> > > // Copyright (C) 2014-2015 Pali Roh?r <[email protected]>
> > >
> > > ... for even less boilerplate ;)
>
> Thanks a lot for the link.
> Do you want to me to update along with SPDX in this patchset with V2 or we will update in the next patchset once this patch merge?
If anyone does conversions of these license texts
you have to be _extremely_ careful about what
specific license identifier you use.
This license includes the text
"or (at your option) any later version"
which is not the same as using
SPDX-License-Identifier: GPL-2.0
Hi Joe,
________________________________________
> From: Joe Perches <[email protected]>
> Sent: Wednesday, December 13, 2017 5:55 PM
> To: Dhaval Shah; Philippe Ombredanne
> Cc: [email protected]; Darren Hart; Andy Shevchenko; [email protected]; LKML
> Subject: Re: [PATCH] platform/x86: dell-rbtn: Block comments use * on subsequent lines
>
> On Wed, 2017-12-13 at 10:53 +0000, Dhaval Shah wrote:
> > > > This is different issue. If we need to fix the issue of this if we update SPDX licensing as well.
> > > > I am agree with you on the use of the new SPDX IDs instead of the long legalese.
> > > > For this, we have to add the text "// SPDX-License-Identifier: GPL-2.0" at the top of the file and remove the
> > > > whole license part and keep on copyright and Author name related info. After change, it looks like
> > > > // SPDX-License-Identifier: GPL-2.0
> > > > /*
> > > > * Dell Airplane Mode Switch driver
> > > > * Copyright (C) 2014-2015 Pali Roh?r <[email protected]>
> > > > */
> > >
> > > And for a short comment block like this, Linus preference is to avoid
> > > mixing comment style and go all the way C++ style as in:
> > >
> > > > // SPDX-License-Identifier: GPL-2.0
> > > > // Dell Airplane Mode Switch driver
> > > // Copyright (C) 2014-2015 Pali Roh?r <[email protected]>
> > >
> > > ... for even less boilerplate ;)
> >
> > Thanks a lot for the link.
> > Do you want to me to update along with SPDX in this patchset with V2 or we will update in the next patchset once this > > >patch merge?
>
> If anyone does conversions of these license texts
> you have to be _extremely_ careful about what
> specific license identifier you use.
> This license includes the text
> "or (at your option) any later version"
> which is not the same as using
> SPDX-License-Identifier: GPL-2.0
I have given this SPDX ID based on the current license info in the driver. Yeah. I give the authority to author [email protected] to decide the SPDX text.
Dhaval,
On Wed, Dec 13, 2017 at 11:47 AM, Dhaval Shah
<[email protected]> wrote:
<snip>
> > And for a short comment block like this, Linus preference is to avoid
> > mixing comment style and go all the way C++ style as in:
> >
> > > // SPDX-License-Identifier: GPL-2.0
> > > // Dell Airplane Mode Switch driver
> > > // Copyright (C) 2014-2015 Pali Rohár <[email protected]>
> > >
> > > ... for even less boilerplate ;)
>
> Thanks a lot for the link.
> Do you want to me to update along with SPDX in this patchset with V2 or we will update in the next patchset once this patch merge?
That's not for me to decide, I am just a lowly licensing elf helper scruffy!
PS: you might want to avoid sending HTML email on lkml, FWIW.
--
Cordially
Philippe Ombredanne
On Wednesday 13 December 2017 12:41:03 Dhaval Shah wrote:
> I have given this SPDX ID based on the current license info in the driver. Yeah. I give the authority to author [email protected] to decide the SPDX text.
Hi! I think that license of this module is clear and does not need more
comments. Part "either version 2 of the License, or (at your option) any
later version." expresses what it exactly mean and it is standard
license also used by other projects.
--
Pali Rohár
[email protected]
Hi Philippe,
________________________________________
> From: Philippe Ombredanne <[email protected]>
> Sent: Wednesday, December 13, 2017 6:27 PM
> To: Dhaval Shah
> Cc: [email protected]; Darren Hart; Andy Shevchenko; [email protected]; LKML
> Subject: Re: [PATCH] platform/x86: dell-rbtn: Block comments use * on subsequent lines
>
> Dhaval,
>
> On Wed, Dec 13, 2017 at 11:47 AM, Dhaval Shah
> <[email protected]> wrote:
>
> <snip>
> > > And for a short comment block like this, Linus preference is to avoid
> > > mixing comment style and go all the way C++ style as in:
> > >
> > > > // SPDX-License-Identifier: GPL-2.0
> > > > // Dell Airplane Mode Switch driver
> > > > // Copyright (C) 2014-2015 Pali Roh?r <[email protected]>
> > > >
> > > > ... for even less boilerplate ;)
> >
> > Thanks a lot for the link.
> > Do you want to me to update along with SPDX in this patchset with V2 or we will update in the next patchset once this patch merge?
>
> That's not for me to decide, I am just a lowly licensing elf helper scruffy!
I can assume that current patch is accepted or I have to provide you any changes
> PS: you might want to avoid sending HTML email on lkml, FWIW.
That is done by mistake. sorry for that. I will take care of that.
--
Cordially
Philippe Ombredanne
On Wed, 2017-12-13 at 12:41 +0000, Dhaval Shah wrote:
> > If anyone does conversions of these license texts
> > you have to be _extremely_ careful about what
> > specific license identifier you use.
> > This license includes the text
> > "or (at your option) any later version"
> > which is not the same as using
> > SPDX-License-Identifier: GPL-2.0
>
> I have given this SPDX ID based on the current license info in the driver.
No, you have not.
You have done it incorrectly.
For a short description, read:
https://spdx.org/sites/cpstandard/files/pages/files/using_spdx_license_list_short_identifiers.pdf
page 5
________________________________________
> From: Joe Perches <[email protected]>
> Sent: Wednesday, December 13, 2017 9:09 PM
> To: Dhaval Shah; Philippe Ombredanne
> Cc: [email protected]; Darren Hart; Andy Shevchenko; [email protected]; LKML
> Subject: Re: [PATCH] platform/x86: dell-rbtn: Block comments use * on subsequent lines
>
> On Wed, 2017-12-13 at 12:41 +0000, Dhaval Shah wrote:
> > > If anyone does conversions of these license texts
> > > you have to be _extremely_ careful about what
> > > specific license identifier you use.
> > > This license includes the text
> > > "or (at your option) any later version"
> > > which is not the same as using
> > > SPDX-License-Identifier: GPL-2.0
> >
> > I have given this SPDX ID based on the current license info in the driver.
>
> No, you have not.
> You have done it incorrectly.
>
> For a short description, read:
> https://spdx.org/sites/cpstandard/files/pages/files/using_spdx_license_list_short_identifiers.pdf
> page 5
I got your point. SPDX text should be "SPDX-License-Identifier: GPL-2.0+".
Am i Right?
On Thu, Dec 14, 2017 at 11:12 AM, Dhaval Shah
<[email protected]> wrote:
> I got your point. SPDX text should be "SPDX-License-Identifier: GPL-2.0+".
> Am i Right?
Whatever you come up with I will apply taking into consideration the following:
1. Linus' preference over comment style.
2. Module author(s) blessing for the change.
Without above on the table I would not even look into the message.
P.S. And personally I'm a bit sceptical about standalone (coment)
style changes. I would rather not to apply anything like this without
real need.
--
With Best Regards,
Andy Shevchenko