[Intel-gfx] [PATCH 9/10] drm/i915: Add debugfs entry for DRRS

[Rodrigo Vivi]

On Sun, Jan 11, 2015 at 4:40 AM, Chris Wilson <chris at chris-wilson.co.uk> 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 at intel.com>
>> ---
>>  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 at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Rodrigo Vivi
Blog: http://blog.vivi.eng.br