arm64: dts: qcom: milos-nothing-asteroids: Drop simplefb

[PATCH] arm64: dts: qcom: milos-nothing-asteroids: Drop simplefb

Posted by Alexander Koskovich 4 days, 17 hours ago

The dummy panel compatible 'boe,bf068mwm-td0' is missing documentation
and gives a warning. The 'interconnects' property is also missing from
the binding.

Dropping the nodes here and will just submit panel binding & driver for
use with MSM DRM.

Signed-off-by: Alexander Koskovich <akoskovich@pm.me>
---
 .../boot/dts/qcom/milos-nothing-asteroids.dts      | 38 ----------------------
 1 file changed, 38 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/milos-nothing-asteroids.dts b/arch/arm64/boot/dts/qcom/milos-nothing-asteroids.dts
index 7393978926e4..787acef4a4c8 100644
--- a/arch/arm64/boot/dts/qcom/milos-nothing-asteroids.dts
+++ b/arch/arm64/boot/dts/qcom/milos-nothing-asteroids.dts
@@ -25,31 +25,7 @@ aliases {
 	};
 
 	chosen {
-		#address-cells = <2>;
-		#size-cells = <2>;
-		ranges;
-
 		stdout-path = "serial0:115200n8";
-
-		framebuffer0: framebuffer@e3940000 {
-			compatible = "simple-framebuffer";
-			reg = <0x0 0xe3940000 0x0 (2392 * 1080 * 4)>;
-			width = <1080>;
-			height = <2392>;
-			stride = <(1080 * 4)>;
-			format = "a8r8g8b8";
-			panel = <&panel>;
-			interconnects = <&mmss_noc MASTER_MDP QCOM_ICC_TAG_ALWAYS
-					 &mc_virt SLAVE_EBI1 QCOM_ICC_TAG_ALWAYS>;
-			clocks = <&gcc GCC_DISP_HF_AXI_CLK>;
-
-			/* Dummy panel for simple-framebuffer dimension info */
-			panel: dummy-panel {
-				compatible = "boe,bf068mwm-td0";
-				height-mm = <157>;
-				width-mm = <71>;
-			};
-		};
 	};
 
 	gpio-keys {
@@ -131,15 +107,6 @@ ramoops@81f20000 {
 			mem-type = <2>;
 			pmsg-size = <0x200000>;
 		};
-
-		/*
-		 * ABL is powering down display and controller if this node is
-		 * not named exactly "splash_region".
-		 */
-		splash_region@e3940000 {
-			reg = <0x0 0xe3940000 0x0 0x2b00000>;
-			no-map;
-		};
 	};
 };
 
@@ -507,11 +474,6 @@ vreg_l12f: ldo12 {
 	};
 };
 
