Re: [PATCH 0/5] SPDX license identifiers in all DD drivers
On Wed, Mar 21, 2018 at 06:29:48PM +0100, Daniel Scheller wrote: > Hi Greg, > > Am Wed, 21 Mar 2018 10:49:32 +0100 > schrieb Greg KH : > > > On Tue, Mar 20, 2018 at 10:01:27PM +0100, Daniel Scheller wrote: > > > From: Daniel Scheller > > > > > > This series adds SPDX license identifiers to all source files which are > > > copyright by either Digital Devices GmbH or Metzlerbros GbR, who are > > > the original authors of the ddbridge, ngene, cxd2099, mxl5xx, stv0910 > > > and stv6111 bridge/demod/tuner drivers, with the mxl5xx driver being > > > based on source code released by MaxLinear. > > > [...] > > > The original intention was to fully replace all the licensing headers > > > with only the SPDX License Identifiers as it is done in a lot of other > > > in-tree drivers nowadays. However, Digital Devices disagreed to do this > > > and expressed major concerns regarding this, in that a machine readable > > > license tag instead of a full license boilerplate won't hold up equally, > > > so we agreed to keep the license boilerplate text as is right now. > > > > That's really odd, who at that company can I talk to about this? Or > > really, what lawyer at that company can I point my lawyer at to talk > > about this, that's the only way this is going to get resolved. > > I'm not entirely sure, but I guess for a first start it's best to > contact Ralph (from Metzlerbros) and Manfred (from Digital > Devices), being the authors and copyright owners of the DDDVB driver > package where the drivers originate from and thus is the upstream for > the mainlined copies of the mentioned drivers. Both are in the Cc list > (rjkm and mvoelkel) of this thread. > > > If it helps, _ALL_ of the major companies that are kernel developers are > > onboard with the removal of the crazy boiler-plate text, so this tiny > > holdout should be easy to resolve. > > > > > Greg, I'm Cc'ing you on this due to the last paragraph, as AFAIK you're > > > one of the initiators of the SPDX tagging initiative, and you even added > > > tags to 10k+ files all over the tree :-) so we maybe can discuss this > > > further, also with DD, in the hopes you're fine with this - sorry in > > > advance if not. > > > > See my review of your first patch here, this needs to be done a lot > > differently... > > Check. Thanks for reviewing. The intent was to do a full cleanup of all > licensing things in one go, per driver. Will do one patch for SPDX and > eventual boilerplate cleanup for all drivers, one for MODULE_LICENSE > and one for missing headers in the next iteration. Though I'd wait > with that for now if you like to contact Ralph and Manfred, and do a v2 > based on the outcome. You can always just do the "add a SPDX line" patches now, that touch nothing else. No one can get upset at that. thanks, greg k-h
Re: [PATCH 0/5] SPDX license identifiers in all DD drivers
Hi Greg, Am Wed, 21 Mar 2018 10:49:32 +0100 schrieb Greg KH : > On Tue, Mar 20, 2018 at 10:01:27PM +0100, Daniel Scheller wrote: > > From: Daniel Scheller > > > > This series adds SPDX license identifiers to all source files which are > > copyright by either Digital Devices GmbH or Metzlerbros GbR, who are > > the original authors of the ddbridge, ngene, cxd2099, mxl5xx, stv0910 > > and stv6111 bridge/demod/tuner drivers, with the mxl5xx driver being > > based on source code released by MaxLinear. > > [...] > > The original intention was to fully replace all the licensing headers > > with only the SPDX License Identifiers as it is done in a lot of other > > in-tree drivers nowadays. However, Digital Devices disagreed to do this > > and expressed major concerns regarding this, in that a machine readable > > license tag instead of a full license boilerplate won't hold up equally, > > so we agreed to keep the license boilerplate text as is right now. > > That's really odd, who at that company can I talk to about this? Or > really, what lawyer at that company can I point my lawyer at to talk > about this, that's the only way this is going to get resolved. I'm not entirely sure, but I guess for a first start it's best to contact Ralph (from Metzlerbros) and Manfred (from Digital Devices), being the authors and copyright owners of the DDDVB driver package where the drivers originate from and thus is the upstream for the mainlined copies of the mentioned drivers. Both are in the Cc list (rjkm and mvoelkel) of this thread. > If it helps, _ALL_ of the major companies that are kernel developers are > onboard with the removal of the crazy boiler-plate text, so this tiny > holdout should be easy to resolve. > > > Greg, I'm Cc'ing you on this due to the last paragraph, as AFAIK you're > > one of the initiators of the SPDX tagging initiative, and you even added > > tags to 10k+ files all over the tree :-) so we maybe can discuss this > > further, also with DD, in the hopes you're fine with this - sorry in > > advance if not. > > See my review of your first patch here, this needs to be done a lot > differently... Check. Thanks for reviewing. The intent was to do a full cleanup of all licensing things in one go, per driver. Will do one patch for SPDX and eventual boilerplate cleanup for all drivers, one for MODULE_LICENSE and one for missing headers in the next iteration. Though I'd wait with that for now if you like to contact Ralph and Manfred, and do a v2 based on the outcome. Thanks & best regards, Daniel Scheller -- https://github.com/herrnst
Re: [PATCH 0/5] SPDX license identifiers in all DD drivers
Em Tue, 20 Mar 2018 22:01:27 +0100 Daniel Scheller escreveu: > Ralph, Manfred, I'm Cc'ing you on these five patches aswell, so if there > are any concerns left regarding the changes proposed by these patches, > please raise them. It should be the reverse: Ralph/Manfred, When Daniel submits a version 2 of this patch series doing the changes requested by Greg, please reply with "Acked-by: your name " in order to clearly state that you're OK with such changes. Thanks, Mauro
Re: [PATCH 0/5] SPDX license identifiers in all DD drivers
On Tue, Mar 20, 2018 at 10:01:27PM +0100, Daniel Scheller wrote: > From: Daniel Scheller > > This series adds SPDX license identifiers to all source files which are > copyright by either Digital Devices GmbH or Metzlerbros GbR, who are > the original authors of the ddbridge, ngene, cxd2099, mxl5xx, stv0910 > and stv6111 bridge/demod/tuner drivers, with the mxl5xx driver being > based on source code released by MaxLinear. > > All source code either carries the license text "redistribute and/or > modify it under the terms of the GNU GPL version 2 only as published > by the FSF", or simply "... GPL version 2" in the case of the mxl5xx > driver, which all should equal to the SPDX License Identifier > "GPL-2.0-only" as of SPDX License List Version 3.0 published on > December the 28th, 2017, which is applied as license tag to all files > of the mentioned drivers by this series. > > During checking of those modules I noticed that the module info carries > the "GPL" version tag, which, according to include/linux/module.h, equals > to "GPL version 2 or later", which (I believe) in turn is a mismatch to > what is written in the file header's license boilerplates. This series > corrects this by setting all MODULE_LICENSE() descriptors to "GPL v2", > which equals to the "GNU GPL version 2 only" phrase. > > Besides that, this fixes some whitespace cosmetics in the headers, and > removes the link to gnu.org (if existing), which points to the GPLv3 > license anyway. > > The original intention was to fully replace all the licensing headers > with only the SPDX License Identifiers as it is done in a lot of other > in-tree drivers nowadays. However, Digital Devices disagreed to do this > and expressed major concerns regarding this, in that a machine readable > license tag instead of a full license boilerplate won't hold up equally, > so we agreed to keep the license boilerplate text as is right now. That's really odd, who at that company can I talk to about this? Or really, what lawyer at that company can I point my lawyer at to talk about this, that's the only way this is going to get resolved. If it helps, _ALL_ of the major companies that are kernel developers are onboard with the removal of the crazy boiler-plate text, so this tiny holdout should be easy to resolve. > Greg, I'm Cc'ing you on this due to the last paragraph, as AFAIK you're > one of the initiators of the SPDX tagging initiative, and you even added > tags to 10k+ files all over the tree :-) so we maybe can discuss this > further, also with DD, in the hopes you're fine with this - sorry in > advance if not. See my review of your first patch here, this needs to be done a lot differently... thanks, greg k-h
[PATCH 0/5] SPDX license identifiers in all DD drivers
From: Daniel Scheller This series adds SPDX license identifiers to all source files which are copyright by either Digital Devices GmbH or Metzlerbros GbR, who are the original authors of the ddbridge, ngene, cxd2099, mxl5xx, stv0910 and stv6111 bridge/demod/tuner drivers, with the mxl5xx driver being based on source code released by MaxLinear. All source code either carries the license text "redistribute and/or modify it under the terms of the GNU GPL version 2 only as published by the FSF", or simply "... GPL version 2" in the case of the mxl5xx driver, which all should equal to the SPDX License Identifier "GPL-2.0-only" as of SPDX License List Version 3.0 published on December the 28th, 2017, which is applied as license tag to all files of the mentioned drivers by this series. During checking of those modules I noticed that the module info carries the "GPL" version tag, which, according to include/linux/module.h, equals to "GPL version 2 or later", which (I believe) in turn is a mismatch to what is written in the file header's license boilerplates. This series corrects this by setting all MODULE_LICENSE() descriptors to "GPL v2", which equals to the "GNU GPL version 2 only" phrase. Besides that, this fixes some whitespace cosmetics in the headers, and removes the link to gnu.org (if existing), which points to the GPLv3 license anyway. The original intention was to fully replace all the licensing headers with only the SPDX License Identifiers as it is done in a lot of other in-tree drivers nowadays. However, Digital Devices disagreed to do this and expressed major concerns regarding this, in that a machine readable license tag instead of a full license boilerplate won't hold up equally, so we agreed to keep the license boilerplate text as is right now. Greg, I'm Cc'ing you on this due to the last paragraph, as AFAIK you're one of the initiators of the SPDX tagging initiative, and you even added tags to 10k+ files all over the tree :-) so we maybe can discuss this further, also with DD, in the hopes you're fine with this - sorry in advance if not. Ralph, Manfred, I'm Cc'ing you on these five patches aswell, so if there are any concerns left regarding the changes proposed by these patches, please raise them. Daniel Scheller (5): [media] stv0910/stv6111: add SPDX license headers [media] dvb-frontends/mxl5xx: add SPDX license headers [media] dvb-frontends/cxd2099: add SPDX license headers [media] ddbridge: add SPDX license headers [media] ngene: add SPDX license headers drivers/media/dvb-frontends/cxd2099.c | 5 +++-- drivers/media/dvb-frontends/cxd2099.h | 3 ++- drivers/media/dvb-frontends/mxl5xx.c | 6 +++--- drivers/media/dvb-frontends/mxl5xx.h | 13 + drivers/media/dvb-frontends/mxl5xx_defs.h | 1 + drivers/media/dvb-frontends/mxl5xx_regs.h | 4 ++-- drivers/media/dvb-frontends/stv0910.c | 5 +++-- drivers/media/dvb-frontends/stv0910.h | 9 + drivers/media/dvb-frontends/stv6111.c | 6 +++--- drivers/media/dvb-frontends/stv6111.h | 7 +++ drivers/media/pci/ddbridge/Makefile| 2 +- drivers/media/pci/ddbridge/ddbridge-ci.c | 6 ++ drivers/media/pci/ddbridge/ddbridge-ci.h | 6 ++ drivers/media/pci/ddbridge/ddbridge-core.c | 8 ++-- drivers/media/pci/ddbridge/ddbridge-hw.c | 4 ++-- drivers/media/pci/ddbridge/ddbridge-hw.h | 4 ++-- drivers/media/pci/ddbridge/ddbridge-i2c.c | 4 ++-- drivers/media/pci/ddbridge/ddbridge-i2c.h | 4 ++-- drivers/media/pci/ddbridge/ddbridge-io.h | 4 ++-- drivers/media/pci/ddbridge/ddbridge-main.c | 8 drivers/media/pci/ddbridge/ddbridge-max.c | 4 ++-- drivers/media/pci/ddbridge/ddbridge-max.h | 4 ++-- drivers/media/pci/ddbridge/ddbridge-regs.h | 7 ++- drivers/media/pci/ddbridge/ddbridge.h | 7 ++- drivers/media/pci/ngene/Makefile | 2 +- drivers/media/pci/ngene/ngene-cards.c | 10 +++--- drivers/media/pci/ngene/ngene-core.c | 8 ++-- drivers/media/pci/ngene/ngene-dvb.c| 8 ++-- drivers/media/pci/ngene/ngene-i2c.c| 8 ++-- drivers/media/pci/ngene/ngene.h| 7 ++- 30 files changed, 87 insertions(+), 87 deletions(-) -- 2.16.1