* [PATCH -next] ASoC: samsung: midas_wm1811: Fix build error when CONFIG_GPIOLIB is not set
@ 2021-05-26 14:43 Wei Yongjun
2021-05-26 15:49 ` Krzysztof Kozlowski
0 siblings, 1 reply; 2+ messages in thread
From: Wei Yongjun @ 2021-05-26 14:43 UTC (permalink / raw)
To: weiyongjun1, Simon Shields, Krzysztof Kozlowski,
Sylwester Nawrocki, Liam Girdwood, Mark Brown, Jaroslav Kysela,
Takashi Iwai
Cc: alsa-devel, linux-kernel, kernel-janitors, Hulk Robot
Fix build error when CONFIG_GPIOLIB is not set:
sound/soc/samsung/midas_wm1811.c: In function 'midas_fm_set':
sound/soc/samsung/midas_wm1811.c:205:3: error:
implicit declaration of function 'gpiod_set_value_cansleep';
did you mean 'gpio_set_value_cansleep'? [-Werror=implicit-function-declaration]
205 | gpiod_set_value_cansleep(priv->gpio_fm_sel, 1);
| ^~~~~~~~~~~~~~~~~~~~~~~~
| gpio_set_value_cansleep
sound/soc/samsung/midas_wm1811.c: In function 'midas_probe':
sound/soc/samsung/midas_wm1811.c:445:22: error:
implicit declaration of function 'devm_gpiod_get_optional';
did you mean 'devm_clk_get_optional'? [-Werror=implicit-function-declaration]
445 | priv->gpio_fm_sel = devm_gpiod_get_optional(dev, "fm-sel", GPIOD_OUT_HIGH);
| ^~~~~~~~~~~~~~~~~~~~~~~
| devm_clk_get_optional
sound/soc/samsung/midas_wm1811.c:445:61:
error: 'GPIOD_OUT_HIGH' undeclared (first use in this function);
did you mean 'GPIOF_INIT_HIGH'?
445 | priv->gpio_fm_sel = devm_gpiod_get_optional(dev, "fm-sel", GPIOD_OUT_HIGH);
| ^~~~~~~~~~~~~~
| GPIOF_INIT_HIGH
Fixes: fd0ea9cd9698 ("ASoC: samsung: Add sound support for Midas boards")
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
---
sound/soc/samsung/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/sound/soc/samsung/Kconfig b/sound/soc/samsung/Kconfig
index a2221ebb1b6a..22be597b1dd9 100644
--- a/sound/soc/samsung/Kconfig
+++ b/sound/soc/samsung/Kconfig
@@ -228,6 +228,7 @@ config SND_SOC_SAMSUNG_ARIES_WM8994
config SND_SOC_SAMSUNG_MIDAS_WM1811
tristate "SoC I2S Audio support for Midas boards"
depends on SND_SOC_SAMSUNG
+ depends on GPIOLIB
select SND_SAMSUNG_I2S
select SND_SOC_WM8994
help
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH -next] ASoC: samsung: midas_wm1811: Fix build error when CONFIG_GPIOLIB is not set
2021-05-26 14:43 [PATCH -next] ASoC: samsung: midas_wm1811: Fix build error when CONFIG_GPIOLIB is not set Wei Yongjun
@ 2021-05-26 15:49 ` Krzysztof Kozlowski
0 siblings, 0 replies; 2+ messages in thread
From: Krzysztof Kozlowski @ 2021-05-26 15:49 UTC (permalink / raw)
To: Wei Yongjun, Simon Shields, Sylwester Nawrocki, Liam Girdwood,
Mark Brown, Jaroslav Kysela, Takashi Iwai
Cc: alsa-devel, linux-kernel, kernel-janitors, Hulk Robot
On 26/05/2021 10:43, Wei Yongjun wrote:
> Fix build error when CONFIG_GPIOLIB is not set:
>
> sound/soc/samsung/midas_wm1811.c: In function 'midas_fm_set':
> sound/soc/samsung/midas_wm1811.c:205:3: error:
> implicit declaration of function 'gpiod_set_value_cansleep';
> did you mean 'gpio_set_value_cansleep'? [-Werror=implicit-function-declaration]
> 205 | gpiod_set_value_cansleep(priv->gpio_fm_sel, 1);
> | ^~~~~~~~~~~~~~~~~~~~~~~~
> | gpio_set_value_cansleep
> sound/soc/samsung/midas_wm1811.c: In function 'midas_probe':
> sound/soc/samsung/midas_wm1811.c:445:22: error:
> implicit declaration of function 'devm_gpiod_get_optional';
> did you mean 'devm_clk_get_optional'? [-Werror=implicit-function-declaration]
> 445 | priv->gpio_fm_sel = devm_gpiod_get_optional(dev, "fm-sel", GPIOD_OUT_HIGH);
> | ^~~~~~~~~~~~~~~~~~~~~~~
> | devm_clk_get_optional
The fix does not match the error. The consumer.h has necessary stubs for
!GPIOLIB case.
To fix this error, you do not depend on GPIOLIB, but include proper
header (consumer.h). The dependency (with compile test) is still nice,
but for runtime fix, not build time, which is a separate patch.
Best regards,
Krzysztof
> sound/soc/samsung/midas_wm1811.c:445:61:
> error: 'GPIOD_OUT_HIGH' undeclared (first use in this function);
> did you mean 'GPIOF_INIT_HIGH'?
> 445 | priv->gpio_fm_sel = devm_gpiod_get_optional(dev, "fm-sel", GPIOD_OUT_HIGH);
> | ^~~~~~~~~~~~~~
> | GPIOF_INIT_HIGH
>
> Fixes: fd0ea9cd9698 ("ASoC: samsung: Add sound support for Midas boards")
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
> ---
> sound/soc/samsung/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/sound/soc/samsung/Kconfig b/sound/soc/samsung/Kconfig
> index a2221ebb1b6a..22be597b1dd9 100644
> --- a/sound/soc/samsung/Kconfig
> +++ b/sound/soc/samsung/Kconfig
> @@ -228,6 +228,7 @@ config SND_SOC_SAMSUNG_ARIES_WM8994
> config SND_SOC_SAMSUNG_MIDAS_WM1811
> tristate "SoC I2S Audio support for Midas boards"
> depends on SND_SOC_SAMSUNG
> + depends on GPIOLIB
> select SND_SAMSUNG_I2S
> select SND_SOC_WM8994
> help
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-05-26 15:50 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-26 14:43 [PATCH -next] ASoC: samsung: midas_wm1811: Fix build error when CONFIG_GPIOLIB is not set Wei Yongjun
2021-05-26 15:49 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).