From patchwork Wed Apr 8 18:55:18 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 11480663 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id C7A0B92C for ; Wed, 8 Apr 2020 18:56:42 +0000 (UTC) Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id E42FC206BE for ; Wed, 8 Apr 2020 18:56:41 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=alsa-project.org header.i=@alsa-project.org header.b="DXEgdHfB" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E42FC206BE Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=arndb.de Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=alsa-devel-bounces@alsa-project.org Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id B29ED85D; Wed, 8 Apr 2020 20:55:54 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz B29ED85D DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1586372199; bh=gqW/Hz2ckDAETLJ13g+seTEXfKmRuqqqEmg90oygsgg=; h=From:To:Subject:Date:Cc:List-Id:List-Unsubscribe:List-Archive: List-Post:List-Help:List-Subscribe:From; b=DXEgdHfBxUpUA1kPBl3sGknmy4UfZm/+aFv2qRPkkxXI8oHBGfPd8lerXI8zcUCyj Q+tYgUEyDu750s1AwZMG8M0w1rc3FIyoTTCR/eAApXbYQdsYokhWSjMOagjf36Ei/n qgjp8+OwISnj+UW0qiLAOh9WBwKV+59tlp+j00r8= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id 4027EF80142; Wed, 8 Apr 2020 20:55:54 +0200 (CEST) X-Original-To: alsa-devel@alsa-project.org Delivered-To: alsa-devel@alsa-project.org Received: by alsa1.perex.cz (Postfix, from userid 50401) id 30B39F80143; Wed, 8 Apr 2020 20:55:52 +0200 (CEST) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on alsa1.perex.cz X-Spam-Level: X-Spam-Status: No, score=0.0 required=5.0 tests=SPF_HELO_NONE,SPF_NONE, URIBL_BLOCKED autolearn=disabled version=3.4.0 Received: from mout.kundenserver.de (mout.kundenserver.de [217.72.192.73]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id 683E2F80107 for ; Wed, 8 Apr 2020 20:55:47 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 683E2F80107 Received: from threadripper.lan ([149.172.19.189]) by mrelayeu.kundenserver.de (mreue106 [212.227.15.145]) with ESMTPA (Nemesis) id 1MwxRN-1j6lFQ1JAh-00ySED; Wed, 08 Apr 2020 20:55:31 +0200 From: Arnd Bergmann To: Oder Chiou , Mark Brown Subject: [PATCH] ASoC: rt5682: fix building without I2C Date: Wed, 8 Apr 2020 20:55:18 +0200 Message-Id: <20200408185527.308213-1-arnd@arndb.de> X-Mailer: git-send-email 2.26.0 MIME-Version: 1.0 X-Provags-ID: V03:K1:4AoJDCXVBq/rxRcJ104peoQYrzLDNcb3p1vOQ7Y937Nq+mHuurc mPlJhXivK+4f42v46kcfEnrMJRG5r7Kg1E11iAC1mT8ndXQcRJ/JgROVMhddj6HJ0f5p945 QZ0AA8Lwne9UnABWVM4cRmiqzcMXmdiTLU/0vD+vcv3ZS6WapO13Bk7ccWW3jrGmOBCn0wy mL1AMEBlueU/zYwcFhylQ== X-UI-Out-Filterresults: notjunk:1;V03:K0:29YYqW2EZs8=:TYqEai8QnFj5T50xeEt3TC 0eDWr8s6Y96tKufknGNLHrTGNVcUoADmJ42LvoZXlHCcHqE14lsxa8rZDJLNOAegH6UHETfqX 2MccJQ9/v2QV14PSEDFB7pXlxyqMDHT+C4dIQNqPYg6Jt4Jt/KGYnIUjJa+iiwXgM03rRc7BN 4WwJZgOFVZKfOvfpSEuMivmdbo/z4nuvqXPdi2zidkGnu601FNvB6gqWY+6S+EoF9SROZUdDp G99v7sJ9pPlXa91hVpyGA98KBHi4HcjBmL7gfz1zq9PiCi6hbB05ZsdcWQolvAkwhGkV+jsGw M3tKV26GK6+5pOCrHAoV9A0ymf+qP9QHFF6I5t75syoPG3hT4O7vQDKdwKzIAHXofi3dPV7cr ti1quVNknN6FhoC4o1RuaDJzKzcyK8r79n89cq2JoIRzXmmNVqsF1tkQ0DGsl/4y/VYkY3doU S1kqkOYLfM7q8735C4c+rIRPf/q3qZ5bCXMudUxaGa5qmDpVId1DNnnh4vuedmushCnt9AjMD OTl7HTcmRJwh72dVSxUZISZ7Yd94IMIvWegrPrIiv7ARxJ3geEza3tWzVhxl+XKDmTQx8twF9 Rl4fDpb68SxVf744Zyo6Oz2Cq6D38NeTvLi+Th1bhby1ZfCUjjlJQ81o5Yyo+ydMO2UGlvSCy 8jC4WBg62+9/VNHkjHxmGwOfE6IHzv/1u6yrVydOA9j4c5HoeELs5mtHEgAljbGad35dR4I/c qXXrPHp0vPJ46gmUmkovehQ7UTvQRjB09oad4/icz5+HvY0OZceoHGJE/J4RKz59yQx2IHBUW ReNJEMSjW0l+VQrj3Vs5mvTeh+sQFbcA/jmb2Q0Pp82RdEnetc= Cc: Pierre-Louis Bossart , alsa-devel@alsa-project.org, kbuild test robot , Arnd Bergmann , linux-kernel@vger.kernel.org, Liam Girdwood , Takashi Iwai , Derek Fang , Shuming Fan X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" When I2C is disabled, building rt5682 produces a compile-time error: sound/soc/codecs/rt5682.c:3716:1: warning: data definition has no type or storage class 3716 | module_i2c_driver(rt5682_i2c_driver); | ^~~~~~~~~~~~~~~~~ sound/soc/codecs/rt5682.c:3716:1: error: type defaults to 'int' in declaration of 'module_i2c_driver' [-Werror=implicit-int] sound/soc/codecs/rt5682.c:3716:1: warning: parameter names (without types) in function declaration sound/soc/codecs/rt5682.c:3706:26: error: 'rt5682_i2c_driver' defined but not used [-Werror=unused-variable] 3706 | static struct i2c_driver rt5682_i2c_driver = { | ^~~~~~~~~~~~~~~~~ Move the i2c specific entry points into an #ifdef section so it can actually be built standalone. Fixes: 5549ea647997 ("ASoC: rt5682: fix unmet dependencies") Signed-off-by: Arnd Bergmann --- sound/soc/codecs/rt5682.c | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/sound/soc/codecs/rt5682.c b/sound/soc/codecs/rt5682.c index c9268a230daa..a6548f870807 100644 --- a/sound/soc/codecs/rt5682.c +++ b/sound/soc/codecs/rt5682.c @@ -3224,12 +3224,6 @@ static const struct regmap_config rt5682_regmap = { .use_single_write = true, }; -static const struct i2c_device_id rt5682_i2c_id[] = { - {"rt5682", 0}, - {} -}; -MODULE_DEVICE_TABLE(i2c, rt5682_i2c_id); - static int rt5682_parse_dt(struct rt5682_priv *rt5682, struct device *dev) { @@ -3526,7 +3520,7 @@ int rt5682_io_init(struct device *dev, struct sdw_slave *slave) EXPORT_SYMBOL_GPL(rt5682_io_init); #endif -static int rt5682_i2c_probe(struct i2c_client *i2c, +static int __maybe_unused rt5682_i2c_probe(struct i2c_client *i2c, const struct i2c_device_id *id) { struct rt5682_platform_data *pdata = dev_get_platdata(&i2c->dev); @@ -3680,13 +3674,14 @@ static int rt5682_i2c_probe(struct i2c_client *i2c, rt5682_dai, ARRAY_SIZE(rt5682_dai)); } -static void rt5682_i2c_shutdown(struct i2c_client *client) +static void __maybe_unused rt5682_i2c_shutdown(struct i2c_client *client) { struct rt5682_priv *rt5682 = i2c_get_clientdata(client); rt5682_reset(rt5682); } +#ifdef CONFIG_I2C #ifdef CONFIG_OF static const struct of_device_id rt5682_of_match[] = { {.compatible = "realtek,rt5682i"}, @@ -3703,6 +3698,12 @@ static const struct acpi_device_id rt5682_acpi_match[] = { MODULE_DEVICE_TABLE(acpi, rt5682_acpi_match); #endif +static const struct i2c_device_id rt5682_i2c_id[] = { + {"rt5682", 0}, + {} +}; +MODULE_DEVICE_TABLE(i2c, rt5682_i2c_id); + static struct i2c_driver rt5682_i2c_driver = { .driver = { .name = "rt5682", @@ -3714,6 +3715,7 @@ static struct i2c_driver rt5682_i2c_driver = { .id_table = rt5682_i2c_id, }; module_i2c_driver(rt5682_i2c_driver); +#endif MODULE_DESCRIPTION("ASoC RT5682 driver"); MODULE_AUTHOR("Bard Liao ");