On 2022-01-06 2:39 PM, Cezary Rojewski wrote:
...
Your comments and review is much appreciated. While we did separate the
series into chunks, I'm keen to agree we could have moved a little bit
further with the separation. Below you'll find the list of patches and
my thoughts after taking your feedback into consideration. There's also
a TLDR if there's not enough coffee in the pot to cover the summary.
...
TLDR:
Separate series #1:
1/37 ALSA: hda: Add snd_hdac_ext_bus_link_at() helper
2/37 ALSA: hda: Update and expose snd_hda_codec_device_init()
3/37 ALSA: hda: Update and expose codec register procedures
4/37 ALSA: hda: Expose codec cleanup and power-save functions
6/37 ASoC: Export DAI register and widget ctor and dctor functions
Separate series #2:
<everything else not listed here>
Note: patches 21-24/37 get reordered to prepend topology and path
management (currently, patches 18/37 and 19/37 respectively). While
right now I don't see a reason for doing so, this also provides a
possibility for separation or division of these last two mentioned
patches if need be.
Separate series #3:
20/37 ASoC: Intel: avs: Conditional-path support
29/37 ASoC: Intel: avs: Coredump and recovery flow
30/37 ASoC: Intel: avs: Prepare for firmware tracing
31/37 ASoC: Intel: avs: D0ix power state support
32/37 ASoC: Intel: avs: Event tracing
33/37 ASoC: Intel: avs: Machine board registration
36/37 ASoC: Intel: avs: SKL-based platforms support
37/37 ASoC: Intel: avs: APL-based platforms support
The last three could be separated too as all of them touch on isolated
subject: recognize ID: XXX to support YYY.
Hello,
Is the proposal described in my previous message acceptable on your end
Mark?
Regards,
Czarek
