[GIT,PULL,1/2] MediaTek ARM64 DTS updates for v6.16

AngeloGioacchino Del Regno angelogioacchino.delregno at collabora.com
Wed May 14 01:23:33 PDT 2025


Il 09/05/25 22:54, Arnd Bergmann ha scritto:
> On Tue, May 6, 2025, at 11:17, AngeloGioacchino Del Regno wrote:
> 
>>     - Since SCP firmwares are available on linux-firmware, the
>>       necessary nodes for the dual-core SCP were added to devicetree
>>       along with the firmware-name property in mt8390-genio-common,
>>       enabling hardware video encoders and decoders on the MediaTek
>>       Genio 510 and Genio 700 EVKs.
> 
> This looks wrong to me: the devicetree should never care about
> particular file names in a Linux repository.
> 
> Usually what happens is that the device driver must work with
> a particular firmware version, so the driver is the one that
> constructs a name based on what firmware is knows and what
> hardware it is running on, so a newer version of the driver
> can attempt to load a newer firmware that may be incompatible
> with the old driver.
> 
> The other patches in the branch all look fine to me.
> 
> Can you resend the pull request without these patches and
> try to come up with a different approach?
> 
>       Arnd

Uhm, I'm not sure if there was a misunderstanding - not only the MediaTek
devicetrees do that, but also Qualcomm ones are doing the same with their
adsp, cdsp, modem and adreno zap firmwares...

....not sure, should I drop those commits for now?

Thanks,
Angelo



More information about the Linux-mediatek mailing list