Re: [PATCH 11/13] arm: dts: mt8188: add framebuffer and splash reserved-memory

On 7/28/26 7:29 AM, Julien Stephan wrote:
> These regions are specific to U-Boot, so define them in
> mt8188-u-boot.dtsi. They are placed at 0x70000000 to stay clear of the
> reserved-memory pools declared by the boards using this SoC (the Genio
> EVKs reserve the range 0x60000000-0x61100000 for the ADSP and AFE DMA
> pools) and of the kernel load window (the Genio kernel FIT images load
> and decompress the kernel at 0x64000000).
> 
> Signed-off-by: Julien Stephan <[email protected]>
> ---
>  arch/arm/dts/mt8188-u-boot.dtsi | 19 +++++++++++++++++++
>  1 file changed, 19 insertions(+)
> 
> diff --git a/arch/arm/dts/mt8188-u-boot.dtsi b/arch/arm/dts/mt8188-u-boot.dtsi
> index fd1121a72cd..923e52522de 100644
> --- a/arch/arm/dts/mt8188-u-boot.dtsi
> +++ b/arch/arm/dts/mt8188-u-boot.dtsi
> @@ -5,6 +5,25 @@
>   */
>  
>  / {
> +	reserved-memory {
> +		/*
> +		 * Keep these regions clear of the board reserved-memory
> +		 * pools (the boards using this SoC reserve regions up to
> +		 * 0x61100000 for the adsp/afe DMA pools) and of the kernel
> +		 * load window: the kernel FIT images load and decompress
> +		 * the kernel at 0x64000000.
> +		 */
> +		splashscreen: memory@70000000 {
> +			reg = <0 0x70000000 0 0x02000000>;
> +			no-map;
> +		};
> +
> +		framebuffer: memory@72000000 {
> +			reg = <0 0x72000000 0 0x02000000>;
> +			no-map;
> +		};
> +	};
> +

I'm not sure everyone using mt8188 compatible chips will want/need this
reserved memory if they don't need the splashscreen. Maybe this is better
placed in mt8390-genio-common-u-boot.dtsi (following upstream dts naming
convention)?


>  	vdosys1_pipeline: vdosys1-pipeline {
>  		compatible = "mediatek,mt8188-vdosys1-pipeline";
>  		reg = <0 0x1c100000 0 0x1000>;
>