On Tue, May 10, 2022 at 04:18:32PM +0200, Michael Walle wrote:
From: Alison Wang alison.wang@nxp.com
This patch adds config support for HDP firmware loading on LS1028A.
FWIW, I really don't like this as this is just for a downstream kernel.
This => what? I only see a config option. I searched the list and the source code for CONFIG_VIDEO_LS_HDP_LOAD and I don't see any user...
For the upstream kernel the firmware should be loaded by the kernel itself.
In principle I agree.
But unfortunately, there is no development there. (I have some half baked patches though and can confirm, that firmware loading is working from linux). Instead of this I'd like to see some work to support the DP PHY in upstream linux as this is really the last missing piece to get graphics working.
It is? What did I miss?
Now this is purely board related code, so it is probably fine. But in the past the handling of purly downstream stuff was removed from upstream u-boot anyway in favor of the upstream method. i.e. fixup of the ls1028a-gpu node.
-michael