Re: [PATCH v4 00/12] ALSA: hda: Adding support for CS35L56 on HDA systems

From: Takashi Iwai
Date: Mon Jun 05 2023 - 03:46:52 EST


On Thu, 01 Jun 2023 17:47:19 +0200,
Richard Fitzgerald wrote:
>
> This set of patches adds support for using the CS35L56 boosted smart
> amplifier on HDA systems. In these systems the CS35L56 audio is
> routed through a HDA-to-I2S bridge codec.
>
> This doesn't include the changes to the Realtek driver to actually hook
> up the CS35L56 driver, because we don't yet have the QUIRK IDs to
> associate it with. But we want to publish the driver now so that it is
> available for bringing up hardware with the CS35L56.
>
> The first 9 patches are moving code out of the ASoC driver and into the
> shared library so that it can be shared with the HDA driver.
>
> Patch #10 fixes missing #includes in the HDA headers so that the CS35L56
> driver doesn't have to #include headers that it doesn't use.
>
> Finally, #11 and #12 actually add the support for CS35L56 on HDA.
>
> CHANGES SINCE V3
> Patch 01:
> - Fix double-free of IRQ caused by passing wrong cookie to devm_irq_free()
> Patch 12:
> - Fix memory leak of struct firmware object and firmware name strings.
> - Find and open firmware files before taking the irq_lock mutex and pm_runtime.
> - Don't bother executing the firmware loading code if no firmware files were
> found.
>
> Richard Fitzgerald (4):
> ASoC: cs35l56: Move runtime suspend/resume to shared library
> ASoC: cs35l56: Move cs_dsp init into shared library
> ASoC: cs35l56: Move part of cs35l56_init() to shared library
> ALSA: hda: Fix missing header dependencies
>
> Simon Trimmer (8):
> ASoC: cs35l56: Move shared data into a common data structure
> ASoC: cs35l56: Make cs35l56_system_reset() code more generic
> ASoC: cs35l56: Convert utility functions to use common data structure
> ASoC: cs35l56: Move utility functions to shared file
> ASoC: cs35l56: Make common function for control port wait
> ASoC: cs35l56: Make a common function to shutdown the DSP
> ALSA: hda: Add mute_hook to hda_component
> ALSA: hda/cs35l56: Add driver for Cirrus Logic CS35L56 amplifier

It'd be likely easier to merge all those via ASoC tree, I suppose, as
some patches seem to have prerequisite in the recent ASoC changes.
So, Mark, could you take those through yours?

Feel free to put my ack:

Reviewed-by: Takashi Iwai <tiwai@xxxxxxx>


thanks,

Takashi