[PATCH 2/7] dt-bindings: rng: add binding for Rockchip RK3588 RNG

[Krzysztof Kozlowski]

On 30/01/2025 17:31, Nicolas Frattaroli wrote:
> +title: Rockchip RK3588 TRNG
> +
> +description: True Random Number Generator on Rockchip RK3588 SoC
> +
> +maintainers:
> +  - Nicolas Frattaroli <nicolas.frattaroli at collabora.com>
> +
> +properties:
> +  compatible:
> +    enum:
> +      - rockchip,rk3588-rng
> +
> +  reg:
> +    maxItems: 1
> +
> +  clocks:
> +    items:
> +      - description: TRNG AHB clock
> +
> +  # Optional, not used by some driver implementations

What driver implementations? Downstream? They do not matter, because
they are full of all sort of crap.

Can this block have interrupt really disconnected? This is the question
you should answer.


> +  interrupts:
> +    maxItems: 1
> +
> +  # Optional, hardware works without explicit reset

Just because bootloader did something? With that reasoning nothing is
ever required because firmware can abstract it. Either you have there a
reset or not. In this particular case your driver is irrelevant.

> +  resets:
> +    maxItems: 1
> +
> +required:
> +  - compatible
> +  - reg
> +  - clocks
> +


BTW, there is a binding for Rockchip TRNG, with a bit different clocks
so I have feeling yours is incomplete here.

> +additionalProperties: false
> +
> +examples:
> +  - |
> +    #include <dt-bindings/clock/rockchip,rk3588-cru.h>
> +    #include <dt-bindings/interrupt-controller/arm-gic.h>
> +    #include <dt-bindings/interrupt-controller/irq.h>
> +    #include <dt-bindings/reset/rockchip,rk3588-cru.h>
> +    bus {
> +      #address-cells = <2>;
> +      #size-cells = <2>;
> +
> +      rng at fe378000 {
> +        compatible = "rockchip,rk3588-rng";
> +        reg = <0x0 0xfe378000 0x0 0x200>;
> +        interrupts = <GIC_SPI 400 IRQ_TYPE_LEVEL_HIGH 0>;
> +        clocks = <&scmi_clk SCMI_HCLK_SECURE_NS>;
> +        resets = <&scmi_reset SCMI_SRST_H_TRNG_NS>;
> +        status = "disabled";

Examples cannot be disabled.

> +      };
> +    };
> +
> +...
> diff --git a/MAINTAINERS b/MAINTAINERS
> index bc8ce7af3303f747e0ef028e5a7b29b0bbba99f4..7daf9bfeb0cb4e9e594b809012c7aa243b0558ae 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -20420,8 +20420,10 @@ F:	include/uapi/linux/rkisp1-config.h
>  ROCKCHIP RK3568 RANDOM NUMBER GENERATOR SUPPORT
>  M:	Daniel Golle <daniel at makrotopia.org>
>  M:	Aurelien Jarno <aurelien at aurel32.net>
> +M:	Nicolas Frattaroli <nicolas.frattaroli at collabora.com>

Like Conor said, this is not really relevant and should be a separate patch.



Best regards,
Krzysztof