On Sat, Aug 20, 2016 at 11:46:19AM +0200, David Weinehall wrote: > On Fri, Aug 19, 2016 at 04:33:49PM -0700, Manasi Navare wrote: > > Get the PLLs for HSW/BDW using the platform specific function > > and add hooks for enabling upfront link training on HSW and BDW. > > > > Signed-off-by: Manasi Navare <[email protected]> > > --- > > drivers/gpu/drm/i915/intel_ddi.c | 2 ++ > > drivers/gpu/drm/i915/intel_dp.c | 4 +++- > > 2 files changed, 5 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/i915/intel_ddi.c > > b/drivers/gpu/drm/i915/intel_ddi.c > > index ef63b4b..1d3ab8a 100644 > > --- a/drivers/gpu/drm/i915/intel_ddi.c > > +++ b/drivers/gpu/drm/i915/intel_ddi.c > > @@ -2410,6 +2410,8 @@ intel_ddi_get_link_dpll(struct intel_dp *intel_dp, > > int clock) > > } > > } else if (IS_SKYLAKE(dev_priv)) { > > pll = skl_find_link_pll(dev_priv, clock); > > + } else if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) { > > + pll = hsw_ddi_dp_get_dpll(encoder, clock); > > } > > return pll; > > } > > diff --git a/drivers/gpu/drm/i915/intel_dp.c > > b/drivers/gpu/drm/i915/intel_dp.c > > index fe156fb..a1cea4d 100644 > > --- a/drivers/gpu/drm/i915/intel_dp.c > > +++ b/drivers/gpu/drm/i915/intel_dp.c > > @@ -5757,8 +5757,10 @@ intel_dp_init_connector(struct intel_digital_port > > *intel_dig_port, > > > > /* Initialize upfront link training vfunc for DP */ > > if (intel_encoder->type != INTEL_OUTPUT_EDP) { > > - if (IS_BROXTON(dev) || IS_SKYLAKE(dev)) > > + if (IS_BROXTON(dev) || IS_SKYLAKE(dev) || > > + IS_BROADWELL(dev) || IS_HASWELL(dev)) > > intel_dp->upfront_link_train = > > intel_ddi_upfront_link_train; > > + > > } > > > > /* eDP only on port B and/or C on vlv/chv */ > > s/dev/dev_priv/ in feature macros. To keep the diff down to a minimum > you can ignore the rest of them, but I certainly wouldn't mind if > you cleanup them all (and change INTEL_INFO()->gen to INTEL_GEN()). > > > Regards: David Weinehall
Thanks for the review David. I will change the dev to dev_priv in the usages in these patches. Cleaning them up all could be a separate patch, not part of this patch series. Regards Manasi > -- > /) David Weinehall <[email protected]> /) Northern lights wander (\ > // Maintainer of the v2.0 kernel // Dance across the winter sky // > \) http://www.acc.umu.se/~tao/ (/ Full colour fire (/ _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
