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

On Wed, Jun 17, 2015 at 02:53:26PM -0700, Matt Ranostay wrote:

On Wed, Jun 17, 2015 at 1:59 AM, Daniel Mack [off-list ref] wrote:

quoted

On 06/17/2015 08:49 AM, Matt Ranostay wrote:

quoted

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.

Thanks.

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

On Wed, Jun 17, 2015 at 3:38 PM, Dmitry Torokhov
[off-list ref] wrote:

On Wed, Jun 17, 2015 at 02:53:26PM -0700, Matt Ranostay wrote:

quoted

On Wed, Jun 17, 2015 at 1:59 AM, Daniel Mack [off-list ref] wrote:

quoted

On 06/17/2015 08:49 AM, Matt Ranostay wrote:

quoted

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