diff mbox series

[v2,1/7] Revert "dt-bindings: crypto: qcom,prng: Add SM8450"

Message ID 20230824-topic-sm8550-rng-v2-1-dfcafbb16a3e@linaro.org
State Accepted
Commit 85b9bf9a514d991fcecb118d0a8a35e754ff9265
Headers show
Series arm64: qcom: sm8550: enable RNG | expand

Commit Message

Neil Armstrong Aug. 24, 2023, 11:33 a.m. UTC
This reverts commit b9296bb41275 ("dt-bindings: crypto: qcom,prng: Add SM8450"),
since the RNG HW on the SM8450 SoC is in fact a True Random Number Generator,
a more appropriate compatible should be instead as reported at [1].

[1] https://lore.kernel.org/all/20230818161720.3644424-1-quic_omprsing@quicinc.com/

Suggested-by: Om Prakash Singh <quic_omprsing@quicinc.com>
Suggested-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
---
 .../devicetree/bindings/crypto/qcom,prng.yaml      | 24 +++++-----------------
 1 file changed, 5 insertions(+), 19 deletions(-)

Comments

Herbert Xu Aug. 25, 2023, 11:07 a.m. UTC | #1
On Thu, Aug 24, 2023 at 01:33:20PM +0200, Neil Armstrong wrote:
> This reverts commit b9296bb41275 ("dt-bindings: crypto: qcom,prng: Add SM8450"),
> since the RNG HW on the SM8450 SoC is in fact a True Random Number Generator,
> a more appropriate compatible should be instead as reported at [1].
> 
> [1] https://lore.kernel.org/all/20230818161720.3644424-1-quic_omprsing@quicinc.com/
> 
> Suggested-by: Om Prakash Singh <quic_omprsing@quicinc.com>
> Suggested-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> ---
>  .../devicetree/bindings/crypto/qcom,prng.yaml      | 24 +++++-----------------
>  1 file changed, 5 insertions(+), 19 deletions(-)

Patch applied.  Thanks.
Om Prakash Singh Aug. 28, 2023, 6:25 a.m. UTC | #2
On 8/25/2023 2:59 AM, Rob Herring wrote:
> On Thu, Aug 24, 2023 at 01:33:20PM +0200, Neil Armstrong wrote:
>> This reverts commit b9296bb41275 ("dt-bindings: crypto: qcom,prng: Add SM8450"),
>> since the RNG HW on the SM8450 SoC is in fact a True Random Number Generator,
>> a more appropriate compatible should be instead as reported at [1].
>>
>> [1] https://lore.kernel.org/all/20230818161720.3644424-1-quic_omprsing@quicinc.com/
>>
>> Suggested-by: Om Prakash Singh <quic_omprsing@quicinc.com>
>> Suggested-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> 
> Is it just me or looks like an unrenderable character in these.
Yes there are spacial character before Om and Konrad that should have 
been removed.
> 
>> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
>> ---
>>   .../devicetree/bindings/crypto/qcom,prng.yaml      | 24 +++++-----------------
>>   1 file changed, 5 insertions(+), 19 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/crypto/qcom,prng.yaml b/Documentation/devicetree/bindings/crypto/qcom,prng.yaml
>> index 36b0ebd9a44b..bb42f4588b40 100644
>> --- a/Documentation/devicetree/bindings/crypto/qcom,prng.yaml
>> +++ b/Documentation/devicetree/bindings/crypto/qcom,prng.yaml
>> @@ -11,13 +11,9 @@ maintainers:
>>   
>>   properties:
>>     compatible:
>> -    oneOf:
>> -      - enum:
>> -          - qcom,prng  # 8916 etc.
>> -          - qcom,prng-ee  # 8996 and later using EE
>> -      - items:
>> -          - const: qcom,sm8450-prng-ee
>> -          - const: qcom,prng-ee
>> +    enum:
>> +      - qcom,prng  # 8916 etc.
>> +      - qcom,prng-ee  # 8996 and later using EE
>>   
>>     reg:
>>       maxItems: 1
>> @@ -32,18 +28,8 @@ properties:
>>   required:
>>     - compatible
>>     - reg
>> -
>> -allOf:
>> -  - if:
>> -      not:
>> -        properties:
>> -          compatible:
>> -            contains:
>> -              const: qcom,sm8450-prng-ee
>> -    then:
>> -      required:
>> -        - clocks
>> -        - clock-names
>> +  - clocks
>> +  - clock-names
>>   
>>   additionalProperties: false
>>   
>>
>> -- 
>> 2.34.1
>>
Neil Armstrong Aug. 28, 2023, 8:01 a.m. UTC | #3
Hi,

