From: Anders Roxell <anders.roxell@linaro.org>
To: perex@perex.cz, tiwai@suse.com, brian.austin@cirrus.com,
Paul.Handrigan@cirrus.com, lgirdwood@gmail.com,
broonie@kernel.org
Cc: alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org,
Anders Roxell <anders.roxell@linaro.org>
Subject: [PATCH 1/3] ASoC: cs4341: fix waring unused-function
Date: Mon, 14 Jan 2019 10:55:40 +0100 [thread overview]
Message-ID: <20190114095540.22956-1-anders.roxell@linaro.org> (raw)
The driver cs4341 can be built with SPI and/or I2C, but it has to be one
of them at least. When I2C is set as a module we see the warning below:
sound/soc/codecs/cs4341.c:213:12: warning: ‘cs4341_probe’
defined but not used [-Wunused-function]
static int cs4341_probe(struct device *dev)
^~~~~~~~~~~~
Rework so that we use IS_ENABLED instead of defined. Also change so
SND_SOC_CS4341 depends on SND_SOC_I2C_AND_SPI to we dont' get a link
error when SND_SOC_CS4341=y, I2C=m and REGMAP_I2C=m is set.
Suggested-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
---
sound/soc/codecs/Kconfig | 2 +-
sound/soc/codecs/cs4341.c | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig
index 87cb9c51e6f5..87ecb38e57f4 100644
--- a/sound/soc/codecs/Kconfig
+++ b/sound/soc/codecs/Kconfig
@@ -546,7 +546,7 @@ config SND_SOC_CS43130
config SND_SOC_CS4341
tristate "Cirrus Logic CS4341 CODEC"
- depends on I2C || SPI_MASTER
+ depends on SND_SOC_I2C_AND_SPI
select REGMAP_I2C if I2C
select REGMAP_SPI if SPI_MASTER
diff --git a/sound/soc/codecs/cs4341.c b/sound/soc/codecs/cs4341.c
index d2e616a89fd4..ade7477d04f1 100644
--- a/sound/soc/codecs/cs4341.c
+++ b/sound/soc/codecs/cs4341.c
@@ -223,7 +223,7 @@ static int cs4341_probe(struct device *dev)
&cs4341_dai, 1);
}
-#if defined(CONFIG_I2C)
+#if IS_ENABLED(CONFIG_I2C)
static int cs4341_i2c_probe(struct i2c_client *i2c,
const struct i2c_device_id *id)
{
@@ -317,7 +317,7 @@ static int __init cs4341_init(void)
{
int ret = 0;
-#if defined(CONFIG_I2C)
+#if IS_ENABLED(CONFIG_I2C)
ret = i2c_add_driver(&cs4341_i2c_driver);
if (ret)
return ret;
@@ -332,7 +332,7 @@ module_init(cs4341_init);
static void __exit cs4341_exit(void)
{
-#if defined(CONFIG_I2C)
+#if IS_ENABLED(CONFIG_I2C)
i2c_del_driver(&cs4341_i2c_driver);
#endif
#if defined(CONFIG_SPI_MASTER)
--
2.19.2
next reply other threads:[~2019-01-14 9:55 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-14 9:55 Anders Roxell [this message]
2019-01-14 11:24 ` [PATCH 1/3] ASoC: cs4341: fix waring unused-function Mark Brown
2019-01-14 11:24 ` Mark Brown
2019-01-14 12:49 ` Anders Roxell
2019-01-14 14:08 ` Mark Brown
2019-01-14 14:08 ` Mark Brown
2019-01-14 23:04 ` Applied "ASoC: cs4341: fix waring unused-function" to the asoc tree Mark Brown
2019-01-14 23:04 ` Mark Brown
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=20190114095540.22956-1-anders.roxell@linaro.org \
--to=anders.roxell@linaro.org \
--cc=Paul.Handrigan@cirrus.com \
--cc=alsa-devel@alsa-project.org \
--cc=brian.austin@cirrus.com \
--cc=broonie@kernel.org \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=perex@perex.cz \
--cc=tiwai@suse.com \
/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: link
Be 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.