Message ID | 1409830075-11139-66-git-send-email-damien.lespiau@intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
2014-09-04 8:27 GMT-03:00 Damien Lespiau <damien.lespiau@intel.com>: > From: Satheeshakrishna M <satheeshakrishna.m@intel.com> > > DPLL0 is not part of the shared PLL infrastructure. We'll use on for > eDP and rely on what the BIOS does for now. > > Signed-off-by: Satheeshakrishna M <satheeshakrishna.m@intel.com> > Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> > --- > drivers/gpu/drm/i915/intel_dp.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c > index 5755f59..93bd9bf 100644 > --- a/drivers/gpu/drm/i915/intel_dp.c > +++ b/drivers/gpu/drm/i915/intel_dp.c > @@ -1005,7 +1005,10 @@ found: > &pipe_config->dp_m2_n2); > } > > - if (IS_HASWELL(dev) || IS_BROADWELL(dev)) > + if (IS_SKYLAKE(dev)) { > + if (is_edp(intel_dp)) > + pipe_config->ddi_pll_sel = SKL_DPLL0; So we just don't assign anything else for the other ports? Why exactly? > + } else if (IS_HASWELL(dev) || IS_BROADWELL(dev)) > hsw_dp_set_ddi_pll_sel(pipe_config, intel_dp->link_bw); > else > intel_dp_set_clock(encoder, pipe_config, intel_dp->link_bw); > -- > 1.8.3.1 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
On 9/23/2014 8:37 PM, Paulo Zanoni wrote: > 2014-09-04 8:27 GMT-03:00 Damien Lespiau<damien.lespiau@intel.com>: >> From: Satheeshakrishna M<satheeshakrishna.m@intel.com> >> >> DPLL0 is not part of the shared PLL infrastructure. We'll use on for >> eDP and rely on what the BIOS does for now. >> >> Signed-off-by: Satheeshakrishna M<satheeshakrishna.m@intel.com> >> Signed-off-by: Damien Lespiau<damien.lespiau@intel.com> >> --- >> drivers/gpu/drm/i915/intel_dp.c | 5 ++++- >> 1 file changed, 4 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c >> index 5755f59..93bd9bf 100644 >> --- a/drivers/gpu/drm/i915/intel_dp.c >> +++ b/drivers/gpu/drm/i915/intel_dp.c >> @@ -1005,7 +1005,10 @@ found: >> &pipe_config->dp_m2_n2); >> } >> >> - if (IS_HASWELL(dev) || IS_BROADWELL(dev)) >> + if (IS_SKYLAKE(dev)) { >> + if (is_edp(intel_dp)) >> + pipe_config->ddi_pll_sel = SKL_DPLL0; > So we just don't assign anything else for the other ports? Why exactly? Other ports follow the shared dpll logic and assignment happens in intel_ddi > >> + } else if (IS_HASWELL(dev) || IS_BROADWELL(dev)) >> hsw_dp_set_ddi_pll_sel(pipe_config, intel_dp->link_bw); >> else >> intel_dp_set_clock(encoder, pipe_config, intel_dp->link_bw); >> -- >> 1.8.3.1 >> >> _______________________________________________ >> Intel-gfx mailing list >> Intel-gfx@lists.freedesktop.org >> http://lists.freedesktop.org/mailman/listinfo/intel-gfx >
diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index 5755f59..93bd9bf 100644 --- a/drivers/gpu/drm/i915/intel_dp.c +++ b/drivers/gpu/drm/i915/intel_dp.c @@ -1005,7 +1005,10 @@ found: &pipe_config->dp_m2_n2); } - if (IS_HASWELL(dev) || IS_BROADWELL(dev)) + if (IS_SKYLAKE(dev)) { + if (is_edp(intel_dp)) + pipe_config->ddi_pll_sel = SKL_DPLL0; + } else if (IS_HASWELL(dev) || IS_BROADWELL(dev)) hsw_dp_set_ddi_pll_sel(pipe_config, intel_dp->link_bw); else intel_dp_set_clock(encoder, pipe_config, intel_dp->link_bw);