It will give better output atleast. On Mon, May 11, at I will keep it in my tree for now until I figure out what to do with KEY macro for matrix keyboards should not be long though. The Maxim MAX is a I2C interfaced key switch controller which provides microprocessors with management of up to 64 key switches. Hello, On Tuesday, July 14, 5: I don’t see this driver picked up yet in your -next branch.

Uploader: Morr
Date Added: 17 August 2013
File Size: 27.39 Mb
Operating Systems: Windows NT/2000/XP/2003/2003/7/8/10 MacOS 10/X
Downloads: 15814
Price: Free* [*Free Regsitration Required]

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

Hi Dmitry, On Fri, Jun 19, at The ‘v3’ version worked fine here. I made a bunch of random changes, mostly to streamline the probe code belowhopfully I ma7x359 not break anything.

We should target this driver to be mainlined in next merge window. I’ve did the tests.

Trilok, I also wanted to thank you for doing reviews of llinux drivers posted on the linux-input, it is much appreciated. Thanks again for you reviewing. I had to apply the threaded irq patch on top of the patch Dmitry has sent me.

This patch adds support for the MAX key switch controller. No events are reported. Joonyoung Shim has pointed me that I was wrong. Hi Dmitry, On Tue, Jul 14, at 1: Hope this makes some sense Hello, On Tuesday, July 14, I don’t see this driver picked up yet in your -next branch.

You are not supposed to add it until somebody explicitely asks you to do that, same with Acked-by Hi Trilok, On Sun, May 10, at 2: On Mon, Jul 13, at It looks that I missed a patch that added some additional events, because I don’t think that the threaded irq patch would cause this.

In reply to this post by Dmitry Torokhov Below is the latest version of it. Hello, On Tuesday, July 14, 5: MAX keypad driver works after your patch, but reports much more events than the previous version.

Tests has been done on 2.

Re: [PATCH] Input: add MAX7359 key switch controller driver, v2

In this test I pressed quickly the first button on the keypad. Sincerely, Kyuwon — To unsubscribe from this list: It ilnux give better output atleast. If possible, I want Dmitry to add “Reviewed-by” when he sends this driver to mainline, or I will send new patch again.

Search everywhere only in this topic.

Linux Kernel: drivers/input/keyboard/max_keypad.c Source File

On Mon, May 11, at Hi Kim, On Sat, May 9, at 7: I confirmed this streamlined driver is nicely working on my S3C board and of course I mostly like your changes. On Sun, May 10, at 5: Great, thanks for testing. Event type 20 Repeat Testing