Re: [PATCH v3] compositor-drm: Add the aspect ratio parsing support

2017-11-10 Thread Pekka Paalanen
On Thu, 9 Nov 2017 19:24:00 +0530 "Nautiyal, Ankit K" wrote: > > On 11/3/2017 3:52 PM, Pekka Paalanen wrote: > > (I fixed the subject line to contain v3.) > > > > On Mon, 16 Oct 2017 20:25:23 +0530 > > Nautiyal Ankit wrote: > > > >> @@ -2376,6 +2424,7 @@ static struct drm_mode * > >> drm_o

Re: [PATCH v3] compositor-drm: Add the aspect ratio parsing support

2017-11-09 Thread Nautiyal, Ankit K
Hi Pekka, Many thanks for the review comments, and suggestions. I will address these in the next patch. Please find my clarifications/comments inline below. On 11/3/2017 3:52 PM, Pekka Paalanen wrote: (I fixed the subject line to contain v3.) On Mon, 16 Oct 2017 20:25:23 +0530 Nautiyal Ank

Re: [PATCH v3] compositor-drm: Add the aspect ratio parsing support

2017-11-03 Thread Pekka Paalanen
(I fixed the subject line to contain v3.) On Mon, 16 Oct 2017 20:25:23 +0530 Nautiyal Ankit wrote: > From: Ankit Nautiyal > > DRM layer populates aspect ratio information for CEA video modes, > but we lose it while parsing modeline (Aspect ratio patch series > in drm layer: https://patchwork.f