2018-02-01 07:48:00

by Kuninori Morimoto

[permalink] [raw]
Subject: [PATCH] drm/panel: lvds: tidyup header explanation


From: Kuninori Morimoto <[email protected]>

panel-lvds.c is for LVDS Panel Driver,
not R-Car Display Unit CRTCs

Reported-by: Koji Matsuoka <[email protected]>
Signed-off-by: Kuninori Morimoto <[email protected]>
---
drivers/gpu/drm/panel/panel-lvds.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/panel/panel-lvds.c b/drivers/gpu/drm/panel/panel-lvds.c
index b5e3994..c8075a8 100644
--- a/drivers/gpu/drm/panel/panel-lvds.c
+++ b/drivers/gpu/drm/panel/panel-lvds.c
@@ -1,5 +1,5 @@
/*
- * rcar_du_crtc.c -- R-Car Display Unit CRTCs
+ * panel-lvds.c -- LVDS Panel Driver
*
* Copyright (C) 2016 Laurent Pinchart
* Copyright (C) 2016 Renesas Electronics Corporation
--
1.9.1



2018-02-02 09:21:25

by Laurent Pinchart

[permalink] [raw]
Subject: Re: [PATCH] drm/panel: lvds: tidyup header explanation

Hi Morimoto-san,

Thank you for your patch.

On Thursday, 1 February 2018 09:45:36 EET Kuninori Morimoto wrote:
> From: Kuninori Morimoto <[email protected]>
>
> panel-lvds.c is for LVDS Panel Driver,
> not R-Car Display Unit CRTCs
>
> Reported-by: Koji Matsuoka <[email protected]>
> Signed-off-by: Kuninori Morimoto <[email protected]>

A similar patch has been posted already:

https://patchwork.freedesktop.org/series/36953/

> ---
> drivers/gpu/drm/panel/panel-lvds.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/panel/panel-lvds.c
> b/drivers/gpu/drm/panel/panel-lvds.c index b5e3994..c8075a8 100644
> --- a/drivers/gpu/drm/panel/panel-lvds.c
> +++ b/drivers/gpu/drm/panel/panel-lvds.c
> @@ -1,5 +1,5 @@
> /*
> - * rcar_du_crtc.c -- R-Car Display Unit CRTCs
> + * panel-lvds.c -- LVDS Panel Driver
> *
> * Copyright (C) 2016 Laurent Pinchart
> * Copyright (C) 2016 Renesas Electronics Corporation


--
Regards,

Laurent Pinchart


2018-02-05 00:17:58

by Kuninori Morimoto

[permalink] [raw]
Subject: Re: [PATCH] drm/panel: lvds: tidyup header explanation


Hi Laurent

> Hi Morimoto-san,
>
> Thank you for your patch.
>
> On Thursday, 1 February 2018 09:45:36 EET Kuninori Morimoto wrote:
> > From: Kuninori Morimoto <[email protected]>
> >
> > panel-lvds.c is for LVDS Panel Driver,
> > not R-Car Display Unit CRTCs
> >
> > Reported-by: Koji Matsuoka <[email protected]>
> > Signed-off-by: Kuninori Morimoto <[email protected]>
>
> A similar patch has been posted already:
>
> https://patchwork.freedesktop.org/series/36953/

Thanks.
Nice to know

Best regards
---
Kuninori Morimoto