From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomi Valkeinen Subject: Re: [PATCHv3 29/30] drm/omap: use drm_atomic_helper_shutdown() Date: Wed, 29 Mar 2017 12:22:23 +0300 Message-ID: References: <1490706496-4959-1-git-send-email-tomi.valkeinen@ti.com> <7170884.JmdPD2UAPW@avalon> <22bc7111-ece5-5fb4-5a23-016d6612c61a@ti.com> <2600860.ZYdxJgX7Qa@avalon> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1279293643==" Return-path: Received: from fllnx209.ext.ti.com (fllnx209.ext.ti.com [198.47.19.16]) by gabe.freedesktop.org (Postfix) with ESMTPS id 0767E6E122 for ; Wed, 29 Mar 2017 09:23:04 +0000 (UTC) In-Reply-To: <2600860.ZYdxJgX7Qa@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 --===============1279293643== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="MjXp9goXJ5XivhoTkMUKb0wDIIGgJF4CI" --MjXp9goXJ5XivhoTkMUKb0wDIIGgJF4CI Content-Type: multipart/mixed; boundary="1hrpfflnbFdKv4RIBoIf5wQmaW4uUTto7"; protected-headers="v1" From: Tomi Valkeinen To: Laurent Pinchart Cc: dri-devel@lists.freedesktop.org, Jyri Sarha , Daniel Vetter Message-ID: Subject: Re: [PATCHv3 29/30] drm/omap: use drm_atomic_helper_shutdown() References: <1490706496-4959-1-git-send-email-tomi.valkeinen@ti.com> <7170884.JmdPD2UAPW@avalon> <22bc7111-ece5-5fb4-5a23-016d6612c61a@ti.com> <2600860.ZYdxJgX7Qa@avalon> In-Reply-To: <2600860.ZYdxJgX7Qa@avalon> --1hrpfflnbFdKv4RIBoIf5wQmaW4uUTto7 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 29/03/17 12:11, Laurent Pinchart wrote: >> The hw is disabled before drm_atomic_helper_shutdown() is called only = if >> you have fbdev. And fbdev on all the connectors. If that's not the cas= e, >> we need drm_atomic_helper_shutdown() to disable the hw. >=20 > But if you have no fbdev, as userspace must not hold the device node op= en=20 > here, isn't the device disabled already ? Unfortunately not. See the "drm/atomic: Introduce drm_atomic_helper_shutdown" thread on dri-devel. Tomi --1hrpfflnbFdKv4RIBoIf5wQmaW4uUTto7-- --MjXp9goXJ5XivhoTkMUKb0wDIIGgJF4CI 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 iQIcBAEBCAAGBQJY23zPAAoJEPo9qoy8lh71Ff4P/A/Tbx4zZPrqBPGm0nsgqUnw x/k/0OYzSTXRKrHpABlNDuaXUL5NPTLEEj2NHasARhmTO5mgySs3bYmUb6ACV6au I4GpQneXDPtFfpR+QA5X7Xztv8U7dNmpttQzcQAKaUUQTpPF/Ve+BR/4BWgsqMal 9a4t7cX1K1hklP4M12oYkexVTuJqeskqxpmwSrULn2Zjv4bowZ3uWUjCJIaXg9Gu UH4AYFRq9WmP9YTVvtcQ66LB7ahKB8hdyCfVicmR/HHTL5/Z0eGJW3tQyGHkdIt/ oLx6fdsSRRRYILcnMrxx569fXi66uLzSe+uF7ThRVWbamZIBy01JKFQ1EK3GfNZn oqYF8ypyGCFCCaKijeTOdsTkClNhYqNqiwS/xzsi1t/ttn6AoTyttDD7wsQtYo3I Si2Za5QEcM9n09bN5hDbq3aqWdKTtzVTDWBjWgz2y10MwRd8mhc+2hzym/KKZCYj k/zDpVjMg6iUp2u+xtBjnTcMl6g2snN7upZyIeGZTBeHy7hE0eNf9Fu77GN3drMf gMWDXeDqSPaFZ72rjylxCniB3RPUkT35hHsACxd2yIIkLz1LDiiveyyg9WuY4+mA zZEy90XM/+O7hfymNtdXtFIda4obg2Ti5vESf3iBtUdZmvCq7bB67oaZXDimO6JA GictdezGMZ2jZWBF/zHE =NsVl -----END PGP SIGNATURE----- --MjXp9goXJ5XivhoTkMUKb0wDIIGgJF4CI-- --===============1279293643== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== --===============1279293643==--