Re: [PATCH v2 00/14] ASoC: Intel: Remove skylake driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]



On 2024-08-14 4:42 PM, Andy Shevchenko wrote:
On Wed, Aug 14, 2024 at 10:39:15AM +0200, Cezary Rojewski wrote:
A spiritual successor to haswell/baytrail removal series [1].

The avs-driver found in sound/soc/intel/avs is a direct replacement to
the existing skylake-driver. It covers all features supported by it and
more and aligns with the recommended flows and requirements based on
Windows driver equivalent.

The skylake-driver related UAPI has been removed with "ASoC: Drop
soc-topology ABI v4 support" [2].

For the official kernel tree the deprecation begun with v6.0. Most
skylake-drivers users moved to avs- or SOF-driver when AudioDSP
capabilities are available on the platform or to snd-hda-intel
(sound/pci/hda) when such capabilities are not.

For the supported trees the deprecation begun with v5.4 with v5.15 being
the first where the skylake-driver is disabled entirely.

All machine board drivers that consume this DSP driver have their
replacements present within sound/soc/intel/avs/boards/ directory.

Changes in v2:
- added two new patches (leading the series)
- the first moves all device entries previously under the skylake-driver
   dependency under the avs-driver
- the second patch drops struct skl_machine_pdata usage to fix
   compilation errors as pointed out by IKP
- no further changes, retained Acked-by tags for these

Hmm... Have you received my tag for the v1?

Acked-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>

Yeah, I've left the tags alone for patches 3-14 but for the newly added two I did not as that would be inappropriate. The first two patches address the problems reported by IKP bot - I've not been dealing with sound/soc/intel/common/ for a while and have forgotten about its existence.

Thank you for reviewing the series!

Kind regards,
Czarek




[Index of Archives]     [Pulseaudio]     [Linux Audio Users]     [ALSA Devel]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux