Re: [PATCH v1 05/10] platform/x86: thinkpad_acpi: Stop setting acpi_device_class()

From: Andy Shevchenko

Date: Fri Jul 03 2026 - 08:24:34 EST


On Fri, Jul 03, 2026 at 01:30:00PM +0200, Rafael J. Wysocki wrote:

> To facilitate the removal of device_class from struct acpi_device_pnp
> in the future, make the driver store the device class string used for
> generating ACPI netlink events in a new device_class field in struct
> ibm_struct.
>
> No intentional functional impact.

...

> acpi_bus_generate_netlink_event(
> - ibm->acpi->device->pnp.device_class,
> + ibm->device_class,
> dev_name(&ibm->acpi->device->dev),
> event, 0);

In other cases you condensed the code a bit, not sure if you want to do the
similar here

acpi_bus_generate_netlink_event(ibm->device_class,
dev_name(&ibm->acpi->device->dev),
event, 0);

> return;

...

> if (send_acpi_ev) {
> acpi_bus_generate_netlink_event(
> - ibm->acpi->device->pnp.device_class,
> + ibm->device_class,
> dev_name(&ibm->acpi->device->dev),
> event, hkey);
> }

Ditto.

--
With Best Regards,
Andy Shevchenko