lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 23 Sep 2021 09:43:07 +0200 From: Geert Uytterhoeven <geert@...ux-m68k.org> To: Kieran Bingham <kieran.bingham@...asonboard.com> Cc: Linux-Renesas <linux-renesas-soc@...r.kernel.org>, Laurent Pinchart <laurent.pinchart@...asonboard.com>, Kieran Bingham <kieran.bingham+renesas@...asonboard.com>, David Airlie <airlied@...ux.ie>, Daniel Vetter <daniel@...ll.ch>, Rob Herring <robh+dt@...nel.org>, "open list:DRM DRIVERS FOR RENESAS" <dri-devel@...ts.freedesktop.org>, "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@...r.kernel.org>, open list <linux-kernel@...r.kernel.org> Subject: Re: [PATCH v3 1/6] dt-bindings: display: renesas,du: Provide bindings for r8a779a0 Hi Kieran, On Thu, Sep 23, 2021 at 1:47 AM Kieran Bingham <kieran.bingham@...asonboard.com> wrote: > From: Kieran Bingham <kieran.bingham+renesas@...asonboard.com> > > Extend the Renesas DU display bindings to support the r8a779a0 V3U. > > Reviewed-by: Laurent Pinchart <laurent.pinchart@...asonboard.com> > Signed-off-by: Kieran Bingham <kieran.bingham+renesas@...asonboard.com> > > --- > v2: > - Collected Laurent's tag > - Remove clock-names requirement > - Specify only a single clock > > v3: > - Use clocknames: 'du.0' instead of 'du' to remain consistent Thanks for the update! > --- a/Documentation/devicetree/bindings/display/renesas,du.yaml > +++ b/Documentation/devicetree/bindings/display/renesas,du.yaml > @@ -39,6 +39,7 @@ properties: > - renesas,du-r8a77980 # for R-Car V3H compatible DU > - renesas,du-r8a77990 # for R-Car E3 compatible DU > - renesas,du-r8a77995 # for R-Car D3 compatible DU > + - renesas,du-r8a779a0 # for R-Car V3U compatible DU > > reg: > maxItems: 1 > @@ -773,6 +774,55 @@ allOf: > - reset-names > - renesas,vsps > > + - if: > + properties: > + compatible: > + contains: > + enum: > + - renesas,du-r8a779a0 > + then: > + properties: > + clocks: > + items: > + - description: Functional clock > + > + clock-names: > + maxItems: 1 > + items: > + - const: du.0 > + > + interrupts: > + maxItems: 2 > + > + resets: > + maxItems: 1 > + > + reset-names: > + items: > + - const: du.0 > + > + ports: > + properties: > + port@0: > + description: DSI 0 > + port@1: > + description: DSI 1 > + port@2: false > + port@3: false > + > + required: > + - port@0 > + - port@1 > + > + renesas,vsps: > + minItems: 2 > + > + required: > + - interrupts > + - resets > + - reset-names > + - renesas,vsps clock-names, for consistency? > + > additionalProperties: false > > examples: With the above fixed: Reviewed-by: Geert Uytterhoeven <geert+renesas@...der.be> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-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
Powered by blists - more mailing lists