On 09/12/2022 15:29, Luca Weiss wrote:
> The code in ufs-qcom-ice.c needs the ICE reg to be named "ice". Add this
> in the bindings so the existing dts can validate successfully.
>
> Also sm8450 is using ICE since commit 276ee34a40c1 ("arm64: dts: qcom:
> sm8450: add Inline Crypto Engine registers and clock") so move the
> compatible to the correct if.
>
> Signed-off-by: Luca Weiss <luca.weiss@xxxxxxxxxxxxx>
> ---
> (no cover subject)
>
> The only remaining validation issues I see is the following on sc8280xp-crd.dtb
> and sa8540p-ride.dtb:
>
> Unevaluated properties are not allowed ('required-opps', 'dma-coherent' were unexpected)
>
> Maybe someone who knows something about this can handle this?
>
> And the patch adding qcom,sm6115-ufshc hasn't been applied yet.
> ---
> Documentation/devicetree/bindings/ufs/qcom,ufs.yaml | 8 +++++++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml b/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml
> index f2d6298d926c..58a2fb2c83c3 100644
> --- a/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml
> +++ b/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml
> @@ -102,7 +102,6 @@ allOf:
> - qcom,sc8280xp-ufshc
> - qcom,sm8250-ufshc
> - qcom,sm8350-ufshc
> - - qcom,sm8450-ufshc
> then:
> properties:
> clocks:
> @@ -130,6 +129,7 @@ allOf:
> - qcom,sdm845-ufshc
> - qcom,sm6350-ufshc
> - qcom,sm8150-ufshc
> + - qcom,sm8450-ufshc
> then:
> properties:
> clocks:
> @@ -149,6 +149,12 @@ allOf:
> reg:
> minItems: 2
> maxItems: 2
> + reg-names:
There are no reg-names in top-level, so it's surprising to see its
customized here. It seems no one ever documented that usage...
> + items:
> + - const: std
> + - const: ice
> + required:
> + - reg-names
>
> - if:
> properties:
>
> ---
> base-commit: f925116b24c0c42dc6d5ab5111c55fd7f74e8dc7
> change-id: 20221209-dt-binding-ufs-2d7f64797ff2
>
> Best regards,
Best regards,
Krzysztof
