On 6/2/2023 3:08 AM, David E. Box wrote:
> Hi Michal,
>
> On Thu, 2023-06-01 at 15:21 +0200, Michal Wilczynski wrote:
>> As callback has been replaced by drivers installing their handlers in
>> .add it's presence is not useful anymore.
>>
>> Remove .notify callback and flags variable from struct acpi_driver,
>> as they're not needed anymore.
>>
>> Signed-off-by: Michal Wilczynski <michal.wilczyn...@intel.com>
>> ---
>> include/acpi/acpi_bus.h | 3 ---
>> 1 file changed, 3 deletions(-)
>>
>> diff --git a/include/acpi/acpi_bus.h b/include/acpi/acpi_bus.h
>> index 7fb411438b6f..3326794d5b70 100644
>> --- a/include/acpi/acpi_bus.h
>> +++ b/include/acpi/acpi_bus.h
>> @@ -151,12 +151,10 @@ struct acpi_hotplug_context {
>>
>> typedef int (*acpi_op_add) (struct acpi_device * device);
>> typedef void (*acpi_op_remove) (struct acpi_device *device);
>> -typedef void (*acpi_op_notify) (struct acpi_device * device, u32 event);
>>
>> struct acpi_device_ops {
>> acpi_op_add add;
>> acpi_op_remove remove;
>> - acpi_op_notify notify;
>> };
>>
>> #define ACPI_DRIVER_ALL_NOTIFY_EVENTS 0x1 /* system AND device events
>> */
>> @@ -165,7 +163,6 @@ struct acpi_driver {
>> char name[80];
>> char class[80];
>> const struct acpi_device_id *ids; /* Supported Hardware IDs */
>> - unsigned int flags;
> Can ACPI_DRIVER_ALL_NOTIFY_EVENTS be removed as well?
Hi David,
Oh yeah, it should be removed as well,
Thanks !
>
>> struct acpi_device_ops ops;
>> struct device_driver drv;
>> struct module *owner;