Re: Fwd: Issue with arm-devel tree

2008-01-14 Thread Andre Haupt
On Sat, Jan 12, 2008 at 02:54:49PM +, Russell King wrote: > It might be a good idea if there was some co-ordination with people > involved in the duplicate include removal work... Sorry for the inconvenience! I did not realise that Lucas was working on this and hence missed his patches. Best

Re: Fwd: Issue with arm-devel tree

2008-01-14 Thread Andre Haupt
On Sat, Jan 12, 2008 at 02:54:49PM +, Russell King wrote: It might be a good idea if there was some co-ordination with people involved in the duplicate include removal work... Sorry for the inconvenience! I did not realise that Lucas was working on this and hence missed his patches. Best

[PATCH][KJ] input: remove duplicate includes

2008-01-08 Thread Andre Haupt
Signed-off-by: Andre Haupt <[EMAIL PROTECTED]> --- drivers/input/gameport/gameport.c |1 - drivers/input/keyboard/bf54x-keys.c |1 - drivers/input/keyboard/jornada720_kbd.c |1 - 3 files changed, 0 insertions(+), 3 deletions(-) diff --git a/drivers/input/gameport/game

[PATCH][KJ] input: remove duplicate includes

2008-01-08 Thread Andre Haupt
Signed-off-by: Andre Haupt [EMAIL PROTECTED] --- drivers/input/gameport/gameport.c |1 - drivers/input/keyboard/bf54x-keys.c |1 - drivers/input/keyboard/jornada720_kbd.c |1 - 3 files changed, 0 insertions(+), 3 deletions(-) diff --git a/drivers/input/gameport/gameport.c

[PATCH][KJ] 8250: remove unnecessary variable tmout from wait_for_xmitr()

2007-12-13 Thread Andre Haupt
This fixes a sparse warning about symbol tmout shadowing an earlier one. Signed-off-by: Andre Haupt <[EMAIL PROTECTED]> --- drivers/serial/8250.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/drivers/serial/8250.c b/drivers/serial/8250.c index f94109c..284757d

[PATCH][KJ] 8250: remove unnecessary variable tmout from wait_for_xmitr()

2007-12-13 Thread Andre Haupt
This fixes a sparse warning about symbol tmout shadowing an earlier one. Signed-off-by: Andre Haupt [EMAIL PROTECTED] --- drivers/serial/8250.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/drivers/serial/8250.c b/drivers/serial/8250.c index f94109c..284757d 100644

[PATCH 2/2] fbmon: cleanup trailing whitespaces

2007-11-27 Thread Andre Haupt
From: Andre Haupt <[EMAIL PROTECTED]> Signed-off-by: Andre Haupt <[EMAIL PROTECTED]> --- drivers/video/fbmon.c | 116 1 files changed, 58 insertions(+), 58 deletions(-) diff --git a/drivers/video/fbmon.c b/drivers/video/fbmon.c i

[PATCH 1/2] fbmon: remove unnecessary local variable

2007-11-27 Thread Andre Haupt
This fixes a sparse warning about symbol 'i' shadowing an earlier one. From: Andre Haupt <[EMAIL PROTECTED]> Signed-off-by: Andre Haupt <[EMAIL PROTECTED]> --- drivers/video/fbmon.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/video/fbmon.c b/d

[PATCH 1/2] fbmon: remove unnecessary local variable

2007-11-27 Thread Andre Haupt
This fixes a sparse warning about symbol 'i' shadowing an earlier one. From: Andre Haupt [EMAIL PROTECTED] Signed-off-by: Andre Haupt [EMAIL PROTECTED] --- drivers/video/fbmon.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/video/fbmon.c b/drivers/video

[PATCH 2/2] fbmon: cleanup trailing whitespaces

2007-11-27 Thread Andre Haupt
From: Andre Haupt [EMAIL PROTECTED] Signed-off-by: Andre Haupt [EMAIL PROTECTED] --- drivers/video/fbmon.c | 116 1 files changed, 58 insertions(+), 58 deletions(-) diff --git a/drivers/video/fbmon.c b/drivers/video/fbmon.c index 3f3a4e9..038ebf1

[PATCH] sdio_uart: fix sign of paramter status in sdio_uart_receive_chars()

