On Sat, Jan 30, 2010 at 07:23:43PM +0100, Wolfram Sang wrote: > On Sat, Jan 30, 2010 at 06:42:57PM +0100, Maxime Bizon wrote: > > The bcm6358 CPU has two uarts, make it possible to use the second one. > > > > Signed-off-by: Maxime Bizon <mbizon@xxxxxxxxxx> > > --- > > drivers/serial/bcm63xx_uart.c | 5 +++-- > > 1 files changed, 3 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/serial/bcm63xx_uart.c b/drivers/serial/bcm63xx_uart.c > > index f78ede8..6ab959a 100644 > > --- a/drivers/serial/bcm63xx_uart.c > > +++ b/drivers/serial/bcm63xx_uart.c > > @@ -35,7 +35,7 @@ > > #include <bcm63xx_regs.h> > > #include <bcm63xx_io.h> > > > > -#define BCM63XX_NR_UARTS 1 > > +#define BCM63XX_NR_UARTS 2 > > > > static struct uart_port ports[BCM63XX_NR_UARTS]; > > > > @@ -784,7 +784,7 @@ static struct uart_driver bcm_uart_driver = { > > .dev_name = "ttyS", > > .major = TTY_MAJOR, > > .minor = 64, > > - .nr = 1, > > + .nr = 2, > > Err, why not using the #define here? Good idea, I've tweaked the patch to do that. thanks, greg k-h