linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] media: dt-bindings: media: renesas,drif: Fix fck definition
@ 2021-04-08 20:24 Fabrizio Castro
  2021-04-09 18:54 ` Rob Herring
  2021-06-09  1:04 ` Rob Herring
  0 siblings, 2 replies; 7+ messages in thread
From: Fabrizio Castro @ 2021-04-08 20:24 UTC (permalink / raw)
  To: Mauro Carvalho Chehab, Rob Herring, Geert Uytterhoeven
  Cc: Fabrizio Castro, Ramesh Shanmugasundaram, linux-media,
	devicetree, linux-kernel, linux-renesas-soc, Chris Paterson,
	Biju Das, Prabhakar Mahadev Lad, Laurent Pinchart

dt_binding_check reports the below error with the latest schema:

Documentation/devicetree/bindings/media/renesas,drif.yaml:
  properties:clock-names:maxItems: False schema does not allow 1
Documentation/devicetree/bindings/media/renesas,drif.yaml:
  ignoring, error in schema: properties: clock-names: maxItems

This patch fixes the problem.

Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com>
---
 Documentation/devicetree/bindings/media/renesas,drif.yaml | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/media/renesas,drif.yaml b/Documentation/devicetree/bindings/media/renesas,drif.yaml
index f1bdaeab4053..e28396f35ee6 100644
--- a/Documentation/devicetree/bindings/media/renesas,drif.yaml
+++ b/Documentation/devicetree/bindings/media/renesas,drif.yaml
@@ -67,9 +67,7 @@ properties:
     maxItems: 1
 
   clock-names:
-    maxItems: 1
-    items:
-      - const: fck
+    const: fck
 
   resets:
     maxItems: 1
-- 
2.25.1


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

* Re: [PATCH] media: dt-bindings: media: renesas,drif: Fix fck definition
  2021-04-08 20:24 [PATCH] media: dt-bindings: media: renesas,drif: Fix fck definition Fabrizio Castro
@ 2021-04-09 18:54 ` Rob Herring
  2021-04-27 18:02   ` Rob Herring
  2021-06-09  1:04 ` Rob Herring
  1 sibling, 1 reply; 7+ messages in thread
From: Rob Herring @ 2021-04-09 18:54 UTC (permalink / raw)
  To: Fabrizio Castro
  Cc: Ramesh Shanmugasundaram, linux-media, Rob Herring, linux-kernel,
	linux-renesas-soc, devicetree, Chris Paterson, Biju Das,
	Prabhakar Mahadev Lad, Laurent Pinchart, Mauro Carvalho Chehab,
	Geert Uytterhoeven

On Thu, 08 Apr 2021 21:24:36 +0100, Fabrizio Castro wrote:
> dt_binding_check reports the below error with the latest schema:
> 
> Documentation/devicetree/bindings/media/renesas,drif.yaml:
>   properties:clock-names:maxItems: False schema does not allow 1
> Documentation/devicetree/bindings/media/renesas,drif.yaml:
>   ignoring, error in schema: properties: clock-names: maxItems
> 
> This patch fixes the problem.
> 
> Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com>
> ---
>  Documentation/devicetree/bindings/media/renesas,drif.yaml | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

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

* Re: [PATCH] media: dt-bindings: media: renesas,drif: Fix fck definition
  2021-04-09 18:54 ` Rob Herring
@ 2021-04-27 18:02   ` Rob Herring
  2021-04-27 20:08     ` Laurent Pinchart
  0 siblings, 1 reply; 7+ messages in thread
From: Rob Herring @ 2021-04-27 18:02 UTC (permalink / raw)
  To: Fabrizio Castro, Mauro Carvalho Chehab, Sakari Ailus, Laurent Pinchart
  Cc: Ramesh Shanmugasundaram, Linux Media Mailing List, linux-kernel,
	open list:MEDIA DRIVERS FOR RENESAS - FCP, devicetree,
	Chris Paterson, Biju Das, Prabhakar Mahadev Lad,
	Geert Uytterhoeven

On Fri, Apr 9, 2021 at 1:54 PM Rob Herring <robh@kernel.org> wrote:
>
> On Thu, 08 Apr 2021 21:24:36 +0100, Fabrizio Castro wrote:
> > dt_binding_check reports the below error with the latest schema:
> >
> > Documentation/devicetree/bindings/media/renesas,drif.yaml:
> >   properties:clock-names:maxItems: False schema does not allow 1
> > Documentation/devicetree/bindings/media/renesas,drif.yaml:
> >   ignoring, error in schema: properties: clock-names: maxItems
> >
> > This patch fixes the problem.
> >
> > Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com>
> > ---
> >  Documentation/devicetree/bindings/media/renesas,drif.yaml | 4 +---
> >  1 file changed, 1 insertion(+), 3 deletions(-)
> >
>
> Reviewed-by: Rob Herring <robh@kernel.org>

Still failing in linux-next...

Rob

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

* Re: [PATCH] media: dt-bindings: media: renesas,drif: Fix fck definition
  2021-04-27 18:02   ` Rob Herring
@ 2021-04-27 20:08     ` Laurent Pinchart
  2021-06-04 19:38       ` Rob Herring
  0 siblings, 1 reply; 7+ messages in thread
From: Laurent Pinchart @ 2021-04-27 20:08 UTC (permalink / raw)
  To: Rob Herring
  Cc: Fabrizio Castro, Mauro Carvalho Chehab, Sakari Ailus,
	Ramesh Shanmugasundaram, Linux Media Mailing List, linux-kernel,
	open list:MEDIA DRIVERS FOR RENESAS - FCP, devicetree,
	Chris Paterson, Biju Das, Prabhakar Mahadev Lad,
	Geert Uytterhoeven

Hi Rob,

On Tue, Apr 27, 2021 at 01:02:57PM -0500, Rob Herring wrote:
> On Fri, Apr 9, 2021 at 1:54 PM Rob Herring wrote:
> > On Thu, 08 Apr 2021 21:24:36 +0100, Fabrizio Castro wrote:
> > > dt_binding_check reports the below error with the latest schema:
> > >
> > > Documentation/devicetree/bindings/media/renesas,drif.yaml:
> > >   properties:clock-names:maxItems: False schema does not allow 1
> > > Documentation/devicetree/bindings/media/renesas,drif.yaml:
> > >   ignoring, error in schema: properties: clock-names: maxItems
> > >
> > > This patch fixes the problem.
> > >
> > > Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com>
> > > ---
> > >  Documentation/devicetree/bindings/media/renesas,drif.yaml | 4 +---
> > >  1 file changed, 1 insertion(+), 3 deletions(-)
> > >
> >
> > Reviewed-by: Rob Herring <robh@kernel.org>
> 
> Still failing in linux-next...

Sorry, as a standalone fix, I incorrectly thought you'd merge it.

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

and applied to my tree. I'll send a pull request to linux-media when the
merge window closes.

-- 
Regards,

Laurent Pinchart

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

* Re: [PATCH] media: dt-bindings: media: renesas,drif: Fix fck definition
  2021-04-27 20:08     ` Laurent Pinchart
@ 2021-06-04 19:38       ` Rob Herring
  2021-06-04 22:05         ` Laurent Pinchart
  0 siblings, 1 reply; 7+ messages in thread
From: Rob Herring @ 2021-06-04 19:38 UTC (permalink / raw)
  To: Laurent Pinchart
  Cc: Fabrizio Castro, Mauro Carvalho Chehab, Sakari Ailus,
	Ramesh Shanmugasundaram, Linux Media Mailing List, linux-kernel,
	open list:MEDIA DRIVERS FOR RENESAS - FCP, devicetree,
	Chris Paterson, Biju Das, Prabhakar Mahadev Lad,
	Geert Uytterhoeven

On Tue, Apr 27, 2021 at 3:08 PM Laurent Pinchart
<laurent.pinchart@ideasonboard.com> wrote:
>
> Hi Rob,
>
> On Tue, Apr 27, 2021 at 01:02:57PM -0500, Rob Herring wrote:
> > On Fri, Apr 9, 2021 at 1:54 PM Rob Herring wrote:
> > > On Thu, 08 Apr 2021 21:24:36 +0100, Fabrizio Castro wrote:
> > > > dt_binding_check reports the below error with the latest schema:
> > > >
> > > > Documentation/devicetree/bindings/media/renesas,drif.yaml:
> > > >   properties:clock-names:maxItems: False schema does not allow 1
> > > > Documentation/devicetree/bindings/media/renesas,drif.yaml:
> > > >   ignoring, error in schema: properties: clock-names: maxItems
> > > >
> > > > This patch fixes the problem.
> > > >
> > > > Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com>
> > > > ---
> > > >  Documentation/devicetree/bindings/media/renesas,drif.yaml | 4 +---
> > > >  1 file changed, 1 insertion(+), 3 deletions(-)
> > > >
> > >
> > > Reviewed-by: Rob Herring <robh@kernel.org>
> >
> > Still failing in linux-next...
>
> Sorry, as a standalone fix, I incorrectly thought you'd merge it.
>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>
> and applied to my tree. I'll send a pull request to linux-media when the
> merge window closes.

Still failing in linux-next. It's also a failure in Linus' tree, so it
needs to go in as a fix.

Rob

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

* Re: [PATCH] media: dt-bindings: media: renesas,drif: Fix fck definition
  2021-06-04 19:38       ` Rob Herring
@ 2021-06-04 22:05         ` Laurent Pinchart
  0 siblings, 0 replies; 7+ messages in thread
From: Laurent Pinchart @ 2021-06-04 22:05 UTC (permalink / raw)
  To: Rob Herring
  Cc: Fabrizio Castro, Mauro Carvalho Chehab, Sakari Ailus,
	Ramesh Shanmugasundaram, Linux Media Mailing List, linux-kernel,
	open list:MEDIA DRIVERS FOR RENESAS - FCP, devicetree,
	Chris Paterson, Biju Das, Prabhakar Mahadev Lad,
	Geert Uytterhoeven

Hi Rob,

On Fri, Jun 04, 2021 at 02:38:36PM -0500, Rob Herring wrote:
> On Tue, Apr 27, 2021 at 3:08 PM Laurent Pinchart wrote:
> > On Tue, Apr 27, 2021 at 01:02:57PM -0500, Rob Herring wrote:
> > > On Fri, Apr 9, 2021 at 1:54 PM Rob Herring wrote:
> > > > On Thu, 08 Apr 2021 21:24:36 +0100, Fabrizio Castro wrote:
> > > > > dt_binding_check reports the below error with the latest schema:
> > > > >
> > > > > Documentation/devicetree/bindings/media/renesas,drif.yaml:
> > > > >   properties:clock-names:maxItems: False schema does not allow 1
> > > > > Documentation/devicetree/bindings/media/renesas,drif.yaml:
> > > > >   ignoring, error in schema: properties: clock-names: maxItems
> > > > >
> > > > > This patch fixes the problem.
> > > > >
> > > > > Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com>
> > > > > ---
> > > > >  Documentation/devicetree/bindings/media/renesas,drif.yaml | 4 +---
> > > > >  1 file changed, 1 insertion(+), 3 deletions(-)
> > > > >
> > > >
> > > > Reviewed-by: Rob Herring <robh@kernel.org>
> > >
> > > Still failing in linux-next...
> >
> > Sorry, as a standalone fix, I incorrectly thought you'd merge it.
> >
> > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> >
> > and applied to my tree. I'll send a pull request to linux-media when the
> > merge window closes.
> 
> Still failing in linux-next. It's also a failure in Linus' tree, so it
> needs to go in as a fix.

Sorry, been overwhelmed. Feel free to take the patch through your tree
as a fix.

-- 
Regards,

Laurent Pinchart

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

* Re: [PATCH] media: dt-bindings: media: renesas,drif: Fix fck definition
  2021-04-08 20:24 [PATCH] media: dt-bindings: media: renesas,drif: Fix fck definition Fabrizio Castro
  2021-04-09 18:54 ` Rob Herring
@ 2021-06-09  1:04 ` Rob Herring
  1 sibling, 0 replies; 7+ messages in thread
From: Rob Herring @ 2021-06-09  1:04 UTC (permalink / raw)
  To: Fabrizio Castro
  Cc: Geert Uytterhoeven, Chris Paterson, devicetree, Rob Herring,
	linux-kernel, linux-renesas-soc, linux-media, Biju Das,
	Ramesh Shanmugasundaram, Laurent Pinchart, Prabhakar Mahadev Lad,
	Mauro Carvalho Chehab

On Thu, 08 Apr 2021 21:24:36 +0100, Fabrizio Castro wrote:
> dt_binding_check reports the below error with the latest schema:
> 
> Documentation/devicetree/bindings/media/renesas,drif.yaml:
>   properties:clock-names:maxItems: False schema does not allow 1
> Documentation/devicetree/bindings/media/renesas,drif.yaml:
>   ignoring, error in schema: properties: clock-names: maxItems
> 
> This patch fixes the problem.
> 
> Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com>
> ---
>  Documentation/devicetree/bindings/media/renesas,drif.yaml | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 

Applied, thanks!

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

end of thread, other threads:[~2021-06-09  1:04 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-08 20:24 [PATCH] media: dt-bindings: media: renesas,drif: Fix fck definition Fabrizio Castro
2021-04-09 18:54 ` Rob Herring
2021-04-27 18:02   ` Rob Herring
2021-04-27 20:08     ` Laurent Pinchart
2021-06-04 19:38       ` Rob Herring
2021-06-04 22:05         ` Laurent Pinchart
2021-06-09  1:04 ` Rob Herring

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).