mbox series

[v3,0/5] Support _UID matching for integer types

Message ID 20231123100617.28020-1-raag.jadav@intel.com
Headers show
Series Support _UID matching for integer types | expand

Message

Raag Jadav Nov. 23, 2023, 10:06 a.m. UTC 
This series updates the standard ACPI helpers to support _UID matching
for both integer and string types, and uses them in a couple of places.

Changes since v2:
- Drop __builtin functions to reduce complexity (Rafael)
- Update tags

Changes since v1:
- Fix build errors

Raag Jadav (5):
  ACPI: bus: update acpi_dev_uid_match() to support multiple types
  ACPI: bus: update acpi_dev_hid_uid_match() to support multiple types
  ACPI: LPSS: use acpi_dev_uid_match() for matching _UID
  efi: dev-path-parser: use acpi_dev_uid_match() for matching _UID
  perf: arm_cspmu: drop redundant acpi_dev_uid_to_integer()

 drivers/acpi/acpi_lpss.c               | 16 ++-----
 drivers/acpi/utils.c                   | 48 -------------------
 drivers/firmware/efi/dev-path-parser.c |  7 +--
 drivers/perf/arm_cspmu/arm_cspmu.c     |  4 +-
 include/acpi/acpi_bus.h                | 65 +++++++++++++++++++++++++-
 include/linux/acpi.h                   | 15 ++----
 6 files changed, 74 insertions(+), 81 deletions(-)


base-commit: accf6bb711d6819cce89a122ccde4811c58c160e

Comments

Raag Jadav Dec. 5, 2023, 4:10 a.m. UTC | #1 
On Thu, Nov 23, 2023 at 03:36:12PM +0530, Raag Jadav wrote:
> This series updates the standard ACPI helpers to support _UID matching
> for both integer and string types, and uses them in a couple of places.
> 
> Changes since v2:
> - Drop __builtin functions to reduce complexity (Rafael)
> - Update tags
> 
> Changes since v1:
> - Fix build errors

Bump.

Anything I can do to move this forward?

Raag
Rafael J. Wysocki Dec. 5, 2023, 11:58 a.m. UTC | #2 
On Tue, Dec 5, 2023 at 5:11 AM Raag Jadav <raag.jadav@intel.com> wrote:
>
> On Thu, Nov 23, 2023 at 03:36:12PM +0530, Raag Jadav wrote:
> > This series updates the standard ACPI helpers to support _UID matching
> > for both integer and string types, and uses them in a couple of places.
> >
> > Changes since v2:
> > - Drop __builtin functions to reduce complexity (Rafael)
> > - Update tags
> >
> > Changes since v1:
> > - Fix build errors
>
> Bump.
>
> Anything I can do to move this forward?

It is on my list of pending material, I'll get to it this week (I think).

Thanks!
Rafael J. Wysocki Dec. 6, 2023, 7:44 p.m. UTC | #3 
On Thu, Nov 23, 2023 at 11:07 AM Raag Jadav <raag.jadav@intel.com> wrote:
>
> This series updates the standard ACPI helpers to support _UID matching
> for both integer and string types, and uses them in a couple of places.
>
> Changes since v2:
> - Drop __builtin functions to reduce complexity (Rafael)
> - Update tags
>
> Changes since v1:
> - Fix build errors
>
> Raag Jadav (5):
>   ACPI: bus: update acpi_dev_uid_match() to support multiple types
>   ACPI: bus: update acpi_dev_hid_uid_match() to support multiple types
>   ACPI: LPSS: use acpi_dev_uid_match() for matching _UID
>   efi: dev-path-parser: use acpi_dev_uid_match() for matching _UID
>   perf: arm_cspmu: drop redundant acpi_dev_uid_to_integer()

All applied as 6.8 material, but I renamed the auxiliary macros in the
first patch and moved the kerneldoc comment below them.

TBH, I'm not entirely sure about the value of the second of those
auxiliary macros, but then I don't particularly dislike it either.