Fix warnings
Warning (avoid_unnecessary_addr_size): /soc@0/display-subsystem@ae00000/dsi@ae94000:
unnecessary #address-cells/#size-cells without "ranges", "dma-ranges" or child "reg" property

Fixes: 73db32b01c9f ("arm64: dts: qcom: sa8775p: add Display Serial Interface device nodes")
Signed-off-by: Eugen Hristev <eugen.hristev@linaro.org>
---
I haven't found a pending patch for this, so here it goes, sorry if someone
already sent.

 arch/arm64/boot/dts/qcom/lemans.dtsi | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/lemans.dtsi b/arch/arm64/boot/dts/qcom/lemans.dtsi
index fd6eb6fbe29a..a551f2a274a2 100644
--- a/arch/arm64/boot/dts/qcom/lemans.dtsi
+++ b/arch/arm64/boot/dts/qcom/lemans.dtsi
@@ -4524,9 +4524,6 @@ mdss0_dsi0: dsi@ae94000 {
 				operating-points-v2 = <&mdss_dsi_opp_table>;
 				power-domains = <&rpmhpd SA8775P_MMCX>;
 
-				#address-cells = <1>;
-				#size-cells = <0>;
-
 				status = "disabled";
 
 				ports {
@@ -4606,9 +4603,6 @@ mdss0_dsi1: dsi@ae96000 {
 				operating-points-v2 = <&mdss_dsi_opp_table>;
 				power-domains = <&rpmhpd SA8775P_MMCX>;
 
-				#address-cells = <1>;
-				#size-cells = <0>;
-
 				status = "disabled";
 
 				ports {
-- 
2.43.0

On Tue, Sep 16, 2025 at 05:19:31PM +0300, Eugen Hristev wrote:
> Fix warnings
> Warning (avoid_unnecessary_addr_size): /soc@0/display-subsystem@ae00000/dsi@ae94000:
> unnecessary #address-cells/#size-cells without "ranges", "dma-ranges" or child "reg" property
> 
> Fixes: 73db32b01c9f ("arm64: dts: qcom: sa8775p: add Display Serial Interface device nodes")
> Signed-off-by: Eugen Hristev <eugen.hristev@linaro.org>
> ---
> I haven't found a pending patch for this, so here it goes, sorry if someone
> already sent.

Thanks, but no. The nodes are correct. I don't think we should be having
the -cells boilerplate each time somebody adds a panel under the DSI
node.

> 
>  arch/arm64/boot/dts/qcom/lemans.dtsi | 6 ------
>  1 file changed, 6 deletions(-)
> 

-- 
With best wishes
Dmitry

On 9/16/25 18:03, Dmitry Baryshkov wrote:
> On Tue, Sep 16, 2025 at 05:19:31PM +0300, Eugen Hristev wrote:
>> Fix warnings
>> Warning (avoid_unnecessary_addr_size): /soc@0/display-subsystem@ae00000/dsi@ae94000:
>> unnecessary #address-cells/#size-cells without "ranges", "dma-ranges" or child "reg" property
>>
>> Fixes: 73db32b01c9f ("arm64: dts: qcom: sa8775p: add Display Serial Interface device nodes")
>> Signed-off-by: Eugen Hristev <eugen.hristev@linaro.org>
>> ---
>> I haven't found a pending patch for this, so here it goes, sorry if someone
>> already sent.
> 
> Thanks, but no. The nodes are correct. I don't think we should be having
> the -cells boilerplate each time somebody adds a panel under the DSI
> node.
> 

I understand your point. But leaving out warnings for a possible future
node is not ok from my point of view.

>>
>>  arch/arm64/boot/dts/qcom/lemans.dtsi | 6 ------
>>  1 file changed, 6 deletions(-)
>>
>

On Tue, Sep 16, 2025 at 06:11:07PM +0300, Eugen Hristev wrote:
> 
> 
> On 9/16/25 18:03, Dmitry Baryshkov wrote:
> > On Tue, Sep 16, 2025 at 05:19:31PM +0300, Eugen Hristev wrote:
> >> Fix warnings
> >> Warning (avoid_unnecessary_addr_size): /soc@0/display-subsystem@ae00000/dsi@ae94000:
> >> unnecessary #address-cells/#size-cells without "ranges", "dma-ranges" or child "reg" property
> >>
> >> Fixes: 73db32b01c9f ("arm64: dts: qcom: sa8775p: add Display Serial Interface device nodes")
> >> Signed-off-by: Eugen Hristev <eugen.hristev@linaro.org>
> >> ---
> >> I haven't found a pending patch for this, so here it goes, sorry if someone
> >> already sent.
> > 
> > Thanks, but no. The nodes are correct. I don't think we should be having
> > the -cells boilerplate each time somebody adds a panel under the DSI
> > node.
> > 
> 
> I understand your point. But leaving out warnings for a possible future
> node is not ok from my point of view.

Then please fix the compiler not to produce those warnings. This applies
to most if not all DSI hosts, not just DSI controllers on Qualcomm
lemans platform.

> 
> >>
> >>  arch/arm64/boot/dts/qcom/lemans.dtsi | 6 ------
> >>  1 file changed, 6 deletions(-)
> >>
> > 
> 

-- 
With best wishes
Dmitry