From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Subject: Re: [RFC v2 3/5] drm: Add HDMI infoframe helpers Date: Thu, 6 Dec 2012 17:02:34 +0100 Message-ID: <20121206160234.GB13614@avionic-0098.adnet.avionic-design.de> References: <1354725944-1862-1-git-send-email-thierry.reding@avionic-design.de> <1354725944-1862-4-git-send-email-thierry.reding@avionic-design.de> <20121206140900.GT11556@phenom.ffwll.local> <20121206142803.GB15982@avionic-0098.adnet.avionic-design.de> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1579891536==" Return-path: Received: from moutng.kundenserver.de (moutng.kundenserver.de [212.227.126.171]) by gabe.freedesktop.org (Postfix) with ESMTP id 68E81E6765 for ; Thu, 6 Dec 2012 08:02:38 -0800 (PST) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Daniel Vetter Cc: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============1579891536== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="Bn2rw/3z4jIqBvZU" Content-Disposition: inline --Bn2rw/3z4jIqBvZU Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Dec 06, 2012 at 04:44:38PM +0100, Daniel Vetter wrote: > On Thu, Dec 6, 2012 at 3:28 PM, Thierry Reding > wrote: > >> Note that the intel avi infoframe in intel_hdmi_set_avi_infoframe also > >> sets the pixel repeat for double clocked modes with: > >> > >> if (adjusted_mode->flags & DRM_MODE_FLAG_DBLCLK) > >> avi_if.body.avi.YQ_CN_PR |=3D DIP_AVI_PR_2; > > > > I left that in place in the Intel driver, but I suppose we can just as > > well move it to the infoframe helpers since it should be a generic > > option. >=20 > Yeah, noticed that while reading through the drm/i915 patch ;-) If you > move I think that would be good in a 2nd patch, just in case it blows > up somewhere it's easier for other drivers it'd be easier to > root-cause. Okay, will do. Thierry --Bn2rw/3z4jIqBvZU Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.19 (GNU/Linux) iQIcBAEBAgAGBQJQwMGaAAoJEN0jrNd/PrOhvkcP/2QZNYDfpADBxjIuEaR5Geo7 4rsBrvUp3NQeZPYXU/+7HZS4Fpkb0sfB6ZF57gRlyUOVjWPYL6fHhn+EUmE8j3jA /EdQoYwCEy8VrNwLLNBL/3BhIw8925Ud6/gjtNnqxYf9ZTv0itD87bSAmH4dQBgC ecmK2KtlaLp8hBP3xwN7GKOX1/WiRt0ul7P9rvER8aolWiWePd9bvhuSxvMSfyOf fxrAatvS1f7x4qMz0IH0h/a+o0xDEKyRWtT+wAkqEdRGdd0jdyqTm2sFmozvUy5H avksJrRSun9qgxI7Xn7+UcwjJeH2ENursZSOKDABu+kgxH7qhSGxI6XCa3Di3PwC RPgDxabE3jOd1RDJfwSsz9x0DRIhjj6a5yNuTofa4j+tqDMfyO1848F0HQkOO7Lt 2CoWX9LFjJGrYSKA+teS0i3zNH4fN265FODHwXLW8mql5BeI5isP0n+U6/DCTHZ4 Oy8jFT0Bd42uMWDcYBlW5iBZudPgyMgAYNwaUvLZvuq8NhdI2/mI1ajzMF68dv7I eo/gep7E+wM0oOkNfdqv0s7s8k19KG9BtL3yfxfG38QQjbm1ya0tqWBPFE7zirmg IcjsA5dq7eQDPuZKpzl/zRKBqQk9vbRMlHOGOOEytXYaRor8YhkpALZFB2swRL5R EdAkysmT2qULM/qj5AWm =vFiR -----END PGP SIGNATURE----- --Bn2rw/3z4jIqBvZU-- --===============1579891536== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel --===============1579891536==--