From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jyri Sarha Subject: Re: [PATCH 2/4] drm/omapdrm: Add gamma table support to DSS dispc Date: Fri, 20 May 2016 16:14:52 +0300 Message-ID: <573F0DCC.9040500@ti.com> References: <573F0868.8090700@ti.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1220862589==" Return-path: Received: from bear.ext.ti.com (bear.ext.ti.com [198.47.19.11]) by gabe.freedesktop.org (Postfix) with ESMTPS id 287FD6EAC1 for ; Fri, 20 May 2016 13:22:28 +0000 (UTC) In-Reply-To: <573F0868.8090700@ti.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Tomi Valkeinen , dri-devel@lists.freedesktop.org Cc: peter.ujfalusi@ti.com, laurent.pinchart@ideasonboard.com List-Id: dri-devel@lists.freedesktop.org --===============1220862589== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="5eA6F40ogeEc93tkqu9SDEmtVqBH7mqvI" --5eA6F40ogeEc93tkqu9SDEmtVqBH7mqvI Content-Type: multipart/mixed; boundary="VTk3Sw2ghPcApnIKid4cD5v7q2AjsGBmt" From: Jyri Sarha To: Tomi Valkeinen , dri-devel@lists.freedesktop.org Cc: airlied@linux.ie, daniel@ffwll.ch, peter.ujfalusi@ti.com, bparrot@ti.com, laurent.pinchart@ideasonboard.com Message-ID: <573F0DCC.9040500@ti.com> Subject: Re: [PATCH 2/4] drm/omapdrm: Add gamma table support to DSS dispc References: <573F0868.8090700@ti.com> In-Reply-To: <573F0868.8090700@ti.com> --VTk3Sw2ghPcApnIKid4cD5v7q2AjsGBmt Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 05/20/16 15:51, Tomi Valkeinen wrote: >> +u32 dispc_mgr_gamma_size(enum omap_channel channel) >> > +{ >> > + const struct dispc_gamma_desc *gdesc =3D &mgr_desc[channel].gamma;= >> > + >> > + if (!dss_has_feature(FEAT_GAMMA_TABLE)) >> > + return 0; >> > + > This should probably check the availability of LCD2/LCD3? >=20 That would be a double check (the function should never be called with channels that are not there), but I am happy to add it. --VTk3Sw2ghPcApnIKid4cD5v7q2AjsGBmt-- --5eA6F40ogeEc93tkqu9SDEmtVqBH7mqvI 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.0.22 (GNU/Linux) iQIcBAEBAgAGBQJXPw3SAAoJEJA2s1DX1hlBFwUP/2ScRc4PbuILXzF2xRP7qJ5T yAeK9JXAx71sWiZAFDiAobkusWrkeU5IEYlX+kbHqno+xQXWiL6RRSlFUb9sMrQo JmRNBeODBikzBKLal0FGTSUyft4ZIRA/cwFSe9HJoS5K2covjd5ROG+Rrt8egk5a sdECfZhvqAmCkl7ixpS+zvWu4w9vOzGZGu0hDBtUWebgMyFFJK/GZAheWb6Pe0O5 LQehDn8xvPtxr1jS2xBMzhXlI6PETJN2v5IVQVxoIqPiPawdtQaEIdyxFqFSRFZu MKAH9N3xVfmm+efVhw66NCJ0e2LCJAARDvDpxpBJDB4slG3i0zkWCJpZBBJiwvt5 +4JNYkWDEpxtM/msqiXAMenyUhkcQ/HytQj3oCLmsYveIE9HwjVZEzrhTEeU49jn uNqMe2WpH/e7jiwjve1igHrAs+XyUu5fjf/PfVDRcVYoFfG9Ry+Gr5sR6oRV9Y6x Rn+Pdi61anwXOMh3SK8OTffuYJ4JgDSmxsZUHtqD8Iqodt+9KRfDVMYr4WciKECP ztxL3DmdQjmWTIFSk1TD6WSilUZKFMp3vjqjKV18nIBO5C8LAxvknM+bGBVyy+7d cIK5H7tIKmu1ShQ5OHaRM8eLpimk27KJYlklyfo73XhGeploxOPSds4TMmHRsDyQ wCID0Fsn40O7v8I86SUr =+Q/a -----END PGP SIGNATURE----- --5eA6F40ogeEc93tkqu9SDEmtVqBH7mqvI-- --===============1220862589== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== --===============1220862589==--