Re: [PATCH] drm/i915: add psr toggle to debugfs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Apr 20, 2015 at 8:41 AM, Daniel Vetter <daniel@xxxxxxxx> wrote:
> On Thu, Apr 16, 2015 at 09:10:11AM -0700, Eric Caruso wrote:
>> On Thu, Apr 16, 2015 at 8:58 AM, Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> wrote:
>> > I'm not sure that that property will be accepted...
>> > if so it can be changed to toggle, but for now it is informative only
>> > (DRM_MODE_PROP_IMMUTABLE).
>> >
>>
>> Okay, that changes things somewhat. There'll have to be some discussion over
>> what the sysfs interface would be, then. It's supposed to be stable, if I
>> understand correctly.
>
> Imo if it's just for debugging, then debugfs is the correct place. Sysfs
> means ABI with all the requirements (igt tests, open-source userspace,
> ...) that entails.

Good points, so:
Acked-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>

Although "i915.enable_psr = enable;" is useless and change the meaning
of this parameter.
I'd remove this...

And probably check for  pipe_config psr ready if/when merged...

> -Daniel
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch



-- 
Rodrigo Vivi
Blog: http://blog.vivi.eng.br
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux