Re: [PATCH 0/8] ALSA: hda: Add AW88399 HDA side codec driver for Lenovo Legion
Takashi Iwai <[email protected]> Thu, 23 Jul 2026 14:53:16 +0200
| Newsgroups | org.kernel.vger.platform-driver-x86,org.kernel.vger.linux-acpi,org.kernel.vger.linux-kernel,org.kernel.vger.linux-sound |
|---|---|
| Message-ID | <[email protected]> |
On Thu, 23 Jul 2026 14:37:09 +0200, Marco Giunta wrote: > > Hi Takashi, > > Thanks for your useful feedback. > > > Through a quick glance, the code changes look mostly OK. > > But I believe the potential memory leak Sashiko suggested in the patch > > 7 is a real issue, and it should be fixed. > > I agree with this point. I will prepare a v2 and send it in a few days. > > > In anyway, as this series spans over different areas, we'd need proper > > ACK's from ASoC, ACPI and platform people in order to get those merged > > to the upstream. > > Since Mark has already applied patches 1-5 (ASoC library changes) > to broonie/sound for-7.3, should I rebase the remaining 3 patches > (ACPI/platform, HDA driver, realtek fixups) on his tree and send > a v2 just for those? Or would you prefer I do something different? Please just resubmit for the remaining parts with the given Acks. Mark, could you send a pull request including your merged stuff, so that I can apply on the top? thanks, Takashi