Re: [PATCH v1 1/2] ALSA: hda/tas2781: Update tas2781 HDA driver

From: Andy Shevchenko
Date: Mon Sep 04 2023 - 06:17:37 EST


On Sun, Sep 03, 2023 at 10:37:57PM +0800, Shenghao Ding wrote:
> Support ACPI_ID both TXNW2781 and TIAS2781, and revert structure
> cs35l41_dev_name.

...

> + /* exactly match either TXNW2781 or TIAS2781 */
> + /* the rest must be exact matching */
> + snprintf(tmp, sizeof(tmp), "-%s:00", "TXNW2781");
> +
> + if (!strcmp(d + n, tmp))
> + return 1;
> +
> /* the rest must be exact matching */
> - snprintf(tmp, sizeof(tmp), "-%s:00", p->hid);
> + snprintf(tmp, sizeof(tmp), "-%s:00", "TIAS2781");

Basically this should use acpi_dev_match infra like Intel drivers do.
But I haven't read code deeply to understand if it's the case here or
not.

--
With Best Regards,
Andy Shevchenko