On Tue, Jun 04, 2024 at 05:46:14PM +0300, Dan Carpenter wrote: > On Tue, Jun 04, 2024 at 12:56:09PM +0100, Mark Brown wrote: > > Is anyone getting any value from these MODULE_DESCRIPTION()s? This all > > just seems like a huge amount of noise and I'm having trouble thinking > > of a use case. > The missing MODULE_DESCRIPTION() warnings are very annoying. We > recently missed a link error issue because the warning was drowned out > in MODULE_DESCRIPTION() warnings. > https://lore.kernel.org/all/202405182038.ncf1mL7Z-lkp@xxxxxxxxx/ Yeah, adding the warning seems premature without having first added the descriptions and I'm not clear in what situation anyone would actually care given how rare it is for someone to manually interact with module loading. The number of cases where anyone would not just look at the source seems vanishingly small.
Attachment:
signature.asc
Description: PGP signature
- References:
- [PATCH] ASoC: qcom: add missing MODULE_DESCRIPTION() macro
- From: Jeff Johnson
- Re: [PATCH] ASoC: qcom: add missing MODULE_DESCRIPTION() macro
- From: Mark Brown
- Re: [PATCH] ASoC: qcom: add missing MODULE_DESCRIPTION() macro
- From: Dan Carpenter
- [PATCH] ASoC: qcom: add missing MODULE_DESCRIPTION() macro
- Prev by Date: Re: [PATCH 1/2] ASoC: dt-bindings: tlv320adc3xxx: Add MICBIAS-as-GPO properties
- Next by Date: Re: [PATCH v1 1/1] treewide: Align match_string() with sysfs_match_string()
- Previous by thread: Re: [PATCH] ASoC: qcom: add missing MODULE_DESCRIPTION() macro
- Next by thread: Re: [PATCH] ASoC: qcom: add missing MODULE_DESCRIPTION() macro
- Index(es):