Re: [PATCH 04/23] ASoC: soc-compress: Implement trigger FE-BE sequencing as with normal PCMs

[Vinod Koul <vkoul@kernel.org>]

On 07-09-26, 21:55, Pierre-Louis Bossart wrote:
> On 9/7/26 18:53, Vinod Koul wrote:
> > On 07-09-26, 17:03, Péter Ujfalusi wrote:
> >>
> >>
> >> On 07/09/2026 14:26, Pierre-Louis Bossart wrote:
> >>>>> ...otherwise it'll end up with -EINVAL instead of getting passed
> >>>>> through.  Also a switch statement would be a bit more idiomatic.
> >>>>
> >>>> OK, I will add the NEXT_TRACK and convert the if to switch.
> >>>
> >>> ICYMI for testing, NEXT_TRACK and PARTIAL_DRAIN are only enabled in the
> >>> 'fcplay' application with the '-p 1' argument for gapless playback, not
> >>> in the initial cplay application.
> >>
> >> With SOF/Intel/IPC4 we can only support DRAIN, NEXT_TRACK and
> >> PARTIAL_DRAIN are not supported, but there were mismatch between fcplay
> > 
> > Those should be optional always. Gapless support should not be mandatory
> well in practice it's a non-negotiable requirement from users... We had
> to add this as the request of the Android team, and it's a standard
> feature on most players, not a nice-to-have.

It may be non-negotiable requirement from user/OS but should not be
mandatory from API :-)

> Note that the NEXT_TRACK only works if the two consecutive tracks have
> the same format, if the platform wants 'perceived' gapless playback with
> a x-fade then two devices are required.

Correct and given the fact the people use apps on this ecosystem, I
would expect the formats will be same for an app

-- 
~Vinod