RE: One USB mouse problem with runtime power management enabled

2013-07-04 Thread Chen Peter-B29397
> > > > Have you opened the mouse by apps (like evtest)?, for USB mouse, > > the usbhid->intf->needs_remote_wakeup is only set at .open/.close > > API. That means if you have not opened mouse, the runtime pm status > > for mouse will changed to RPM_ACTIVE (choose_wakeup -> > pm_runtime_resume) >

[git pull] Input updates for 3.11-rc0

2013-07-04 Thread Dmitry Torokhov
Hi Linus, Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git for-linus or master.kernel.org:/pub/scm/linux/kernel/git/dtor/input.git for-linus to receive first round of updates for the input subsystem. You will get a new touchsreen driver for Cypress

Re: One USB mouse problem with runtime power management enabled

2013-07-04 Thread Alan Stern
On Thu, 4 Jul 2013, Peter Chen wrote: > > I tried doing the same thing on a slightly modified 3.10 kernel. It > > worked okay. But my mouse was attached to a UHCI controller rather > > than EHCI, which may make a difference. > > > > Have you tried running this test with 3.10? > > > > Have y

Re: [PATCH 04/22] input/ti_am33x_tsc: Order of TSC wires, made configurable

2013-07-04 Thread Sekhar Nori
On 7/4/2013 7:20 PM, Sebastian Andrzej Siewior wrote: > On 07/04/2013 03:39 PM, Sekhar Nori wrote: >> Yes, I noticed that after sending the mail. To me it does not make sense >> to make changes to accept something as platform data only to remove >> platform data itself later. > > The patches were

Re: [PATCH 14/22] arm/am33xx: add TSC/ADC mfd device support

2013-07-04 Thread Sebastian Andrzej Siewior
On 07/04/2013 03:49 PM, Sekhar Nori wrote: >> diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi >> index 1460d9b..4ad7797 100644 >> --- a/arch/arm/boot/dts/am33xx.dtsi >> +++ b/arch/arm/boot/dts/am33xx.dtsi >> @@ -404,6 +404,24 @@ >> ti,hwmods = "wkup_m

Re: [PATCH 04/22] input/ti_am33x_tsc: Order of TSC wires, made configurable

2013-07-04 Thread Sebastian Andrzej Siewior
On 07/04/2013 03:39 PM, Sekhar Nori wrote: > Yes, I noticed that after sending the mail. To me it does not make sense > to make changes to accept something as platform data only to remove > platform data itself later. The patches were made earlier and it was easier that way to take everything and

Re: [PATCH 14/22] arm/am33xx: add TSC/ADC mfd device support

2013-07-04 Thread Sekhar Nori
On 6/11/2013 5:01 PM, Sebastian Andrzej Siewior wrote: > From: "Patil, Rachna" > > Add support for core multifunctional device along > with its clients touchscreen and ADC. > > [ pa...@antoniou-consulting.com : make sure status is > set to 'disabled' in dtsi file. ] > > Signed-off-by: Pa

Re: [PATCH 04/22] input/ti_am33x_tsc: Order of TSC wires, made configurable

2013-07-04 Thread Sekhar Nori
On 7/4/2013 5:03 PM, Sebastian Andrzej Siewior wrote: > On 07/04/2013 01:14 PM, Sekhar Nori wrote: >> >> On 6/11/2013 5:00 PM, Sebastian Andrzej Siewior wrote: >>> From: "Patil, Rachna" >>> >>> The current driver expected touchscreen input >>> wires(XP,XN,YP,YN) to be connected in a particular ord

Re: [PATCH] Input: cyttsp4 - use 16bit address for I2C/SPI communication

2013-07-04 Thread Javier Martinez Canillas
On Wed, Jul 3, 2013 at 10:36 PM, Ferruh Yigit wrote: > In TSG4, register map is 512bytes long and to access all of it, > one bit from address byte is used (which bit to use differs for > I2C and SPI); > > Since common code used for TSG3 and TSG4 for I2C, this parameter > wrongly used as u8. TSG3 d

Re: [PATCH 04/22] input/ti_am33x_tsc: Order of TSC wires, made configurable

2013-07-04 Thread Sebastian Andrzej Siewior
On 07/04/2013 01:14 PM, Sekhar Nori wrote: > > On 6/11/2013 5:00 PM, Sebastian Andrzej Siewior wrote: >> From: "Patil, Rachna" >> >> The current driver expected touchscreen input >> wires(XP,XN,YP,YN) to be connected in a particular order. >> Making changes to accept this as platform data > > Th

Re: [PATCH 01/22] mfd/ti_am335x_tscadc: remove regmap

2013-07-04 Thread Sebastian Andrzej Siewior
On 07/04/2013 12:45 PM, Mark Brown wrote: > On Thu, Jul 04, 2013 at 11:02:41AM +0200, Sebastian Andrzej Siewior > wrote: > >> The driver here does not use atomic updates but read followed by >> write so your locking here is futile. So the API/regmap alone >> does not make > > Doesn't that sound l

Re: [PATCH 04/22] input/ti_am33x_tsc: Order of TSC wires, made configurable

2013-07-04 Thread Sekhar Nori
On 6/11/2013 5:00 PM, Sebastian Andrzej Siewior wrote: > From: "Patil, Rachna" > > The current driver expected touchscreen input > wires(XP,XN,YP,YN) to be connected in a particular order. > Making changes to accept this as platform data The platform data part of this driver will never get used

Re: [PATCH 01/22] mfd/ti_am335x_tscadc: remove regmap

2013-07-04 Thread Mark Brown
On Thu, Jul 04, 2013 at 11:02:41AM +0200, Sebastian Andrzej Siewior wrote: > The driver here does not use atomic updates but read followed by write > so your locking here is futile. So the API/regmap alone does not make Doesn't that sound like the driver ought to be using a r/m/w primitive though

Re: One USB mouse problem with runtime power management enabled

2013-07-04 Thread Peter Chen
On Wed, Jul 03, 2013 at 12:29:08PM -0400, Alan Stern wrote: > On Wed, 3 Jul 2013, Peter Chen wrote: > > > Hi Jiri and Alan, > > > > I have run below below at 3.5.7 kernel, but after checking upstream > > kernel, it may also be existed. > > > > 1. evtest /dev/input/event1 & > > 2. Let mouse auto

Re: [PATCH 01/22] mfd/ti_am335x_tscadc: remove regmap

2013-07-04 Thread Sebastian Andrzej Siewior
Sorry for the long pause… On 06/17/2013 06:03 PM, Mark Brown wrote: >> This is a lot of for a simple mmio access. In terms of >> performance: If I add a trace point to my read and write I have >> still less code which is called and it can be disabled. This >> regmap overhead is always there chasin