Re: [PATCH v3 1/1] arm64: dts: qcom: sm8550: remove address/size-cells from mdss_dsi1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 12/19/2023 3:17 PM, Krzysztof Kozlowski wrote:
On 19/12/2023 01:31, Tengfei Fan wrote:
The address/size-cells in mdss_dsi1 node have not ranges and child also
have not reg, then this leads to dtc W=1 warnings:

Comments can be more readable:
"mdss_dsi1" node don't have "ranges" or child "reg" property, while it have address/size-cells properties. This caused "avoid_unnecessary_addr_size" warning from dtb check.
Remove address/size-cells properties for "mdss_dsi1" node.

I cannot parse it. Address/size cells never have ranges or children.
They cannot have. These are uint32 properties.
Pls help to comment on the revised commit message. Every time I write a commit message, also takes a while for me to double confirm whether others can understand me correctly as well. Feel free to let us know if it is not readable to you. It will help us as non-English native developers.


   sm8550.dtsi:2937.27-2992.6: Warning (avoid_unnecessary_addr_size): /soc@0/display-subsystem@ae00000/dsi@ae96000:
     unnecessary #address-cells/#size-cells without "ranges" or child "reg" property


Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
Signed-off-by: Tengfei Fan <quic_tengfan@xxxxxxxxxxx>
---

I disagreed with the patch before. You resubmit it without really
addressing my concerns.

I am not sure if this is correct fix and I want to fix all of such
errors (there are multiple of them) in the same way. Feel free to
propose common solution based on arguments.
Per my understanding, "qcom,mdss-dsi-ctrl" driver node like "mdss_dsi1" don't need to have address/size-cells properties. Feel free to let us know whether there is different idea of "address/size-cells" needed for the "qcom,mdss-dsi-ctrl" driver node.

If there is no valid reason to add "address/size-cells" properties for "qcom,mdss-dsi-ctrl" driver node, the plan is that: Remove address/size-cells properties for "qcom,mdss-dsi-ctrl" compatible node. While there are about 22 different soc dtsi and it's document binding files needed to be fixed.Shall we fix it for all qcom related soc usage in one patch, or we'd better to split into different patches according to soc specifically?

I don't really want to NAKit but since you are resending without
finishing the discussion, which is quite impolite, then let's be like that:
Acked your feelings. Feel free to NAK when you thought it is right thing to do. :)

NAK

Best regards,
Krzysztof


--
Thx and BRs,
Aiqun(Maria) Yu




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux