On 08/02/2023 19:37, Vladimir Zapolskiy wrote: > On newer Qualcomm SoCs the crypto engine clocks are enabled by default > by security firmware. To drop clocks and clock-names from the list of > required properties use 'qcom,sm8150-qce' compatible name. > > The change is based on Neil Armstrong's observation and an original change. > > Cc: Neil Armstrong <neil.armstrong@xxxxxxxxxx> > Signed-off-by: Vladimir Zapolskiy <vladimir.zapolskiy@xxxxxxxxxx> > --- > .../devicetree/bindings/crypto/qcom-qce.yaml | 15 +++++++++++++-- > 1 file changed, 13 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/crypto/qcom-qce.yaml b/Documentation/devicetree/bindings/crypto/qcom-qce.yaml > index f6f1759a2f6e..d0f6b830a5dd 100644 > --- a/Documentation/devicetree/bindings/crypto/qcom-qce.yaml > +++ b/Documentation/devicetree/bindings/crypto/qcom-qce.yaml > @@ -79,11 +79,22 @@ properties: > required: > - compatible > - reg > - - clocks > - - clock-names > - dmas > - dma-names > > +if: This should be in allOf, like I wrote in last discussion. Best regards, Krzysztof