On Mon, Jul 22, 2019 at 02:07:47AM +0000, Shuming [范書銘] wrote:
On Fri, Jul 19, 2019 at 02:33:02PM +0800, shumingf@realtek.com 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@realtek.com - * - */ +// +// rt1308.c -- RT1308 ALSA SoC amplifier component driver +// +// Copyright 2019 Realtek Semiconductor Corp. +// Author: Derek Fang derek.fang@realtek.com +//