[PATCH v2 2/3] dt-bindings: rng: meson: add meson-rng-s4 compatible

Alexey Romanov posted 3 patches 2 years, 1 month ago
There is a newer version of this series
[PATCH v2 2/3] dt-bindings: rng: meson: add meson-rng-s4 compatible
Posted by Alexey Romanov 2 years, 1 month ago
Now the driver has a separate algo for S4 SoC.

Signed-off-by: Alexey Romanov <avromanov@sberdevices.ru>
---
 Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml b/Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml
index 457a6e43d810..afa52af442a7 100644
--- a/Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml
+++ b/Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml
@@ -14,6 +14,7 @@ properties:
   compatible:
     enum:
       - amlogic,meson-rng
+      - amlogic,meson-s4-rng
 
   reg:
     maxItems: 1
-- 
2.38.1
Re: [PATCH v2 2/3] dt-bindings: rng: meson: add meson-rng-s4 compatible
Posted by Conor Dooley 2 years, 1 month ago
On Mon, Aug 07, 2023 at 04:06:10PM +0300, Alexey Romanov wrote:
> Now the driver has a separate algo for S4 SoC.
> 
> Signed-off-by: Alexey Romanov <avromanov@sberdevices.ru>

Acked-by: Conor Dooley <conor.dooley@microchip.com>

Thanks,
Conor.

> ---
>  Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml b/Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml
> index 457a6e43d810..afa52af442a7 100644
> --- a/Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml
> +++ b/Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml
> @@ -14,6 +14,7 @@ properties:
>    compatible:
>      enum:
>        - amlogic,meson-rng
> +      - amlogic,meson-s4-rng
>  
>    reg:
>      maxItems: 1
> -- 
> 2.38.1
> 
Re: [PATCH v2 2/3] dt-bindings: rng: meson: add meson-rng-s4 compatible
Posted by Conor Dooley 2 years, 1 month ago
On Tue, Aug 08, 2023 at 01:09:45PM +0100, Conor Dooley wrote:
> On Mon, Aug 07, 2023 at 04:06:10PM +0300, Alexey Romanov wrote:
> > Now the driver has a separate algo for S4 SoC.
> > 
> > Signed-off-by: Alexey Romanov <avromanov@sberdevices.ru>
> 
> Acked-by: Conor Dooley <conor.dooley@microchip.com>

Actually, one comment, please, for bindings, write commit messages that
relate to the hardware rather than drivers. The bindings describe the
hardware, after all.
Re: [PATCH v2 2/3] dt-bindings: rng: meson: add meson-rng-s4 compatible
Posted by Alexey Romanov 2 years ago
Hi, Conor!

Thank you for the review.

On Tue, Aug 08, 2023 at 01:11:24PM +0100, Conor Dooley wrote:
> On Tue, Aug 08, 2023 at 01:09:45PM +0100, Conor Dooley wrote:
> > On Mon, Aug 07, 2023 at 04:06:10PM +0300, Alexey Romanov wrote:
> > > Now the driver has a separate algo for S4 SoC.
> > > 
> > > Signed-off-by: Alexey Romanov <avromanov@sberdevices.ru>
> > 
> > Acked-by: Conor Dooley <conor.dooley@microchip.com>
> 
> Actually, one comment, please, for bindings, write commit messages that
> relate to the hardware rather than drivers. The bindings describe the
> hardware, after all.

Yeah, will fix this in V3.

-- 
Thank you,
Alexey