From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pekka Paalanen Subject: Re: [RFC 0/7] drm/omap: Module parameter for display order configuration Date: Thu, 5 Oct 2017 12:56:27 +0300 Message-ID: <20171005125627.08fa3890@eldfell> References: <20170829073218.11097-1-peter.ujfalusi@ti.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1206356378==" Return-path: Received: from mail-lf0-x244.google.com (mail-lf0-x244.google.com [IPv6:2a00:1450:4010:c07::244]) by gabe.freedesktop.org (Postfix) with ESMTPS id 232236E1AE for ; Thu, 5 Oct 2017 09:56:37 +0000 (UTC) Received: by mail-lf0-x244.google.com with SMTP id j73so5480675lfg.0 for ; Thu, 05 Oct 2017 02:56:37 -0700 (PDT) In-Reply-To: <20170829073218.11097-1-peter.ujfalusi@ti.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Peter Ujfalusi Cc: tomi.valkeinen@ti.com, laurent.pinchart@ideasonboard.com, dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============1206356378== Content-Type: multipart/signed; micalg=pgp-sha256; boundary="Sig_/mX/EOhSH8FMIVgUTIJpwvwE"; protocol="application/pgp-signature" --Sig_/mX/EOhSH8FMIVgUTIJpwvwE Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Tue, 29 Aug 2017 10:32:11 +0300 Peter Ujfalusi wrote: > =EF=BB=BFHi >=20 > The series adds support for changing the order of the displays defined by= DT > display aliases. >=20 > The motivation to do such a thing is that for example the fb emulation is > treating the first display/crtc as the 'main' display and will create the > fb emulation based on the first display's properties. > There are many custom applications using DRM directly and they assume tha= t the > first connector is the 'main' display. > Afaik weston provides no means either to change the 'main/preferred' disp= lay. Hi, that's because Weston does not have a concept of main or preferred display to begin with. If what you refer to involves running Weston with the fbdev-backend, then Weston has nothing to do with the issue. Weston only uses /dev/fb0, whatever that might be and however that might work, as set up by the kernel. Thanks, pq >=20 > It should be the work of user space application (except the fb emulation)= to > somehow deal with the 'main' display selection for their needs, but > unfortunately they are not capable of diong so for some reason. >=20 > We have boards with LCD panel and HDMI for example and in DT the LCD is s= et as > display0, but in certain useage scenarios it is desired to have the HDMI = as the > 'main' display instead of the LCD. >=20 > With the kernel cmd line parameter it is possible to change the pre defin= ed > order without recompiling the kernel/DT. >=20 > If the board have two active displays: > 0 - LCD > 1 - HDMI > then: > omapdrm.displays=3D0,1 - represents the original order (LCD, HDMI) > omapdrm.displays=3D1,0 - represents reverse order (HDMI, LCD) > omapdrm.displays=3D0 - only the LCD is enabled > omapdrm.displays=3D1 - only the HDMI is enabled > omapdrm.displays=3D-1 - disable all displays >=20 > The first 6 patch of the series is doing some generic clean up and prepar= es the > code so the display ordering is going to be easy to add. >=20 > Regards, > Peter > --- > Peter Ujfalusi (7): > drm/omap: Use devm_kzalloc() to allocate omap_drm_private > drm/omap: Allocate drm_device earlier and unref it as last step > drm/omap: Manage the usable omap_dss_device list within > omap_drm_private > drm/omap: Separate the dssdevs array setup from the connect function > drm/omap: Do dss_device (display) ordering in omap_drv.c > drm/omap: dss: Remove display ordering from dss/display.c > drm/omap: Add kernel parameter to specify the desired display order >=20 > drivers/gpu/drm/omapdrm/dss/display.c | 15 +-- > drivers/gpu/drm/omapdrm/dss/omapdss.h | 3 - > drivers/gpu/drm/omapdrm/omap_drv.c | 244 ++++++++++++++++++++++++----= ------ > drivers/gpu/drm/omapdrm/omap_drv.h | 3 + > 4 files changed, 183 insertions(+), 82 deletions(-) >=20 --Sig_/mX/EOhSH8FMIVgUTIJpwvwE Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEJQjwWQChkWOYOIONI1/ltBGqqqcFAlnWAcsACgkQI1/ltBGq qqfMBhAAmmJ7/QhlqsVW8zn32WGoQWTitIzi1G2fV6M+2XCErdKqWCJPugQAZ8H6 gpAsapO6UyXkKP5ZywEEMk54U0bpjznhp+baAEZkgjvh0KtxrK+JkSyJjpS8/Fpq 43jPR8FJaJlf/lJ2v/2YcF1ZLr8UVjZOghFX7heZiNrcl3Q3fx0e6YU/2l6NCp50 /TsF1+mJhMrti14TAu3I+X+XaYDFPKMVSEFHoJiRNbNb5j8BrMKXtF0gnWzea6Z7 5m+B+ObdpuTz7Se1zduWG+SUDnHQr4EtKE22x9V7EjGgRuYJRc1q2v1tNMPBJWbz r9w6QhDMD11uhYZ7Am/Ypph2BDfC7Nf8wX5Xew8011xH+JNbin5loB3vR70sUpjf TpOFLXPkkQqkNp1umimcKE1X4jCCPVE3nb97kCABnI+soxXHb/qj+JLqMtae1arP eyPVCkF9d4W7kW0QI2khKNkdh7tOetHWB93wIWExVmj/0vVbiFZSSpMlux0bsbPX ZRdATvetp4wNzx8RfiGjplrYssfUJ3xLTJn1HCNxum2+zE1GgJlLUE2PU6+M7HGF PcQc1wITzgTqqGm2PJz8O5B8F0qEhfq9UyUIzWCVBgXNIdmGIQKa9yoyNGcjUI7E qeROtzqP3IlYsY/IkijoJnHvHzlH2lccDgJMPBulDhenZ9vQLE4= =/1YP -----END PGP SIGNATURE----- --Sig_/mX/EOhSH8FMIVgUTIJpwvwE-- --===============1206356378== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== --===============1206356378==--