Re: [PATCH V3 2/2] ASoC: codecs: Add aw88083 amplifier driver

[Luca Ceresoli <luca.ceresoli@xxxxxxxxxxx>]

Hello Weidong,

On Tue, 24 Dec 2024 11:18:23 +0800
wangweidong.a@xxxxxxxxxx wrote:

> From: Weidong Wang <wangweidong.a@xxxxxxxxxx>
> 
> The driver is for amplifiers aw88083 of Awinic Technology
> Corporation. The AW88083 is an intelligent digital audio
> amplifier with low noise.
> 
> Reported-by: kernel test robot <lkp@xxxxxxxxx>
> Closes: https://lore.kernel.org/oe-kbuild-all/202412201745.fBpf3Ui5-lkp@xxxxxxxxx/

You should not add these two lines. From the kernel test robot report:

> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <lkp@xxxxxxxxx>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202412201745.fBpf3Ui5-lkp@xxxxxxxxx/

If you think about someone looking at git log in the future, they will
see the final version of your patch, they will never see v2. So the
final version (v3, v4, whatever) is not fixing any error as the error in
v2 will never be applied.

Luca

-- 
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com