Re: [patch] [git pull] Input updates for 2.6.25-rc0

2008-02-08 Thread Randy Dunlap
Andrew Morton wrote: On Fri, 8 Feb 2008 15:37:41 -0800 Randy Dunlap <[EMAIL PROTECTED]> wrote: I think someone already sent a patch to select the LEDS I did... and more. Who will merge it? (below) --- From: Randy Dunlap <[EMAIL PROTECTED]> Add I2C to config since the driver makes several

Re: [patch] [git pull] Input updates for 2.6.25-rc0

2008-02-08 Thread Andrew Morton
On Fri, 8 Feb 2008 15:37:41 -0800 Randy Dunlap <[EMAIL PROTECTED]> wrote: > > I think someone already sent a patch to select the LEDS > > I did... and more. Who will merge it? (below) > > --- > From: Randy Dunlap <[EMAIL PROTECTED]> > > Add I2C to config since the driver makes several i2c*()

Re: [patch] [git pull] Input updates for 2.6.25-rc0

2008-02-08 Thread Jesper Juhl
On 09/02/2008, Randy Dunlap <[EMAIL PROTECTED]> wrote: ... > > On Sat, 9 Feb 2008 00:25:36 +0100 > > Ingo Molnar <[EMAIL PROTECTED]> wrote: > > ... > > I think someone already sent a patch to select the LEDS > > I did... and more. Who will merge it? (below) > > --- > From: Randy Dunlap <[EMAIL

[patch] [git pull] Input updates for 2.6.25-rc0

2008-02-08 Thread Randy Dunlap
On Fri, 8 Feb 2008 15:28:12 -0800 Stephen Hemminger wrote: > On Sat, 9 Feb 2008 00:25:36 +0100 > Ingo Molnar <[EMAIL PROTECTED]> wrote: > > > > > * Dmitry Torokhov <[EMAIL PROTECTED]> wrote: > > > > > Stephen Hemminger (2): > > > Input: add driver for Fujitsu application buttons > > > >

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-08 Thread Stephen Hemminger
On Sat, 9 Feb 2008 00:25:36 +0100 Ingo Molnar <[EMAIL PROTECTED]> wrote: > > * Dmitry Torokhov <[EMAIL PROTECTED]> wrote: > > > Stephen Hemminger (2): > > Input: add driver for Fujitsu application buttons > > this change broke the build on x86 in randconfig testing: > >

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-08 Thread Ingo Molnar
* Dmitry Torokhov <[EMAIL PROTECTED]> wrote: > Stephen Hemminger (2): > Input: add driver for Fujitsu application buttons this change broke the build on x86 in randconfig testing: drivers/built-in.o: In function `apanel_detach_client': apanel.c:(.text+0x15c120): undefined reference

Re: [patch] [git pull] Input updates for 2.6.25-rc0

2008-02-08 Thread Andrew Morton
On Fri, 8 Feb 2008 15:37:41 -0800 Randy Dunlap [EMAIL PROTECTED] wrote: I think someone already sent a patch to select the LEDS I did... and more. Who will merge it? (below) --- From: Randy Dunlap [EMAIL PROTECTED] Add I2C to config since the driver makes several i2c*() calls.

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-08 Thread Ingo Molnar
* Dmitry Torokhov [EMAIL PROTECTED] wrote: Stephen Hemminger (2): Input: add driver for Fujitsu application buttons this change broke the build on x86 in randconfig testing: drivers/built-in.o: In function `apanel_detach_client': apanel.c:(.text+0x15c120): undefined reference to

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-08 Thread Stephen Hemminger
On Sat, 9 Feb 2008 00:25:36 +0100 Ingo Molnar [EMAIL PROTECTED] wrote: * Dmitry Torokhov [EMAIL PROTECTED] wrote: Stephen Hemminger (2): Input: add driver for Fujitsu application buttons this change broke the build on x86 in randconfig testing: drivers/built-in.o: In

Re: [patch] [git pull] Input updates for 2.6.25-rc0

2008-02-08 Thread Jesper Juhl
On 09/02/2008, Randy Dunlap [EMAIL PROTECTED] wrote: ... On Sat, 9 Feb 2008 00:25:36 +0100 Ingo Molnar [EMAIL PROTECTED] wrote: ... I think someone already sent a patch to select the LEDS I did... and more. Who will merge it? (below) --- From: Randy Dunlap [EMAIL PROTECTED] Add

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-07 Thread Andrew Morton
On Thu, 7 Feb 2008 15:05:57 -0500 Dmitry Torokhov <[EMAIL PROTECTED]> wrote: > On Tue, Feb 05, 2008 at 10:32:42PM -0800, Andrew Morton wrote: > > > > Looks OK. Minorish things from a quick scan: > > > > *ping* > > Linus, > > Do you want me to redo the pull or is it still acceptable as is? >

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-07 Thread Dmitry Torokhov
On Tue, Feb 05, 2008 at 10:32:42PM -0800, Andrew Morton wrote: > > Looks OK. Minorish things from a quick scan: > *ping* Linus, Do you want me to redo the pull or is it still acceptable as is? I am hesitant touching the tree after sending a pull request... Thanks, Dmitry -- Dmitry -- To

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-07 Thread Dmitry Torokhov
On Tue, Feb 05, 2008 at 10:32:42PM -0800, Andrew Morton wrote: Looks OK. Minorish things from a quick scan: *ping* Linus, Do you want me to redo the pull or is it still acceptable as is? I am hesitant touching the tree after sending a pull request... Thanks, Dmitry -- Dmitry -- To

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-07 Thread Andrew Morton
On Thu, 7 Feb 2008 15:05:57 -0500 Dmitry Torokhov [EMAIL PROTECTED] wrote: On Tue, Feb 05, 2008 at 10:32:42PM -0800, Andrew Morton wrote: Looks OK. Minorish things from a quick scan: *ping* Linus, Do you want me to redo the pull or is it still acceptable as is? I am hesitant

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-06 Thread Dmitry Torokhov
On Wed, Feb 06, 2008 at 08:56:35AM +0100, Sam Ravnborg wrote: > > > > > > > > > > > > > > i8042_platform_init(): > > > > > > +#if defined(__i386__) || defined(__x86_64__) > > > > > > use #ifdef CONFIG_X86? > > > > > > > I considered it but above was tested and in line with the style of

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-06 Thread Dmitry Torokhov
On Wed, Feb 06, 2008 at 08:56:35AM +0100, Sam Ravnborg wrote: i8042_platform_init(): +#if defined(__i386__) || defined(__x86_64__) use #ifdef CONFIG_X86? I considered it but above was tested and in line with the style of the rest of the file... Then

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-05 Thread Sam Ravnborg
> > > > > > > > > i8042_platform_init(): > > > > +#if defined(__i386__) || defined(__x86_64__) > > > > use #ifdef CONFIG_X86? > > > > I considered it but above was tested and in line with the style of the > rest of the file... Then please change the rest of the file so it is consistent

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-05 Thread Andrew Morton
On Wed, 6 Feb 2008 01:08:30 -0500 Dmitry Torokhov <[EMAIL PROTECTED]> wrote: > Andrew Morton (1): > Input: i8042 - non-x86 build fix Also, please prefer to fold patches like this into the patch which they fix, to avoid breaking git-bisect, thanks. (I add a little one-line mention in the

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-05 Thread Dmitry Torokhov
On Wednesday 06 February 2008 01:32, Andrew Morton wrote: > > Looks OK. Minorish things from a quick scan: > > > > tosakbd_scankeyboard() looks like it'll spend a perfectly wicked amount of > time under spin_lock_irqsave(). > I think you are right. I will check with Dmitry if it can be

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-05 Thread Andrew Morton
Looks OK. Minorish things from a quick scan: tosakbd_scankeyboard() looks like it'll spend a perfectly wicked amount of time under spin_lock_irqsave(). This code, in tosakbd_probe(): +fail2: + while (--i >= 0) + gpio_free(TOSA_GPIO_KEY_STROBE(i)); + + i =

[git pull] Input updates for 2.6.25-rc0

2008-02-05 Thread Dmitry Torokhov
Hi Linus, Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git for-linus or master.kernel.org:/pub/scm/linux/kernel/git/dtor/input.git for-linus to receive updates for the input subsystem. Most of the changes were in -mm for some time with the exception

[git pull] Input updates for 2.6.25-rc0

2008-02-05 Thread Dmitry Torokhov
Hi Linus, Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git for-linus or master.kernel.org:/pub/scm/linux/kernel/git/dtor/input.git for-linus to receive updates for the input subsystem. Most of the changes were in -mm for some time with the exception

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-05 Thread Andrew Morton
Looks OK. Minorish things from a quick scan: tosakbd_scankeyboard() looks like it'll spend a perfectly wicked amount of time under spin_lock_irqsave(). This code, in tosakbd_probe(): +fail2: + while (--i = 0) + gpio_free(TOSA_GPIO_KEY_STROBE(i)); + + i =

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-05 Thread Dmitry Torokhov
On Wednesday 06 February 2008 01:32, Andrew Morton wrote: Looks OK. Minorish things from a quick scan: tosakbd_scankeyboard() looks like it'll spend a perfectly wicked amount of time under spin_lock_irqsave(). I think you are right. I will check with Dmitry if it can be relaxed a

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-05 Thread Andrew Morton
On Wed, 6 Feb 2008 01:08:30 -0500 Dmitry Torokhov [EMAIL PROTECTED] wrote: Andrew Morton (1): Input: i8042 - non-x86 build fix Also, please prefer to fold patches like this into the patch which they fix, to avoid breaking git-bisect, thanks. (I add a little one-line mention in the

Re: [git pull] Input updates for 2.6.25-rc0

2008-02-05 Thread Sam Ravnborg
i8042_platform_init(): +#if defined(__i386__) || defined(__x86_64__) use #ifdef CONFIG_X86? I considered it but above was tested and in line with the style of the rest of the file... Then please change the rest of the file so it is consistent with the usual style to