Re: [PATCH 2/2] input: tsc2007: Don't require an interrupt to load

2015-03-31 Thread Aaron Sierra
- Original Message - > From: "Dmitry Torokhov" > Sent: Tuesday, March 31, 2015 4:35:18 PM > > Hi Aaron, > > On Tue, Mar 31, 2015 at 10:06:05AM -0500, Aaron Sierra wrote: > > When an IRQ isn't available, poll the device based on poll_period. > > What is the point in using such mode? Why

Re: [PATCH 1/2] input: tsc2007: Convert msecs to jiffies only once

2015-03-31 Thread Dmitry Torokhov
On Tue, Mar 31, 2015 at 10:05:59AM -0500, Aaron Sierra wrote: > Eliminate redundant calculations by performing millisecond to jiffy > calculations once during driver initialization. > > Signed-off-by: Aaron Sierra Applied, thank you. > --- > drivers/input/touchscreen/tsc2007.c | 11 ++-

Re: [PATCH 3/3] tty: remove buf parameter from tty_name()

2015-03-31 Thread Dmitry Torokhov
On Tue, Mar 31, 2015 at 03:55:59PM +0200, Rasmus Villemoes wrote: > tty_name no longer uses the buf parameter, so remove it along with all > the 64 byte stack buffers that used to be passed in. > > Mostly generated by the coccinelle script > > @depends on patch@ > identifier buf; > constant C; >

[PATCH] Input: elan_i2c - enable ELAN0600 acpi panels

2015-03-31 Thread Benjamin Tissoires
ELAN0600 seems to work just fine in mouse emulation mode through i2c-hid, but to have full raw touch support we need to register it in elan_i2c.ko Found on a Lenovo Yoga 3 11". Reported-and-tested-by: Alessio Treglia Signed-off-by: Benjamin Tissoires --- Hi, Alessio reported this touchpad on

Re: [PATCH 2/2] input: tsc2007: Don't require an interrupt to load

2015-03-31 Thread Dmitry Torokhov
Hi Aaron, On Tue, Mar 31, 2015 at 10:06:05AM -0500, Aaron Sierra wrote: > When an IRQ isn't available, poll the device based on poll_period. What is the point in using such mode? Why would not IRQ be available? If this is for device bringup I'd say just carry this locally. Thanks. -- Dmitry --

Re: Using gpio_keys with regmapped gpio?

2015-03-31 Thread Thor Thayer
On 03/31/2015 03:17 PM, Dmitry Torokhov wrote: On Tue, Mar 31, 2015 at 02:57:35PM -0500, Thor Thayer wrote: Hi, I have a SPI gpio expander chip that is using the regmap framework. There are some pushbuttons that I'd like to use the gpio_keys framework for but only the first button press creat

Re: [PATCH 5/6] Input: evdev - Enable runtime PM of the evdev input handler

2015-03-31 Thread Dmitry Torokhov
On Tue, Mar 31, 2015 at 06:14:49PM +0200, Tomeu Vizoso wrote: > So ancestor devices can remain runtime-suspended when the system goes > into a sleep state, they and all of their descendant devices need to > have runtime PM enabled. I am confused. Input devices are not runtime-PM-enabled, so what e

Using gpio_keys with regmapped gpio?

2015-03-31 Thread Thor Thayer
Hi, I have a SPI gpio expander chip that is using the regmap framework. There are some pushbuttons that I'd like to use the gpio_keys framework for but only the first button press creates an input event. It seems like my problem occurs because I'm only getting events on a button press. The

Re: Using gpio_keys with regmapped gpio?

2015-03-31 Thread Dmitry Torokhov
On Tue, Mar 31, 2015 at 02:57:35PM -0500, Thor Thayer wrote: > Hi, > > I have a SPI gpio expander chip that is using the regmap framework. > There are some pushbuttons that I'd like to use the gpio_keys > framework for but only the first button press creates an input > event. > > It seems like my

Re: Using gpio_keys with regmapped gpio?

2015-03-31 Thread Mark Brown
On Tue, Mar 31, 2015 at 02:57:35PM -0500, Thor Thayer wrote: > Which brings up my next question. Can the gpio_keys framework be used with a > regmapped gpio? I haven't been able to find any examples of gpio_keys with > an external gpio expander and maybe this isn't valid usage. Why would there be

Re: [PATCH] hid-ids.h: fix typo

2015-03-31 Thread Oliver Neukum
On Tue, 2015-03-31 at 22:00 +0200, Jiri Slaby wrote: > On 03/31/2015, 02:36 PM, Oliver Neukum wrote: > > There is a typo in a device ID > > Hi, this should have gone to jikos instead ;). It has. Unless I am very mistaken. I made a version against your tree driven by embarasment. Regards

Re: [PATCH] hid-ids.h: fix typo

2015-03-31 Thread Jiri Slaby
On 03/31/2015, 02:36 PM, Oliver Neukum wrote: > There is a typo in a device ID Hi, this should have gone to jikos instead ;). > Signed-off-by: Oliver Neukum > CC: sta...@vger.kernel.org > --- > drivers/hid/hid-ids.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drive

[PATCH 3/6] Input: Implement dev_pm_ops.prepare()

2015-03-31 Thread Tomeu Vizoso
Have it return 1 in both input_dev_type and input_class (for evdev handlers) so that input devices that are runtime-suspended won't be suspended when the system goes to a sleep state. This can make resume times considerably shorter because these devices don't need to be resumed when the system is a

[PATCH 5/6] Input: evdev - Enable runtime PM of the evdev input handler

2015-03-31 Thread Tomeu Vizoso
So ancestor devices can remain runtime-suspended when the system goes into a sleep state, they and all of their descendant devices need to have runtime PM enabled. Signed-off-by: Tomeu Vizoso --- drivers/input/evdev.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/input/evdev.c b

[PATCH 0/6] Allow UVC devices to remain runtime-suspended when sleeping

