From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomi Valkeinen Subject: Re: [PATCH v6 1/4] drm: drm_helper_crtc_enable_color_mgmt() => drm_crtc_enable_color_mgmt() Date: Tue, 7 Jun 2016 19:54:35 +0300 Message-ID: <5756FC4B.1050700@ti.com> References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0120111948==" Return-path: Received: from devils.ext.ti.com (devils.ext.ti.com [198.47.26.153]) by gabe.freedesktop.org (Postfix) with ESMTPS id B20926E110 for ; Tue, 7 Jun 2016 16:54:44 +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, daniel@ffwll.ch Cc: laurent.pinchart@ideasonboard.com List-Id: dri-devel@lists.freedesktop.org --===============0120111948== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="OxOLniU627a0cljfur5V1eSHwdxVvET4R" --OxOLniU627a0cljfur5V1eSHwdxVvET4R Content-Type: multipart/mixed; boundary="M3RhARN6aNak8LUpA1gpTe5jWJrWnD37c" From: Tomi Valkeinen To: Jyri Sarha , dri-devel@lists.freedesktop.org, daniel@ffwll.ch Cc: airlied@linux.ie, bparrot@ti.com, laurent.pinchart@ideasonboard.com, lionel.g.landwerlin@intel.com Message-ID: <5756FC4B.1050700@ti.com> Subject: Re: [PATCH v6 1/4] drm: drm_helper_crtc_enable_color_mgmt() => drm_crtc_enable_color_mgmt() References: In-Reply-To: --M3RhARN6aNak8LUpA1gpTe5jWJrWnD37c Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 07/06/16 15:09, Jyri Sarha wrote: > Add drm_crtc_enable_color_mgmt(), remove drm_helper_crtc_enable_color_m= gmt() > and update drm/i915-driver (the only user of the old function). >=20 > The new function is more flexible. It allows driver to enable only the > features it has without forcing to enable all three color management > properties: degamma lut, csc matrix (ctm), and gamma lut. >=20 > Suggested-by: Daniel Vetter > Signed-off-by: Jyri Sarha > --- > drivers/gpu/drm/drm_crtc.c | 45 ++++++++++++++++++++++++++++++= ++++++++ > drivers/gpu/drm/drm_crtc_helper.c | 33 ---------------------------- > drivers/gpu/drm/i915/intel_color.c | 3 ++- > include/drm/drm_crtc.h | 5 ++++- > include/drm/drm_crtc_helper.h | 3 --- > 5 files changed, 51 insertions(+), 38 deletions(-) Looks good to me. I can queue this up with the omapdrm patches in this series, if no one complains. Tomi --M3RhARN6aNak8LUpA1gpTe5jWJrWnD37c-- --OxOLniU627a0cljfur5V1eSHwdxVvET4R 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 iQIcBAEBCAAGBQJXVvxLAAoJEPo9qoy8lh71KwAP/ioRzLyLLTwAymDZrrWLoNGn yRcT4AJbez6E5/nxaWRMDZuvbDXQ/kkf23xFmt2e7htAa26MPdQ5YEynOMVZxAQP E+U3WaTkS/ZFfIw8sa50YfOV8ckz4lJckzUNVjI2vTKxyCUtg/g9d4Aqexrb8QRw QwXK2/iO0MZwb9+8DhdlX375xlXL50x3PnL82nTRtpnZ5Bm2ssDX15/rKUfziHw+ DAZpRwrRJOf4kako0mCYMdb+yUNgza0dAgBNoksVybLRayIU9I4di371OQ64yyA3 3BeqgpuTkLDkW/FR3uiPkdiSoDdmbmCWWvVKc3PwLIbbz4VBePJhJ0Ihh7VD0Vd/ SofaS8/3tEIcOHitVyLefLrVcXxotlwdWeOTB4iMhLaXN7BPpNu/Jfl9ehUlOJT2 IA711jTTtD4cnuo0ddWnAOLUPDh/khZIpio7M+h+e/L+T20MHP50bEQqzz8rvFBr wPA7VxB42ZM56QDSX6UFR+FX+PWELR9stfZ4xaFb1c80pXoGd2EoWp5NVoATZ01d T1WfLdUXiVt826snQkoJFmcab2+iig4+aPmrSongedPTRDLjHvXVaFwQBCGM7qHl 43fL8Jyff9bFtZFW1VNGpLhUDYFMONQlWac1Rdj5Ei4lB7QxrI4fiPv1aYTkSBGV 6HRym+ZjKFE4NFdFDvqw =ULtM -----END PGP SIGNATURE----- --OxOLniU627a0cljfur5V1eSHwdxVvET4R-- --===============0120111948== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== --===============0120111948==--