On 28/08/2023 08:25, Om Prakash Singh wrote:
> 
> 
> On 8/25/2023 2:59 AM, Rob Herring wrote:
>> On Thu, Aug 24, 2023 at 01:33:20PM +0200, Neil Armstrong wrote:
>>> This reverts commit b9296bb41275 ("dt-bindings: crypto: qcom,prng: Add SM8450"),
>>> since the RNG HW on the SM8450 SoC is in fact a True Random Number Generator,
>>> a more appropriate compatible should be instead as reported at [1].
>>>
>>> [1] https://lore.kernel.org/all/20230818161720.3644424-1-quic_omprsing@quicinc.com/
>>>
>>> Suggested-by: Om Prakash Singh <quic_omprsing@quicinc.com>
>>> Suggested-by: Konrad Dybcio <konrad.dybcio@linaro.org>
>>
>> Is it just me or looks like an unrenderable character in these.
> Yes there are spacial character before Om and Konrad that should have been removed.

Let me resend a v2 without this space and the review tags.

Neil

>>
>>> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
>>> ---
>>>   .../devicetree/bindings/crypto/qcom,prng.yaml      | 24 +++++-----------------
>>>   1 file changed, 5 insertions(+), 19 deletions(-)
>>>
>>> diff --git a/Documentation/devicetree/bindings/crypto/qcom,prng.yaml b/Documentation/devicetree/bindings/crypto/qcom,prng.yaml
>>> index 36b0ebd9a44b..bb42f4588b40 100644
>>> --- a/Documentation/devicetree/bindings/crypto/qcom,prng.yaml
>>> +++ b/Documentation/devicetree/bindings/crypto/qcom,prng.yaml
>>> @@ -11,13 +11,9 @@ maintainers:
>>>   properties:
>>>     compatible:
>>> -    oneOf:
>>> -      - enum:
>>> -          - qcom,prng  # 8916 etc.
>>> -          - qcom,prng-ee  # 8996 and later using EE
>>> -      - items:
>>> -          - const: qcom,sm8450-prng-ee
>>> -          - const: qcom,prng-ee
>>> +    enum:
>>> +      - qcom,prng  # 8916 etc.
>>> +      - qcom,prng-ee  # 8996 and later using EE
>>>     reg:
>>>       maxItems: 1
>>> @@ -32,18 +28,8 @@ properties:
>>>   required:
>>>     - compatible
>>>     - reg
>>> -
>>> -allOf:
>>> -  - if:
>>> -      not:
>>> -        properties:
>>> -          compatible:
>>> -            contains:
>>> -              const: qcom,sm8450-prng-ee
>>> -    then:
>>> -      required:
>>> -        - clocks
>>> -        - clock-names
>>> +  - clocks
>>> +  - clock-names
>>>   additionalProperties: false
>>>
>>> -- 
>>> 2.34.1
>>>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/crypto/qcom,prng.yaml b/Documentation/devicetree/bindings/crypto/qcom,prng.yaml
index 36b0ebd9a44b..bb42f4588b40 100644
--- a/Documentation/devicetree/bindings/crypto/qcom,prng.yaml
+++ b/Documentation/devicetree/bindings/crypto/qcom,prng.yaml
@@ -11,13 +11,9 @@  maintainers:
 
 properties:
   compatible:
-    oneOf:
-      - enum:
-          - qcom,prng  # 8916 etc.
-          - qcom,prng-ee  # 8996 and later using EE
-      - items:
-          - const: qcom,sm8450-prng-ee
-          - const: qcom,prng-ee
+    enum:
+      - qcom,prng  # 8916 etc.
+      - qcom,prng-ee  # 8996 and later using EE
 
   reg:
     maxItems: 1
@@ -32,18 +28,8 @@  properties:
 required:
   - compatible
   - reg
-
-allOf:
-  - if:
-      not:
-        properties:
-          compatible:
-            contains:
-              const: qcom,sm8450-prng-ee
-    then:
-      required:
-        - clocks
-        - clock-names
+  - clocks
+  - clock-names
 
 additionalProperties: false