-&dispcc {
-	/* Disable for now so simple-framebuffer continues working */
-	status = "disabled";
-};
-
 &gcc {
 	protected-clocks = <GCC_PCIE_1_AUX_CLK>, <GCC_PCIE_1_AUX_CLK_SRC>,
 			   <GCC_PCIE_1_CFG_AHB_CLK>, <GCC_PCIE_1_MSTR_AXI_CLK>,

---
base-commit: b7bee4ca5688e30ca50fbc87b1b8f7eed7006c17
change-id: 20260603-asteroids-drop-simplefb-0aeb1d1bbadb

Best regards,
-- 
Alexander Koskovich <akoskovich@pm.me>

Re: [PATCH] arm64: dts: qcom: milos-nothing-asteroids: Drop simplefb

Posted by Dmitry Baryshkov 4 days, 17 hours ago

On Wed, Jun 03, 2026 at 12:41:34PM +0000, Alexander Koskovich wrote:
> The dummy panel compatible 'boe,bf068mwm-td0' is missing documentation
> and gives a warning. The 'interconnects' property is also missing from
> the binding.
> 
> Dropping the nodes here and will just submit panel binding & driver for
> use with MSM DRM.
> 
> Signed-off-by: Alexander Koskovich <akoskovich@pm.me>
> ---
>  .../boot/dts/qcom/milos-nothing-asteroids.dts      | 38 ----------------------
>  1 file changed, 38 deletions(-)
> 
> @@ -131,15 +107,6 @@ ramoops@81f20000 {
>  			mem-type = <2>;
>  			pmsg-size = <0x200000>;
>  		};
> -
> -		/*
> -		 * ABL is powering down display and controller if this node is
> -		 * not named exactly "splash_region".
> -		 */

You might want to keep this node.

> -		splash_region@e3940000 {
> -			reg = <0x0 0xe3940000 0x0 0x2b00000>;
> -			no-map;
> -		};
>  	};
>  };
>  
> @@ -507,11 +474,6 @@ vreg_l12f: ldo12 {
>  	};
>  };
>  
> -&dispcc {
> -	/* Disable for now so simple-framebuffer continues working */
> -	status = "disabled";
> -};
> -
>  &gcc {
>  	protected-clocks = <GCC_PCIE_1_AUX_CLK>, <GCC_PCIE_1_AUX_CLK_SRC>,
>  			   <GCC_PCIE_1_CFG_AHB_CLK>, <GCC_PCIE_1_MSTR_AXI_CLK>,
> 
> ---
> base-commit: b7bee4ca5688e30ca50fbc87b1b8f7eed7006c17
> change-id: 20260603-asteroids-drop-simplefb-0aeb1d1bbadb
> 
> Best regards,
> -- 
> Alexander Koskovich <akoskovich@pm.me>
> 
> 

-- 
With best wishes
Dmitry

Re: [PATCH] arm64: dts: qcom: milos-nothing-asteroids: Drop simplefb

Posted by Alexander Koskovich 4 days, 17 hours ago

On Wednesday, June 3rd, 2026 at 8:54 AM, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote:

> On Wed, Jun 03, 2026 at 12:41:34PM +0000, Alexander Koskovich wrote:
> > The dummy panel compatible 'boe,bf068mwm-td0' is missing documentation
> > and gives a warning. The 'interconnects' property is also missing from
> > the binding.
> >
> > Dropping the nodes here and will just submit panel binding & driver for
> > use with MSM DRM.
> >
> > Signed-off-by: Alexander Koskovich <akoskovich@pm.me>
> > ---
> >  .../boot/dts/qcom/milos-nothing-asteroids.dts      | 38 ----------------------
> >  1 file changed, 38 deletions(-)
> >
> > @@ -131,15 +107,6 @@ ramoops@81f20000 {
> >  			mem-type = <2>;
> >  			pmsg-size = <0x200000>;
> >  		};
> > -
> > -		/*
> > -		 * ABL is powering down display and controller if this node is
> > -		 * not named exactly "splash_region".
> > -		 */
> 
> You might want to keep this node.

I have it dropped in my working tree and works fine with MSM DRM,
unless you're thinking of keeping it for out of tree simplefb users?

> 
> > -		splash_region@e3940000 {
> > -			reg = <0x0 0xe3940000 0x0 0x2b00000>;
> > -			no-map;
> > -		};
> >  	};
> >  };
> >
> > @@ -507,11 +474,6 @@ vreg_l12f: ldo12 {
> >  	};
> >  };
> >
> > -&dispcc {
> > -	/* Disable for now so simple-framebuffer continues working */
> > -	status = "disabled";
> > -};
> > -
> >  &gcc {
> >  	protected-clocks = <GCC_PCIE_1_AUX_CLK>, <GCC_PCIE_1_AUX_CLK_SRC>,
> >  			   <GCC_PCIE_1_CFG_AHB_CLK>, <GCC_PCIE_1_MSTR_AXI_CLK>,
> >
> > ---
> > base-commit: b7bee4ca5688e30ca50fbc87b1b8f7eed7006c17
> > change-id: 20260603-asteroids-drop-simplefb-0aeb1d1bbadb
> >
> > Best regards,
> > --
> > Alexander Koskovich <akoskovich@pm.me>
> >
> >
> 
> --
> With best wishes
> Dmitry
> 

Thanks,
Alex

Re: [PATCH] arm64: dts: qcom: milos-nothing-asteroids: Drop simplefb

Posted by Dmitry Baryshkov 4 days, 17 hours ago

On Wed, Jun 03, 2026 at 12:59:44PM +0000, Alexander Koskovich wrote:
> On Wednesday, June 3rd, 2026 at 8:54 AM, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote:
> 
> > On Wed, Jun 03, 2026 at 12:41:34PM +0000, Alexander Koskovich wrote:
> > > The dummy panel compatible 'boe,bf068mwm-td0' is missing documentation
> > > and gives a warning. The 'interconnects' property is also missing from
> > > the binding.
> > >
> > > Dropping the nodes here and will just submit panel binding & driver for
> > > use with MSM DRM.
> > >
> > > Signed-off-by: Alexander Koskovich <akoskovich@pm.me>
> > > ---
> > >  .../boot/dts/qcom/milos-nothing-asteroids.dts      | 38 ----------------------
> > >  1 file changed, 38 deletions(-)
> > >
> > > @@ -131,15 +107,6 @@ ramoops@81f20000 {
> > >  			mem-type = <2>;
> > >  			pmsg-size = <0x200000>;
> > >  		};
> > > -
> > > -		/*
> > > -		 * ABL is powering down display and controller if this node is
> > > -		 * not named exactly "splash_region".
> > > -		 */
> > 
> > You might want to keep this node.
> 
> I have it dropped in my working tree and works fine with MSM DRM,
> unless you're thinking of keeping it for out of tree simplefb users?

I was thinking of making it useful for possible future usage, but in the
end we can always revert it, if the future use arives.


Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>


-- 
With best wishes
Dmitry