2007-11-21 Thread Andre Haupt
I think, the status paramter should be unsigned. Is this correct? This also fixes a sparse warning about different signedness. Only compile tested, because i do not have the hardware. From: Andre Haupt <[EMAIL PROTECTED]> Signed-off-by: Andre Haupt <[EMAIL PROTECTED]> --- driv

[PATCH] sdio_uart: fix sign of paramter status in sdio_uart_receive_chars()

2007-11-21 Thread Andre Haupt
I think, the status paramter should be unsigned. Is this correct? This also fixes a sparse warning about different signedness. Only compile tested, because i do not have the hardware. From: Andre Haupt [EMAIL PROTECTED] Signed-off-by: Andre Haupt [EMAIL PROTECTED] --- drivers/mmc/card

[patch 1/1] omap2: remove double include

2007-11-05 Thread Andre Haupt
From: Andre Haupt <[EMAIL PROTECTED]> Signed-off-by: Andre Haupt <[EMAIL PROTECTED]> --- arch/arm/mach-omap2/board-apollon.c |1 - 1 file changed, 1 deletion(-) Index: linus/arch/arm/mach-omap2/boa

[patch 1/1] remove double includes

2007-11-05 Thread Andre Haupt
From: Andre Haupt <[EMAIL PROTECTED]> Signed-off-by: Andre Haupt <[EMAIL PROTECTED]> --- arch/arm/mach-omap1/board-nokia770.c |1 - arch/arm/mach-omap1/board-osk.c |1 - arch/arm/mach-omap1/board-palmtt.c |1 - arch/arm/mach-omap1/board-palmz71.c |1 - 4 file

[patch 1/1] ixp4xx: remove double include

2007-11-05 Thread Andre Haupt
From: Andre Haupt <[EMAIL PROTECTED]> Signed-off-by: Andre Haupt <[EMAIL PROTECTED]> --- arch/arm/mach-ixp4xx/nslu2-power.c |1 - 1 file changed, 1 deletion(-) Index: linus/arch/arm/mach-ixp4xx/nslu2-power.c === --

[patch 1/1] ixp23xx: fix some double includes

2007-11-05 Thread Andre Haupt
From: Andre Haupt <[EMAIL PROTECTED]> Signed-off-by: Andre Haupt <[EMAIL PROTECTED]> --- arch/arm/mach-ixp23xx/core.c |1 - arch/arm/mach-ixp23xx/espresso.c |2 -- arch/arm/mach-ixp23xx/ixdp2351.c |2 -- arch/arm/mach-ixp23xx/roadrunner.c |2 -- 4 file

[patch 1/1] ep93xx: fix double include of linux/serial.h

2007-11-05 Thread Andre Haupt
Signed-off-by: Andre Haupt <[EMAIL PROTECTED]> --- arch/arm/mach-ep93xx/core.c |1 - 1 file changed, 1 deletion(-) Index: linus/arch/arm/mach-ep93xx/core.c === --- linus.orig/arch/arm/mach-ep93xx/core.c 2007-11-05

[patch 1/1] ep93xx: fix double include of linux/serial.h

2007-11-05 Thread Andre Haupt
Signed-off-by: Andre Haupt [EMAIL PROTECTED] --- arch/arm/mach-ep93xx/core.c |1 - 1 file changed, 1 deletion(-) Index: linus/arch/arm/mach-ep93xx/core.c === --- linus.orig/arch/arm/mach-ep93xx/core.c 2007-11-05 14:48

[patch 1/1] ixp23xx: fix some double includes

2007-11-05 Thread Andre Haupt
From: Andre Haupt [EMAIL PROTECTED] Signed-off-by: Andre Haupt [EMAIL PROTECTED] --- arch/arm/mach-ixp23xx/core.c |1 - arch/arm/mach-ixp23xx/espresso.c |2 -- arch/arm/mach-ixp23xx/ixdp2351.c |2 -- arch/arm/mach-ixp23xx/roadrunner.c |2 -- 4 files changed, 7 deletions

[patch 1/1] ixp4xx: remove double include

2007-11-05 Thread Andre Haupt
From: Andre Haupt [EMAIL PROTECTED] Signed-off-by: Andre Haupt [EMAIL PROTECTED] --- arch/arm/mach-ixp4xx/nslu2-power.c |1 - 1 file changed, 1 deletion(-) Index: linus/arch/arm/mach-ixp4xx/nslu2-power.c === --- linus.orig/arch

[patch 1/1] remove double includes

