On Tue, Feb 17, 2015 at 02:44:56PM +0000, Damien Lespiau wrote:
> On Fri, Feb 06, 2015 at 08:26:35PM +0530, akash.goel@xxxxxxxxx wrote:
> > + if (IS_GEN9(dev_priv->dev)) {
> > + limits = (dev_priv->rps.max_freq_softlimit * GEN9_FREQ_SCALER) << 23;
> > + if (val <= dev_priv->rps.min_freq_softlimit)
> > + limits |= (dev_priv->rps.min_freq_softlimit * GEN9_FREQ_SCALER) << 14;
>
> I believe the values here are in 16.666 Mhz, the power spec I have gives
> examples:
> [31:23]=54d: No interrupt if already >=900MHz
> [22:14]=18d: No interrupt if already <= 300MHz
>
> and 54 * 16.66.. = 900.
Right, so RP_STATE_CAP is documented using 50Mhz units, so the above is
correct. How about unifying all values for SKL to the same units (always
16.66 Mhz units)? that would remove some of the confusion a bit.
--
Damien
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
