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

2017-10-16 Thread Nautiyal, Ankit K
Hi Pekka, I have sent the next version of the patch with the review comments addressed, it got a bit delayed, due to addition of the DRM CLIENT CAP for aspect-ratio. So Weston will call now the drmSetClientCap to show that it understands the aspect_ratio flags from the DRM layer. If for so

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

2017-10-05 Thread Pekka Paalanen
On Thu, 5 Oct 2017 16:28:37 +0530 "Nautiyal, Ankit K" wrote: > Hi Pekka, > > Thanks a lot for your review comments and the suggestions, and pointing > out the things I had missed. > > I agree to most of the changes, please find my response inline. > > I will shortly send the next version of t

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

2017-10-05 Thread Nautiyal, Ankit K
Hi Pekka, Thanks a lot for your review comments and the suggestions, and pointing out the things I had missed. I agree to most of the changes, please find my response inline. I will shortly send the next version of the patch in a couple of days. On 10/4/2017 2:13 PM, Pekka Paalanen wrote: O

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

2017-10-04 Thread Pekka Paalanen
On Wed, 20 Sep 2017 17:33:00 +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.freedesktop.org/series/10850). > The flag bi

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

2017-09-20 Thread Nautiyal Ankit
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.freedesktop.org/series/10850). The flag bits 19-22 of the connector modes, provide the aspect ratio info. This in