Re: [PATCH 00/31] sound: Phase out hybrid PCI devres API

On Wed, Apr 16, 2025 at 03:12:10PM +0200, Philipp Stanner wrote:
Hi,
a year ago we spent quite some work trying to get PCI into better shape. Some pci_ functions can be sometimes managed with devres, which is obviously bad. We want to provide an obvious API, where pci_ functions are never, and pcim_ functions are always managed.
Thus, everyone enabling his device with pcim_enable_device() must be ported to pcim_ functions. Porting all users will later enable us to significantly simplify parts of the PCI subsystem. See here [1] for details.
This patch series does that for sound.
AFAIK the ASoC and ALSA maintained by different people and perhaps you would need to split, but I'm not the guy in charge, so wait for them to tell you their preferences.

On Wed, Apr 16, 2025 at 06:34:35PM +0300, Andy Shevchenko wrote:
On Wed, Apr 16, 2025 at 03:12:10PM +0200, Philipp Stanner wrote:
a year ago we spent quite some work trying to get PCI into better shape. Some pci_ functions can be sometimes managed with devres, which is obviously bad. We want to provide an obvious API, where pci_ functions are never, and pcim_ functions are always managed.
Thus, everyone enabling his device with pcim_enable_device() must be ported to pcim_ functions. Porting all users will later enable us to significantly simplify parts of the PCI subsystem. See here [1] for details.
This patch series does that for sound.
AFAIK the ASoC and ALSA maintained by different people and perhaps you would need to split, but I'm not the guy in charge, so wait for them to tell you their preferences.
Seems split is a good to have as I just looked to all of them and ALSA are pretty straightforward, while for every patch in ASoC I have a comment.
So, here is a formal tag for all ALSA patches. Feel free to send a v2 with them and the tag.
Reviewed-by: Andy Shevchenko andriy.shevchenko@linux.intel.com
participants (1)
-
Andy Shevchenko