Re: [PATCH 0/8] Reup: SM8350 and SC8280XP venus support

[Johan Hovold <johan@kernel.org>]

Hi Bryan,

On Fri, Apr 04, 2025 at 10:02:47AM +0100, Bryan O'Donoghue wrote:
> On 04/04/2025 06:24, Vikash Garodia wrote:
> >>> How different is this from sm8250 which is already enabled on iris driver ?
> >> As far as I remember, SM8250 support in Iris did not reach
> >> feature-parity yet. So in my opinion it is fine to add new platforms to
> >> the Venus driver, that will later migrate to the Iris driver.
> > I would say, from decoder side all codecs are there now on Iris. H264 merged,
> > while h265 and VP9 dec are posted as RFC, there is one compliance failure which
> > is under debug to post them as regular patches.
> > If we are mainly looking for decode usecases, then we should be on Iris.
> > Preference would be to stay on Iris, otherwise we would have that extra ask to
> > port it later from venus to iris.
> 
> Right now venus represents 9/20 - 45% of the patches being churned for 
> sc8280xp.
> 
> https://github.com/jhovold/linux/tree/wip/sc8280xp-6.14-rc7
> 
> This is a good debate to have, however my memory of what we collectively 
> agreed both in public and private was to continue to merge new silicon 
> <= HFI6XX into venus unless and until iris hit feature parity for HFI6XX 
> and to continue with venus at that point for < HFI6XX.
> 
> So merging sc8280xp - HFI6XX is consistent with our agreement, the right 
> thing to do for our users and a big win in terms of technical debt 
> reduction.
> 
> I will post an update to this series ASAP.

It seems things may be moving again on the firmware front, so could you
please respin this series so we can have video acceleration support for
the X13s in 6.17?

Johan