On Tue, Sep 24, 2019 at 10:37:36PM +0300, Andy Shevchenko wrote:
> There are users outside of ACPI realm which reimplementing the comparator
> function to check if the given device matches to given HID and UID.
>
> For better utilization, introduce a helper for everyone to use.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
>
> ---
> drivers/acpi/utils.c | 25 +++++++++++++++++++++++++
> include/acpi/acpi_bus.h | 2 ++
> include/linux/acpi.h | 6 ++++++
> 3 files changed, 33 insertions(+)
>
> diff --git a/drivers/acpi/utils.c b/drivers/acpi/utils.c
> index dbd1c4cfd7d1..804ac0df58ec 100644
> --- a/drivers/acpi/utils.c
> +++ b/drivers/acpi/utils.c
> @@ -701,6 +701,31 @@ bool acpi_check_dsm(acpi_handle handle, const guid_t *guid, u64 rev, u64 funcs)
> }
> EXPORT_SYMBOL(acpi_check_dsm);
>
> +/**
> + * acpi_dev_hid_uid_match - Match device by supplied HID and UID
> + * @adev: ACPI device to match.
> + * @hid2: Hardware ID of the device.
> + * @uid2: Unique ID of the device, pass NULL to not check _UID.
> + *
> + * Matches HID and UID in @adev with given @hid2 and @uid2.
> + * Returns true if matches.
> + */
> +bool acpi_dev_hid_uid_match(struct acpi_device *adev,
> + const char *hid2, const char *uid2)
> +{
> + const char *hid1 = acpi_device_hid(adev);
> + const char *uid1 = acpi_device_uid(adev);
> +
> + if (strcmp(hid1, hid2))
> + return false;
> +
> + if (!uid2)
> + return true;
> +
> + return uid1 && !strcmp(uid1, uid2);
> +}
> +EXPORT_SYMBOL(acpi_dev_hid_uid_match);
Should this be _GPL?
In any case looks good,
Reviewed-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>
