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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <87o7luic7c.wl-tiwai@suse.de>
Date:   Mon, 05 Jun 2023 09:45:59 +0200
From:   Takashi Iwai <tiwai@...e.de>
To:     Richard Fitzgerald <rf@...nsource.cirrus.com>
Cc:     <tiwai@...e.com>, <perex@...ex.cz>, <broonie@...nel.org>,
        <alsa-devel@...a-project.org>, <linux-kernel@...r.kernel.org>,
        <patches@...nsource.cirrus.com>
Subject: Re: [PATCH v4 00/12] ALSA: hda: Adding support for CS35L56 on HDA systems

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@...e.de>


thanks,

Takashi