* [PATCH v1] platform/loongarch: laptop: Stop setting acpi_device_class()
@ 2026-07-06 12:33 Rafael J. Wysocki
2026-07-08 14:30 ` Andy Shevchenko
0 siblings, 1 reply; 2+ messages in thread
From: Rafael J. Wysocki @ 2026-07-06 12:33 UTC (permalink / raw)
To: loongarch, Linux ACPI
Cc: Huacai Chen, WANG Xuerui, Hans de Goede, Andy Shevchenko, LKML
From: "Rafael J. Wysocki" <rafael.j.wysocki@intel.com>
The driver populates acpi_device_class() which is never read afterward,
so make it stop doing that and drop the symbol defined specifically for
this purpose.
No intentional functional impact.
This will facilitate the removal of device_class from struct
acpi_device_pnp in the future.
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
---
drivers/platform/loongarch/loongson-laptop.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/platform/loongarch/loongson-laptop.c b/drivers/platform/loongarch/loongson-laptop.c
index 61b18ac206c9..f2dcc6d78051 100644
--- a/drivers/platform/loongarch/loongson-laptop.c
+++ b/drivers/platform/loongarch/loongson-laptop.c
@@ -30,7 +30,6 @@
#define LOONGSON_ACPI_HKEY_HID "LOON0000"
#define ACPI_LAPTOP_NAME "loongson-laptop"
-#define ACPI_LAPTOP_ACPI_EVENT_PREFIX "loongson"
#define MAX_ACPI_ARGS 3
#define GENERIC_HOTKEY_MAP_MAX 64
@@ -167,8 +166,6 @@ static int __init setup_acpi_notify(struct generic_sub_driver *sub_driver)
}
sub_driver->device->driver_data = sub_driver;
- sprintf(acpi_device_class(sub_driver->device), "%s/%s",
- ACPI_LAPTOP_ACPI_EVENT_PREFIX, sub_driver->name);
status = acpi_install_notify_handler(*sub_driver->handle,
sub_driver->type, dispatch_acpi_notify, sub_driver);
--
2.51.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v1] platform/loongarch: laptop: Stop setting acpi_device_class()
2026-07-06 12:33 [PATCH v1] platform/loongarch: laptop: Stop setting acpi_device_class() Rafael J. Wysocki
@ 2026-07-08 14:30 ` Andy Shevchenko
0 siblings, 0 replies; 2+ messages in thread
From: Andy Shevchenko @ 2026-07-08 14:30 UTC (permalink / raw)
To: Rafael J. Wysocki
Cc: loongarch, Linux ACPI, Huacai Chen, WANG Xuerui, Hans de Goede,
LKML
On Mon, Jul 06, 2026 at 02:33:23PM +0200, Rafael J. Wysocki wrote:
> The driver populates acpi_device_class() which is never read afterward,
> so make it stop doing that and drop the symbol defined specifically for
> this purpose.
>
> No intentional functional impact.
>
> This will facilitate the removal of device_class from struct
> acpi_device_pnp in the future.
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-07-08 14:30 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-06 12:33 [PATCH v1] platform/loongarch: laptop: Stop setting acpi_device_class() Rafael J. Wysocki
2026-07-08 14:30 ` Andy Shevchenko
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox