* Re: [v4,1/9] net-next: phy: new Asix Electronics PHY driver [not found] <1524103526-12240-2-git-send-email-schmitzmic@gmail.com> @ 2019-01-18 10:22 ` Thomas Gleixner 2019-01-20 17:43 ` Andrew Lunn 0 siblings, 1 reply; 3+ messages in thread From: Thomas Gleixner @ 2019-01-18 10:22 UTC (permalink / raw) To: Michael Schmitz Cc: netdev, andrew, fthain, geert, f.fainelli, linux-m68k, Michael.Karcher, LKML, Greg KH, Kate Stewart Michael, On Thu, 19 Apr 2018, Michael Schmitz wrote: > --- /dev/null > +++ b/drivers/net/phy/asix.c > @@ -0,0 +1,63 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* Driver for Asix PHYs > + * > + * Author: Michael Schmitz <schmitzmic@gmail.com> > + * > + * 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 license information is broken. The SPDX license identifier and the boiler plate text are contradicting. The SPDX id is GPL v2 only and the boiler plate says v2 or later. Please decide which version you want and fix ASAP. If you fix that up please add a Fixes: tag and cc stable. While at it please remove the boiler plate text as the SPDX id is sufficient and the boiler plate is redundant information. See Documentation/process/license-rules.txt Thanks, tglx ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [v4,1/9] net-next: phy: new Asix Electronics PHY driver 2019-01-18 10:22 ` [v4,1/9] net-next: phy: new Asix Electronics PHY driver Thomas Gleixner @ 2019-01-20 17:43 ` Andrew Lunn 2019-01-20 22:44 ` Michael Schmitz 0 siblings, 1 reply; 3+ messages in thread From: Andrew Lunn @ 2019-01-20 17:43 UTC (permalink / raw) To: Thomas Gleixner Cc: Michael Schmitz, netdev, fthain, geert, f.fainelli, linux-m68k, Michael.Karcher, LKML, Greg KH, Kate Stewart On Fri, Jan 18, 2019 at 11:22:39AM +0100, Thomas Gleixner wrote: > Michael, > > On Thu, 19 Apr 2018, Michael Schmitz wrote: > > > --- /dev/null > > +++ b/drivers/net/phy/asix.c > > @@ -0,0 +1,63 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > +/* Driver for Asix PHYs > > + * > > + * Author: Michael Schmitz <schmitzmic@gmail.com> > > + * > > + * 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 license information is broken. The SPDX license identifier and the > boiler plate text are contradicting. The SPDX id is GPL v2 only and the > boiler plate says v2 or later. Hi Thomas Please see: https://www.spinics.net/lists/netdev/msg544312.html The first two patches are simple SPDX converstions. Then it gets interesting trying to sort out license inconsistencies. Andrew ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [v4,1/9] net-next: phy: new Asix Electronics PHY driver 2019-01-20 17:43 ` Andrew Lunn @ 2019-01-20 22:44 ` Michael Schmitz 0 siblings, 0 replies; 3+ messages in thread From: Michael Schmitz @ 2019-01-20 22:44 UTC (permalink / raw) To: Andrew Lunn, Thomas Gleixner Cc: netdev, fthain, geert, f.fainelli, linux-m68k, Michael.Karcher, LKML, Greg KH, Kate Stewart Hi Andrew, no objection for you to pick this up as part as a larger cleanup. I've tried to reconstruct how this happened (i.e. what other phy driver file I used as a 'template' for asix.c) - all I can say is that the 2.0+ boiler plate text was in my initial commit, and the incorrect SPDX tag was added in response to checkpath complaints. So 2.0+ would be correct. Thomas: does that suit your purpose? Cheers, Michael On 21/01/19 6:43 AM, Andrew Lunn wrote: > On Fri, Jan 18, 2019 at 11:22:39AM +0100, Thomas Gleixner wrote: >> Michael, >> >> On Thu, 19 Apr 2018, Michael Schmitz wrote: >> >>> --- /dev/null >>> +++ b/drivers/net/phy/asix.c >>> @@ -0,0 +1,63 @@ >>> +// SPDX-License-Identifier: GPL-2.0 >>> +/* Driver for Asix PHYs >>> + * >>> + * Author: Michael Schmitz <schmitzmic@gmail.com> >>> + * >>> + * 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 license information is broken. The SPDX license identifier and the >> boiler plate text are contradicting. The SPDX id is GPL v2 only and the >> boiler plate says v2 or later. > Hi Thomas > > Please see: > > https://www.spinics.net/lists/netdev/msg544312.html > > The first two patches are simple SPDX converstions. Then it gets > interesting trying to sort out license inconsistencies. > > Andrew ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-01-21 8:59 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <1524103526-12240-2-git-send-email-schmitzmic@gmail.com> 2019-01-18 10:22 ` [v4,1/9] net-next: phy: new Asix Electronics PHY driver Thomas Gleixner 2019-01-20 17:43 ` Andrew Lunn 2019-01-20 22:44 ` Michael Schmitz
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).