drm: rcar-du: lvds: add R8A774C0 support
diff mbox series

Message ID 1544732616-7322-1-git-send-email-fabrizio.castro@bp.renesas.com
State New
Delegated to: Kieran Bingham
Headers show
Series
  • drm: rcar-du: lvds: add R8A774C0 support
Related show

Commit Message

Fabrizio Castro Dec. 13, 2018, 8:23 p.m. UTC
The LVDS implementation on the RZ/G2E (a.k.a. R8A774C0) is very similar
to the one found on R-Car E3 (a.k.a. R8A77990), therefore add RZ/G2E
LVDS support to the LVDS encoder driver in a similar fashion to what
done for R-Car E3.

Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
---
 drivers/gpu/drm/rcar-du/rcar_lvds.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Laurent Pinchart Dec. 13, 2018, 9:37 p.m. UTC | #1
Hi Fabrizio,

Thank you for the patch.

On Thursday, 13 December 2018 22:23:36 EET Fabrizio Castro wrote:
> The LVDS implementation on the RZ/G2E (a.k.a. R8A774C0) is very similar
> to the one found on R-Car E3 (a.k.a. R8A77990), therefore add RZ/G2E
> LVDS support to the LVDS encoder driver in a similar fashion to what

s/what/what is/

> done for R-Car E3.
> 
> Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>

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

and applied to my tree with the above fix.

> ---
>  drivers/gpu/drm/rcar-du/rcar_lvds.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/rcar-du/rcar_lvds.c
> b/drivers/gpu/drm/rcar-du/rcar_lvds.c index 173d7ad..329dfca 100644
> --- a/drivers/gpu/drm/rcar-du/rcar_lvds.c
> +++ b/drivers/gpu/drm/rcar-du/rcar_lvds.c
> @@ -785,6 +785,7 @@ static const struct rcar_lvds_device_info
> rcar_lvds_r8a77995_info = {
> 
>  static const struct of_device_id rcar_lvds_of_table[] = {
>  	{ .compatible = "renesas,r8a7743-lvds", .data = &rcar_lvds_gen2_info },
> +	{ .compatible = "renesas,r8a774c0-lvds", .data = &rcar_lvds_r8a77990_info
> }, { .compatible = "renesas,r8a7790-lvds", .data = &rcar_lvds_r8a7790_info
> }, { .compatible = "renesas,r8a7791-lvds", .data = &rcar_lvds_gen2_info },
> { .compatible = "renesas,r8a7793-lvds", .data = &rcar_lvds_gen2_info },
Fabrizio Castro Dec. 14, 2018, 8:11 a.m. UTC | #2
Hi Laurent,

> -----Original Message-----
> From: linux-renesas-soc-owner@vger.kernel.org <linux-renesas-soc-owner@vger.kernel.org> On Behalf Of Laurent Pinchart
> Sent: 13 December 2018 21:38
> To: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> Cc: David Airlie <airlied@linux.ie>; Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>; dri-devel@lists.freedesktop.org;
> linux-renesas-soc@vger.kernel.org; linux-kernel@vger.kernel.org; Simon Horman <horms@verge.net.au>; Geert Uytterhoeven
> <geert+renesas@glider.be>; Chris Paterson <Chris.Paterson2@renesas.com>; Biju Das <biju.das@bp.renesas.com>
> Subject: Re: [PATCH] drm: rcar-du: lvds: add R8A774C0 support
>
> Hi Fabrizio,
>
> Thank you for the patch.
>
> On Thursday, 13 December 2018 22:23:36 EET Fabrizio Castro wrote:
> > The LVDS implementation on the RZ/G2E (a.k.a. R8A774C0) is very similar
> > to the one found on R-Car E3 (a.k.a. R8A77990), therefore add RZ/G2E
> > LVDS support to the LVDS encoder driver in a similar fashion to what
>
> s/what/what is/
>
> > done for R-Car E3.
> >
> > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>
> and applied to my tree with the above fix.

Thank you for fixing.

Fab

>
> > ---
> >  drivers/gpu/drm/rcar-du/rcar_lvds.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/gpu/drm/rcar-du/rcar_lvds.c
> > b/drivers/gpu/drm/rcar-du/rcar_lvds.c index 173d7ad..329dfca 100644
> > --- a/drivers/gpu/drm/rcar-du/rcar_lvds.c
> > +++ b/drivers/gpu/drm/rcar-du/rcar_lvds.c
> > @@ -785,6 +785,7 @@ static const struct rcar_lvds_device_info
> > rcar_lvds_r8a77995_info = {
> >
> >  static const struct of_device_id rcar_lvds_of_table[] = {
> >  { .compatible = "renesas,r8a7743-lvds", .data = &rcar_lvds_gen2_info },
> > +{ .compatible = "renesas,r8a774c0-lvds", .data = &rcar_lvds_r8a77990_info
> > }, { .compatible = "renesas,r8a7790-lvds", .data = &rcar_lvds_r8a7790_info
> > }, { .compatible = "renesas,r8a7791-lvds", .data = &rcar_lvds_gen2_info },
> > { .compatible = "renesas,r8a7793-lvds", .data = &rcar_lvds_gen2_info },
>
>
> --
> Regards,
>
> Laurent Pinchart
>
>



[https://www2.renesas.eu/media/email/unicef.jpg]

This Christmas, instead of sending out cards, Renesas Electronics Europe have decided to support Unicef with a donation. For further details click here<https://www.unicef.org/> to find out about the valuable work they do, helping children all over the world.
We would like to take this opportunity to wish you a Merry Christmas and a prosperous New Year.



Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
Simon Horman Dec. 17, 2018, 11:58 a.m. UTC | #3
On Thu, Dec 13, 2018 at 08:23:36PM +0000, Fabrizio Castro wrote:
> The LVDS implementation on the RZ/G2E (a.k.a. R8A774C0) is very similar
> to the one found on R-Car E3 (a.k.a. R8A77990), therefore add RZ/G2E
> LVDS support to the LVDS encoder driver in a similar fashion to what
> done for R-Car E3.
> 
> Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>

Reviewed-by: Simon Horman <horms+renesas@verge.net.au>

Patch
diff mbox series

diff --git a/drivers/gpu/drm/rcar-du/rcar_lvds.c b/drivers/gpu/drm/rcar-du/rcar_lvds.c
index 173d7ad..329dfca 100644
--- a/drivers/gpu/drm/rcar-du/rcar_lvds.c
+++ b/drivers/gpu/drm/rcar-du/rcar_lvds.c
@@ -785,6 +785,7 @@  static const struct rcar_lvds_device_info rcar_lvds_r8a77995_info = {
 
 static const struct of_device_id rcar_lvds_of_table[] = {
 	{ .compatible = "renesas,r8a7743-lvds", .data = &rcar_lvds_gen2_info },
+	{ .compatible = "renesas,r8a774c0-lvds", .data = &rcar_lvds_r8a77990_info },
 	{ .compatible = "renesas,r8a7790-lvds", .data = &rcar_lvds_r8a7790_info },
 	{ .compatible = "renesas,r8a7791-lvds", .data = &rcar_lvds_gen2_info },
 	{ .compatible = "renesas,r8a7793-lvds", .data = &rcar_lvds_gen2_info },