On Mon, 2015-06-15 at 15:15 +0200, Daniel Vetter wrote: > On Sun, Jun 07, 2015 at 06:32:23PM +0530, akash.goel@xxxxxxxxx wrote: > > From: Akash Goel <akash.goel@xxxxxxxxx> > > > > Corrected the platform checks in i915_ring_freq_table debugfs function > > so as to allow the read of ring frequency table for BDW and disallow for VLV > > > > Issue: VIZ-5144 > > Signed-off-by: Akash Goel <akash.goel@xxxxxxxxx> > > --- > > drivers/gpu/drm/i915/i915_debugfs.c | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c > > index 47636f3..1c83596 100644 > > --- a/drivers/gpu/drm/i915/i915_debugfs.c > > +++ b/drivers/gpu/drm/i915/i915_debugfs.c > > @@ -1779,7 +1779,8 @@ static int i915_ring_freq_table(struct seq_file *m, void *unused) > > int ret = 0; > > int gpu_freq, ia_freq; > > > > - if (!(IS_GEN6(dev) || IS_GEN7(dev))) { > > + if (!(IS_GEN6(dev) || (IS_GEN7(dev) && !IS_VALLEYVIEW(dev)) || > > + IS_BROADWELL(dev))) { > > This is really hard to read with the double negation. What about > > if (gen < 6 || IS_VLV(dev)) > /* not supported */ > > instaed? Presuming I decoded this correctly ... Yes this way also it is fine. Will make this change. Accordingly the next patch will also change, like this if (gen < 6 || IS_VLV(dev) || IS_BROXTON(dev)) /* not supported */ Best regards Akash > -Daniel > > seq_puts(m, "unsupported on this chipset\n"); > > return 0; > > } > > -- > > 1.9.2 > > > > _______________________________________________ > > Intel-gfx mailing list > > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > > http://lists.freedesktop.org/mailman/listinfo/intel-gfx > _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx