From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [Intel-gfx] [PATCH v3] drm/edid: Fill PAR in AVI infoframe based on CEA mode list Date: Tue, 1 Apr 2014 19:04:56 +0200 Message-ID: <20140401170456.GL22327@phenom.ffwll.local> References: <20140401135446.GG21652@intel.com> <1396362141-5220-1-git-send-email-vandana.kannan@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Content-Disposition: inline In-Reply-To: <1396362141-5220-1-git-send-email-vandana.kannan@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Vandana Kannan Cc: intel-gfx@lists.freedesktop.org, Jesse Barnes , dri-devel@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org 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 foun= d. > 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 > Cc: Jesse Barnes > Cc: Vijay Purushothaman > Cc: Ville Syrj=E4l=E4 > Cc: intel-gfx@lists.freedesktop.org > Reviewed-by: Jesse Barnes 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