From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj2-f7.google.com (mail-pj2-f7.google.com [74.125.227.135]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DE1B5377A82 for ; Thu, 20 Aug 2026 03:37:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.227.135 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787197033; cv=none; b=XlPQY0FvTcsx9MlWEAsP03vZ+cR+LGmZPYPyapX9n2D803hXY2lwKRQbUw8iETQ2vrUU1rtlZRohCiOskjr2tva2onNbeGznLfiXXof205RcvKxw+ucWjuIUUoi/C0FPKbzgbDfkjFpSretI8KR0ooa3lp//IbvdR0MGfWDxK4Q= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787197033; c=relaxed/simple; bh=ofPrnzg+ZXWkEkhbKcspbON3ebrXzoTrIj07Zp2uxtQ=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=FdKjZjq9wEgl0Mr2+4ovj1mfVaERQoOTh6kQITBCurR/VNGZSY3e94BBzgaT1BlQDmUFksbpbYC5WF17I0SrO85vUhXiu2QHLQk7MQxvKnw3aol/omia/n4ofEXe/4pdu05vlecBRIlUbb0K1jpHdcdlfPiVq9Bpkm33djnZBo4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=JXcaJpA0; arc=none smtp.client-ip=74.125.227.135 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="JXcaJpA0" Received: by mail-pj2-f7.google.com with SMTP id d9443c01a7336-2d561173f9fso11616175ad.0 for ; Wed, 19 Aug 2026 20:37:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1787197031; x=1787801831; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to:content-type; bh=SkTtVaUvkrndBFZ8UGOVBPX/3VqVO7aSr6OqB3qIqCU=; b=JXcaJpA0fIwEmHxPwhCbPSmpl3zmrdLmbeJ3Kp8A6A76D8gGRNCdTlUJ4+n6VJlUOb 73cU/V8glpUFO57CHhQJ6ZnNCh2xhxmjwx0qFalqXs0dgKAFLNNVJtliWqwxrD/OrNYO kDI50jdOmyTQyfL31LiJ+qo64JhbcZzkhAoqwUJpWLHoQ1B5KEkOiREGM7Ej1B2jq81o VFQrFLXj2t6/Re8gM9a2j7QqxxKyXsoA1l3xqkF8gtLbhZNrOBuyDOUT68G36pGFCA31 o2ilR2/9HrlKJ4USXeqZNGT4PZ5F0+UgxSoGSU8ad+0zLO4yxtEyIR/+1aexauGqG98z ZYbA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1787197031; x=1787801831; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=SkTtVaUvkrndBFZ8UGOVBPX/3VqVO7aSr6OqB3qIqCU=; b=Bs4oio8DmJv2b2O34HdDzsGs9Jy7uJvQ/PQpG7wvsKnVVUmXUdY3824WQW9jaAddp5 CeEeQceAfNY6ZGTFEDPydz0N76gf5IKSmB6IDNhk4KBKB1tnvHDV7GJcAl2rfjaNKwgR J2ie30taTsdAiG4GsjSoBBFXYmlQ6Kkb833N2XkKTUdmOV35W5/vYIyDAdjw4JzDMjEZ qBwdtspdbx5RY5+NCBFdB8GyVm2NcDTjVHgLZgboglMrKwoGxGuIGXg32CZDo2aIKOKB jGPPJ3PoPt3Pk9ZZfpVUF3vjXkchCa6ir5/0nt8B/MznfgDS7Umw/KwqoFyTohxNPGJG wQfg== X-Forwarded-Encrypted: i=1; AHgh+RralGEXTO4gcbON3PPW+yDnNvdlqYO5n0q3U5iDr0Md9VRHBk1qFQnIl94bko4hTPZiSZvKrv3PGJWAaCI=@vger.kernel.org X-Gm-Message-State: AFuF++k9HS2kEfQz+i61lZ5h539TWGf6T7NcuKxZXMBenmrLdW1QWO2s qBrx/TsIsAfOU9GYSK+XD2O695zWSETqINe4RTWRpLHk4yWLuesd5HQ30gVjn+dRWA== X-Gm-Gg: AR+sD11/35XcrW8jazdblKZ2aR24sJTkC08gnTW5r1uwpMGxS5aYkELB3a6jb1FtTZg 0l5sJ9x9+zac3iBnAQB1DBkxqz4ltOT87PKREiUkryBEcvGpw7yaXTor4UPBUkSpNNBLAxP7ODV sW1xRLJ5pqsneksJ6omgoBvT6D0Q2gq/RAnoHaZ45e98d+ENJxfarN1jBq4egXAEHn7mcErsaYN P/PG6sUnRCN2tzdi8j3gOo+QJ0zXTEI/S59P5iVgEtXzk7B5C+Q0M2SjynS9Zd8vE9p6u6RcP2R JodAirJi4NYw/BKi0GJE1gj0EAbb8dJJ47IzbEOzQG7eYPY15gEAGG4LxyHBLfX/0+T9UwUs7qM Ffqt/riemk/iR0Oee0XK/lre+VBXQSu72LgQ4ZDhvKnHEGawLGMTnnPNQDjjh6SDzSZ4FsxL7ET mCmEFK6ib9WKjXYzYQ1+Gf33ld4GaTs9O7X2+mFH+KPCy5ggKd61tjkxsH+g/o6TRREC371qtnE g== X-Received: by 2002:a17:903:b4e:b0:2d5:ea01:429d with SMTP id d9443c01a7336-2d60197e7b6mr192855385ad.14.1787197031007; Wed, 19 Aug 2026 20:37:11 -0700 (PDT) Received: from DL2XHKPB4.dl.net ([103.63.104.162]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2d62e36e1f9sm1438415ad.43.2026.08.19.20.37.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 19 Aug 2026 20:37:10 -0700 (PDT) From: Triet Hoang To: rafael@kernel.org Cc: lenb@kernel.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, Triet Hoang Subject: [PATCH] ACPI: button: use DEFINE_SIMPLE_DEV_PM_OPS Date: Thu, 20 Aug 2026 10:36:53 +0700 Message-ID: <20260820033653.1682804-1-triet.hoang.dev@gmail.com> X-Mailer: git-send-email 2.53.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Convert the deprecated SIMPLE_DEV_PM_OPS to DEFINE_SIMPLE_DEV_PM_OPS Also use pm_sleep_ptr() when assigning the ops to the driver. This allows the removal of the ifdef CONFIG_PM_SLEEP and unnecessary forward declarations of 4 functions. No intentional functional impact. Signed-off-by: Triet Hoang --- drivers/acpi/button.c | 36 ++++++++++++------------------------ 1 file changed, 12 insertions(+), 24 deletions(-) diff --git a/drivers/acpi/button.c b/drivers/acpi/button.c index 3836ee75dd66..d4cb942b754f 100644 --- a/drivers/acpi/button.c +++ b/drivers/acpi/button.c @@ -147,28 +147,6 @@ static const struct dmi_system_id dmi_lid_quirks[] = { {} }; -static int acpi_button_probe(struct platform_device *pdev); -static void acpi_button_remove(struct platform_device *pdev); - -#ifdef CONFIG_PM_SLEEP -static int acpi_button_suspend(struct device *dev); -static int acpi_button_resume(struct device *dev); -#else -#define acpi_button_suspend NULL -#define acpi_button_resume NULL -#endif -static SIMPLE_DEV_PM_OPS(acpi_button_pm, acpi_button_suspend, acpi_button_resume); - -static struct platform_driver acpi_button_driver = { - .probe = acpi_button_probe, - .remove = acpi_button_remove, - .driver = { - .name = "acpi-button", - .acpi_match_table = button_device_ids, - .pm = &acpi_button_pm, - }, -}; - struct acpi_button { struct acpi_device *adev; struct device *dev; /* physical button device */ @@ -491,7 +469,6 @@ static u32 acpi_button_event(void *data) return ACPI_INTERRUPT_HANDLED; } -#ifdef CONFIG_PM_SLEEP static int acpi_button_suspend(struct device *dev) { struct acpi_button *button = dev_get_drvdata(dev); @@ -521,7 +498,6 @@ static int acpi_button_resume(struct device *dev) } return 0; } -#endif static int acpi_lid_input_open(struct input_dev *input) { @@ -753,6 +729,18 @@ static void acpi_button_remove(struct platform_device *pdev) acpi_lid_forget(button->adev); } +static DEFINE_SIMPLE_DEV_PM_OPS(acpi_button_pm, acpi_button_suspend, acpi_button_resume); + +static struct platform_driver acpi_button_driver = { + .probe = acpi_button_probe, + .remove = acpi_button_remove, + .driver = { + .name = "acpi-button", + .acpi_match_table = button_device_ids, + .pm = pm_sleep_ptr(&acpi_button_pm), + }, +}; + static int param_set_lid_init_state(const char *val, const struct kernel_param *kp) { -- 2.53.0