Re: [PATCH v3] Input: add touchscreen driver for MELFAS MCS-5000 controller

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

 



On 7/14/2009 3:04 PM, Dmitry Torokhov wrote:
> Hi Joonyoung,
> 
> On Fri, Jul 10, 2009 at 11:10:12AM +0900, Joonyoung Shim wrote:
>> The MELPAS MCS-5000 is the touchscreen controller. The overview of this
>> controller can see at the following website:
>>
>> http://www.melfas.com/product/product01.asp?k_r=eng_
>>
>> This driver is tested on s3c6410 NCP board and supports only the i2c
>> interface.
>>
>> Signed-off-by: Joonyoung Shim <jy0922.shim@xxxxxxxxxxx>
>> ---
>> Dear Dmitry,
>>
>> I sent the patch v2, but couldn't get any response.
>> This patch v3 is updated a little bit from v2.
>>
> 
> While I was sittign on the patch mainline acquired threaded IRQ support
> which fits the bill here and simplifies logic quite a bit, could you
> please try the patch below and tell me if the touchscreen still works?
> 

I've tested your patch, but it does not work. If i touch the pannel,
mcs5000_ts_hardirq is called once, but mcs5000_ts_interrupt is not
called, so the interrupt is not occurred any longer.

>> I have one question. The MCS-5000 controller don't support PRESSURE, so
>> i removed a fake PRESSURE report on driver, but tslib v1.0
>> (http://tslib.berlios.de) require the PRESSURE report from driver. I
>> wonder whether it is make sense to report a fake PRESSURE for tslib.
>>
> 
> I believe the adjustments to use BTN_TOUCH in absebse of ABS_PRESSURE
> has been applied to tslib, you need to talk to the maintainers about
> releasing updated version instead of adding fake events to the kernel.
> 

I see, thank you.

> Thanks.
> 

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

[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux