[Intel-gfx] [PATCH] drm/i915: Pick the first unused PLL once again
Lucas De Marchi
lucas.demarchi at intel.com
Thu Jan 31 01:24:07 UTC 2019
On Wed, Jan 30, 2019 at 08:13:59PM +0200, Ville Syrjälä wrote:
>From: Ville Syrjälä <ville.syrjala at linux.intel.com>
>
>commit 5b0bd14dcc6b ("drm/i915/icl: keep track of unused pll while
>looping") inadvertently (I presume) changed the code to pick the
>last unused dpll rather than the first unused one like we did before.
>
>While there should most likely be no harm in changing the order
>let's change back just to avoid a change in the behaviour. At
>least it might reduce the confusion when staring at logs (took
>me a while to figure out why DPLL1 being picked over DPLL0
>when the latter was most definitely available).
I assumed the no harm and simplest approach so I didn't bother
checking if the it was already set. So I'd not say it was
"inadvertently", it was on purpose.
But if it makes life easier to read the logs:
Acked-by: Lucas De Marchi <lucas.demarchi at intel.com>
Lucas De Marchi
>
>Cc: Lucas De Marchi <lucas.demarchi at intel.com>
>Cc: Paulo Zanoni <paulo.r.zanoni at intel.com>
>Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
>---
> drivers/gpu/drm/i915/intel_dpll_mgr.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
>diff --git a/drivers/gpu/drm/i915/intel_dpll_mgr.c b/drivers/gpu/drm/i915/intel_dpll_mgr.c
>index 8f70838ac7d8..0a42d11c4c33 100644
>--- a/drivers/gpu/drm/i915/intel_dpll_mgr.c
>+++ b/drivers/gpu/drm/i915/intel_dpll_mgr.c
>@@ -258,7 +258,8 @@ intel_find_shared_dpll(struct intel_crtc *crtc,
>
> /* Only want to check enabled timings first */
> if (shared_dpll[i].crtc_mask == 0) {
>- unused_pll = pll;
>+ if (!unused_pll)
>+ unused_pll = pll;
> continue;
> }
>
>--
>2.19.2
>
More information about the Intel-gfx
mailing list