On 3/25/24 5:32 AM, Andy Shevchenko wrote:
> While looking for something else in the scan.c I noticed the room
> of improvement. Hence this series. Also bus.c patches, which some
> how were related to my research, but I think are independent from
> the scan.c improvements.
Changes looks good to me.
Reviewed-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com>
>
> Andy Shevchenko (7):
> ACPI: bus: Make container_of() no-op where it makes sense
> ACPI: bus: Don't use "proxy" headers
> ACPI: scan: Replace infinite for-loop with finite while-loop
> ACPI: scan: Use list_first_entry_or_null() in acpi_device_hid()
> ACPI: scan: Move misleading comment to acpi_dma_configure_id()
> ACPI: scan: Use standard error checking pattern
> ACPI: scan: Introduce typedef:s for struct acpi_hotplug_context
> members
>
> drivers/acpi/dock.c | 48 +++++++++++++++--------------------------
> drivers/acpi/scan.c | 42 +++++++++++++++++-------------------
> include/acpi/acpi_bus.h | 28 +++++++++++++++---------
> 3 files changed, 55 insertions(+), 63 deletions(-)
>
--
Sathyanarayanan Kuppuswamy
Linux Kernel Developer