From nobody Sat Apr 11 21:30:22 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 07A87C19F2A for ; Sun, 7 Aug 2022 14:56:38 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234449AbiHGO4g (ORCPT ); Sun, 7 Aug 2022 10:56:36 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52328 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234539AbiHGO4L (ORCPT ); Sun, 7 Aug 2022 10:56:11 -0400 Received: from aposti.net (aposti.net [89.234.176.197]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D7570E9E for ; Sun, 7 Aug 2022 07:55:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=crapouillou.net; s=mail; t=1659883987; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=9Wcqw8v4yEtQTQLoqAM27e1h0VJ0qSd8BKRCHlYay5o=; b=Ok0I2oLiNXtVv+CN6haNu485gyrbK6Xd2UsofgcOKfD1g0KnQLAWf1GndyaSKFYLxV2y5h pRCxoQLsKsO6vGM0UfJzwmsXAlnYwCc4Luyl+zS+9dP+PDT70j7C6YvSbWNo9T7M1B+QrC sph1EAovSJ9J3wjopDJRewlmwDHBVzI= From: Paul Cercueil To: Lee Jones Cc: linux-kernel@vger.kernel.org, Paul Cercueil , patches@opensource.cirrus.com Subject: [PATCH 20/28] mfd: arizona: Remove #ifdef guards for PM related functions Date: Sun, 7 Aug 2022 16:52:39 +0200 Message-Id: <20220807145247.46107-21-paul@crapouillou.net> In-Reply-To: <20220807145247.46107-1-paul@crapouillou.net> References: <20220807145247.46107-1-paul@crapouillou.net> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Only export the arizona_pm_ops if CONFIG_PM is set, but leave the suspend/resume functions (and related code) outside #ifdef guards. If CONFIG_PM is not set, the arizona_pm_ops will be defined as "static __maybe_unused", and the structure plus all the callbacks will be automatically dropped by the compiler. The advantage is then that these functions are now always compiled independently of any Kconfig option, and thanks to that bugs and regressions are easier to catch. Signed-off-by: Paul Cercueil Cc: patches@opensource.cirrus.com Reported-by: kernel test robot --- drivers/mfd/arizona-core.c | 21 +++++++++++---------- drivers/mfd/arizona-i2c.c | 2 +- drivers/mfd/arizona-spi.c | 2 +- 3 files changed, 13 insertions(+), 12 deletions(-) diff --git a/drivers/mfd/arizona-core.c b/drivers/mfd/arizona-core.c index cbf1dd90b70d..c1acc9521f83 100644 --- a/drivers/mfd/arizona-core.c +++ b/drivers/mfd/arizona-core.c @@ -480,7 +480,6 @@ static int wm5102_clear_write_sequencer(struct arizona = *arizona) return 0; } =20 -#ifdef CONFIG_PM static int arizona_isolate_dcvdd(struct arizona *arizona) { int ret; @@ -742,9 +741,7 @@ static int arizona_runtime_suspend(struct device *dev) =20 return 0; } -#endif =20 -#ifdef CONFIG_PM_SLEEP static int arizona_suspend(struct device *dev) { struct arizona *arizona =3D dev_get_drvdata(dev); @@ -784,17 +781,21 @@ static int arizona_resume(struct device *dev) =20 return 0; } -#endif =20 +#ifndef CONFIG_PM +static __maybe_unused +#endif const struct dev_pm_ops arizona_pm_ops =3D { - SET_RUNTIME_PM_OPS(arizona_runtime_suspend, - arizona_runtime_resume, - NULL) - SET_SYSTEM_SLEEP_PM_OPS(arizona_suspend, arizona_resume) - SET_NOIRQ_SYSTEM_SLEEP_PM_OPS(arizona_suspend_noirq, - arizona_resume_noirq) + RUNTIME_PM_OPS(arizona_runtime_suspend, + arizona_runtime_resume, + NULL) + SYSTEM_SLEEP_PM_OPS(arizona_suspend, arizona_resume) + NOIRQ_SYSTEM_SLEEP_PM_OPS(arizona_suspend_noirq, + arizona_resume_noirq) }; +#ifdef CONFIG_PM EXPORT_SYMBOL_GPL(arizona_pm_ops); +#endif =20 #ifdef CONFIG_OF static int arizona_of_get_core_pdata(struct arizona *arizona) diff --git a/drivers/mfd/arizona-i2c.c b/drivers/mfd/arizona-i2c.c index 6d83e6b9a692..8799d9183bee 100644 --- a/drivers/mfd/arizona-i2c.c +++ b/drivers/mfd/arizona-i2c.c @@ -119,7 +119,7 @@ static const struct of_device_id arizona_i2c_of_match[]= =3D { static struct i2c_driver arizona_i2c_driver =3D { .driver =3D { .name =3D "arizona", - .pm =3D &arizona_pm_ops, + .pm =3D pm_ptr(&arizona_pm_ops), .of_match_table =3D of_match_ptr(arizona_i2c_of_match), }, .probe =3D arizona_i2c_probe, diff --git a/drivers/mfd/arizona-spi.c b/drivers/mfd/arizona-spi.c index 941b0267d09d..da05b966d48c 100644 --- a/drivers/mfd/arizona-spi.c +++ b/drivers/mfd/arizona-spi.c @@ -282,7 +282,7 @@ static const struct of_device_id arizona_spi_of_match[]= =3D { static struct spi_driver arizona_spi_driver =3D { .driver =3D { .name =3D "arizona", - .pm =3D &arizona_pm_ops, + .pm =3D pm_ptr(&arizona_pm_ops), .of_match_table =3D of_match_ptr(arizona_spi_of_match), .acpi_match_table =3D ACPI_PTR(arizona_acpi_match), }, --=20 2.35.1