Message ID | 87o9ggdd94.wl-kuninori.morimoto.gx@renesas.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Hi Kuninori-san, On Mon, Jun 11, 2018 at 10:11 PM, Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> wrote: > > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > --- > sound/soc/sh/fsi.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/sound/soc/sh/fsi.c b/sound/soc/sh/fsi.c > index 3bae06dd121f..40bafba2f3b3 100644 > --- a/sound/soc/sh/fsi.c > +++ b/sound/soc/sh/fsi.c > @@ -1,3 +1,4 @@ > +// SPDX-License-Identifier: GPL-2.0 > /* > * Fifo-attached Serial Interface (FSI) support for SH7724 > * > @@ -6,10 +7,6 @@ > * > * Based on ssi.c > * Copyright (c) 2007 Manuel Lauss <mano@roarinelk.homelinux.net> > - * > - * 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. > */ Mark prefers that the entire block is converted to // style: // SPDX-License-Identifier: GPL-2.0 // // Fifo-attached Serial Interface (FSI) support for SH7724 // // Copyright (C) 2009 Renesas Solutions Corp. // Kuninori Morimoto <morimoto.kuninori@renesas.com> // // Based on ssi.c // Copyright (c) 2007 Manuel Lauss <mano@roarinelk.homelinux.net>
Hi Fabio > Mark prefers that the entire block is converted to // style: > > // SPDX-License-Identifier: GPL-2.0 > // > // Fifo-attached Serial Interface (FSI) support for SH7724 > // > // Copyright (C) 2009 Renesas Solutions Corp. > // Kuninori Morimoto <morimoto.kuninori@renesas.com> > // > // Based on ssi.c > // Copyright (c) 2007 Manuel Lauss <mano@roarinelk.homelinux.net> Thanks. will post v2
On Tue, 12 Jun 2018 03:15:05 +0200, Fabio Estevam wrote: > > Hi Kuninori-san, > > On Mon, Jun 11, 2018 at 10:11 PM, Kuninori Morimoto > <kuninori.morimoto.gx@renesas.com> wrote: > > > > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > --- > > sound/soc/sh/fsi.c | 5 +---- > > 1 file changed, 1 insertion(+), 4 deletions(-) > > > > diff --git a/sound/soc/sh/fsi.c b/sound/soc/sh/fsi.c > > index 3bae06dd121f..40bafba2f3b3 100644 > > --- a/sound/soc/sh/fsi.c > > +++ b/sound/soc/sh/fsi.c > > @@ -1,3 +1,4 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > /* > > * Fifo-attached Serial Interface (FSI) support for SH7724 > > * > > @@ -6,10 +7,6 @@ > > * > > * Based on ssi.c > > * Copyright (c) 2007 Manuel Lauss <mano@roarinelk.homelinux.net> > > - * > > - * 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. > > */ > > Mark prefers that the entire block is converted to // style: > > // SPDX-License-Identifier: GPL-2.0 > // > // Fifo-attached Serial Interface (FSI) support for SH7724 > // > // Copyright (C) 2009 Renesas Solutions Corp. > // Kuninori Morimoto <morimoto.kuninori@renesas.com> > // > // Based on ssi.c > // Copyright (c) 2007 Manuel Lauss <mano@roarinelk.homelinux.net> ... is this really the consensus as the preferred style in the whole Linux kernel tree? To be honest, I find it rather annoying. It's a food for bikeshed, and in general I'd postpone such changes unless we really have to convert. thanks, Takashi
On Tue, Jun 12, 2018 at 3:53 AM, Takashi Iwai <tiwai@suse.de> wrote: > ... is this really the consensus as the preferred style in the whole > Linux kernel tree? To be honest, I find it rather annoying. > > It's a food for bikeshed, and in general I'd postpone such changes > unless we really have to convert. Putting everything as // is Linus Torvalds' preferred style: https://lkml.org/lkml/2017/11/25/133 Regards, Fabio Estevam
On Tue, Jun 12, 2018 at 08:53:29AM +0200, Takashi Iwai wrote: > Fabio Estevam wrote: > > Hi Kuninori-san, > > // SPDX-License-Identifier: GPL-2.0 > > // > > // Fifo-attached Serial Interface (FSI) support for SH7724 > ... is this really the consensus as the preferred style in the whole > Linux kernel tree? To be honest, I find it rather annoying. I'm not a big fan either but at least the first line absolutely needs to be a C++ comment, this has been decided. > It's a food for bikeshed, and in general I'd postpone such changes > unless we really have to convert. Everything is supposed to move to the C++ style comments.
On Tue, 12 Jun 2018 16:07:25 +0200, Mark Brown wrote: > > On Tue, Jun 12, 2018 at 08:53:29AM +0200, Takashi Iwai wrote: > > Fabio Estevam wrote: > > > Hi Kuninori-san, > > > > // SPDX-License-Identifier: GPL-2.0 > > > // > > > // Fifo-attached Serial Interface (FSI) support for SH7724 > > > ... is this really the consensus as the preferred style in the whole > > Linux kernel tree? To be honest, I find it rather annoying. > > I'm not a big fan either but at least the first line absolutely needs to > be a C++ comment, this has been decided. > > > It's a food for bikeshed, and in general I'd postpone such changes > > unless we really have to convert. > > Everything is supposed to move to the C++ style comments. Fair enough, let's go ahead in that high way. thanks, Takashi
diff --git a/sound/soc/sh/fsi.c b/sound/soc/sh/fsi.c index 3bae06dd121f..40bafba2f3b3 100644 --- a/sound/soc/sh/fsi.c +++ b/sound/soc/sh/fsi.c @@ -1,3 +1,4 @@ +// SPDX-License-Identifier: GPL-2.0 /* * Fifo-attached Serial Interface (FSI) support for SH7724 * @@ -6,10 +7,6 @@ * * Based on ssi.c * Copyright (c) 2007 Manuel Lauss <mano@roarinelk.homelinux.net> - * - * 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. */ #include <linux/delay.h>