All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: arm: renesas: Document RZ/A2M EVB
@ 2019-01-23 13:48 Chris Brandt
  2019-01-23 13:59   ` Rob Herring
  0 siblings, 1 reply; 11+ messages in thread
From: Chris Brandt @ 2019-01-23 13:48 UTC (permalink / raw)
  To: Simon Horman, Rob Herring, Mark Rutland
  Cc: devicetree, linux-renesas-soc, Chris Brandt

Add the RZ/A2M Evaluation board.

Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
---
 Documentation/devicetree/bindings/arm/renesas.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/renesas.yaml b/Documentation/devicetree/bindings/arm/renesas.yaml
index 5e9d4864a600..474eb2647829 100644
--- a/Documentation/devicetree/bindings/arm/renesas.yaml
+++ b/Documentation/devicetree/bindings/arm/renesas.yaml
@@ -29,6 +29,8 @@ properties:
           - const: renesas,r7s72100
 
       - description: RZ/A2 (R7S9210)
+          - enum:
+              - renesas,rza2mevb # RZ/A2M Eval Board (RTK7921053S00000BE)
         items:
           - const: renesas,r7s9210
 
-- 
2.16.1

^ permalink raw reply related	[flat|nested] 11+ messages in thread

* Re: [PATCH] dt-bindings: arm: renesas: Document RZ/A2M EVB
  2019-01-23 13:48 [PATCH] dt-bindings: arm: renesas: Document RZ/A2M EVB Chris Brandt
@ 2019-01-23 13:59   ` Rob Herring
  0 siblings, 0 replies; 11+ messages in thread
From: Rob Herring @ 2019-01-23 13:59 UTC (permalink / raw)
  To: Chris Brandt
  Cc: Simon Horman, Mark Rutland, devicetree,
	open list:MEDIA DRIVERS FOR RENESAS - FCP

On Wed, Jan 23, 2019 at 7:48 AM Chris Brandt <chris.brandt@renesas.com> wrote:
>
> Add the RZ/A2M Evaluation board.
>
> Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
> ---
>  Documentation/devicetree/bindings/arm/renesas.yaml | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/renesas.yaml b/Documentation/devicetree/bindings/arm/renesas.yaml
> index 5e9d4864a600..474eb2647829 100644
> --- a/Documentation/devicetree/bindings/arm/renesas.yaml
> +++ b/Documentation/devicetree/bindings/arm/renesas.yaml
> @@ -29,6 +29,8 @@ properties:
>            - const: renesas,r7s72100
>
>        - description: RZ/A2 (R7S9210)
> +          - enum:
> +              - renesas,rza2mevb # RZ/A2M Eval Board (RTK7921053S00000BE)

Please check your changes with 'dt_binding_check' target. This will
fail. 'description' already has a value, so you can't add a list to
it. And description has to be a string.

>          items:

Move the 2 lines here.

>            - const: renesas,r7s9210
>
> --
> 2.16.1
>

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] dt-bindings: arm: renesas: Document RZ/A2M EVB
@ 2019-01-23 13:59   ` Rob Herring
  0 siblings, 0 replies; 11+ messages in thread
From: Rob Herring @ 2019-01-23 13:59 UTC (permalink / raw)
  To: Chris Brandt
  Cc: Simon Horman, Mark Rutland, devicetree,
	open list:MEDIA DRIVERS FOR RENESAS - FCP

On Wed, Jan 23, 2019 at 7:48 AM Chris Brandt <chris.brandt@renesas.com> wrote:
>
> Add the RZ/A2M Evaluation board.
>
> Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
> ---
>  Documentation/devicetree/bindings/arm/renesas.yaml | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/renesas.yaml b/Documentation/devicetree/bindings/arm/renesas.yaml
> index 5e9d4864a600..474eb2647829 100644
> --- a/Documentation/devicetree/bindings/arm/renesas.yaml
> +++ b/Documentation/devicetree/bindings/arm/renesas.yaml
> @@ -29,6 +29,8 @@ properties:
>            - const: renesas,r7s72100
>
>        - description: RZ/A2 (R7S9210)
> +          - enum:
> +              - renesas,rza2mevb # RZ/A2M Eval Board (RTK7921053S00000BE)

Please check your changes with 'dt_binding_check' target. This will
fail. 'description' already has a value, so you can't add a list to
it. And description has to be a string.

>          items:

Move the 2 lines here.

>            - const: renesas,r7s9210
>
> --
> 2.16.1
>

^ permalink raw reply	[flat|nested] 11+ messages in thread

* RE: [PATCH] dt-bindings: arm: renesas: Document RZ/A2M EVB
  2019-01-23 13:59   ` Rob Herring
