From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Anholt Subject: Re: [PATCH] drm/vc4: Fix with pm_runtime synchronization on DSI Date: Tue, 09 Apr 2019 09:24:56 -0700 Message-ID: <87ftqrno2v.fsf@anholt.net> References: <20190329012913.29538-1-hoegeun.kwon@samsung.com> <87bm1p1usk.fsf@anholt.net> <9adb599d-3be3-2103-0169-dcd9a497b78a@samsung.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0017885956==" Return-path: Received: from anholt.net (anholt.net [50.246.234.109]) by gabe.freedesktop.org (Postfix) with ESMTP id 7533C89229 for ; Tue, 9 Apr 2019 16:24:58 +0000 (UTC) In-Reply-To: <9adb599d-3be3-2103-0169-dcd9a497b78a@samsung.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: airlied@linux.ie, daniel@ffwll.ch, dri-devel@lists.freedesktop.org Cc: Hoegeun Kwon List-Id: dri-devel@lists.freedesktop.org --===============0017885956== Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" --=-=-= Content-Type: text/plain Hoegeun Kwon writes: > On 4/2/19 2:48 AM, Eric Anholt wrote: >> Hoegeun Kwon writes: >> >>> There is a problem when often dpms goes from off to on. pm idle is not >>> in sync and the problem occurs. Modify pm_runtime_put from >>> asynchronous to synchronous. >> Why would we need the power domain to go to off before we try to come >> back? Any idea? Also, please specify what "the problem" is. > > Hi Eric, > > > First thank you for your review. > > There is a problem failed to runtime PM enable on DSI when often dpms What do you mean by "failed to runtime PM enable"? The pm_runtime_enable() returned an error? Have you investigated the source that error, if so? --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEE/JuuFDWp9/ZkuCBXtdYpNtH8nugFAlysx1gACgkQtdYpNtH8 nujJAA/+LhrJ8BmIwMzcgIHPnEd6upZMinswslAIR7NL1+7CcDqlpIAIt/2ajJ6C iO6WR4+ThsuOz7NywS+AWXUxRvbM4JUvSZTu2uvOH1141/z6EHyLlLhwSKJAxAvP mFOX5RtLo/jot1WRh569rYcI5MnBhOcJhhhhK6lI8n2gC+tu8fVfX7Q3MPcsr0Ef 8/XqMHmRX9kB7Wr20pTfQDJgNaIfbApLSt9Vy1Na3Idu4nX8EpggC+6sJYgz4JVM MY0rAcU2iCHBJQkq04PVSXiZaDvSJ9U5gDedbK+6veV1HQUrxaiegMj21PdBdcEX EkxRkbNOAlETckPdNRLqNGhGZ0luo8hoMlMkr+9GSbnWajbQYw7zKPcJQ1D33Eey MFpvgTChjvPuvcJNYDO08/L19p/EmOnxdWuDmt7GbTPNFp6kyDMvSnpl1It1WLJ7 8ux+wZFVL9oIN2XN9mMGc89QygiR2ZPcA+HMCTq4GX6kSpqLuXUJoVyut+wl+0AM G+yKJU9MB1orlFvRkbY1nP/uWv+kS0RucdiSYSRhisI+Uo3yaL2aABr9D4QB1qAO kBAMLgI8iIyUGq5ksEfrxiKVLdKzE2Cl18RxwzDffaqgqMNZPGD1CK/ZjumvRXbH d6RsDNgLFXwOatlmV9QTLlIlZocJye6nWJB4ElnN7Rn2Xj4o+Jw= =Ucmo -----END PGP SIGNATURE----- --=-=-=-- --===============0017885956== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVs --===============0017885956==--