From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomi Valkeinen Subject: Re: [PATCH v5 0/4] drm/omapdrm: gamma table support + drm_crtc_enable_color_mgmt() Date: Thu, 26 May 2016 14:15:35 +0300 Message-ID: <5746DAD7.1070404@ti.com> References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0579156766==" Return-path: Received: from bear.ext.ti.com (bear.ext.ti.com [198.47.19.11]) by gabe.freedesktop.org (Postfix) with ESMTPS id 2E0F66EAA3 for ; Thu, 26 May 2016 11:15:45 +0000 (UTC) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Jyri Sarha , dri-devel@lists.freedesktop.org Cc: peter.ujfalusi@ti.com, laurent.pinchart@ideasonboard.com List-Id: dri-devel@lists.freedesktop.org --===============0579156766== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="B7ipPXcwhNQ6v9I57ABXv81vS1MKcif52" --B7ipPXcwhNQ6v9I57ABXv81vS1MKcif52 Content-Type: multipart/mixed; boundary="VQM9Dfw2MWMaU6tELV3pLHtp4qh63cPr6" From: Tomi Valkeinen To: Jyri Sarha , dri-devel@lists.freedesktop.org Cc: airlied@linux.ie, daniel@ffwll.ch, peter.ujfalusi@ti.com, bparrot@ti.com, laurent.pinchart@ideasonboard.com, lionel.g.landwerlin@intel.com Message-ID: <5746DAD7.1070404@ti.com> Subject: Re: [PATCH v5 0/4] drm/omapdrm: gamma table support + drm_crtc_enable_color_mgmt() References: In-Reply-To: --VQM9Dfw2MWMaU6tELV3pLHtp4qh63cPr6 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 26/05/16 11:35, Jyri Sarha wrote: > Implements gamma tables for OMAP4, OMAP5, and dra7xx SoCs and adds a > workaround for errata that may break LCD1 channel if gamma tables > are in use. >=20 > Also adds new drm_crtc_enable_color_mgmt() as suggested[1] by Daniel > Vetter and get rid of the old drm_helper_crtc_enable_color_mgmt(). I > have not tested the change to i915 driver, only compiled it, but > functionally it should be exactly the same. Tested the v4, which should be identical to this one, on Panda, and works fine. This version looks good too: Reviewed-by: Tomi Valkeinen Tomi --VQM9Dfw2MWMaU6tELV3pLHtp4qh63cPr6-- --B7ipPXcwhNQ6v9I57ABXv81vS1MKcif52 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJXRtrYAAoJEPo9qoy8lh71P/wP/1J4m8d86nqr91juMLd2GntH YKUI5ibCKcS2OQCa6hR0M7Ed+5a4dIzgXEF3KUvltSLv8UeiJjSJdRd9JPhrA6zh KO5xe0bBdD5JdqY9HOlG3AwFPUTR54ZdDIxmMOBBMfLURm85/WfjPm5OfOUCLcjt DzR7ID6Dd4lIVX2sHnFfyV5bDJBJn1JhECClcaD7HqMCXEvOfKq9EbftGBOXPs+2 QpR0BIJUy/+3fp8b6bb+it7LLFPA14Ktw+Qr0L5Yp0vw5OrrdDB5dDUrC9m/Ce7w FqTjm9jGqIPWDsRLV923xul5FtYHUv+hLygDU8om0vDWfqb8G/PO07XqlBwmb6Ga 4WHil0QCizTuowpLwlv7Ot9t4MvgYw0hB/ZWwwcqoYaHxEKUHjQFm/hTITy2eEOu wj7kR88ib0Mqd63Fb/fxmY5GkyhuwxFRqTshFSVgUxJ5nQ9tuvpQgaQQnc7L9mCr G5S26q+IdMyGyIPOTUYpseHH90NlWJev5fme1FHguoFWDJLTLv4htaLuzlBilIzB 9RSKRsYvZjXV1ljCrrGCtYZm79vSLpknhcrOmWuYXb6UgxEKgCVG05Ytfb4ivYj9 0fXkLmgqkgJCsdPLcubSSE84nGghlfgso4MmS1QMbF94mGKy+x1DcquleFTDh1xI ydBkb7Sriyn5iu6xodTV =a4yQ -----END PGP SIGNATURE----- --B7ipPXcwhNQ6v9I57ABXv81vS1MKcif52-- --===============0579156766== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== --===============0579156766==--