@ 2019-01-23 14:09     ` Chris Brandt
  -1 siblings, 0 replies; 11+ messages in thread
From: Chris Brandt @ 2019-01-23 14:09 UTC (permalink / raw)
  To: Rob Herring
  Cc: Simon Horman, Mark Rutland, devicetree,
	open list:MEDIA DRIVERS FOR RENESAS - FCP

On Wednesday, January 23, 2019, Rob Herring wrote:
> Please check your changes with 'dt_binding_check' target. This will
> fail. 'description' already has a value, so you can't add a list to
> it. And description has to be a string.
> 
> >          items:
> 
> Move the 2 lines here.
> 
> >            - const: renesas,r7s9210

I can't find 'dt_binding_check' (I guess it's only in the DT tree).

But, I can see my mistake (now that you pointed it out).

Chris


^ permalink raw reply	[flat|nested] 11+ messages in thread

* RE: [PATCH] dt-bindings: arm: renesas: Document RZ/A2M EVB
@ 2019-01-23 14:09     ` Chris Brandt
  0 siblings, 0 replies; 11+ messages in thread
From: Chris Brandt @ 2019-01-23 14:09 UTC (permalink / raw)
  To: Rob Herring
  Cc: Simon Horman, Mark Rutland, devicetree,
	open list:MEDIA DRIVERS FOR RENESAS - FCP

On Wednesday, January 23, 2019, Rob Herring wrote:
> Please check your changes with 'dt_binding_check' target. This will
> fail. 'description' already has a value, so you can't add a list to
> it. And description has to be a string.
> 
> >          items:
> 
> Move the 2 lines here.
> 
> >            - const: renesas,r7s9210

I can't find 'dt_binding_check' (I guess it's only in the DT tree).

But, I can see my mistake (now that you pointed it out).

Chris


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] dt-bindings: arm: renesas: Document RZ/A2M EVB
  2019-01-23 14:09     ` Chris Brandt
@ 2019-01-23 14:59       ` Geert Uytterhoeven
  -1 siblings, 0 replies; 11+ messages in thread
From: Geert Uytterhoeven @ 2019-01-23 14:59 UTC (permalink / raw)
  To: Chris Brandt
  Cc: Rob Herring, Simon Horman, Mark Rutland, devicetree,
	open list:MEDIA DRIVERS FOR RENESAS - FCP

Hi Chris,

On Wed, Jan 23, 2019 at 3:10 PM Chris Brandt <Chris.Brandt@renesas.com> wrote:
> On Wednesday, January 23, 2019, Rob Herring wrote:
> > Please check your changes with 'dt_binding_check' target. This will
> > fail. 'description' already has a value, so you can't add a list to
> > it. And description has to be a string.
> >
> > >          items:
> >
> > Move the 2 lines here.
> >
> > >            - const: renesas,r7s9210
>
> I can't find 'dt_binding_check' (I guess it's only in the DT tree).

This seems to be a make target: "make dt_binding_check"
(included in last renesas-drivers).

This needs "dt-mk-schema", which you can find in
https://github.com/robherring/yaml-bindings

Cloned, patch sent for missing python3-pip dependency, installed, put in $PATH,
and now it seems to like your v2.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] dt-bindings: arm: renesas: Document RZ/A2M EVB
@ 2019-01-23 14:59       ` Geert Uytterhoeven
  0 siblings, 0 replies; 11+ messages in thread
From: Geert Uytterhoeven @ 2019-01-23 14:59 UTC (permalink / raw)
  To: Chris Brandt
  Cc: Rob Herring, Simon Horman, Mark Rutland, devicetree,
	open list:MEDIA DRIVERS FOR RENESAS - FCP

Hi Chris,

On Wed, Jan 23, 2019 at 3:10 PM Chris Brandt <Chris.Brandt@renesas.com> wrote:
> On Wednesday, January 23, 2019, Rob Herring wrote:
> > Please check your changes with 'dt_binding_check' target. This will
> > fail. 'description' already has a value, so you can't add a list to
> > it. And description has to be a string.
> >
> > >          items:
> >
> > Move the 2 lines here.
> >
> > >            - const: renesas,r7s9210
>
> I can't find 'dt_binding_check' (I guess it's only in the DT tree).

This seems to be a make target: "make dt_binding_check"
(included in last renesas-drivers).

This needs "dt-mk-schema", which you can find in
https://github.com/robherring/yaml-bindings

Cloned, patch sent for missing python3-pip dependency, installed, put in $PATH,
and now it seems to like your v2.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 11+ messages in thread

* RE: [PATCH] dt-bindings: arm: renesas: Document RZ/A2M EVB
  2019-01-23 14:59       ` Geert Uytterhoeven
