Re: [PATCH v4 2/2] cap11xx: add LED support

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

 




On Wed, Jun 17, 2015 at 3:38 PM, Dmitry Torokhov
<dmitry.torokhov@xxxxxxxxx> wrote:
> On Wed, Jun 17, 2015 at 02:53:26PM -0700, Matt Ranostay wrote:
>> On Wed, Jun 17, 2015 at 1:59 AM, Daniel Mack <daniel@xxxxxxxxxx> wrote:
>> > On 06/17/2015 08:49 AM, Matt Ranostay wrote:
>> >> Actually after further reading of the datasheet and testing with
>> >> i2ctools it seems cap11xx_input_close() isn't putting the device in
>> >> DEEP SLEEP mode.
>> >>
>> >> Any thoughts why?
>> >
>> > No, sorry. IIRC, the board I wrote the driver for didn't give me the
>> > ability to measure power consumption.
>> >
>>
>> I can confirm it never goes to deep sleep or cap11xx_input_close()
>> gets called. Does evdev kernel module increment/input_open_device even
>> if nothing in userspace is accessing the /dev/event/inputX nodes?
>
> No it does not. On the other hand there is VT that does keep all
> keyboard-like devices open.

Ah yes is that is likely the issue. I'll disable CONFIG_VT, and update
in v5 to handle to keep the device from going to deep sleep when one
or more LEDS is on.

- Matt

>
> Thanks.
>
> --
> Dmitry
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux