diff mbox series

dt-bindings: imx-rng: Allow passing only "fsl,imx31-rnga"

Message ID 20241009125144.560941-1-festevam@gmail.com (mailing list archive)
State Accepted
Delegated to: Herbert Xu
Headers show
Series dt-bindings: imx-rng: Allow passing only "fsl,imx31-rnga" | expand

Commit Message

Fabio Estevam Oct. 9, 2024, 12:51 p.m. UTC 
From: Fabio Estevam <festevam@denx.de>

On imx31.dtsi the rng compatible string contains "fsl,imx31-rnga" only.

Adjust the binding to accept passing "fsl,imx31-rnga" only.

This fixes the following dt-schema warning:

imx31-lite.dtb: rng@53fb0000: compatible: 'oneOf' conditional failed, one must be fixed:
	['fsl,imx31-rnga'] is too short
	'fsl,imx21-rnga' was expected
	'fsl,imx25-rngb' was expected
	'fsl,imx31-rnga' is not one of ['fsl,imx6sl-rngb', 'fsl,imx6sll-rngb', 'fsl,imx6ull-rngb']
	'fsl,imx35-rngc' was expected

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 Documentation/devicetree/bindings/rng/imx-rng.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Rob Herring (Arm) Oct. 9, 2024, 9:11 p.m. UTC | #1 
On Wed, 09 Oct 2024 09:51:44 -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> On imx31.dtsi the rng compatible string contains "fsl,imx31-rnga" only.
> 
> Adjust the binding to accept passing "fsl,imx31-rnga" only.
> 
> This fixes the following dt-schema warning:
> 
> imx31-lite.dtb: rng@53fb0000: compatible: 'oneOf' conditional failed, one must be fixed:
> 	['fsl,imx31-rnga'] is too short
> 	'fsl,imx21-rnga' was expected
> 	'fsl,imx25-rngb' was expected
> 	'fsl,imx31-rnga' is not one of ['fsl,imx6sl-rngb', 'fsl,imx6sll-rngb', 'fsl,imx6ull-rngb']
> 	'fsl,imx35-rngc' was expected
> 
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---
>  Documentation/devicetree/bindings/rng/imx-rng.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>
Herbert Xu Oct. 19, 2024, 11:57 a.m. UTC | #2 
On Wed, Oct 09, 2024 at 09:51:44AM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> On imx31.dtsi the rng compatible string contains "fsl,imx31-rnga" only.
> 
> Adjust the binding to accept passing "fsl,imx31-rnga" only.
> 
> This fixes the following dt-schema warning:
> 
> imx31-lite.dtb: rng@53fb0000: compatible: 'oneOf' conditional failed, one must be fixed:
> 	['fsl,imx31-rnga'] is too short
> 	'fsl,imx21-rnga' was expected
> 	'fsl,imx25-rngb' was expected
> 	'fsl,imx31-rnga' is not one of ['fsl,imx6sl-rngb', 'fsl,imx6sll-rngb', 'fsl,imx6ull-rngb']
> 	'fsl,imx35-rngc' was expected
> 
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---
>  Documentation/devicetree/bindings/rng/imx-rng.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Patch applied.  Thanks.
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/rng/imx-rng.yaml b/Documentation/devicetree/bindings/rng/imx-rng.yaml
index 07f6ff89bcc1..252fa9a41abe 100644
--- a/Documentation/devicetree/bindings/rng/imx-rng.yaml
+++ b/Documentation/devicetree/bindings/rng/imx-rng.yaml
@@ -14,8 +14,8 @@  properties:
     oneOf:
       - const: fsl,imx21-rnga
       - const: fsl,imx25-rngb
+      - const: fsl,imx31-rnga
       - items:
-          - const: fsl,imx31-rnga
           - const: fsl,imx21-rnga
       - items:
           - enum: