| Message ID | 20210408202436.3706-1-fabrizio.castro.jz@renesas.com |
|---|---|
| State | Accepted |
| Commit | 8929ef8d4dfd53a05913e22561784ece5f6419c7 |
| Headers | show |
| Series | media: dt-bindings: media: renesas,drif: Fix fck definition | expand |
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>
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
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.
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
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
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!
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
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(-)