| Message ID | 20221117122547.809644-3-tomi.valkeinen@ideasonboard.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Geert Uytterhoeven |
| Headers | show |
| Series | Renesas V4H DSI & DP output support | expand |
Quoting Tomi Valkeinen (2022-11-17 12:25:41) > From: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com> > > Extend the Renesas DSI display bindings to support the r8a779g0 V4H. > > Signed-off-by: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com> > --- > .../bindings/display/bridge/renesas,dsi-csi2-tx.yaml | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml b/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml > index afeeb967393d..bc3101f77e5a 100644 > --- a/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml > +++ b/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml > @@ -11,13 +11,14 @@ maintainers: > > description: | > This binding describes the MIPI DSI/CSI-2 encoder embedded in the Renesas > - R-Car V3U SoC. The encoder can operate in either DSI or CSI-2 mode, with up > + R-Car V3U/V4H SoC. The encoder can operate in either DSI or CSI-2 mode, with up That's 81 chars I think ... so perhaps the 'up' should go 'down' a line ... but hey it's one char, I don't care too much, but I can't resist an up/down reference :D Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> > to four data lanes. > > properties: > compatible: > enum: > - renesas,r8a779a0-dsi-csi2-tx # for V3U > + - renesas,r8a779g0-dsi-csi2-tx # for V4H > > reg: > maxItems: 1 > -- > 2.34.1 >
Hi Tomi, On Thu, Nov 17, 2022 at 1:26 PM Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> wrote: > From: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com> > > Extend the Renesas DSI display bindings to support the r8a779g0 V4H. > > Signed-off-by: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com> > --- > .../bindings/display/bridge/renesas,dsi-csi2-tx.yaml | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml b/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml > index afeeb967393d..bc3101f77e5a 100644 > --- a/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml > +++ b/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml > @@ -11,13 +11,14 @@ maintainers: > > description: | > This binding describes the MIPI DSI/CSI-2 encoder embedded in the Renesas > - R-Car V3U SoC. The encoder can operate in either DSI or CSI-2 mode, with up > + R-Car V3U/V4H SoC. The encoder can operate in either DSI or CSI-2 mode, with up Perhaps "R-Car Gen4 SoCs", so we stay within 80 chars, and don't have to update this when the next member of the family is around the block? Is there anything that might be SoC-specific? If not, perhaps the time is ripe for a family-specific compatible value? > to four data lanes. > > properties: > compatible: > enum: > - renesas,r8a779a0-dsi-csi2-tx # for V3U > + - renesas,r8a779g0-dsi-csi2-tx # for V4H > > reg: > maxItems: 1 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
On 17/11/2022 13:25, Tomi Valkeinen wrote: > From: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com> > > Extend the Renesas DSI display bindings to support the r8a779g0 V4H. > > Signed-off-by: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On Thu, Nov 17, 2022 at 04:14:21PM +0100, Geert Uytterhoeven wrote: > On Thu, Nov 17, 2022 at 1:26 PM Tomi Valkeinen wrote: > > From: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com> > > > > Extend the Renesas DSI display bindings to support the r8a779g0 V4H. > > > > Signed-off-by: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com> > > --- > > .../bindings/display/bridge/renesas,dsi-csi2-tx.yaml | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml b/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml > > index afeeb967393d..bc3101f77e5a 100644 > > --- a/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml > > +++ b/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml > > @@ -11,13 +11,14 @@ maintainers: > > > > description: | > > This binding describes the MIPI DSI/CSI-2 encoder embedded in the Renesas > > - R-Car V3U SoC. The encoder can operate in either DSI or CSI-2 mode, with up > > + R-Car V3U/V4H SoC. The encoder can operate in either DSI or CSI-2 mode, with up > > Perhaps "R-Car Gen4 SoCs", so we stay within 80 chars, and don't have > to update this when the next member of the family is around the block? Sounds good. > Is there anything that might be SoC-specific? > If not, perhaps the time is ripe for a family-specific compatible value? That's hard to tell, I have little visibility into what surprises other SoCs will bring :-S > > to four data lanes. > > > > properties: > > compatible: > > enum: > > - renesas,r8a779a0-dsi-csi2-tx # for V3U > > + - renesas,r8a779g0-dsi-csi2-tx # for V4H Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > > > reg: > > maxItems: 1
On 17/11/2022 17:14, Geert Uytterhoeven wrote: > Hi Tomi, > > On Thu, Nov 17, 2022 at 1:26 PM Tomi Valkeinen > <tomi.valkeinen@ideasonboard.com> wrote: >> From: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com> >> >> Extend the Renesas DSI display bindings to support the r8a779g0 V4H. >> >> Signed-off-by: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com> >> --- >> .../bindings/display/bridge/renesas,dsi-csi2-tx.yaml | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml b/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml >> index afeeb967393d..bc3101f77e5a 100644 >> --- a/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml >> +++ b/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml >> @@ -11,13 +11,14 @@ maintainers: >> >> description: | >> This binding describes the MIPI DSI/CSI-2 encoder embedded in the Renesas >> - R-Car V3U SoC. The encoder can operate in either DSI or CSI-2 mode, with up >> + R-Car V3U/V4H SoC. The encoder can operate in either DSI or CSI-2 mode, with up > > Perhaps "R-Car Gen4 SoCs", so we stay within 80 chars, and don't have > to update this when the next member of the family is around the block? Is V3U gen 4? Or do you mean "R-Car V3U and Gen 4 SoCs"? > Is there anything that might be SoC-specific? > If not, perhaps the time is ripe for a family-specific compatible value? At least v3u and v4h DSIs are slightly different. Well, the DSI IP block itself looks the same, but the PLL and PHY are different. Tomi
Hi Tomi, On Tue, Nov 22, 2022 at 9:20 AM Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> wrote: > On 17/11/2022 17:14, Geert Uytterhoeven wrote: > > On Thu, Nov 17, 2022 at 1:26 PM Tomi Valkeinen > > <tomi.valkeinen@ideasonboard.com> wrote: > >> From: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com> > >> > >> Extend the Renesas DSI display bindings to support the r8a779g0 V4H. > >> > >> Signed-off-by: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com> > >> --- > >> .../bindings/display/bridge/renesas,dsi-csi2-tx.yaml | 3 ++- > >> 1 file changed, 2 insertions(+), 1 deletion(-) > >> > >> diff --git a/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml b/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml > >> index afeeb967393d..bc3101f77e5a 100644 > >> --- a/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml > >> +++ b/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml > >> @@ -11,13 +11,14 @@ maintainers: > >> > >> description: | > >> This binding describes the MIPI DSI/CSI-2 encoder embedded in the Renesas > >> - R-Car V3U SoC. The encoder can operate in either DSI or CSI-2 mode, with up > >> + R-Car V3U/V4H SoC. The encoder can operate in either DSI or CSI-2 mode, with up > > > > Perhaps "R-Car Gen4 SoCs", so we stay within 80 chars, and don't have > > to update this when the next member of the family is around the block? > > Is V3U gen 4? Or do you mean "R-Car V3U and Gen 4 SoCs"? Despite the name, R-Car V3U is the first member of the R-Car Gen4 family... https://www.renesas.com/us/en/products/automotive-products/automotive-system-chips-socs/r-car-v3u-best-class-r-car-v3u-asil-d-system-chip-automated-driving > > Is there anything that might be SoC-specific? > > If not, perhaps the time is ripe for a family-specific compatible value? > > At least v3u and v4h DSIs are slightly different. Well, the DSI IP block > itself looks the same, but the PLL and PHY are different. I noticed, when I saw the dsi-csi2 driver changes. So no family-specific compatible value is needed. 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
diff --git a/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml b/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml index afeeb967393d..bc3101f77e5a 100644 --- a/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml +++ b/Documentation/devicetree/bindings/display/bridge/renesas,dsi-csi2-tx.yaml @@ -11,13 +11,14 @@ maintainers: description: | This binding describes the MIPI DSI/CSI-2 encoder embedded in the Renesas - R-Car V3U SoC. The encoder can operate in either DSI or CSI-2 mode, with up + R-Car V3U/V4H SoC. The encoder can operate in either DSI or CSI-2 mode, with up to four data lanes. properties: compatible: enum: - renesas,r8a779a0-dsi-csi2-tx # for V3U + - renesas,r8a779g0-dsi-csi2-tx # for V4H reg: maxItems: 1