Linux Input/HID development
 help / color / mirror / Atom feed
From: Joonyoung Shim <jy0922.shim@samsung.com>
To: Marek Szyprowski <m.szyprowski@samsung.com>
Cc: 'Dmitry Torokhov' <dmitry.torokhov@gmail.com>,
	linux-input@vger.kernel.org, kyungmin.park@samsung.com,
	Tomasz Fujak <t.fujak@samsung.com>
Subject: Re: [PATCH v3] Input: add touchscreen driver for MELFAS MCS-5000 controller
Date: Tue, 14 Jul 2009 16:26:00 +0900	[thread overview]
Message-ID: <4A5C3308.7000404@samsung.com> (raw)
In-Reply-To: <001901ca044f$0314e930$093ebb90$%szyprowski@samsung.com>

On 7/14/2009 3:47 PM, Marek Szyprowski wrote:
> Hello,
> 
> On Tuesday, July 14, 2009 8:04 AM, 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@samsung.com>
>>> ---
>>> 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 have no idea what I do wrong, but this patch also does not apply here:
> 
> $ git apply touchscreen-update.diff
> error: patch failed: drivers/input/touchscreen/mcs5000_ts.c:259
> error: drivers/input/touchscreen/mcs5000_ts.c: patch does not apply
> 
> $ patch -p1 < touchscreen-update.diff
> patching file drivers/input/touchscreen/mcs5000_ts.c
> Hunk #6 FAILED at 196.
> 1 out of 7 hunks FAILED -- saving rejects to file drivers/input/touchscreen/mcs5000_ts.c.rej

Hmm, I've tried '$ patch -p1' also, but i have no problem.

Marek,
This patch of Dmitry needs the latest kernel source, but our source 
doesn't seem latest, so i cannot compile.  Can you update to the latest
kernel?

> 
> I've tried it on clean 'v3' patch posted in '[PATCH v3] Input: add
> touchscreen driver for MELFAS MCS-5000 controller' mail on Friday 
> 2009-07-10 04:10.
> 
> Best regards
> --
> Marek Szyprowski
> Samsung Poland R&D Center
> 
> 
> 
> 


  reply	other threads:[~2009-07-14  7:26 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-10  2:10 [PATCH v3] Input: add touchscreen driver for MELFAS MCS-5000 controller Joonyoung Shim
2009-07-14  6:04 ` Dmitry Torokhov
2009-07-14  6:47   ` Marek Szyprowski
2009-07-14  7:26     ` Joonyoung Shim [this message]
2009-07-15  7:51   ` Joonyoung Shim
2009-07-15 15:30     ` Dmitry Torokhov
2009-07-21  5:29       ` Joonyoung Shim
2009-07-21  7:45         ` Dmitry Torokhov
2009-07-21 12:15           ` Joonyoung Shim

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4A5C3308.7000404@samsung.com \
    --to=jy0922.shim@samsung.com \
    --cc=dmitry.torokhov@gmail.com \
    --cc=kyungmin.park@samsung.com \
    --cc=linux-input@vger.kernel.org \
    --cc=m.szyprowski@samsung.com \
    --cc=t.fujak@samsung.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox