Re: [PATCH 1/4] drm: add picture aspect ratio flags

Sharma, Shashank
2016-08-09T13:57:00Z

Thanks Daniel.

Hi Joes,
As Daniel mentioned, I have already written a patch which adds all HDMI 2.0 modes in CEA DB, and it also completes all the VICs in the DB from 1-107 (we have 1-64 now).

I was myself waiting for aspect ratio patches to get accepted, as I needed them for the modes.
Will keep you in CC.

I am planning to send a new patch set by tonight.

Regards
Shashank

-----Original Message-----
From: daniel.vetter@ffwll.ch [mailto:daniel.vetter@ffwll.ch] On Behalf Of Daniel Vetter
Sent: Tuesday, August 9, 2016 7:14 PM
To: Jose Abreu <Jose.Abreu@synopsys.com>
Cc: Sharma, Shashank <shashank.sharma@intel.com>; Emil Velikov <emil.l.velikov@gmail.com>; Vetter, Daniel <daniel.vetter@intel.com>; intel-gfx@lists.freedesktop.org; ML dri-devel <dri-devel@lists.freedesktop.org>
Subject: Re: [Intel-gfx] [PATCH 1/4] drm: add picture aspect ratio flags

On Tue, Aug 9, 2016 at 3:12 PM, Jose Abreu <Jose.Abreu@synopsys.com> wrote:
> Will you send a new version of these patches? I have a patch ready
> that adds the new HDMI 2.0 modes to the CEA modes list in DRM but
> these modes require the addition of the new picture aspect ratio flags
> (64:27, 256:135). I can either wait that your patches get accepted or
> I can add to my patch set one that adds the new PAR flags.

Please base your work on top of Sharma's entire patch series - I suspect that Sharma's patch series already covers all you really need.
With the exception of then correctly encoding the chosen aspect ratio into the infoframes.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch