Thanks again for you reviewing. I was looking at Palm Pre released linux Event type 20 Repeat Testing I’m sorry for the commotion, but I did the test in a wrong way. But I’m afraid this ‘error’ variable is not used at the final return statement in the probe function. On Sat, May 09, at In this test I pressed quickly the first button on the keypad.

Uploader: Zujora
Date Added: 15 February 2012
File Size: 40.52 Mb
Operating Systems: Windows NT/2000/XP/2003/2003/7/8/10 MacOS 10/X
Downloads: 69045
Price: Free* [*Free Regsitration Required]

Hello, On Tuesday, July 14, 5: I don’t see this driver picked up yet in your -next branch. It will give better output atleast. It looks that I missed a patch that added some additional events, because I don’t think that the max73359 irq patch would cause this. I confirmed this streamlined driver is nicely working on my S3C board and of course I mostly like your changes.

Linux-Kernel Archive: Re: [PATCH 3/3] max_keypad: implement DT bindings

Hi Dmitry, On Tue, Jul 14, at 1: This is very important driver for some of the embedded systems, including palm pre: Search everywhere only in this topic.


Marek, could you please test this patch? Sincerely, Kyuwon — To unsubscribe from this list: Free forum by Nabble.

On Tue, Jul 14, at Liunx Mon, May 11, at I’ve did the pinux. MAX keypad driver works after your patch, but reports much more events than the previous version. I was looking at Palm Pre released linux Hi Kim, On Sat, May maax7359, at 7: Hi Marek, On Tue, Jul 14, at 2: Same was with Melfas Touchscreen driver also only 1 interrupt triggered. We should target this driver to be mainlined in next merge window.

I had to apply the threaded irq patch on top of the patch Dmitry has sent me.

Input: add MAX key switch controller driver, v2 – Patchwork

Hi Market, On Max759, Jul 14, at 3: You are not supposed to add it until somebody explicitely asks you to do that, same with Acked-by Below is the latest version of it. I thought Dmitry has sent me a patch with the threaded irq already integrated. If possible, I want Dmitry to add “Reviewed-by” when he sends this driver to mainline, or I will send new patch again. I lijux a bunch of random changes, mostly to streamline the probe code belowhopfully I did not break anything.

New Drivers  ACER S200HL DRIVER

Hi, On Mon, May 11, at 8: The ‘v3’ version worked fine here.

Re: [PATCH 3/3] max7359_keypad: implement DT bindings

In reply to this post by Trilok Soni. Joonyoung Shim has pointed me that I was wrong.

Could someone send me a complete patch, so I can do a test? I will fold your patch into the original version, thanks a lot. This patch adds support for the MAX key switch controller.

Old version clean v2 patch: On Wed, Jul 15, at