On Tue, Apr 01, 2014 at 07:52:21PM +0530, Vandana Kannan wrote: > Populate PAR in infoframe structure. If there is a user setting for PAR, then > that value is set. Else, value is taken from CEA mode list if VIC is found. > Else, PAR is calculated from resolution. If none of these conditions are > satisfied, PAR is NONE as per initialization. > > v2: Removed the part which sets PAR according to user input, based on > Daniel's review comments. > > v3: Removed calculation of PAR for non-CEA modes as per discussion with > Ville. > > A separate patch will be submitted to create a property that would enable a > user space app to set aspect ratio for AVI infoframe. > > Signed-off-by: Vandana Kannan <vandana.kannan@xxxxxxxxx> > Cc: Jesse Barnes <jesse.barnes@xxxxxxxxx> > Cc: Vijay Purushothaman <vijay.a.purushothaman@xxxxxxxxx> > Cc: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx > Reviewed-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> I've pulled this into my topic/core-stuff branch so it doesn't get lost. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
