mbox series

[v1,0/7] ACPI: scan: A few ad-hoc cleanups

Message ID 20240325123444.3031851-1-andriy.shevchenko@linux.intel.com (mailing list archive)
Headers show
Series ACPI: scan: A few ad-hoc cleanups | expand

Message

Andy Shevchenko March 25, 2024, 12:32 p.m. UTC
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.

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(-)

Comments

Kuppuswamy Sathyanarayanan March 27, 2024, 5:50 a.m. UTC | #1
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(-)
>