diff mbox series

[v2,3/7] arm64: dts: qcom: sm8550: remove address/size-cells from mdss_dsi1

Message ID 20231207092801.7506-4-quic_tengfan@quicinc.com
State New
Headers show
Series [v2,1/7] dt-bindings: arm: qcom: Document qcs8550 SoC and board | expand

Commit Message

Tengfei Fan Dec. 7, 2023, 9:27 a.m. UTC
Address/size-cells will cause dtbs check warning, because mdss_dsi1 node
have not ranges and child also have not reg, so remove address/size-cells.

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com>
---
 arch/arm64/boot/dts/qcom/sm8550.dtsi | 3 ---
 1 file changed, 3 deletions(-)

Comments

Krzysztof Kozlowski Dec. 11, 2023, 10:25 a.m. UTC | #1
On 07/12/2023 10:27, Tengfei Fan wrote:
> Address/size-cells will cause dtbs check warning, because mdss_dsi1 node
> have not ranges and child also have not reg, so remove address/size-cells.

Paste the excerpt of warning.

> 
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com>
> ---

I am not sure if this is correct. The address/size cells are there for
panels. We have multiple of such warnings, so I would like to have here
consensus for all of them.

Best regards,
Krzysztof
Tengfei Fan Dec. 18, 2023, 9:59 a.m. UTC | #2
在 12/11/2023 6:25 PM, Krzysztof Kozlowski 写道:
> On 07/12/2023 10:27, Tengfei Fan wrote:
>> Address/size-cells will cause dtbs check warning, because mdss_dsi1 node
>> have not ranges and child also have not reg, so remove address/size-cells.
> 
> Paste the excerpt of warning.
Sure, I will paste the excerpt of warning.
> 
>>
>> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>> Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com>
>> ---
> 
> I am not sure if this is correct. The address/size cells are there for
> panels. We have multiple of such warnings, so I would like to have here
> consensus for all of them.
Currently these address/size cells lead to dtc W=1 warning, so these 
should be dropped. These can be re-added if needed at a later date.
> 
> Best regards,
> Krzysztof
>
Krzysztof Kozlowski Dec. 18, 2023, 10:13 a.m. UTC | #3
On 18/12/2023 10:59, Tengfei Fan wrote:
> 
> 
> 在 12/11/2023 6:25 PM, Krzysztof Kozlowski 写道:
>> On 07/12/2023 10:27, Tengfei Fan wrote:
>>> Address/size-cells will cause dtbs check warning, because mdss_dsi1 node
>>> have not ranges and child also have not reg, so remove address/size-cells.
>>
>> Paste the excerpt of warning.
> Sure, I will paste the excerpt of warning.
>>
>>>
>>> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>>> Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com>
>>> ---
>>
>> I am not sure if this is correct. The address/size cells are there for
>> panels. We have multiple of such warnings, so I would like to have here
>> consensus for all of them.
> Currently these address/size cells lead to dtc W=1 warning, so these 

I know.

> should be dropped. 

Why? They are correct.

> These can be re-added if needed at a later date.

Added where? These are rather properties of SoC.

This does not solve any of my concerns.

Best regards,
Krzysztof
Tengfei Fan Dec. 19, 2023, 12:40 a.m. UTC | #4
在 12/18/2023 6:13 PM, Krzysztof Kozlowski 写道:
> On 18/12/2023 10:59, Tengfei Fan wrote:
>>
>>
>> 在 12/11/2023 6:25 PM, Krzysztof Kozlowski 写道:
>>> On 07/12/2023 10:27, Tengfei Fan wrote:
>>>> Address/size-cells will cause dtbs check warning, because mdss_dsi1 node
>>>> have not ranges and child also have not reg, so remove address/size-cells.
>>>
>>> Paste the excerpt of warning.
>> Sure, I will paste the excerpt of warning.
>>>
>>>>
>>>> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>>>> Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com>
>>>> ---
>>>
>>> I am not sure if this is correct. The address/size cells are there for
>>> panels. We have multiple of such warnings, so I would like to have here
>>> consensus for all of them.
>> Currently these address/size cells lead to dtc W=1 warning, so these
> 
> I know.
> 
>> should be dropped.
> 
> Why? They are correct.
> 
>> These can be re-added if needed at a later date.
> 
> Added where? These are rather properties of SoC.
> 
> This does not solve any of my concerns.
Hi Krzysztof,
Sorry, a new patch has been submitted to upstream before seeing your reply.
I will do more sync with internal teams, please ignore new patch series 
before we cannot solve your concerns.

> 
> Best regards,
> Krzysztof
>
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/sm8550.dtsi b/arch/arm64/boot/dts/qcom/sm8550.dtsi
index 17c4f0a7638a..2c6f2ed458bb 100644
--- a/arch/arm64/boot/dts/qcom/sm8550.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8550.dtsi
@@ -2771,9 +2771,6 @@ 
 				phys = <&mdss_dsi1_phy>;
 				phy-names = "dsi";
 
-				#address-cells = <1>;
-				#size-cells = <0>;
-
 				status = "disabled";
 
 				ports {