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

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

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.

If you do need to support IP versions, then use the correct non client
product name and PCI IDs. Don't confuse users and claim client device
names or IDs that you have no way to support. 

> 
> And then we have validation point. intel-dsp-config remains intact
> (no 
> selection changes for MTL/LNL) but still leaves window for the 
> AVS-validation to have high code coverage.

So this is for internal IP testing reasons and its best to carry these
out of tree like everyone else does. 

> 
> Unless you can bring technical arguments, 'NAK' does not apply. The 
> requirements for the project state clearly - upstream the code for
> the 
> architectures we do support.
> 

Sorry, we have not even crossed the threshold for a technical argument,
this is an unsupported driver and an unsupported FW for these devices
on Linux. Period. 

The SOF FW and driver is the supported driver and FW for these devices
on Linux.

Please do point me internally to where AVS is plan of record for Linux
client MTL, LNL and PTL and I will approve. 
 
Thanks

Liam
> 
> Kind regards,
> Czarek