[PATCH] ASoC: core: clarify the driver name initialization

[Mark Brown]

On Thu, Sep 29, 2022 at 10:06:54AM +0200, Jaroslav Kysela wrote:
> The driver field in the struct snd_ctl_card_info is a valid
> user space identifier. Actually, many ASoC drivers do not care
> and let to initialize this field using a standard wrapping method.

This breaks at least an arm multi_v7_defconfig build:

/build/stage/linux/sound/soc/soc-core.c: In function ‘snd_soc_bind_card’:
/build/stage/linux/sound/soc/soc-core.c:2055:36: error: ‘struct snd_soc_card’ ha
s no member named ‘dmi_longname’
 2055 |         if (card->long_name == card->dmi_longname)
      |                                    ^~


> Also, it does not make sense to set the driver field to the
> card name composed from DMI. This card name is longer in most
> (all?) cases. Use a generic "ASoC-DMI" string here.

This should be a separate change, and DMI is a term specific to the
ACPI/EFI so I don't think we should be using it as a generic here, this
seems like a step back.  If we want to make a change there I'd expect it
to be more picking the actual card driver name.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20220929/b91dbb5c/attachment.sig>