Re: [PATCH 3/3] ASoC: rt1308: Convert headers to SPDX

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> On Mon, Jul 22, 2019 at 02:07:47AM +0000, Shuming [范書銘] wrote:
> > > On Fri, Jul 19, 2019 at 02:33:02PM +0800, shumingf@xxxxxxxxxxx wrote:
> 
> > > > @@ -1,12 +1,10 @@
> > > > +// SPDX-License-Identifier: GPL-2.0
> > > >  /*
> > > >   * rt1308.c  --  RT1308 ALSA SoC amplifier component driver
> > > >   *
> 
> > > Please convert the entire comment block to a C++ one so this looks
> > > more intentional.
> 
> > I convert the comment like below.
> > -// SPDX-License-Identifier: GPL-2.0
> > +/* SPDX-License-Identifier: GPL-2.0 */
> 
> That's a C comment, not a C++ comment.  I'm saying convert the whole block
> there to C++ rather than add one random line that's C++ next to some C
> comments.
> 

May I confirm what your point?
I convert the entire comment block to a C++ one like below.
// SPDX-License-Identifier: GPL-2.0
-/*
- * rt1308.c  --  RT1308 ALSA SoC amplifier component driver
- *
- * Copyright 2019 Realtek Semiconductor Corp.
- * Author: Derek Fang <derek.fang@xxxxxxxxxxx>
- *
- */
+//
+// rt1308.c  --  RT1308 ALSA SoC amplifier component driver
+//
+// Copyright 2019 Realtek Semiconductor Corp.
+// Author: Derek Fang <derek.fang@xxxxxxxxxxx>
+//


_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel




[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux