From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Anholt Subject: Re: [PATCH] drm/msm/dsi: workaround for display enabled by bootloader Date: Tue, 17 Oct 2017 13:16:14 -0700 Message-ID: <874lqxttrl.fsf@anholt.net> References: <20171017143918.12518-1-robdclark@gmail.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1194647165==" Return-path: In-Reply-To: <20171017143918.12518-1-robdclark-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: freedreno-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Sender: "Freedreno" To: Rob Clark , dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Cc: linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org List-Id: linux-arm-msm@vger.kernel.org --===============1194647165== Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" --=-=-= Content-Type: text/plain Rob Clark writes: > Bootloader enabled display, when the driver is built-in (rather than a > module loaded after CCF/genpd disable "unused" clocks/powerdomains) > causes problems since the driver thinks the clocks are off, but in fact > they are on. This causes (for example) clk_set_rate() to fail. > > A better solution would be to support display handover from bootloader, > but that will require some CCF+genpd changes before that is possible. > So until then, we need this workaround. Hmm. Couldn't the clk_set_rate() just turn off the clock and then make the rate change when there are no refs on the clk currently? --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEE/JuuFDWp9/ZkuCBXtdYpNtH8nugFAlnmZQ4ACgkQtdYpNtH8 nujSBg//WBy+StX2mbAjepWeQQkLxuO0eIvapkz4R6OvckINvQFy1tldTWz3X1OD haOSt6GsKt87/OlKSesS/Nq7cem4HPfvbO/Tbd+5Iy8m2rvn3pIfgBa82jMXYGhh 0Vx79MJn7cFDzD1wojB5uu5P5Bg4nvlh9P8t24x6E5ugs6+Fr6RyWNj63lxI8aWR oaS5Ch9pgxzRB6wTpqB/hzvmE0/BcKhgjUJRfGgv3VlyXSL7N0Qry7+HNBG7qco3 H00CyCfEJQnYA+lXOB/cxHu/YHPLOlOXyKQfxE2tbPeSlc/Pu48OqN6ZTEZbicAz P0asBUxOOR6Q6MfF6t8GX38W8/6HpMoxtQ9n3tHbNu0W9nQmI+aMRW8z3mp20pZW 44E6rnOExyyMSBt8zdckAasbNq/j0WL83l5FCfQnb0gNWV8M20AvQ11uXu5qnX5M w2xhR8TNA5XjS3RLK9+NhHGE0Tg9KqJKMOSs4sOEavit2VhAO01NbKzjW944AVjO Qrsvg5bfN4P2uNdEESRhmIJkXkEZBu/GtMycYEyISXdN48XM1E1XPqyg/GRKfiwl crPbSxYP4DBoQgRrxBqO7Y57zhGmDErTkfNl3Ro5n1XE5oB5b4nXM1WJWSieEt3j Xb7uZroa7GEQn639lx3vLARuQaTy/VhKbGcOnr412Xjr2bwjD04= =3eoQ -----END PGP SIGNATURE----- --=-=-=-- --===============1194647165== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KRnJlZWRyZW5v IG1haWxpbmcgbGlzdApGcmVlZHJlbm9AbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZnJlZWRyZW5vCg== --===============1194647165==--