sound/oss/dmasound/dmasound_core.c | 2 ++ 1 file changed, 2 insertions(+)
We observed: 'dmasound_setup' defined but not used error with
COMPILER=gcc ARCH=m68k DEFCONFIG=allmodconfig build.
__setup() does not work if MODULE is defined.
Fix it by warpping dmasound_setup with #ifndef MODULES.
Error(s):
sound/oss/dmasound/dmasound_core.c:1431:12: error: 'dmasound_setup' defined but not used [-Werror=unused-function]
Signed-off-by: Miles Chen <miles.chen@mediatek.com>
---
sound/oss/dmasound/dmasound_core.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/sound/oss/dmasound/dmasound_core.c b/sound/oss/dmasound/dmasound_core.c
index 9c48f3a9e3d1..a1b3e71beadf 100644
--- a/sound/oss/dmasound/dmasound_core.c
+++ b/sound/oss/dmasound/dmasound_core.c
@@ -1428,6 +1428,7 @@ void dmasound_deinit(void)
unregister_sound_dsp(sq_unit);
}
+#ifndef MODULE
static int dmasound_setup(char *str)
{
int ints[6], size;
@@ -1470,6 +1471,7 @@ static int dmasound_setup(char *str)
}
__setup("dmasound=", dmasound_setup);
+#endif
/*
* Conversion tables
--
2.18.0
Il 14/04/22 10:11, Miles Chen ha scritto: > We observed: 'dmasound_setup' defined but not used error with > COMPILER=gcc ARCH=m68k DEFCONFIG=allmodconfig build. > > __setup() does not work if MODULE is defined. > Fix it by warpping dmasound_setup with #ifndef MODULES. > > Error(s): > sound/oss/dmasound/dmasound_core.c:1431:12: error: 'dmasound_setup' defined but not used [-Werror=unused-function] > > Signed-off-by: Miles Chen <miles.chen@mediatek.com> > --- > sound/oss/dmasound/dmasound_core.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/sound/oss/dmasound/dmasound_core.c b/sound/oss/dmasound/dmasound_core.c > index 9c48f3a9e3d1..a1b3e71beadf 100644 > --- a/sound/oss/dmasound/dmasound_core.c > +++ b/sound/oss/dmasound/dmasound_core.c > @@ -1428,6 +1428,7 @@ void dmasound_deinit(void) > unregister_sound_dsp(sq_unit); > } > > +#ifndef MODULE > static int dmasound_setup(char *str) Hello Miles, I agree with Takashi, __maybe_unused looks way better. Regards, Angelo
Hi Angelo, > Hello Miles, > I agree with Takashi, __maybe_unused looks way better. No problem, I posted v2 (with __maybe_unused). https://lore.kernel.org/lkml/20220414091940.2216-1-miles.chen@mediatek.com/ Thanks, Miles > > Regards, > Angelo
On Thu, 14 Apr 2022 10:11:18 +0200, Miles Chen wrote: > > We observed: 'dmasound_setup' defined but not used error with > COMPILER=gcc ARCH=m68k DEFCONFIG=allmodconfig build. > > __setup() does not work if MODULE is defined. > Fix it by warpping dmasound_setup with #ifndef MODULES. > > Error(s): > sound/oss/dmasound/dmasound_core.c:1431:12: error: 'dmasound_setup' defined but not used [-Werror=unused-function] > > Signed-off-by: Miles Chen <miles.chen@mediatek.com> This must be a side-effect of the recent fix 9dd7c46346ca ("sound/oss/dmasound: fix build when drivers are mixed =y/=m"). Adding Randy to Cc. IMO, a less uglier way would be to add __maybe_unused to that function. But it's a matter of taste. thanks, Takashi > --- > sound/oss/dmasound/dmasound_core.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/sound/oss/dmasound/dmasound_core.c b/sound/oss/dmasound/dmasound_core.c > index 9c48f3a9e3d1..a1b3e71beadf 100644 > --- a/sound/oss/dmasound/dmasound_core.c > +++ b/sound/oss/dmasound/dmasound_core.c > @@ -1428,6 +1428,7 @@ void dmasound_deinit(void) > unregister_sound_dsp(sq_unit); > } > > +#ifndef MODULE > static int dmasound_setup(char *str) > { > int ints[6], size; > @@ -1470,6 +1471,7 @@ static int dmasound_setup(char *str) > } > > __setup("dmasound=", dmasound_setup); > +#endif > > /* > * Conversion tables > -- > 2.18.0 >
Hi Takashi, > This must be a side-effect of the recent fix 9dd7c46346ca > ("sound/oss/dmasound: fix build when drivers are mixed =y/=m"). > Adding Randy to Cc. > > IMO, a less uglier way would be to add __maybe_unused to that > function. But it's a matter of taste. thanks, I will use __maybe_unused and the fix tag in v2. Thanks, Miles > > > thanks, > > Takashi
© 2016 - 2024 Red Hat, Inc.