Re: [PATCH v2 1/2] dt-bindings: crypto: ice: Document sm8150 inline crypto engine

[Krzysztof Kozlowski]

On Wed, Nov 04, 2026 at 06:38:12PM +0000, Andrew Gigena wrote:
> Document the compatible used for the inline crypto engine found on
> SM8150.
> 
> SM8150 UFS uses a standalone ICE node referenced via qcom,ice, so move
> qcom,sm8150-ufshc from the legacy embedded-ICE UFS binding to the
> standalone-ICE UFS binding. Update the legacy binding example to use
> qcom,sdm845-ufshc, which still uses the embedded ICE representation.
> 

This part should be a separate commit.

> Signed-off-by: Andrew Gigena <mail@xxxxxxxxxxxxxxxx>
> ---
>  .../devicetree/bindings/crypto/qcom,inline-crypto-engine.yaml |  1 +
>  Documentation/devicetree/bindings/ufs/qcom,sc7180-ufshc.yaml  |  2 ++
>  Documentation/devicetree/bindings/ufs/qcom,ufs.yaml           | 11 ++++-------
>  3 files changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/crypto/qcom,inline-crypto-engine.yaml b/Documentation/devicetree/bindings/crypto/qcom,inline-crypto-engine.yaml
> index 23bf98c2b9fa..0d73e4581e6e 100644
> --- a/Documentation/devicetree/bindings/crypto/qcom,inline-crypto-engine.yaml
> +++ b/Documentation/devicetree/bindings/crypto/qcom,inline-crypto-engine.yaml
> @@ -24,6 +24,7 @@ properties:
>            - qcom,sc7180-inline-crypto-engine
>            - qcom,sc7280-inline-crypto-engine
>            - qcom,shikra-inline-crypto-engine
> +          - qcom,sm8150-inline-crypto-engine
>            - qcom,sm8450-inline-crypto-engine
>            - qcom,sm8550-inline-crypto-engine
>            - qcom,sm8650-inline-crypto-engine
> diff --git a/Documentation/devicetree/bindings/ufs/qcom,sc7180-ufshc.yaml b/Documentation/devicetree/bindings/ufs/qcom,sc7180-ufshc.yaml
> index 3c407426d697..572eec814a60 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

And what about eight clock? All existing DTS will report warning? I
don't quite get the need of this move.

Best regards,
Krzysztof