From: Andy Shevchenko <andriy.shevchenko@linux.intel.com> To: Arnd Bergmann <arnd@kernel.org> Cc: linux-kernel@vger.kernel.org, "Liam Girdwood" <lgirdwood@gmail.com>, "Mark Brown" <broonie@kernel.org>, "Jaroslav Kysela" <perex@perex.cz>, "Takashi Iwai" <tiwai@suse.com>, "Claudiu Beznea" <claudiu.beznea@tuxon.dev>, "Nicolas Ferre" <nicolas.ferre@microchip.com>, "Alexandre Belloni" <alexandre.belloni@bootlin.com>, "Oder Chiou" <oder_chiou@realtek.com>, "Srinivas Kandagatla" <srinivas.kandagatla@linaro.org>, "Banajit Goswami" <bgoswami@quicinc.com>, "Sylwester Nawrocki" <s.nawrocki@samsung.com>, "Arnd Bergmann" <arnd@arndb.de>, "Alper Nebi Yasak" <alpernebiyasak@gmail.com>, "Kuninori Morimoto" <kuninori.morimoto.gx@renesas.com>, "Akihiko Odaki" <akihiko.odaki@daynix.com>, "Linus Walleij" <linus.walleij@linaro.org>, "Yinchuan Guo" <guoych37@mail2.sysu.edu.cn>, "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>, "Rob Herring" <robh@kernel.org>, "AngeloGioacchino Del Regno" <angelogioacchino.delregno@collabora.com>, "Krzysztof Kozlowski" <krzysztof.kozlowski@linaro.org>, linux-sound@vger.kernel.org, alsa-devel@alsa-project.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 32/34] ASoC: remove incorrect of_match_ptr/ACPI_PTR annotations Date: Wed, 3 Apr 2024 12:15:13 +0300 [thread overview] Message-ID: <Zg0eIZDS3F_PQLyB@smile.fi.intel.com> (raw) In-Reply-To: <20240403080702.3509288-33-arnd@kernel.org> On Wed, Apr 03, 2024 at 10:06:50AM +0200, Arnd Bergmann wrote: > From: Arnd Bergmann <arnd@arndb.de> > > When building with CONFIG_OF and/or CONFIG_ACPI disabled but W=1 extra > warnings enabled, a lot of driver cause a warning about an unused > ID table: > > sound/soc/atmel/sam9x5_wm8731.c:187:34: error: unused variable 'sam9x5_wm8731_of_match' [-Werror,-Wunused-const-variable] > sound/soc/codecs/rt5514-spi.c:496:34: error: unused variable 'rt5514_of_match' [-Werror,-Wunused-const-variable] > sound/soc/samsung/aries_wm8994.c:524:34: error: unused variable 'samsung_wm8994_of_match' [-Werror,-Wunused-const-variable] > > The fix is always to just remove the of_match_ptr() and ACPI_PTR() wrappers > that remove the reference, rather than adding another #ifdef just for build > testing for a configuration that doesn't matter in practice. ... > index d6cdb6d9fdd6..ffc105759994 100644 > --- a/sound/soc/amd/acp3x-rt5682-max9836.c > +++ b/sound/soc/amd/acp3x-rt5682-max9836.c > @@ -543,7 +543,7 @@ MODULE_DEVICE_TABLE(acpi, acp3x_audio_acpi_match); > static struct platform_driver acp3x_audio = { > .driver = { > .name = "acp3x-alc5682-max98357", > - .acpi_match_table = ACPI_PTR(acp3x_audio_acpi_match), > + .acpi_match_table = acp3x_audio_acpi_match, > .pm = &snd_soc_pm_ops, > }, > .probe = acp3x_probe, Replace acpi_match_device() by device_get_match_data() and acpi.h by mod_devicetable.h + property.h. I really would like to see agnostic drivers (when they don't need of*.h/acpi.h to be included as "proxy" headers). With this, the change probably needs to be separated from this patch. If you address as suggested, Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> ... The rest I haven't checked, it might be possible to do something similar there as well. -- With Best Regards, Andy Shevchenko
WARNING: multiple messages have this Message-ID (diff)
From: Andy Shevchenko <andriy.shevchenko@linux.intel.com> To: Arnd Bergmann <arnd@kernel.org> Cc: "Alexandre Belloni" <alexandre.belloni@bootlin.com>, "Akihiko Odaki" <akihiko.odaki@daynix.com>, "Kuninori Morimoto" <kuninori.morimoto.gx@renesas.com>, "Linus Walleij" <linus.walleij@linaro.org>, alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org, "Srinivas Kandagatla" <srinivas.kandagatla@linaro.org>, "Sylwester Nawrocki" <s.nawrocki@samsung.com>, "Claudiu Beznea" <claudiu.beznea@tuxon.dev>, "Rob Herring" <robh@kernel.org>, "Yinchuan Guo" <guoych37@mail2.sysu.edu.cn>, "Takashi Iwai" <tiwai@suse.com>, "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>, "Arnd Bergmann" <arnd@arndb.de>, linux-sound@vger.kernel.org, "Alper Nebi Yasak" <alpernebiyasak@gmail.com>, "Mark Brown" <broonie@kernel.org>, "Jaroslav Kysela" <perex@perex.cz>, linux-arm-kernel@lists.infradead.org, "AngeloGioacchino Del Regno" <angelogioacchino.delregno@collabora.com>, "Oder Chiou" <oder_chiou@realtek.com>, "Liam Girdwood" <lgirdwood@gmail.com>, "Krzysztof Kozlowski" <krzysztof.kozlowski@linaro.org>, "Banajit Goswami" <bgoswami@quicinc.com> Subject: Re: [PATCH 32/34] ASoC: remove incorrect of_match_ptr/ACPI_PTR annotations Date: Wed, 3 Apr 2024 12:15:13 +0300 [thread overview] Message-ID: <Zg0eIZDS3F_PQLyB@smile.fi.intel.com> (raw) In-Reply-To: <20240403080702.3509288-33-arnd@kernel.org> On Wed, Apr 03, 2024 at 10:06:50AM +0200, Arnd Bergmann wrote: > From: Arnd Bergmann <arnd@arndb.de> > > When building with CONFIG_OF and/or CONFIG_ACPI disabled but W=1 extra > warnings enabled, a lot of driver cause a warning about an unused > ID table: > > sound/soc/atmel/sam9x5_wm8731.c:187:34: error: unused variable 'sam9x5_wm8731_of_match' [-Werror,-Wunused-const-variable] > sound/soc/codecs/rt5514-spi.c:496:34: error: unused variable 'rt5514_of_match' [-Werror,-Wunused-const-variable] > sound/soc/samsung/aries_wm8994.c:524:34: error: unused variable 'samsung_wm8994_of_match' [-Werror,-Wunused-const-variable] > > The fix is always to just remove the of_match_ptr() and ACPI_PTR() wrappers > that remove the reference, rather than adding another #ifdef just for build > testing for a configuration that doesn't matter in practice. ... > index d6cdb6d9fdd6..ffc105759994 100644 > --- a/sound/soc/amd/acp3x-rt5682-max9836.c > +++ b/sound/soc/amd/acp3x-rt5682-max9836.c > @@ -543,7 +543,7 @@ MODULE_DEVICE_TABLE(acpi, acp3x_audio_acpi_match); > static struct platform_driver acp3x_audio = { > .driver = { > .name = "acp3x-alc5682-max98357", > - .acpi_match_table = ACPI_PTR(acp3x_audio_acpi_match), > + .acpi_match_table = acp3x_audio_acpi_match, > .pm = &snd_soc_pm_ops, > }, > .probe = acp3x_probe, Replace acpi_match_device() by device_get_match_data() and acpi.h by mod_devicetable.h + property.h. I really would like to see agnostic drivers (when they don't need of*.h/acpi.h to be included as "proxy" headers). With this, the change probably needs to be separated from this patch. If you address as suggested, Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> ... The rest I haven't checked, it might be possible to do something similar there as well. -- With Best Regards, Andy Shevchenko _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2024-04-03 9:15 UTC|newest] Thread overview: 140+ messages / expand[flat|nested] mbox.gz Atom feed top 2024-04-03 8:06 [PATCH 00/34] address all -Wunused-const warnings Arnd Bergmann 2024-04-03 8:06 ` Arnd Bergmann 2024-04-03 8:06 ` Arnd Bergmann 2024-04-03 8:06 ` [PATCH 01/34] powerpc/fsl-soc: hide unused const variable Arnd Bergmann 2024-04-03 8:06 ` Arnd Bergmann 2024-04-03 8:19 ` Christophe Leroy 2024-04-03 8:19 ` Christophe Leroy 2024-04-03 8:06 ` [PATCH 02/34] ubsan: fix unused variable warning in test module Arnd Bergmann 2024-04-03 21:36 ` (subset) " Kees Cook 2024-04-03 8:06 ` [PATCH 03/34] platform: goldfish: remove ACPI_PTR() annotations Arnd Bergmann 2024-04-03 9:08 ` Andy Shevchenko 2024-04-03 8:06 ` [PATCH 04/34] i2c: pxa: hide unused icr_bits[] variable Arnd Bergmann 2024-04-03 23:17 ` Andi Shyti 2024-04-03 8:06 ` [PATCH 05/34] 3c515: remove unused 'mtu' variable Arnd Bergmann 2024-04-03 8:06 ` [PATCH 06/34] tracing: hide unused ftrace_event_id_fops Arnd Bergmann 2024-04-03 8:06 ` [PATCH 07/34] Input: stmpe-ts - mark OF related data as maybe unused Arnd Bergmann 2024-04-03 8:06 ` Arnd Bergmann 2024-04-03 9:40 ` Andy Shevchenko 2024-04-03 9:40 ` Andy Shevchenko 2024-04-03 9:52 ` Krzysztof Kozlowski 2024-04-03 9:52 ` Krzysztof Kozlowski 2024-04-03 10:03 ` Andy Shevchenko 2024-04-03 10:03 ` Andy Shevchenko 2024-04-03 10:10 ` Krzysztof Kozlowski 2024-04-03 10:10 ` Krzysztof Kozlowski 2024-04-03 13:17 ` Uwe Kleine-König 2024-04-03 13:17 ` Uwe Kleine-König 2024-04-03 13:27 ` Uwe Kleine-König 2024-04-03 13:27 ` Uwe Kleine-König 2024-04-03 8:06 ` [PATCH 08/34] Input: synaptics: hide unused smbus_pnp_ids[] array Arnd Bergmann 2024-04-03 8:06 ` [PATCH 09/34] power: rt9455: hide unused rt9455_boost_voltage_values Arnd Bergmann 2024-04-03 8:06 ` [PATCH 10/34] efi: sysfb: don't build when EFI is disabled Arnd Bergmann 2024-04-03 13:03 ` Ard Biesheuvel 2024-04-03 13:15 ` Javier Martinez Canillas 2024-04-03 8:06 ` [PATCH 11/34] clk: ti: dpll: fix incorrect #ifdef checks Arnd Bergmann 2024-04-03 9:00 ` Tony Lindgren 2024-04-05 21:13 ` Stephen Boyd 2024-04-03 8:06 ` [PATCH 12/34] apm-emulation: hide an unused variable Arnd Bergmann 2024-04-03 12:23 ` Jiri Kosina 2024-04-03 8:06 ` [PATCH 13/34] sisfb: hide unused variables Arnd Bergmann 2024-04-03 12:39 ` Helge Deller 2024-04-03 8:06 ` [PATCH 14/34] dma/congiguous: avoid warning about unused size_bytes Arnd Bergmann 2024-04-03 9:42 ` Christoph Hellwig 2024-04-03 8:06 ` [PATCH 15/34] leds: apu: remove duplicate DMI lookup data Arnd Bergmann 2024-04-11 15:51 ` (subset) " Lee Jones 2024-04-03 8:06 ` [PATCH 16/34] iio: ad5755: hook up of_device_id lookup to platform driver Arnd Bergmann 2024-04-03 9:36 ` Andy Shevchenko 2024-04-03 9:55 ` Krzysztof Kozlowski 2024-04-03 10:01 ` Arnd Bergmann 2024-04-06 15:30 ` Jonathan Cameron 2024-04-06 15:31 ` Jonathan Cameron 2024-04-03 8:06 ` [PATCH 17/34] greybus: arche-ctrl: move device table to its right location Arnd Bergmann 2024-04-30 23:41 ` Alex Elder 2024-04-03 8:06 ` [PATCH 18/34] lib: checksum: hide unused expected_csum_ipv6_magic[] Arnd Bergmann 2024-04-03 8:41 ` Christophe Leroy 2024-04-03 8:58 ` Arnd Bergmann 2024-04-03 9:03 ` Christophe Leroy 2024-04-04 11:43 ` Arnd Bergmann 2024-04-03 8:06 ` [PATCH 19/34] sunrpc: suppress warnings for unused procfs functions Arnd Bergmann 2024-04-03 8:59 ` Jeff Layton 2024-04-03 13:20 ` Chuck Lever 2024-04-03 8:06 ` [PATCH 20/34] comedi: ni_atmio: avoid warning for unused device_ids[] table Arnd Bergmann 2024-04-03 8:06 ` [PATCH 21/34] iwlegacy: don't warn for unused variables with DEBUG_FS=n Arnd Bergmann 2024-04-03 9:33 ` Andy Shevchenko 2024-04-03 14:26 ` Jeff Johnson 2024-04-03 8:06 ` [PATCH 22/34] drm/komeda: don't warn for unused debugfs files Arnd Bergmann 2024-04-03 8:06 ` [PATCH 23/34] firmware: qcom_scm: mark qcom_scm_qseecom_allowlist as __maybe_unused Arnd Bergmann 2024-04-03 9:31 ` Krzysztof Kozlowski 2024-04-03 8:06 ` [PATCH 24/34] crypto: ccp - drop platform ifdef checks Arnd Bergmann 2024-04-03 15:17 ` Tom Lendacky 2024-04-03 15:50 ` Arnd Bergmann 2024-04-12 7:30 ` Herbert Xu 2024-04-03 8:06 ` [PATCH 25/34] usb: gadget: omap_udc: remove unused variable Arnd Bergmann 2024-04-05 22:51 ` Aaro Koskinen 2024-04-03 8:06 ` [PATCH 26/34] isdn: kcapi: don't build unused procfs code Arnd Bergmann 2024-04-03 8:06 ` [PATCH 27/34] cpufreq: intel_pstate: hide unused intel_pstate_cpu_oob_ids[] Arnd Bergmann 2024-04-03 14:43 ` Rafael J. Wysocki 2024-04-03 8:06 ` [PATCH 28/34] net: xgbe: remove extraneous #ifdef checks Arnd Bergmann 2024-04-03 8:06 ` [PATCH 29/34] Input: imagis - remove incorrect ifdef checks Arnd Bergmann 2024-04-03 8:06 ` [PATCH 30/34] sata: mv: drop unnecessary #ifdef checks Arnd Bergmann 2024-04-03 8:29 ` Damien Le Moal 2024-04-03 8:32 ` Damien Le Moal 2024-04-03 8:50 ` Arnd Bergmann 2024-04-04 3:45 ` Damien Le Moal 2024-04-03 9:10 ` Andy Shevchenko 2024-04-03 8:06 ` [PATCH 31/34] spi: remove incorrect of_match_ptr annotations Arnd Bergmann 2024-04-03 8:06 ` Arnd Bergmann 2024-04-03 8:06 ` Arnd Bergmann 2024-04-03 8:06 ` Arnd Bergmann 2024-04-03 9:04 ` Andy Shevchenko 2024-04-03 9:04 ` Andy Shevchenko 2024-04-03 9:04 ` Andy Shevchenko 2024-04-03 9:04 ` Andy Shevchenko 2024-04-03 9:05 ` Krzysztof Kozlowski 2024-04-03 9:05 ` Krzysztof Kozlowski 2024-04-03 9:05 ` Krzysztof Kozlowski 2024-04-03 9:05 ` Krzysztof Kozlowski 2024-04-03 9:56 ` Mark Brown 2024-04-03 9:56 ` Mark Brown 2024-04-03 9:56 ` Mark Brown 2024-04-03 9:56 ` Mark Brown 2024-04-03 21:05 ` Uwe Kleine-König 2024-04-03 21:05 ` Uwe Kleine-König 2024-04-03 21:05 ` Uwe Kleine-König 2024-04-03 21:05 ` Uwe Kleine-König 2024-04-03 21:13 ` Andy Shevchenko 2024-04-03 21:13 ` Andy Shevchenko 2024-04-03 21:13 ` Andy Shevchenko 2024-04-03 21:13 ` Andy Shevchenko 2024-04-03 22:19 ` Arnd Bergmann 2024-04-03 22:19 ` Arnd Bergmann 2024-04-03 22:19 ` Arnd Bergmann 2024-04-03 22:19 ` Arnd Bergmann 2024-04-03 8:06 ` [PATCH 32/34] ASoC: remove incorrect of_match_ptr/ACPI_PTR annotations Arnd Bergmann 2024-04-03 8:06 ` Arnd Bergmann 2024-04-03 9:06 ` Krzysztof Kozlowski 2024-04-03 9:06 ` Krzysztof Kozlowski 2024-04-03 9:15 ` Andy Shevchenko [this message] 2024-04-03 9:15 ` Andy Shevchenko 2024-04-03 8:06 ` [PATCH 33/34] drivers: " Arnd Bergmann 2024-04-03 9:13 ` Krzysztof Kozlowski 2024-04-03 9:30 ` Andy Shevchenko 2024-04-03 12:47 ` Corey Minyard 2024-04-03 13:23 ` Andy Shevchenko 2024-04-03 12:47 ` Corey Minyard 2024-04-03 16:09 ` Jarkko Sakkinen 2024-04-09 13:48 ` Greg Kroah-Hartman 2024-04-23 7:33 ` Xu Yilun 2024-04-23 9:52 ` Peter Rosin 2024-04-03 8:06 ` [PATCH 34/34] kbuild: always enable -Wunused-const-variable Arnd Bergmann 2024-04-03 9:19 ` Andy Shevchenko 2024-04-03 9:26 ` Arnd Bergmann 2024-04-06 5:20 ` [PATCH 00/34] address all -Wunused-const warnings patchwork-bot+netdevbpf 2024-04-06 5:20 ` patchwork-bot+netdevbpf 2024-04-10 8:02 ` (subset) " Sebastian Reichel 2024-04-10 8:02 ` Sebastian Reichel 2024-04-22 8:16 ` Michael Ellerman 2024-04-22 8:16 ` Michael Ellerman 2024-04-22 8:16 ` Michael Ellerman 2024-04-22 8:16 ` Michael Ellerman
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=Zg0eIZDS3F_PQLyB@smile.fi.intel.com \ --to=andriy.shevchenko@linux.intel.com \ --cc=akihiko.odaki@daynix.com \ --cc=alexandre.belloni@bootlin.com \ --cc=alpernebiyasak@gmail.com \ --cc=alsa-devel@alsa-project.org \ --cc=angelogioacchino.delregno@collabora.com \ --cc=arnd@arndb.de \ --cc=arnd@kernel.org \ --cc=bgoswami@quicinc.com \ --cc=broonie@kernel.org \ --cc=claudiu.beznea@tuxon.dev \ --cc=guoych37@mail2.sysu.edu.cn \ --cc=krzysztof.kozlowski@linaro.org \ --cc=kuninori.morimoto.gx@renesas.com \ --cc=lgirdwood@gmail.com \ --cc=linus.walleij@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-sound@vger.kernel.org \ --cc=nicolas.ferre@microchip.com \ --cc=oder_chiou@realtek.com \ --cc=perex@perex.cz \ --cc=robh@kernel.org \ --cc=s.nawrocki@samsung.com \ --cc=srinivas.kandagatla@linaro.org \ --cc=tiwai@suse.com \ --cc=u.kleine-koenig@pengutronix.de \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.