@ 2019-01-23 15:08         ` Chris Brandt
  -1 siblings, 0 replies; 11+ messages in thread
From: Chris Brandt @ 2019-01-23 15:08 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Rob Herring, Simon Horman, Mark Rutland, devicetree,
	open list:MEDIA DRIVERS FOR RENESAS - FCP

Hi Geert,

On Wednesday, January 23, 2019, Geert Uytterhoeven wrote:
> > I can't find 'dt_binding_check' (I guess it's only in the DT tree).
> 
> This seems to be a make target: "make dt_binding_check"
> (included in last renesas-drivers).
> 
> This needs "dt-mk-schema", which you can find in
> https://github.com/robherring/yaml-bindings
> 
> Cloned, patch sent for missing python3-pip dependency, installed, put in
> $PATH,
> and now it seems to like your v2.

Thank you!!!


Chris

^ permalink raw reply	[flat|nested] 11+ messages in thread

* RE: [PATCH] dt-bindings: arm: renesas: Document RZ/A2M EVB
@ 2019-01-23 15:08         ` Chris Brandt
  0 siblings, 0 replies; 11+ messages in thread
From: Chris Brandt @ 2019-01-23 15:08 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Rob Herring, Simon Horman, Mark Rutland, devicetree,
	open list:MEDIA DRIVERS FOR RENESAS - FCP

Hi Geert,

On Wednesday, January 23, 2019, Geert Uytterhoeven wrote:
> > I can't find 'dt_binding_check' (I guess it's only in the DT tree).
> 
> This seems to be a make target: "make dt_binding_check"
> (included in last renesas-drivers).
> 
> This needs "dt-mk-schema", which you can find in
> https://github.com/robherring/yaml-bindings
> 
> Cloned, patch sent for missing python3-pip dependency, installed, put in
> $PATH,
> and now it seems to like your v2.

Thank you!!!


Chris

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] dt-bindings: arm: renesas: Document RZ/A2M EVB
  2019-01-23 14:09     ` Chris Brandt
@ 2019-01-23 15:15       ` Rob Herring
  -1 siblings, 0 replies; 11+ messages in thread
From: Rob Herring @ 2019-01-23 15:15 UTC (permalink / raw)
  To: Chris Brandt
  Cc: Simon Horman, Mark Rutland, devicetree,
	open list:MEDIA DRIVERS FOR RENESAS - FCP

On Wed, Jan 23, 2019 at 8:09 AM Chris Brandt <Chris.Brandt@renesas.com> wrote:
>
> On Wednesday, January 23, 2019, Rob Herring wrote:
> > Please check your changes with 'dt_binding_check' target. This will
> > fail. 'description' already has a value, so you can't add a list to
> > it. And description has to be a string.
> >
> > >          items:
> >
> > Move the 2 lines here.
> >
> > >            - const: renesas,r7s9210
>
> I can't find 'dt_binding_check' (I guess it's only in the DT tree).

It's a make target added in 5.0-rc1. You need 'yaml-bindings'
installed. See Documentation/devicetree/writing-schemas.md for
details.

Rob

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] dt-bindings: arm: renesas: Document RZ/A2M EVB
@ 2019-01-23 15:15       ` Rob Herring
  0 siblings, 0 replies; 11+ messages in thread
From: Rob Herring @ 2019-01-23 15:15 UTC (permalink / raw)
  To: Chris Brandt
  Cc: Simon Horman, Mark Rutland, devicetree,
	open list:MEDIA DRIVERS FOR RENESAS - FCP

On Wed, Jan 23, 2019 at 8:09 AM Chris Brandt <Chris.Brandt@renesas.com> wrote:
>
> On Wednesday, January 23, 2019, Rob Herring wrote:
> > Please check your changes with 'dt_binding_check' target. This will
> > fail. 'description' already has a value, so you can't add a list to
> > it. And description has to be a string.
> >
> > >          items:
> >
> > Move the 2 lines here.
> >
> > >            - const: renesas,r7s9210
>
> I can't find 'dt_binding_check' (I guess it's only in the DT tree).

It's a make target added in 5.0-rc1. You need 'yaml-bindings'
installed. See Documentation/devicetree/writing-schemas.md for
details.

Rob

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2019-01-23 15:15 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-23 13:48 [PATCH] dt-bindings: arm: renesas: Document RZ/A2M EVB Chris Brandt
2019-01-23 13:59 ` Rob Herring
2019-01-23 13:59   ` Rob Herring
2019-01-23 14:09   ` Chris Brandt
2019-01-23 14:09     ` Chris Brandt
2019-01-23 14:59     ` Geert Uytterhoeven
2019-01-23 14:59       ` Geert Uytterhoeven
2019-01-23 15:08       ` Chris Brandt
2019-01-23 15:08         ` Chris Brandt
2019-01-23 15:15     ` Rob Herring
2019-01-23 15:15       ` Rob Herring

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.