[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