From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomi Valkeinen Subject: Re: [PATCHv3 04/30] drm/omap: decrease min width & height Date: Wed, 29 Mar 2017 11:26:45 +0300 Message-ID: <03c89160-af6e-d6fd-7cf3-7f4ae4179ee6@ti.com> References: <1490706496-4959-1-git-send-email-tomi.valkeinen@ti.com> <1934740.S4qNL6kWRQ@avalon> <445c5092-0807-0694-7213-584125891c5d@ti.com> <1867154.MpK1o854Pl@avalon> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1366989987==" Return-path: Received: from fllnx209.ext.ti.com (fllnx209.ext.ti.com [198.47.19.16]) by gabe.freedesktop.org (Postfix) with ESMTPS id 4BD686E6E3 for ; Wed, 29 Mar 2017 08:27:15 +0000 (UTC) In-Reply-To: <1867154.MpK1o854Pl@avalon> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Laurent Pinchart Cc: Jyri Sarha , dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============1366989987== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="1ujmdDGLkIgCeHpmKgKcWBiI79aXmCuvG" --1ujmdDGLkIgCeHpmKgKcWBiI79aXmCuvG Content-Type: multipart/mixed; boundary="3PBTLKHX3xhXVNaE4IeQiVmuxG8qIT3fb"; protected-headers="v1" From: Tomi Valkeinen To: Laurent Pinchart Cc: dri-devel@lists.freedesktop.org, Jyri Sarha Message-ID: <03c89160-af6e-d6fd-7cf3-7f4ae4179ee6@ti.com> Subject: Re: [PATCHv3 04/30] drm/omap: decrease min width & height References: <1490706496-4959-1-git-send-email-tomi.valkeinen@ti.com> <1934740.S4qNL6kWRQ@avalon> <445c5092-0807-0694-7213-584125891c5d@ti.com> <1867154.MpK1o854Pl@avalon> In-Reply-To: <1867154.MpK1o854Pl@avalon> --3PBTLKHX3xhXVNaE4IeQiVmuxG8qIT3fb Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 29/03/17 11:24, Laurent Pinchart wrote: > On Wednesday 29 Mar 2017 11:23:00 Tomi Valkeinen wrote: >> On 29/03/17 11:13, Laurent Pinchart wrote: >>> Hi Tomi, >>> >>> Thank you for the patch. >>> >>> On Tuesday 28 Mar 2017 16:07:50 Tomi Valkeinen wrote: >>>> mode_config's min_width and min_height are both set to 32, which is >>>> overly restrictive. >>>> >>>> The real limits depend on whether we're configuring a crtc or a plan= e, >>>> but a limit of 8x2 is safe for both cases. >>> >>> Strictly speaking it's planes in all cases, you could talk about prim= ary >>> or overlay planes instead. Apart from that, >> >> Hmm, well, on the HW they are different. We have CRTC limits, and we >> have plane limits. >> >> In our HW, I think we can have 1x1 planes (at least with RGB). But CRT= C >> minimum depends on the display used. In some cases CRTC minimum is 1x1= , >> but usually it's 8x1 I think. >> >> And talking about primary or overlay planes is so legacy... With atomi= c, >> we just have planes! =3D) >=20 > CRTCs and planes are KMS concepts, at the hardware level we have backgr= ound=20 > and overlay planes, don't we ? No, on the HW level we have... I don't know what's the proper name, but whatever matches the CRTC. The dss driver calls it overlay manager, but that's a silly name too. It's the video timing generator, plus overlay composition. It does have a default background color, though. And then we have overlays, i.e. planes. You can have the CRTC enabled without any planes. Then you'll see the background color. If you have planes, they can be smaller than the CRTC. Tomi --3PBTLKHX3xhXVNaE4IeQiVmuxG8qIT3fb-- --1ujmdDGLkIgCeHpmKgKcWBiI79aXmCuvG 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 iQIcBAEBCAAGBQJY22/FAAoJEPo9qoy8lh71JSUP/1PD5Os7Qn9M1ozqN8tAabpY c/c1G03DO1IYikpsQzBxiZJlVEEz0eIwjpCAG46lB80qAB8cGSU9zMk7E5JtFcOC ym5J598fDZZhXcyCJ9l8lpQWsOXZ4ydSPNEeXwkoc7ABEyxW6y8DAfTteuqTdoeQ d+Wb7LyHVP64BsbZBr+yrSJ/vYPHAmw5I/SfrcpYM0beLJbNGdMXcesKmXaB6qba eQlqGr8LLoxQ1gN3n3+7cuXTbVieO5bs/R9IU2S5prnRvHCwc8uHpy1q92PCImPn A201YCPWxxJOIwm2dIj2YyhT3kejZuJhvQy66mQp1bp1XtGKJz3qDwiKP7Y9K5AN xcqehpEHNRPaXqCsTtWA3Q9i6t/EtQdceH5ZtruQs2bZK1myPGuhp/74VSLSwg4G xaAVMmern0LQB8eX0Fq23Ty20rHDi8KPfS+uXl8N2qtFxZsWPboWO9XnZNvHabki ipJJlyaYNQp4Lnh2Z0t79dZh1kQZKXRyAbSuV58HKmNioRkKxO17M7ZkpUte1wgE 78iSZTAbcc6TRIql345WIwoq0R1ieYkmwqa+ZZNCeoXS2sky4kRcXdfPuqiSCqu5 q/DQbfgVzfAAWkqXtDwgXUInsHfdIlS3oPlLlykTLGpHRI+yj0dt4GzKemLr7YOd oZTvKW0Iu10SSD3N6oVH =Dqae -----END PGP SIGNATURE----- --1ujmdDGLkIgCeHpmKgKcWBiI79aXmCuvG-- --===============1366989987== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== --===============1366989987==--