Messages in this thread | | | From | Clément Péron <> | Date | Thu, 15 Nov 2018 17:31:44 +0100 | Subject | Re: [PATCH 1/2] ASoC: ak4118: Add support for AK4118 S/PDIF transceiver |
| |
Hi Mark,
On Wed, 14 Nov 2018 at 18:31, Mark Brown <broonie@kernel.org> wrote: > > On Wed, Nov 14, 2018 at 11:55:17AM +0100, Clément Péron wrote: > > On Tue, 13 Nov 2018 at 23:51, Mark Brown <broonie@kernel.org> wrote: > > > On Thu, Nov 08, 2018 at 01:49:33PM +0100, Clément Péron wrote: > > > > > +// SPDX-License-Identifier: GPL-2.0 > > > > +/* > > > > + * ak4118.c -- Asahi Kasei ALSA Soc Audio driver > > > > + * > > > > Please make the entire comment a C++ one so this looks more intentional. > > > I check from other files and also this article says : > > "For normal C source files, the string will be a comment using the > > "//" syntax; header files, instead, use traditional (/* */) comments > > for reasons related to tooling" > > https://lwn.net/Articles/739183/ > > Right, and what I said was that you should make the entire comment a C++ > one which means that the magic line will be C++ as required.
Yes, sorry I was confused between C++ and C99, Regards, Clement
| |