Re: [PATCH 00/12] ASoC: Intel: avs: Add support for MTL-FCL platforms

["Girdwood, Liam R" <liam.r.girdwood@intel.com>]

On Mon, 2025-04-07 at 10:04 +0200, Cezary Rojewski wrote:
> On 2025-04-04 3:28 PM, Girdwood, Liam R wrote:
> > On Fri, 2025-04-04 at 15:09 +0200, Cezary Rojewski wrote:
> > > On 2025-04-04 2:52 PM, Girdwood, Liam R wrote:
> > > > On Fri, 2025-04-04 at 11:49 +0200, Cezary Rojewski wrote:
> > > > > Cezary Rojewski (11):
> > > > >     ALSA: hda: Allow to fetch hlink by ID
> > > > >     ASoC: Intel: avs: Ignore Vendor-space manipulation for
> > > > > ACE
> > > > >     ASoC: Intel: avs: Read HW capabilities when possible
> > > > >     ASoC: Intel: avs: Relocate DSP status registers
> > > > >     ASoC: Intel: avs: MTL-based platforms support
> > > > >     ASoC: Intel: avs: LNL-based platforms support
> > > > >     ASoC: Intel: avs: PTL-based platforms support
> > > > >     ASoC: Intel: avs: PCM operations for LNL-based platforms
> > > > > 
> > > > Sorry - AVS is not officially supported for MTL, LNL and PTL on
> > > > Linux.
> > > > 
> > > > NAK.
> > > 
> > > 
> > > FCL is a PTL-A, PTL-based platform. The driver is officially
> > > supported
> > > on the ACE architecture with clients utilizing it on
> > > distributions
> > > such
> > > as Android and Yocto. As software needs little to no changes to
> > > cover
> > > PTL alone it is preferable to have clear division: ace1.0 <-
> > > ace2.0
> > > <-
> > > ace3.0. The code is maintainable and readable this way.
> > 
> > No argument with FCL or any non client devices.
> 
> I see. So we're talking about pci_ids and their attached descriptors,
> and _not_ removing the functional code that enables avs-driver on ACE
> architecture?

Yes, lets not cause any confusion with duplicated IDs or product names
across different drivers. Not removing any AVS code :)

> 
> The filenames/prefixes shall remain as-is given the driver's design -
> platform that starts given arch, becomes its namesake.
> 

No issue if you want to rename any IP code filename to e.g. ACE2
instead of LNL as this makes it obvious for anyone reading the code and
you can add my  Ack to that if wanted.

Thanks

Liam

> Kind regards,
> Czarek