Re: [PATCH v8 0/9] Enable video decoder & encoder for MT8189

[Nicolas Dufresne <nicolas.dufresne@collabora.com>]

[-- Attachment #1: Type: text/plain, Size: 1530 bytes --]

Kyrie, Yunfei, Irui,

Le vendredi 20 mars 2026 à 13:59 +0800, Kyrie Wu a écrit :
> 
[...]
> This series patches dependent on:
> [1]
> https://patchwork.linuxtv.org/project/linux-media/patch/20260211054149.27249-2-yunfei.dong@mediatek.com/
> [2]
> https://patchwork.linuxtv.org/project/linux-media/patch/20260302035244.8994-2-irui.wang@mediatek.com/

While this one is now ready, it still can't be merge, because the other series
it depends one have not been updated to use the pdata properly. In fact, its
quite likely that some of the patches in this serie must be moved up the chain.
This type of linked submission is making my life extremely difficult, and causes
massive delays.

Basically, in a chain of 3 series (and probably 4 since the first one does not
apply), the refactoring that causes all the series to not be accepted is
happening in the third one. Angelo and I have been clear for weeks, if you need
a constant value for a platform, it goes in the platform data. Don't do switch
of strcmp at runtime, these are constants and we already did string matching to
in the probe to find this pdata. Once the missing data is added, simply make a
final patch adding the pdata structure for the platform. Use this v8 of the
MT8189 support as reference, it is as I said ready, but it does not apply due to
broken dependencies.

I can't easily fix this, so please work together, synchronize your submissions
so that we can get the MT8196 and MT8189 support finally merged.

regards,
Nicolas

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 228 bytes --]