[Intel-gfx] [PATCH] drm/i915: Adjust BXT HDMI port clock limits

Imre Deak imre.deak at intel.com
Fri Jul 10 06:11:14 PDT 2015


On ke, 2015-07-08 at 15:07 +0530, Kannan, Vandana wrote:
> 
> On 7/6/2015 5:14 PM, ville.syrjala at linux.intel.com wrote:
> > From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> >
> > Since
> >   commit e62925567c7926e78bc8ca976cde5c28ea265a49
> >   Author: Vandana Kannan <vandana.kannan at intel.com>
> >   Date:   Wed Jul 1 17:02:57 2015 +0530
> >
> >      drm/i915/bxt: BUNs related to port PLL
> >
> > BXT DPLL can now generate frequencies in the 216-223 MHz range.
> > Adjust the HDMI port clock checks to account for the reduced range
> > of invalid frequencies.
> >
> > Cc: Vandana Kannan <vandana.kannan at intel.com>
> > Cc: Imre Deak <imre.deak at intel.com>
> > Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> > ---
> >   drivers/gpu/drm/i915/intel_hdmi.c | 9 ++++++---
> >   1 file changed, 6 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/intel_hdmi.c b/drivers/gpu/drm/i915/intel_hdmi.c
> > index c7e912b..70bad5b 100644
> > --- a/drivers/gpu/drm/i915/intel_hdmi.c
> > +++ b/drivers/gpu/drm/i915/intel_hdmi.c
> > @@ -1174,9 +1174,12 @@ hdmi_port_clock_valid(struct intel_hdmi *hdmi,
> >   	if (clock > hdmi_port_clock_limit(hdmi, respect_dvi_limit))
> >   		return MODE_CLOCK_HIGH;
> >
> > -	/* CHV/BXT DPLL can't generate 216-240 MHz */
> > -	if ((IS_CHERRYVIEW(dev) || IS_BROXTON(dev)) &&
> > -	    clock > 216000 && clock < 240000)
> > +	/* BXT DPLL can't generate 223-240 MHz */
> > +	if (IS_BROXTON(dev) && clock > 223333 && clock < 240000)
> > +		return MODE_CLOCK_RANGE;
> > +
> BSpec needs an update based on this ?
> It shows
> "
> HDMI/DVI link rates: 0.2 to 3.0 GHz Non-SSC*
> 
> *2.17 - 2.4 GHz not supported
> "

Yes, this needs updating. Could you file a change request for it?

--Imre

> 
> - Vandana
> > +	/* CHV DPLL can't generate 216-240 MHz */
> > +	if (IS_CHERRYVIEW(dev) && clock > 216000 && clock < 240000)
> >   		return MODE_CLOCK_RANGE;
> >
> >   	return MODE_OK;
> >




More information about the Intel-gfx mailing list