Re: [PATCH] spi: s3c64xx: add SPDX identifier

2017-12-13 Thread Mark Brown
On Wed, Dec 13, 2017 at 10:48:54AM +0900, Andi Shyti wrote: > > > +// > > > +// Copyright (c) 2009 Samsung Electronics Co., Ltd. > > > +// Jaswinder Singh > it just looks ugly to me to have something like > // SPDX... > /* >* Copyright (c)... >* Jasw... >*/ > I would

Re: [PATCH] spi: s3c64xx: add SPDX identifier

2017-12-12 Thread Krzysztof Kozlowski
On Wed, Dec 13, 2017 at 2:48 AM, Andi Shyti wrote: > Hi Krzysztof, > >> > Here, this should be as a top line: >> > >> > // SPDX-License-Identifier: GPL-2.0+ >> > >> > >> > So I agree with the SPDX id but also pointing to the use of the C++ // >> > comment style as requested by Linus [1] >> >> Than

Re: [PATCH] spi: s3c64xx: add SPDX identifier

2017-12-12 Thread Andi Shyti
Hi Krzysztof, > > Here, this should be as a top line: > > > > // SPDX-License-Identifier: GPL-2.0+ > > > > > > So I agree with the SPDX id but also pointing to the use of the C++ // > > comment style as requested by Linus [1] > > Thanks for the reference. I see that Linus prefers converting entir

Re: [PATCH] spi: s3c64xx: add SPDX identifier

2017-12-12 Thread Krzysztof Kozlowski
On Tue, Dec 12, 2017 at 4:24 PM, Philippe Ombredanne wrote: > Dear Krzysztof, > > On Tue, Dec 12, 2017 at 4:00 PM, Krzysztof Kozlowski wrote: >> On Tue, Dec 12, 2017 at 3:48 PM, Philippe Ombredanne >> wrote: >>> On Tue, Dec 12, 2017 at 2:45 PM, Krzysztof Kozlowski >>> wrote: On Tue, Dec 1

Re: [PATCH] spi: s3c64xx: add SPDX identifier

2017-12-12 Thread Andi Shyti
Hi Krzysztof, > > - * Copyright (C) 2009 Samsung Electronics Ltd. > > - * Jaswinder Singh > > - * > > - * 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; eit

Re: [PATCH] spi: s3c64xx: add SPDX identifier

2017-12-12 Thread Philippe Ombredanne
Dear Krzysztof, On Tue, Dec 12, 2017 at 4:00 PM, Krzysztof Kozlowski wrote: > On Tue, Dec 12, 2017 at 3:48 PM, Philippe Ombredanne > wrote: >> On Tue, Dec 12, 2017 at 2:45 PM, Krzysztof Kozlowski wrote: >>> On Tue, Dec 12, 2017 at 2:03 PM, Andi Shyti wrote: Hi Krzysztof, > > - *

Re: [PATCH] spi: s3c64xx: add SPDX identifier

2017-12-12 Thread Krzysztof Kozlowski
On Tue, Dec 12, 2017 at 3:48 PM, Philippe Ombredanne wrote: > On Tue, Dec 12, 2017 at 2:45 PM, Krzysztof Kozlowski wrote: >> On Tue, Dec 12, 2017 at 2:03 PM, Andi Shyti wrote: >>> Hi Krzysztof, >>> > - * Copyright (C) 2009 Samsung Electronics Ltd. > - * Jaswinder Singh > - *

Re: [PATCH] spi: s3c64xx: add SPDX identifier

2017-12-12 Thread Philippe Ombredanne
On Tue, Dec 12, 2017 at 2:45 PM, Krzysztof Kozlowski wrote: > On Tue, Dec 12, 2017 at 2:03 PM, Andi Shyti wrote: >> Hi Krzysztof, >> >>> > - * Copyright (C) 2009 Samsung Electronics Ltd. >>> > - * Jaswinder Singh >>> > - * >>> > - * This program is free software; you can redistribute it and/

Re: [PATCH] spi: s3c64xx: add SPDX identifier

2017-12-12 Thread Krzysztof Kozlowski
On Tue, Dec 12, 2017 at 2:03 PM, Andi Shyti wrote: > Hi Krzysztof, > >> > - * Copyright (C) 2009 Samsung Electronics Ltd. >> > - * Jaswinder Singh >> > - * >> > - * This program is free software; you can redistribute it and/or modify >> > - * it under the terms of the GNU General Public Licen

Re: [PATCH] spi: s3c64xx: add SPDX identifier

2017-12-12 Thread Mark Brown
On Tue, Dec 12, 2017 at 11:13:26AM +0100, Krzysztof Kozlowski wrote: > On Tue, Dec 12, 2017 at 11:07 AM, Mark Brown wrote: > > Apparently that's the thing for SPDX headers. > Yes (although in sources one can find /* */ as well) but the rest of > comment does not have to be changed in that case.

Re: [PATCH] spi: s3c64xx: add SPDX identifier

2017-12-12 Thread Krzysztof Kozlowski
On Tue, Dec 12, 2017 at 11:07 AM, Mark Brown wrote: > On Tue, Dec 12, 2017 at 10:17:09AM +0100, Krzysztof Kozlowski wrote: >> On Tue, Dec 12, 2017 at 8:58 AM, Andi Shyti wrote: > >> > +// SPDX-License-Identifier: GPL-2.0 > >> Why changing the comment style? > > Apparently that's the thing for SPD

Re: [PATCH] spi: s3c64xx: add SPDX identifier

2017-12-12 Thread Mark Brown
On Tue, Dec 12, 2017 at 10:17:09AM +0100, Krzysztof Kozlowski wrote: > On Tue, Dec 12, 2017 at 8:58 AM, Andi Shyti wrote: > > +// SPDX-License-Identifier: GPL-2.0 > Why changing the comment style? Apparently that's the thing for SPDX headers. signature.asc Description: PGP signature

Re: [PATCH] spi: s3c64xx: add SPDX identifier

2017-12-12 Thread Krzysztof Kozlowski
On Tue, Dec 12, 2017 at 8:58 AM, Andi Shyti wrote: > Replace the original license statement with the SPDX identifier. > > Signed-off-by: Andi Shyti > --- > drivers/spi/spi-s3c64xx.c | 18 -- > include/linux/platform_data/spi-s3c64xx.h | 4 +--- > 2 files changed,

[PATCH] spi: s3c64xx: add SPDX identifier

2017-12-11 Thread Andi Shyti
Replace the original license statement with the SPDX identifier. Signed-off-by: Andi Shyti --- drivers/spi/spi-s3c64xx.c | 18 -- include/linux/platform_data/spi-s3c64xx.h | 4 +--- 2 files changed, 5 insertions(+), 17 deletions(-) diff --git a/drivers/spi/spi-s