2007-11-05 Thread Andre Haupt
From: Andre Haupt [EMAIL PROTECTED] Signed-off-by: Andre Haupt [EMAIL PROTECTED] --- arch/arm/mach-omap1/board-nokia770.c |1 - arch/arm/mach-omap1/board-osk.c |1 - arch/arm/mach-omap1/board-palmtt.c |1 - arch/arm/mach-omap1/board-palmz71.c |1 - 4 files changed, 4

[patch 1/1] omap2: remove double include

2007-11-05 Thread Andre Haupt
From: Andre Haupt [EMAIL PROTECTED] Signed-off-by: Andre Haupt [EMAIL PROTECTED] --- arch/arm/mach-omap2/board-apollon.c |1 - 1 file changed, 1 deletion(-) Index: linus/arch/arm/mach-omap2/board-apollon.c === --- linus.orig

Re: 2.6.23-git11 compile issues

2007-10-17 Thread Andre Haupt
On Wed, Oct 17, 2007 at 08:33:18AM -0700, Badari Pulavarty wrote: > Known issue ? I saw that too after a git pull a couple of days ago. The following worked for me: make mrproper cp /proc/config.gz .config.gz (i was running 2.6.23 release kernel) gunzip .config.gz make silentoldconfig make

Re: 2.6.23-git11 compile issues

2007-10-17 Thread Andre Haupt
On Wed, Oct 17, 2007 at 08:33:18AM -0700, Badari Pulavarty wrote: Known issue ? I saw that too after a git pull a couple of days ago. The following worked for me: make mrproper cp /proc/config.gz .config.gz (i was running 2.6.23 release kernel) gunzip .config.gz make silentoldconfig make

Re: [PATCH] ISDN: Change pci_find_device() calls to pci_get_device()

2007-09-21 Thread Andre Haupt
On Fri, Sep 21, 2007 at 10:55:08AM +0200, Gregory 'GaLi' Cavelier wrote: > > The following patch removes the obsolete pci_find_device() calls and replaces > them with pci_get_device() for all ISDN drivers. > No, it does not. It replaces the correct pci_get_device() calls in the isdn tree with

Re: [PATCH] ISDN: Change pci_find_device() calls to pci_get_device()

2007-09-21 Thread Andre Haupt
On Fri, Sep 21, 2007 at 10:55:08AM +0200, Gregory 'GaLi' Cavelier wrote: The following patch removes the obsolete pci_find_device() calls and replaces them with pci_get_device() for all ISDN drivers. No, it does not. It replaces the correct pci_get_device() calls in the isdn tree with the

hci_sock.c build failure

2007-09-12 Thread Andre Haupt
Hi, I did a make randconfig and then make C=1. hci_sock.c fails to build because it doesnt know compat_timeval (i guess). find the error message and the config attached. regards, Andre LD net/ax25/ax25.o LD net/ax25/built-in.o CHECK net/bluetooth/af_bluetooth.c CC

hci_sock.c build failure

2007-09-12 Thread Andre Haupt
Hi, I did a make randconfig and then make C=1. hci_sock.c fails to build because it doesnt know compat_timeval (i guess). find the error message and the config attached. regards, Andre LD net/ax25/ax25.o LD net/ax25/built-in.o CHECK net/bluetooth/af_bluetooth.c CC

Re: [KJ][patch 3/3] VIDC20: use abs() from kernel.h instead of own definition

2007-08-30 Thread Andre Haupt
On Thu, Aug 30, 2007 at 05:17:47AM -0600, Matthew Wilcox wrote: > On Thu, Aug 30, 2007 at 12:40:38PM +0200, [EMAIL PROTECTED] wrote: > > - diff_int = my_abs(rate_ext-rate); > > - diff_ext = my_abs(rate_int-rate); > > + diff_int = abs(rate_ext-rate); > > +

Re: [KJ][patch 3/3] VIDC20: use abs() from kernel.h instead of own definition

2007-08-30 Thread Andre Haupt
On Thu, Aug 30, 2007 at 05:17:47AM -0600, Matthew Wilcox wrote: On Thu, Aug 30, 2007 at 12:40:38PM +0200, [EMAIL PROTECTED] wrote: - diff_int = my_abs(rate_ext-rate); - diff_ext = my_abs(rate_int-rate); + diff_int = abs(rate_ext-rate); + diff_ext =