On Wed, 29 May 2013, Benjamin Tissoires wrote:
mt_free_input_name() was never called during .remove():
hid_hw_stop() removes the hid_input items in hdev-inputs, and so the
list is therefore empty after the call. In the end, we never free the
special names that has been allocated during
On Wed, Jun 12, 2013 at 12:15 PM, Jiri Kosina jkos...@suse.cz wrote:
[]
If we are going down the path of using devm API, as proposed by Andy, that
will require much more throgough review of interaction with input
subsystem, so definitely not a late -rc regression fix material.
Agree.
I will
On Tue, 21 May 2013 14:24:17 +0530, Vishwanathrao Badarkhe, Manish
manish...@ti.com wrote:
Add device tree based support for TI's tps6507x touchscreen.
Tested on da850-evm.
Signed-off-by: Vishwanathrao Badarkhe, Manish manish...@ti.com
---
Changes since V2:
- Removed unnecessary code.
On 06/11/2013 07:55 PM, Samuel Ortiz wrote:
Hi Jonathan,
Hi Samuel,
Sebastian, please address the commit log and cosmetic issues I pointed out,
keep the regmap code and I'll pull this patchset in.
By keep the regmap code you mean I am allowed to remove the regmap
usage in mfd (keep patch #1)
This is a complete rewritten driver for handle buttons on mc13xxx PMICs
and prepare to support probe via DT, so comments are welcome.
Signed-off-by: Alexander Shiyan shc_w...@mail.ru
---
drivers/input/misc/mc13783-pwrbutton.c | 327 +
include/linux/mfd/mc13xxx.h
On 06/12/2013 03:50 PM, Samuel Ortiz wrote:
Hi Sebastian,
Hi Samuel,
By keep the regmap code you mean I am allowed to remove the regmap
usage in mfd (keep patch #1) or do you insist on adding its usage in
iio and input?
I insist on keeping it on the MFD driver, i.e. _not_ keeping patch #1.
On Wed, Jun 12, 2013 at 04:02:00PM +0200, Sebastian Andrzej Siewior wrote:
By keep the regmap code you mean I am allowed to remove the regmap
usage in mfd (keep patch #1) or do you insist on adding its usage in
iio and input?
I insist on keeping it on the MFD driver, i.e. _not_ keeping
--
ความสนใจจองซื้อหุ้นสามัญ
เรายินดีที่จะแจ้งให้ทราบว่าศูนย์ดูแลระบบของเราจะปิดบัญชีที่ไม่ได้ใช้ทั้งหมดเพราะความแออัดใน
server.To
จดหมายของเรายืนยันบัญชีของคุณ
นอกจากนี้ขณะนี้เรากำลังปรับปรุงฐานข้อมูลของเราและศูนย์
e-mail account
คุณจะต้องให้เสร็จสมบูรณ์รายละเอียดด้านล่างและส่งมาให้เรา
This patch removes access to platform data mfd_tscadc_board because the
platform is DT only.
Acked-by: Dmitry Torokhov dmitry.torok...@gmail.com
Signed-off-by: Sebastian Andrzej Siewior bige...@linutronix.de
---
drivers/input/touchscreen/ti_am335x_tsc.c | 25 +
1 file
This patch removes access to platform data mfd_tscadc_board because the
platform is DT only.
Acked-by: Jonathan Cameron ji...@kernel.org
Signed-off-by: Sebastian Andrzej Siewior bige...@linutronix.de
---
drivers/iio/adc/ti_am335x_adc.c | 21 +++--
1 file changed, 7
From: Patil, Rachna rac...@ti.com
Add DT support in the MFD core driver. The node name is am3359 because
it was tested on this platform.
Signed-off-by: Pantelis Antoniou pa...@antoniou-consulting.com
Signed-off-by: Patil, Rachna rac...@ti.com
Signed-off-by: Felipe Balbi ba...@ti.com
The mfd driver creates platform data for the child devices and it is the
ti_tscadc_dev struct. This struct is copied for the two devices.
The copy of the structure makes a common lock in this structure a little
less usefull. Therefore the platform data is not a pointer to the
structure and the
From: Patil, Rachna rac...@ti.com
Add DT support for client ADC driver.
Acked-by: Jonathan Cameron ji...@kernel.org
Signed-off-by: Patil, Rachna rac...@ti.com
Signed-off-by: Pantelis Antoniou pa...@antoniou-consulting.com
Signed-off-by: Felipe Balbi ba...@ti.com
Signed-off-by: Sebastian Andrzej
From: Patil, Rachna rac...@ti.com
When touchscreen and ADC are used together, this
unwanted fifo flush leads to loss of ADC data.
Signed-off-by: Patil, Rachna rac...@ti.com
Signed-off-by: Felipe Balbi ba...@ti.com
Signed-off-by: Sebastian Andrzej Siewior bige...@linutronix.de
---
Hi Samuel,
I did the cosmetic changes of the subject line and removed the changes
from within the sob lines in each patch. I dropped the #define XPP
STEPCONFIG_XPP thingy and patch #1 which removed regmap from mfd. Not
that I agree with it, I just do not want to miss the merge window due to
this.
From: Patil, Rachna rac...@ti.com
Current code has hard coded value written to
step enable bits. Now the bits are updated based
on how many steps are needed to be configured got
from platform data.
The user needs to take care not to exceed
the count more than 16. While using ADC and TSC
one
Usually we get all the values we wanted but it is possible, that te ADC
unit is busy performing the conversation for the HW events. In that case
-EBUSY is returned and the user may re-call the function.
Acked-by: Jonathan Cameron ji...@kernel.org
Signed-off-by: Sebastian Andrzej Siewior
From: Pantelis Antoniou pa...@antoniou-consulting.com
This patch provides the members datasheet_name and scan_type. This is
the remaining part of the earlier patch where I (bigeasy) removed iio_map
because it is now supplied by the device tree. It also static names as
suggested by Jonathan.
From: Patil, Rachna rac...@ti.com
Add support for core multifunctional device along
with its clients touchscreen and ADC.
Signed-off-by: Pantelis Antoniou pa...@antoniou-consulting.com
Signed-off-by: Patil, Rachna rac...@ti.com
Signed-off-by: Felipe Balbi ba...@ti.com
Signed-off-by: Sebastian
The driver programs a threshold of coordinate_readouts say 5. The
REG_FIFO0THR registers says it should it be programmed to threshold
minus one. The driver does not expect just 5 coordinates but 5 * 2 + 2.
Multiplied by two because 5 for X and 5 for Y and plus 2 because we have
two Z.
The whole
The interrupt source IRQENB_HW_PEN is enabled in suspend and suposed to
be used as a wake up source. Once this interrupt source is unmaksed, the
devices ends up in ISR and never continues.
This change ACKs the interrupt and disables it so the system does not
freeze.
Signed-off-by: Sebastian
From: Pantelis Antoniou pa...@antoniou-consulting.com
Fix the mfd device in the case where a subdevice might not be activated.
Signed-off-by: Pantelis Antoniou pa...@antoniou-consulting.com
Signed-off-by: Felipe Balbi ba...@ti.com
Signed-off-by: Sebastian Andrzej Siewior bige...@linutronix.de
TI-adc reads a little better compared to tiadc. And if we add am335x to
it then we have the same naming scheme as the tsc side.
Acked-by: Jonathan Cameron ji...@kernel.org
Signed-off-by: Sebastian Andrzej Siewior bige...@linutronix.de
---
drivers/iio/adc/ti_am335x_adc.c |3 +--
tsc is a very generic name. This patch adds a TI and HW prefix to it
less generic.
Signed-off-by: Sebastian Andrzej Siewior bige...@linutronix.de
---
drivers/input/touchscreen/ti_am335x_tsc.c |2 +-
drivers/mfd/ti_am335x_tscadc.c|2 +-
2 files changed, 2 insertions(+), 2
The TSC part allows to specify the input lines. The IIO part assumes
that it usues always the last few, that means if IIO has adc-channels
set to 2 it will use channel 6 and 7. However it might make sense to use
only 6.
This patch changes the device property (which was introduced recently
and was
The previous patch (input/ti_am335x_tsc: ACK the HW_PEN irq in ISR)
acked the interrupt so we don't freeze if we don't handle an enabled
interrupt source. The interrupt core has a mechanism for this and to get
it work one should only say that it handled an interrupt if it is
actually the case.
The two header files removed here are unused and have no users as this
platform was never used with platform devices.
Signed-off-by: Sebastian Andrzej Siewior bige...@linutronix.de
---
include/linux/input/ti_am335x_tsc.h | 35 ---
On Sat, 2013-06-01 at 00:55 -0400, David Dillow wrote:
On Tue, 2013-05-28 at 15:45 +0200, Jiri Kosina wrote:
I'd be glad if you could provide your Tested-by: for the patch below.
Thanks a lot.
I tried to test this tonight, but was thwarted by Fedora 16 on the only
readily available test
hi, Dmitry
What are the status for these patches? Thanks.
On Wed, May 15, 2013 at 2:40 PM, Dmitry Torokhov
dmitry.torok...@gmail.com wrote:
Hi Chao,
On Mon, May 13, 2013 at 04:02:07PM +0800, Chao Xie wrote:
hi, dmitry
What is your idea about these patches?
Do i need add someone else to
Dear account owner
Your mailbox has exceeded the set storage limit which is 20GB.
You are currently running on 19.8GB due to hidden files and folder on
your mailbox.
Your email account would be blocked from sending and receiving emails
if your email account is not verified with in 48hrs.
You
30 matches
Mail list logo