mbox series

[0/7] ALSA: Add missing module description

Message ID 20240508091128.25274-1-tiwai@suse.de (mailing list archive)
Headers show
Series ALSA: Add missing module description | expand

Message

Takashi Iwai May 8, 2024, 9:11 a.m. UTC
Hi,

here is a trivial patch series for adding the missing
MODULE_DESCRIPTION() to the code that can be built as modules
where make W=1 starts complaining the lack of MODULE_DESCRIPTION()
since recently.

Fixes for ASoC follow later.


Takashi

===

Takashi Iwai (7):
  ALSA: ac97: legacy: Add missing module description
  ALSA: ac97: bus: Add missing module description
  ALSA: pcm_dmaengine: Add missing module description
  ALSA: kunit: Add missing module descriptions
  ALSA: pcmtest: Add missing module descriptions
  ALSA: hda: cirrus_scodec_test: Add missing module descriptions
  sound: oss: dmasound: Add missing module descriptions

 sound/ac97/bus.c                    | 1 +
 sound/ac97_bus.c                    | 1 +
 sound/core/pcm_dmaengine.c          | 1 +
 sound/core/sound_kunit.c            | 1 +
 sound/drivers/pcmtest.c             | 1 +
 sound/oss/dmasound/dmasound_atari.c | 2 ++
 sound/oss/dmasound/dmasound_paula.c | 1 +
 sound/pci/hda/cirrus_scodec_test.c  | 1 +
 8 files changed, 9 insertions(+)

Comments

Dragan Simic May 8, 2024, 9:17 a.m. UTC | #1
Hello Takashi,

On 2024-05-08 11:11, Takashi Iwai wrote:
> here is a trivial patch series for adding the missing
> MODULE_DESCRIPTION() to the code that can be built as modules
> where make W=1 starts complaining the lack of MODULE_DESCRIPTION()
> since recently.
> 
> Fixes for ASoC follow later.

The changes are looking good to me.  Thus, please add for the
entire series:

Reviewed-by: Dragan Simic <dsimic@manjaro.org>

> ===
> 
> Takashi Iwai (7):
>   ALSA: ac97: legacy: Add missing module description
>   ALSA: ac97: bus: Add missing module description
>   ALSA: pcm_dmaengine: Add missing module description
>   ALSA: kunit: Add missing module descriptions
>   ALSA: pcmtest: Add missing module descriptions
>   ALSA: hda: cirrus_scodec_test: Add missing module descriptions
>   sound: oss: dmasound: Add missing module descriptions
> 
>  sound/ac97/bus.c                    | 1 +
>  sound/ac97_bus.c                    | 1 +
>  sound/core/pcm_dmaengine.c          | 1 +
>  sound/core/sound_kunit.c            | 1 +
>  sound/drivers/pcmtest.c             | 1 +
>  sound/oss/dmasound/dmasound_atari.c | 2 ++
>  sound/oss/dmasound/dmasound_paula.c | 1 +
>  sound/pci/hda/cirrus_scodec_test.c  | 1 +
>  8 files changed, 9 insertions(+)
Andy Shevchenko May 8, 2024, 9:36 a.m. UTC | #2
On Wed, May 08, 2024 at 11:11:19AM +0200, Takashi Iwai wrote:
> Hi,
> 
> here is a trivial patch series for adding the missing
> MODULE_DESCRIPTION() to the code that can be built as modules
> where make W=1 starts complaining the lack of MODULE_DESCRIPTION()
> since recently.
> 
> Fixes for ASoC follow later.

Probably Reported-by: fits better, but I'm fine with either.
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Andy Shevchenko May 8, 2024, 9:45 a.m. UTC | #3
On Wed, May 08, 2024 at 12:36:10PM +0300, Andy Shevchenko wrote:
> On Wed, May 08, 2024 at 11:11:19AM +0200, Takashi Iwai wrote:
> > Hi,
> > 
> > here is a trivial patch series for adding the missing
> > MODULE_DESCRIPTION() to the code that can be built as modules
> > where make W=1 starts complaining the lack of MODULE_DESCRIPTION()
> > since recently.
> > 
> > Fixes for ASoC follow later.
> 
> Probably Reported-by: fits better, but I'm fine with either.
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

In case you decide to switch to Reported-by, replace also Link by Closes tag.
Takashi Iwai May 9, 2024, 8:13 a.m. UTC | #4
On Wed, 08 May 2024 11:45:57 +0200,
Andy Shevchenko wrote:
> 
> On Wed, May 08, 2024 at 12:36:10PM +0300, Andy Shevchenko wrote:
> > On Wed, May 08, 2024 at 11:11:19AM +0200, Takashi Iwai wrote:
> > > Hi,
> > > 
> > > here is a trivial patch series for adding the missing
> > > MODULE_DESCRIPTION() to the code that can be built as modules
> > > where make W=1 starts complaining the lack of MODULE_DESCRIPTION()
> > > since recently.
> > > 
> > > Fixes for ASoC follow later.
> > 
> > Probably Reported-by: fits better, but I'm fine with either.
> > Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> 
> In case you decide to switch to Reported-by, replace also Link by Closes tag.

OK, I updated in that way and applied now.


thanks,

Takashi