Re: [PATCH 0/8] ALSA: hda: Add AW88399 HDA side codec driver for Lenovo Legion
Marco Giunta <[email protected]> Thu, 23 Jul 2026 14:37:09 +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 | <DS7PR19MB7724E1F838E331DDE7A99E5CFCC02@DS7PR19MB7724.namprd19.prod.outlook.com> |
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? Patch 6 (ACPI/platform) already has Acked-by tags from Rafael and Ilpo. Best regards, Marco