Re: [PATCH 1/3] dt-bindings: crypto: qcom,prng: Add SM8450

From: Krzysztof Kozlowski
Date: Mon Aug 21 2023 - 02:07:49 EST


On 21/08/2023 02:52, Om Prakash Singh wrote:
> I meant first one. using "qcom,rng-ee".

Then please provide some reasons.

>
> I am looking for generic compatible string for all SoCs for which core
> clock can be optional, same as we have "qcom,prng-ee".

There is a generic compatible already... but anyway, is the clock really
optional? Or just configured by firmware?

>
> If we are using SoC name in compatible string, for each SoC support we
> need to update qcom,prng.yaml file.

So you were talking about second case from my email? Still not sure what
you want to propose, but just in case - please always follow DT bindings
guidelines:

https://elixir.bootlin.com/linux/v6.1-rc1/source/Documentation/devicetree/bindings/writing-bindings.rst#L42

Best regards,
Krzysztof