Re: [PATCH] TWL4030 Keypad driver - incorporated review comments

[tony@atomide.com]

* nishanth menon <menon.nishanth@gmail.com> [070511 17:51]:
> Hi Khasim,
> 
> On 5/10/07, Nishanth Menon <menon.nishanth@gmail.com> wrote:
> >Syed Mohammed, Khasim stated on 5/10/2007 7:19 PM:
> >> I feel like not doing this, fundamental reason is we don't have an open 
> >TRM for TWL4030, this code is new to community and I don't want to confuse 
> >developers more. This though takes some kind of space is easy to 
> >understand and comment on. I can definitely revisit once I see some kind 
> >of usage of TWL4030 driver in community and having a public TRM to explain 
> >the same. Also I am just targeting drivers to get into some respectable 
> >form in GIT then we can improve them for more functionality and
> >>
> >I can send a patch for the same if you'd like to save yourself the
> >effort ;).... but just hoping to save git some bytes.. and hoping to see
> Attached is a clean up patch for the same.. - this is a follow on
> patch in mail: Thu 5/10/2007 11:49 PM

Khasim, can you repost one more time with Nishants clean-up patch merged
if you're OK with that? It's pointless to add the driver to the git tree
and then immediately fix it.

Again, that will make it easier for us to use the same patches for
sending upstream without need to rewrite them.

Tony