RE: [PATCH v2 5/9] drm/panel: Add timings field to drm_panel

2019-08-15 Thread Fabrizio Castro
Hi Sam, Thank you for your feedback! > From: Sam Ravnborg > Sent: 15 August 2019 15:14 > Subject: Re: [PATCH v2 5/9] drm/panel: Add timings field to drm_panel > > Hi Fabrizio > > On Thu, Aug 15, 2019 at 12:04:29PM +0100, Fabrizio Castro wrote: > > We need to know

Re: [PATCH v2 5/9] drm/panel: Add timings field to drm_panel

2019-08-15 Thread Sam Ravnborg
Hi Fabrizio On Thu, Aug 15, 2019 at 12:04:29PM +0100, Fabrizio Castro wrote: > We need to know if the panel supports dual-link, similarly > to bridges, therefore add a reference to drm_timings in > drm_panel. Why do we need to know this? Why is it needed in drm_panel and not in some driver specif

RE: [PATCH v2 5/9] drm/panel: Add timings field to drm_panel

2019-08-15 Thread Fabrizio Castro
Hello Laurent, Thank you for your feedback! > From: linux-renesas-soc-ow...@vger.kernel.org > On Behalf Of Laurent Pinchart > Sent: 15 August 2019 13:03 > Subject: Re: [PATCH v2 5/9] drm/panel: Add timings field to drm_panel > > Hi Fabrizio, > > Thank you for the pa

Re: [PATCH v2 5/9] drm/panel: Add timings field to drm_panel

2019-08-15 Thread Laurent Pinchart
Hi Fabrizio, Thank you for the patch. On Thu, Aug 15, 2019 at 12:04:29PM +0100, Fabrizio Castro wrote: > We need to know if the panel supports dual-link, similarly > to bridges, therefore add a reference to drm_timings in > drm_panel. Panels may also need to report setup/hold time, so it's not a

[PATCH v2 5/9] drm/panel: Add timings field to drm_panel

2019-08-15 Thread Fabrizio Castro
We need to know if the panel supports dual-link, similarly to bridges, therefore add a reference to drm_timings in drm_panel. Signed-off-by: Fabrizio Castro --- v1->v2: * new patch include/drm/drm_panel.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/include/drm/drm_panel.h b/include/