2015-03-31 Thread Tomeu Vizoso
Hi, this series contain what I needed to do in order to have my USB webcam to not be resumed when the system resumes, reducing considerably the total time that resuming takes. It makes use of the facility that Rafael Wysocki added in aae4518b3 ("PM / sleep: Mechanism to avoid resuming runtime-

[PATCH 1/2] input: tsc2007: Convert msecs to jiffies only once

2015-03-31 Thread Aaron Sierra
Eliminate redundant calculations by performing millisecond to jiffy calculations once during driver initialization. Signed-off-by: Aaron Sierra --- drivers/input/touchscreen/tsc2007.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/input/touchscreen/tsc200

[PATCH 2/2] input: tsc2007: Don't require an interrupt to load

2015-03-31 Thread Aaron Sierra
When an IRQ isn't available, poll the device based on poll_period. Signed-off-by: Aaron Sierra --- drivers/input/touchscreen/tsc2007.c | 43 +++-- 1 file changed, 36 insertions(+), 7 deletions(-) diff --git a/drivers/input/touchscreen/tsc2007.c b/drivers/input/t

Re: [PATCH v2 0/3] Balanced slots, attempt #2

2015-03-31 Thread Henrik Rydberg
On 03/31/2015 05:07 PM, Benjamin Tissoires wrote: > Hi, > > so this is the v2 with Hans' ack and Henrik's respin of the first patch. The whole series looks good, thank you Benjamin. Acked-by: Henrik Rydberg Henrik -- To unsubscribe from this list: send the line "unsubscribe linux-input" in th

[PATCH v2 3/3] Input: synaptics - allocate 3 slots to keep stability in image sensors

2015-03-31 Thread Benjamin Tissoires
When slowly dropping 1, 2 and then 3 fingers on an image sensor touchpad, we can see that the first finger gets reassigned a new slot while it did not move. This is due to the kernel tracking algorithm which can not assign correctly the 3 touches, being out of slots. Declaring that we support 3 sl

[PATCH v2 0/3] Balanced slots, attempt #2

2015-03-31 Thread Benjamin Tissoires
Hi, so this is the v2 with Hans' ack and Henrik's respin of the first patch. Cheers, Benjamin Benjamin Tissoires (3): Input: MT - make slot assignment work for overcovered solutions Revert "Revert "Input: synaptics - use dmax in input_mt_assign_slots"" Input: synaptics - allocate 3 slots

[PATCH v2 1/3] Input: MT - make slot assignment work for overcovered solutions

2015-03-31 Thread Benjamin Tissoires
The recent inclusion of a deassignment cost in the slot assignment algorithm did not properly account for the corner cases where the solutions are overcovered. This patch makes sure the resulting assignment is unique, allocating new slots when necessary. Signed-off-by: Henrik Rydberg Signed-off-b

[PATCH v2 2/3] Revert "Revert "Input: synaptics - use dmax in input_mt_assign_slots""

2015-03-31 Thread Benjamin Tissoires
This reverts commit 09d042a2eb90 ("Revert "Input: synaptics - use dmax in input_mt_assign_slots"") Now that balanced slots assignments seem to be fixed, let's reenable the use in synaptics.c and wait for users to complain if there are still problems. Signed-off-by: Benjamin Tissoires Acked-by:

Re: [PATCH 1/3] Input: mt - prevent balanced slot assignment to assign twice the slot

2015-03-31 Thread Benjamin Tissoires
Hi Henrik, On Mar 31 2015 or thereabouts, Henrik Rydberg wrote: > Hi Benjamin, > > > If two touches are under the dmax distance, it looks like they can now > > be assigned to the same slot. Add a band aid to prevent such situation > > and be able to use the balanced slot assignment. > > Yes, gre

Re: [PATCH 07/12] HID: hid-lg4ff: Protect concurrent access to the output HID report values with a spinlock.

2015-03-31 Thread Jiri Kosina
On Sat, 21 Mar 2015, Michal Malý wrote: > Since all functions that need to send some data to the device they > manage share the same HID report some synchronization is needed to > prevent sending bogus data to the device. This patch is doing much more than just adding a mutual exclusion mechanis

Re: [PATCH 05/12] HID: hid-lg4ff: Update a warning message for a case where device is incorrectly flagged to be handled by hid-lg4ff in hid-lg.

2015-03-31 Thread Jiri Kosina
On Sat, 21 Mar 2015, Michal Malý wrote: > The original warning message was highly misleading. This warning can be > triggered only if a device is flagged to be handled by hid-lg4ff in > hid-lg but hid-lg4ff lacks support for such device. > > Signed-off-by: Michal Malý > --- > drivers/hid/hid-lg

[PATCH 0/3] tty: eliminate 64 byte stack buffers passed to tty_name

2015-03-31 Thread Rasmus Villemoes
All users of tty_name pass the return value directly to a printf function, so we might as well use tty->name directly instead of copying it to a user-supplied (stack) buffer. allmodconfig compile-tested only. Rasmus Villemoes (3): tty: constify return type of tty_name tty: return tty->name di

[PATCH 3/3] tty: remove buf parameter from tty_name()

2015-03-31 Thread Rasmus Villemoes
tty_name no longer uses the buf parameter, so remove it along with all the 64 byte stack buffers that used to be passed in. Mostly generated by the coccinelle script @depends on patch@ identifier buf; constant C; expression tty; @@ - char buf[C]; <+... - tty_name(tty, buf) + tty_name(tty) ...

[PATCH] hid-ids.h: fix typo

2015-03-31 Thread Oliver Neukum
There is a typo in a device ID Signed-off-by: Oliver Neukum CC: sta...@vger.kernel.org --- drivers/hid/hid-ids.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h index 56a4ed6..4ee0fb0 100644 --- a/drivers/hid/hid-ids.h +++ b/drive

[PATCH] sur40: fix occasional hard freeze due to buffer queue underrun

2015-03-31 Thread Florian Echtler
This patch fixes a kernel panic which occurs when buf_list is empty. This can happen occasionally when user space is under heavy load (e.g. due to image processing on the CPU) and new buffers aren't re-queued fast enough. In that case, vb2_start_streaming_called can return true, but when the spin

Re: [PATCH 1/3] Input: mt - prevent balanced slot assignment to assign twice the slot

2015-03-31 Thread Henrik Rydberg
Hi Benjamin, > If two touches are under the dmax distance, it looks like they can now > be assigned to the same slot. Add a band aid to prevent such situation > and be able to use the balanced slot assignment. Yes, great find. You patch is correct, but it is not a band aid, but a result stemming

Re: [PATCH 0/3] Balanced slots, attempt #2

2015-03-31 Thread Hans de Goede
Hi, On 31-03-15 00:09, Benjamin Tissoires wrote: Hi, So, it occurs that the balanced slots assignment introduced in v4.0 had some troubles and the Synaptics part was reverted. I tried to spend some time today to figure out a solution, and the only way I can get something from it is to add a ban