* [PATCH] Input: stmfts,s6sy671 - add SPDX identifier [not found] <CGME20171212074156epcas2p4fefff65c3316ec1aa9b3d7797aaca550@epcas2p4.samsung.com> @ 2017-12-12 7:41 ` Andi Shyti 2018-01-05 9:57 ` Andi Shyti 0 siblings, 1 reply; 5+ messages in thread From: Andi Shyti @ 2017-12-12 7:41 UTC (permalink / raw) To: Dmitry Torokhov; +Cc: linux-input, linux-kernel, Andi Shyti, Andi Shyti Replace the original license statement with the SPDX identifier. Update also the copyright owner adding myself as co-owner of the copyright. Signed-off-by: Andi Shyti <andi.shyti@samsung.com> --- drivers/input/touchscreen/s6sy761.c | 15 +++++---------- drivers/input/touchscreen/stmfts.c | 15 +++++---------- 2 files changed, 10 insertions(+), 20 deletions(-) diff --git a/drivers/input/touchscreen/s6sy761.c b/drivers/input/touchscreen/s6sy761.c index 26b1cb8a88ec..675efa93d444 100644 --- a/drivers/input/touchscreen/s6sy761.c +++ b/drivers/input/touchscreen/s6sy761.c @@ -1,13 +1,8 @@ -/* - * Copyright (c) 2017 Samsung Electronics Co., Ltd. - * Author: Andi Shyti <andi.shyti@samsung.com> - * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License version 2 as - * published by the Free Software Foundation. - * - * Samsung S6SY761 Touchscreen device driver - */ +// SPDX-License-Identifier: GPL-2.0 +// Samsung S6SY761 Touchscreen device driver +// +// Copyright (c) 2017 Samsung Electronics Co., Ltd. +// Copyright (c) 2017 Andi Shyti <andi.shyti@samsung.com> #include <asm/unaligned.h> #include <linux/delay.h> diff --git a/drivers/input/touchscreen/stmfts.c b/drivers/input/touchscreen/stmfts.c index c12d01899939..2a123e20a42e 100644 --- a/drivers/input/touchscreen/stmfts.c +++ b/drivers/input/touchscreen/stmfts.c @@ -1,13 +1,8 @@ -/* - * Copyright (c) 2017 Samsung Electronics Co., Ltd. - * Author: Andi Shyti <andi.shyti@samsung.com> - * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License version 2 as - * published by the Free Software Foundation. - * - * STMicroelectronics FTS Touchscreen device driver - */ +// SPDX-License-Identifier: GPL-2.0 +// STMicroelectronics FTS Touchscreen device driver +// +// Copyright (c) 2017 Samsung Electronics Co., Ltd. +// Copyright (c) 2017 Andi Shyti <andi.shyti@samsung.com> #include <linux/delay.h> #include <linux/i2c.h> -- 2.15.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] Input: stmfts,s6sy671 - add SPDX identifier 2017-12-12 7:41 ` [PATCH] Input: stmfts,s6sy671 - add SPDX identifier Andi Shyti @ 2018-01-05 9:57 ` Andi Shyti 2018-01-05 16:49 ` Dmitry Torokhov 0 siblings, 1 reply; 5+ messages in thread From: Andi Shyti @ 2018-01-05 9:57 UTC (permalink / raw) To: Dmitry Torokhov; +Cc: linux-input, linux-kernel, Andi Shyti Hi Dmitry, this is a kind ping, would you also mind giving me a feedback to all the previous patches I sent? Thanks, Andi On Tue, Dec 12, 2017 at 04:41:49PM +0900, Andi Shyti wrote: > Replace the original license statement with the SPDX identifier. > > Update also the copyright owner adding myself as co-owner of the > copyright. > > Signed-off-by: Andi Shyti <andi.shyti@samsung.com> > --- > drivers/input/touchscreen/s6sy761.c | 15 +++++---------- > drivers/input/touchscreen/stmfts.c | 15 +++++---------- > 2 files changed, 10 insertions(+), 20 deletions(-) > > diff --git a/drivers/input/touchscreen/s6sy761.c b/drivers/input/touchscreen/s6sy761.c > index 26b1cb8a88ec..675efa93d444 100644 > --- a/drivers/input/touchscreen/s6sy761.c > +++ b/drivers/input/touchscreen/s6sy761.c > @@ -1,13 +1,8 @@ > -/* > - * Copyright (c) 2017 Samsung Electronics Co., Ltd. > - * Author: Andi Shyti <andi.shyti@samsung.com> > - * > - * This program is free software; you can redistribute it and/or modify > - * it under the terms of the GNU General Public License version 2 as > - * published by the Free Software Foundation. > - * > - * Samsung S6SY761 Touchscreen device driver > - */ > +// SPDX-License-Identifier: GPL-2.0 > +// Samsung S6SY761 Touchscreen device driver > +// > +// Copyright (c) 2017 Samsung Electronics Co., Ltd. > +// Copyright (c) 2017 Andi Shyti <andi.shyti@samsung.com> > > #include <asm/unaligned.h> > #include <linux/delay.h> > diff --git a/drivers/input/touchscreen/stmfts.c b/drivers/input/touchscreen/stmfts.c > index c12d01899939..2a123e20a42e 100644 > --- a/drivers/input/touchscreen/stmfts.c > +++ b/drivers/input/touchscreen/stmfts.c > @@ -1,13 +1,8 @@ > -/* > - * Copyright (c) 2017 Samsung Electronics Co., Ltd. > - * Author: Andi Shyti <andi.shyti@samsung.com> > - * > - * This program is free software; you can redistribute it and/or modify > - * it under the terms of the GNU General Public License version 2 as > - * published by the Free Software Foundation. > - * > - * STMicroelectronics FTS Touchscreen device driver > - */ > +// SPDX-License-Identifier: GPL-2.0 > +// STMicroelectronics FTS Touchscreen device driver > +// > +// Copyright (c) 2017 Samsung Electronics Co., Ltd. > +// Copyright (c) 2017 Andi Shyti <andi.shyti@samsung.com> > > #include <linux/delay.h> > #include <linux/i2c.h> > -- > 2.15.1 > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] Input: stmfts,s6sy671 - add SPDX identifier 2018-01-05 9:57 ` Andi Shyti @ 2018-01-05 16:49 ` Dmitry Torokhov 2018-01-09 5:23 ` Andi Shyti 0 siblings, 1 reply; 5+ messages in thread From: Dmitry Torokhov @ 2018-01-05 16:49 UTC (permalink / raw) To: Andi Shyti; +Cc: linux-input, linux-kernel, Andi Shyti Hi Andi, On Fri, Jan 05, 2018 at 06:57:15PM +0900, Andi Shyti wrote: > Hi Dmitry, > > this is a kind ping, would you also mind giving me a feedback to Yes, sorry. Could you please split the patch for each driver individually? Also, until we have an update to the CodingStyle doc mandating the C++ style comments, I'd prefer keeping the original style of comments. So // for the SPDX line and /* */ for the rest. Thanks! > all the previous patches I sent? Could you tell me what reviews I owe you? My mailbox keeps growing :( > > Thanks, > Andi > > On Tue, Dec 12, 2017 at 04:41:49PM +0900, Andi Shyti wrote: > > Replace the original license statement with the SPDX identifier. > > > > Update also the copyright owner adding myself as co-owner of the > > copyright. > > > > Signed-off-by: Andi Shyti <andi.shyti@samsung.com> > > --- > > drivers/input/touchscreen/s6sy761.c | 15 +++++---------- > > drivers/input/touchscreen/stmfts.c | 15 +++++---------- > > 2 files changed, 10 insertions(+), 20 deletions(-) > > > > diff --git a/drivers/input/touchscreen/s6sy761.c b/drivers/input/touchscreen/s6sy761.c > > index 26b1cb8a88ec..675efa93d444 100644 > > --- a/drivers/input/touchscreen/s6sy761.c > > +++ b/drivers/input/touchscreen/s6sy761.c > > @@ -1,13 +1,8 @@ > > -/* > > - * Copyright (c) 2017 Samsung Electronics Co., Ltd. > > - * Author: Andi Shyti <andi.shyti@samsung.com> > > - * > > - * This program is free software; you can redistribute it and/or modify > > - * it under the terms of the GNU General Public License version 2 as > > - * published by the Free Software Foundation. > > - * > > - * Samsung S6SY761 Touchscreen device driver > > - */ > > +// SPDX-License-Identifier: GPL-2.0 > > +// Samsung S6SY761 Touchscreen device driver > > +// > > +// Copyright (c) 2017 Samsung Electronics Co., Ltd. > > +// Copyright (c) 2017 Andi Shyti <andi.shyti@samsung.com> > > > > #include <asm/unaligned.h> > > #include <linux/delay.h> > > diff --git a/drivers/input/touchscreen/stmfts.c b/drivers/input/touchscreen/stmfts.c > > index c12d01899939..2a123e20a42e 100644 > > --- a/drivers/input/touchscreen/stmfts.c > > +++ b/drivers/input/touchscreen/stmfts.c > > @@ -1,13 +1,8 @@ > > -/* > > - * Copyright (c) 2017 Samsung Electronics Co., Ltd. > > - * Author: Andi Shyti <andi.shyti@samsung.com> > > - * > > - * This program is free software; you can redistribute it and/or modify > > - * it under the terms of the GNU General Public License version 2 as > > - * published by the Free Software Foundation. > > - * > > - * STMicroelectronics FTS Touchscreen device driver > > - */ > > +// SPDX-License-Identifier: GPL-2.0 > > +// STMicroelectronics FTS Touchscreen device driver > > +// > > +// Copyright (c) 2017 Samsung Electronics Co., Ltd. > > +// Copyright (c) 2017 Andi Shyti <andi.shyti@samsung.com> > > > > #include <linux/delay.h> > > #include <linux/i2c.h> > > -- > > 2.15.1 > > -- Dmitry ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] Input: stmfts,s6sy671 - add SPDX identifier 2018-01-05 16:49 ` Dmitry Torokhov @ 2018-01-09 5:23 ` Andi Shyti 2018-01-23 1:41 ` Dmitry Torokhov 0 siblings, 1 reply; 5+ messages in thread From: Andi Shyti @ 2018-01-09 5:23 UTC (permalink / raw) To: Dmitry Torokhov Cc: linux-input, linux-kernel, Andi Shyti, Philippe Ombredanne Hi Dmitry, On Fri, Jan 05, 2018 at 08:49:58AM -0800, Dmitry Torokhov wrote: > Hi Andi, > > On Fri, Jan 05, 2018 at 06:57:15PM +0900, Andi Shyti wrote: > > Hi Dmitry, > > > > this is a kind ping, would you also mind giving me a feedback to > > Yes, sorry. Could you please split the patch for each driver > individually? Also, until we have an update to the CodingStyle doc > mandating the C++ style comments, I'd prefer keeping the original style > of comments. So // for the SPDX line and /* */ for the rest. I was actually following Linus guideline [1] [2]. I also had the same discussion previously in another context [3]. Please let me know if you still want a mixed style of comments in the next patch. Thanks, Andi [1] https://marc.info/?l=linux-kernel&m=151163713125320&w=2 ... └-> https://marc.info/?l=linux-kernel&m=151163867325641&w=2 ... └-> https://marc.info/?l=linux-kernel&m=151163867325641&w=2 [2] https://marc.info/?l=linux-kernel&m=150964359922353 [3] https://marc.info/?l=linux-kernel&m=151312974503109&w=2 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] Input: stmfts,s6sy671 - add SPDX identifier 2018-01-09 5:23 ` Andi Shyti @ 2018-01-23 1:41 ` Dmitry Torokhov 0 siblings, 0 replies; 5+ messages in thread From: Dmitry Torokhov @ 2018-01-23 1:41 UTC (permalink / raw) To: Andi Shyti; +Cc: linux-input, linux-kernel, Andi Shyti, Philippe Ombredanne On Tue, Jan 09, 2018 at 02:23:31PM +0900, Andi Shyti wrote: > Hi Dmitry, > > On Fri, Jan 05, 2018 at 08:49:58AM -0800, Dmitry Torokhov wrote: > > Hi Andi, > > > > On Fri, Jan 05, 2018 at 06:57:15PM +0900, Andi Shyti wrote: > > > Hi Dmitry, > > > > > > this is a kind ping, would you also mind giving me a feedback to > > > > Yes, sorry. Could you please split the patch for each driver > > individually? Also, until we have an update to the CodingStyle doc > > mandating the C++ style comments, I'd prefer keeping the original style > > of comments. So // for the SPDX line and /* */ for the rest. > > I was actually following Linus guideline [1] [2]. I also had the > same discussion previously in another context [3]. > > Please let me know if you still want a mixed style of comments in > the next patch. Eh, I guess it does not matter in the grand scheme of things; applied as is. Thank you. > > Thanks, > Andi > > [1] https://marc.info/?l=linux-kernel&m=151163713125320&w=2 > ... > └-> https://marc.info/?l=linux-kernel&m=151163867325641&w=2 > ... > └-> https://marc.info/?l=linux-kernel&m=151163867325641&w=2 > > [2] https://marc.info/?l=linux-kernel&m=150964359922353 > [3] https://marc.info/?l=linux-kernel&m=151312974503109&w=2 -- Dmitry ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-01-23 1:41 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20171212074156epcas2p4fefff65c3316ec1aa9b3d7797aaca550@epcas2p4.samsung.com>
2017-12-12 7:41 ` [PATCH] Input: stmfts,s6sy671 - add SPDX identifier Andi Shyti
2018-01-05 9:57 ` Andi Shyti
2018-01-05 16:49 ` Dmitry Torokhov
2018-01-09 5:23 ` Andi Shyti
2018-01-23 1:41 ` Dmitry Torokhov
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).