| Message ID | 20240828-dts-qcom-label-v1-0-b27b72130247@linaro.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | arm64: dts: qcom: change labels to lower-case | expand |
On 28.08.2024 9:17 AM, Krzysztof Kozlowski wrote: > DTS coding style expects labels to be lowercase. No functional impact. > Verified with comparing decompiled DTB (dtx_diff and fdtdump+diff). > > I am splitting the patchset per few patches doing the same, because > otherwise diffs would be too big and would bounce from Patchwork/mailing > list. > > Best regards, > Krzysztof > > --- I can confirm the changes are a NOP, however: g diff $(git last).. | grep "^+.*&[A-Z]" | wc -l 232 e.g. + domain-idle-states = <&BIG_cpu_sleep_0>; Konrad
On 28/08/2024 12:55, Konrad Dybcio wrote: > On 28.08.2024 9:17 AM, Krzysztof Kozlowski wrote: >> DTS coding style expects labels to be lowercase. No functional impact. >> Verified with comparing decompiled DTB (dtx_diff and fdtdump+diff). >> >> I am splitting the patchset per few patches doing the same, because >> otherwise diffs would be too big and would bounce from Patchwork/mailing >> list. >> >> Best regards, >> Krzysztof >> >> --- > > I can confirm the changes are a NOP, however: > > g diff $(git last).. | grep "^+.*&[A-Z]" | wc -l > 232 > > e.g. > > + domain-idle-states = <&BIG_cpu_sleep_0>; Same as in patch 3. I'll re-check and send a v2 tomorrow. Best regards, Krzysztof
DTS coding style expects labels to be lowercase. No functional impact. Verified with comparing decompiled DTB (dtx_diff and fdtdump+diff). I am splitting the patchset per few patches doing the same, because otherwise diffs would be too big and would bounce from Patchwork/mailing list. Best regards, Krzysztof --- Krzysztof Kozlowski (16): arm64: dts: qcom: ipq: change labels to lower-case arm64: dts: qcom: msm: change labels to lower-case arm64: dts: qcom: sc7180: change labels to lower-case arm64: dts: qcom: sc8280xp: change labels to lower-case arm64: dts: qcom: sc: change labels to lower-case arm64: dts: qcom: sm6115: change labels to lower-case arm64: dts: qcom: sm6350: change labels to lower-case arm64: dts: qcom: sm8150: change labels to lower-case arm64: dts: qcom: sm8350: change labels to lower-case arm64: dts: qcom: sm8250: change labels to lower-case arm64: dts: qcom: sm8450: change labels to lower-case arm64: dts: qcom: sm8550: change labels to lower-case arm64: dts: qcom: sm8650: change labels to lower-case arm64: dts: qcom: sm: change labels to lower-case arm64: dts: qcom: sdm: change labels to lower-case arm64: dts: qcom: change labels to lower-case arch/arm64/boot/dts/qcom/ipq5018.dtsi | 10 +- arch/arm64/boot/dts/qcom/ipq5332.dtsi | 18 +- arch/arm64/boot/dts/qcom/ipq6018.dtsi | 26 +- arch/arm64/boot/dts/qcom/ipq8074.dtsi | 18 +- arch/arm64/boot/dts/qcom/ipq9574.dtsi | 50 +-- arch/arm64/boot/dts/qcom/msm8916.dtsi | 100 +++--- arch/arm64/boot/dts/qcom/msm8939.dtsi | 110 +++---- arch/arm64/boot/dts/qcom/msm8953.dtsi | 68 ++-- arch/arm64/boot/dts/qcom/msm8976.dtsi | 32 +- arch/arm64/boot/dts/qcom/msm8992-lg-h815.dts | 12 +- arch/arm64/boot/dts/qcom/msm8992.dtsi | 4 +- arch/arm64/boot/dts/qcom/msm8994.dtsi | 52 +-- arch/arm64/boot/dts/qcom/msm8996.dtsi | 54 ++-- arch/arm64/boot/dts/qcom/msm8998-clamshell.dtsi | 32 +- arch/arm64/boot/dts/qcom/msm8998.dtsi | 92 +++--- arch/arm64/boot/dts/qcom/qcm2290.dtsi | 68 ++-- arch/arm64/boot/dts/qcom/qcs404.dtsi | 68 ++-- arch/arm64/boot/dts/qcom/qdu1000.dtsi | 76 ++--- arch/arm64/boot/dts/qcom/qrb2210-rb1.dts | 10 +- arch/arm64/boot/dts/qcom/sa8775p.dtsi | 168 +++++----- arch/arm64/boot/dts/qcom/sc7180-firmware-tfa.dtsi | 84 ++--- .../arm64/boot/dts/qcom/sc7180-trogdor-coachz.dtsi | 8 +- .../boot/dts/qcom/sc7180-trogdor-homestar.dtsi | 8 +- .../boot/dts/qcom/sc7180-trogdor-wormdingler.dtsi | 8 +- arch/arm64/boot/dts/qcom/sc7180.dtsi | 344 ++++++++++---------- arch/arm64/boot/dts/qcom/sc7280-chrome-common.dtsi | 6 +- arch/arm64/boot/dts/qcom/sc7280.dtsi | 356 ++++++++++----------- arch/arm64/boot/dts/qcom/sc8180x.dtsi | 146 ++++----- .../dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts | 16 +- arch/arm64/boot/dts/qcom/sc8280xp.dtsi | 140 ++++---- arch/arm64/boot/dts/qcom/sdm630.dtsi | 152 ++++----- arch/arm64/boot/dts/qcom/sdm632.dtsi | 26 +- arch/arm64/boot/dts/qcom/sdm660.dtsi | 16 +- arch/arm64/boot/dts/qcom/sdm670.dtsi | 140 ++++---- arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi | 74 ++--- arch/arm64/boot/dts/qcom/sdm845.dtsi | 160 ++++----- arch/arm64/boot/dts/qcom/sdx75.dtsi | 80 ++--- arch/arm64/boot/dts/qcom/sm4250.dtsi | 16 +- arch/arm64/boot/dts/qcom/sm4450.dtsi | 142 ++++---- arch/arm64/boot/dts/qcom/sm6115.dtsi | 152 ++++----- arch/arm64/boot/dts/qcom/sm6125.dtsi | 52 +-- arch/arm64/boot/dts/qcom/sm6350.dtsi | 172 +++++----- arch/arm64/boot/dts/qcom/sm6375.dtsi | 142 ++++---- arch/arm64/boot/dts/qcom/sm7125.dtsi | 16 +- arch/arm64/boot/dts/qcom/sm7225.dtsi | 16 +- arch/arm64/boot/dts/qcom/sm8150.dtsi | 352 ++++++++++---------- arch/arm64/boot/dts/qcom/sm8250.dtsi | 348 ++++++++++---------- arch/arm64/boot/dts/qcom/sm8350.dtsi | 334 +++++++++---------- arch/arm64/boot/dts/qcom/sm8450.dtsi | 142 ++++---- arch/arm64/boot/dts/qcom/sm8550.dtsi | 144 ++++----- arch/arm64/boot/dts/qcom/sm8650.dtsi | 142 ++++---- arch/arm64/boot/dts/qcom/x1e80100.dtsi | 192 +++++------ 52 files changed, 2597 insertions(+), 2597 deletions(-) --- base-commit: c9795221fa7998bbe2298f904a4c80e0851bbb1d change-id: 20240828-dts-qcom-label-81989fb78b8c Best regards,