From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wolfram Sang Date: Wed, 25 Nov 2015 08:27:36 +0000 Subject: Re: [PATCH] drm: adv7511: really enable interrupts for EDID detection Message-Id: <20151125082736.GA1526@katana> MIME-Version: 1 Content-Type: multipart/mixed; boundary="FCuugMFkClbJLl1L" List-Id: References: <1448376627-12255-1-git-send-email-wsa@the-dreams.de> <20151124210501.GA1584@katana> <20151125064851.GA1511@tetsubishi> In-Reply-To: To: Magnus Damm Cc: dri-devel@lists.freedesktop.org, SH-Linux , Simon Horman , Laurent Pinchart , Geert Uytterhoeven , Kuninori Morimoto , Lars-Peter Clausen --FCuugMFkClbJLl1L Content-Type: text/plain; charset=us-ascii Content-Disposition: inline > I guess you mean that the GPIO callbacks include Runtime PM handling > however for irq_chip Runtime PM may not be hooked up so the GPIO block > is in such case is not powered on / get clock enabled? Yes. There is another drawback when GPIOs are not properly requested. It is still possible to request them from userspace although a kernel driver is using them. I am playing with the idea that the GPIO core auto-requests GPIOs which are not already requested but still set up as interrupts. --FCuugMFkClbJLl1L Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJWVXD4AAoJEBQN5MwUoCm2ZdcP/jkLEKHff0T3VKJarcmcTHmR 5VfQNooH++QSOqBLi1zK/kUHGLNggpZswRqmy6phcXusWu76wb8jX1bME3+2bjZa ITmx221hTiV5MmNsdVwthSogFcN3GFkJy9xvyjKlIIlXkjePz2hWiYr9/e+i3C7y PxtrvIGVLkdunopfzaKvuIAA8ngkyrG5HtVqDBDbmIoACWJGhQbC5FoRnsxz87b1 SBFDQao58V4YbWyOFEyZGP+Rqj95//RVhqKIeNXggDOjgZygWH8a+tmWtAIcH/qZ anfWfvT3ryfno8RGGGlTboJb0oA/gMj+rvUK6yEPxwR06vJo/R2w6ZNsaGsvElg2 CSVyDXeWY9kolSwXzJXl/J1n5ghrQEskMUknTRyYadCEKZqVK2zpisV4zoDGwSKC 4TszLcd/uCC9fIrd6Pw8fnb3P2q0mkprv/fZSsn2YVTKSJ9OpLKYW5VI+uiCDAik Ml9+I6SEXoOYkWXd2yY57wZHsvSimXmlOKednic4JtzsQN50v/kOul4qRSbFpkef 19/1PwpqAV2mVFd+HBp3IuUpglPGhCbE0B5xf3efyllhsaPNxBU3mbch5kdFBulK FXwnLOL4b+C71tkHjVNSLeAB5BQiQ8Db0gnqnd0gXlrtMGWaeoYHRVbbR8H6JtL4 sQL+WiLGVfoddr0uN8Ak =wx56 -----END PGP SIGNATURE----- --FCuugMFkClbJLl1L-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wolfram Sang Subject: Re: [PATCH] drm: adv7511: really enable interrupts for EDID detection Date: Wed, 25 Nov 2015 09:27:36 +0100 Message-ID: <20151125082736.GA1526@katana> References: <1448376627-12255-1-git-send-email-wsa@the-dreams.de> <20151124210501.GA1584@katana> <20151125064851.GA1511@tetsubishi> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="FCuugMFkClbJLl1L" Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-sh-owner@vger.kernel.org To: Magnus Damm Cc: dri-devel@lists.freedesktop.org, SH-Linux , Simon Horman , Laurent Pinchart , Geert Uytterhoeven , Kuninori Morimoto , Lars-Peter Clausen List-Id: dri-devel@lists.freedesktop.org --FCuugMFkClbJLl1L Content-Type: text/plain; charset=us-ascii Content-Disposition: inline > I guess you mean that the GPIO callbacks include Runtime PM handling > however for irq_chip Runtime PM may not be hooked up so the GPIO block > is in such case is not powered on / get clock enabled? Yes. There is another drawback when GPIOs are not properly requested. It is still possible to request them from userspace although a kernel driver is using them. I am playing with the idea that the GPIO core auto-requests GPIOs which are not already requested but still set up as interrupts. --FCuugMFkClbJLl1L Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJWVXD4AAoJEBQN5MwUoCm2ZdcP/jkLEKHff0T3VKJarcmcTHmR 5VfQNooH++QSOqBLi1zK/kUHGLNggpZswRqmy6phcXusWu76wb8jX1bME3+2bjZa ITmx221hTiV5MmNsdVwthSogFcN3GFkJy9xvyjKlIIlXkjePz2hWiYr9/e+i3C7y PxtrvIGVLkdunopfzaKvuIAA8ngkyrG5HtVqDBDbmIoACWJGhQbC5FoRnsxz87b1 SBFDQao58V4YbWyOFEyZGP+Rqj95//RVhqKIeNXggDOjgZygWH8a+tmWtAIcH/qZ anfWfvT3ryfno8RGGGlTboJb0oA/gMj+rvUK6yEPxwR06vJo/R2w6ZNsaGsvElg2 CSVyDXeWY9kolSwXzJXl/J1n5ghrQEskMUknTRyYadCEKZqVK2zpisV4zoDGwSKC 4TszLcd/uCC9fIrd6Pw8fnb3P2q0mkprv/fZSsn2YVTKSJ9OpLKYW5VI+uiCDAik Ml9+I6SEXoOYkWXd2yY57wZHsvSimXmlOKednic4JtzsQN50v/kOul4qRSbFpkef 19/1PwpqAV2mVFd+HBp3IuUpglPGhCbE0B5xf3efyllhsaPNxBU3mbch5kdFBulK FXwnLOL4b+C71tkHjVNSLeAB5BQiQ8Db0gnqnd0gXlrtMGWaeoYHRVbbR8H6JtL4 sQL+WiLGVfoddr0uN8Ak =wx56 -----END PGP SIGNATURE----- --FCuugMFkClbJLl1L--