On Sun, Jan 11, 2015 at 4:40 AM, Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> wrote: > On Sat, Jan 10, 2015 at 02:26:04AM +0530, Vandana Kannan wrote: >> Adding a debugfs entry to determine if DRRS is supported or not >> >> Signed-off-by: Vandana Kannan <vandana.kannan@xxxxxxxxx> >> --- >> drivers/gpu/drm/i915/i915_debugfs.c | 18 ++++++++++++++++++ >> 1 file changed, 18 insertions(+) >> >> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c >> index e515aad..544b4c3 100644 >> --- a/drivers/gpu/drm/i915/i915_debugfs.c >> +++ b/drivers/gpu/drm/i915/i915_debugfs.c >> @@ -2825,6 +2825,23 @@ static int i915_ddb_info(struct seq_file *m, void *unused) >> return 0; >> } >> >> +static int i915_drrs_status(struct seq_file *m, void *unused) >> +{ >> + struct drm_info_node *node = m->private; >> + struct drm_device *dev = node->minor->dev; >> + struct intel_crtc *crtc; >> + >> + for_each_intel_crtc(dev, crtc) { >> + if (crtc->active) { > > Don't you want to know which CRTC this is? Would this not be better > extending display_info with the extra CRTC status? > >> + if (crtc->config.has_drrs) Also it just shows if panel supports drrs, not if it is enabled... >> + seq_puts(m, "DRRS enabled"); >> + else >> + seq_puts(m, "DRRS disabled"); >> + } >> + } > -Chris > > -- > Chris Wilson, Intel Open Source Technology Centre > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Rodrigo Vivi Blog: http://blog.vivi.eng.br _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx