Re: [PATCH v6 14/15] arm64: dts: qcom: sdm845-lg-{judyln, judyp}: reference memory region in fb

[Dmitry Baryshkov]

On Mon, Mar 23, 2026 at 07:42:14PM -0700, Paul Sajna wrote:
> This cleans up the node a bit and prevents duplicating the region specs
> in the reg field

To prevent duplicating the framebuffer adress and size point out the
existing framebuffer memory region instead of specifying the address
manually.

> 
> Signed-off-by: Paul Sajna <sajattack@xxxxxxxxxxxxxxxx>
> ---
>  arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi | 3 +--
>  arch/arm64/boot/dts/qcom/sdm845-lg-judyln.dts  | 2 +-
>  arch/arm64/boot/dts/qcom/sdm845-lg-judyp.dts   | 2 +-
>  3 files changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi b/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi
> index 51fc1735ff49..f42800dd25be 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi
> @@ -98,8 +98,7 @@ spss_mem: memory@99000000 {
>  			no-map;
>  		};
>  
> -		/* Framebuffer region */
> -		memory@9d400000 {
> +		framebuffer_mem: memory@9d400000 {
>  			reg = <0x0 0x9d400000 0x0 0x2400000>;
>  			no-map;
>  		};
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-lg-judyln.dts b/arch/arm64/boot/dts/qcom/sdm845-lg-judyln.dts
> index adf41aa0146a..83e392907244 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-lg-judyln.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm845-lg-judyln.dts
> @@ -16,7 +16,7 @@ / {
>  	chosen {
>  		framebuffer@9d400000 {
>  			compatible = "simple-framebuffer";
> -			reg = <0x0 0x9d400000 0x0 (1440 * 3120 * 4)>;
> +			memory-region = <&framebuffer_mem>;
>  			width = <1440>;
>  			height = <3120>;
>  			stride = <(1440 * 4)>;
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-lg-judyp.dts b/arch/arm64/boot/dts/qcom/sdm845-lg-judyp.dts
> index ffe1da2227f0..ae6cbd132949 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-lg-judyp.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm845-lg-judyp.dts
> @@ -16,7 +16,7 @@ / {
>  	chosen {
>  		framebuffer@9d400000 {
>  			compatible = "simple-framebuffer";
> -			reg = <0x0 0x9d400000 0x0 (1440 * 2880 * 4)>;
> +			memory-region = <&framebuffer_mem>;
>  			width = <1440>;
>  			height = <2880>;
>  			stride = <(1440 * 4)>;
> 
> -- 
> 2.53.0
> 

-- 
With best wishes
Dmitry