Re: [PATCH v3 2/3] dt-bindings: ufs: qcom: classify sm8150-ufshc as standalone ICE

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Medium] dt-bindings: ufs: qcom: moving sm8150-ufshc breaks existing
  2-reg/9-clock DTs
--

commit 830241ce50cfd60c40fb1ab69965eff9ecfda73c
Author: Andrew Gigena <mail@andrewgigena.dev>

dt-bindings: ufs: qcom: classify sm8150-ufshc as standalone ICE

This commit moves the qcom,sm8150-ufshc compatible from
Documentation/devicetree/bindings/ufs/qcom,ufs.yaml to
Documentation/devicetree/bindings/ufs/qcom,sc7180-ufshc.yaml and updates
the example in qcom,ufs.yaml to use qcom,sdm845-ufshc.

> diff --git a/Documentation/devicetree/bindings/ufs/qcom,sc7180-ufshc.yaml b/Documentation/devicetree/bindings/ufs/qcom,sc7180-ufshc.yaml
> index 3c407426d6972..572eec814a604 100644
> --- a/Documentation/devicetree/bindings/ufs/qcom,sc7180-ufshc.yaml
> +++ b/Documentation/devicetree/bindings/ufs/qcom,sc7180-ufshc.yaml
> @@ -23,6 +23,7 @@ select:
>            - qcom,sc7280-ufshc
>            - qcom,sc8180x-ufshc
>            - qcom,sc8280xp-ufshc
> +          - qcom,sm8150-ufshc
>            - qcom,sm8250-ufshc
>            - qcom,sm8350-ufshc
>            - qcom,sm8450-ufshc
> @@ -48,6 +49,7 @@ properties:
>                - qcom,sc7280-ufshc
>                - qcom,sc8180x-ufshc
>                - qcom,sc8280xp-ufshc
> +              - qcom,sm8150-ufshc
>                - qcom,sm8250-ufshc
>                - qcom,sm8350-ufshc
>                - qcom,sm8450-ufshc

[Severity: Medium]
As reported by Krzysztof Kozlowski on v2 [1], does moving
qcom,sm8150-ufshc to qcom,sc7180-ufshc.yaml break dtbs_check for existing
SM8150 and SA8155P device trees?

qcom,sc7180-ufshc.yaml restricts reg to maxItems: 1 and clocks to
maxItems: 8, rejecting the second ice register region and the ninth
ice_core_clk clock used by the embedded-ICE representation (supported
since 462c5c0aa798 ("dt-bindings: ufs: qcom,ufs: convert to dtschema")).
Later commit "arm64: dts: qcom: sm8150: use standalone ICE for UFS" in
this series converts the in-tree sm8150.dtsi to use a standalone ICE node,
but dtbs_check fails at this commit (and across subsystem trees if UFS
bindings and QCOM DTS changes are merged separately) and schema backwards
compatibility with existing 2-reg/9-clock DTBs is broken.

[1] https://lore.kernel.org/r/20261106-offbeat-hawk-of-purring-50a8aa@quoll

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261111-sm8150-ice-rfc-v3-0-f9d5a806fc95@andrewgigena.dev?part=2