On Thu, Feb 26, 2015 at 01:29:09PM +0530, akash.goel@xxxxxxxxx wrote: > From: Akash Goel <akash.goel@xxxxxxxxx> > > The frequency values(Rp0, Rp1, Rpn) reported by RP_STATE_CAP register > are stored, initially by the Driver, inside the dev_priv->rps structure. > Since these values are expected to remain same throughout, there is no real > need to read this register, on dynamic basis, from debugfs/sysfs functions > and the values can be instead retrieved from the dev_priv->rps structure > when needed. Inside debugfs, you want both as we want to compare what the hardware thinks and what we believe it should be. Everywhere else we should indeed be trusting our bookkeeping. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
