Hi Govindraj, Just one non-functional comment below. > -----Original Message----- > From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap- > owner@xxxxxxxxxxxxxxx] On Behalf Of Raja, Govindraj > Sent: Wednesday, September 22, 2010 10:15 AM > To: linux-omap@xxxxxxxxxxxxxxx > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-serial@xxxxxxxxxxxxxxx; > Kevin Hilman; Tony Lindgren; Gadiyar, Anand > Subject: [PATCH 13/13 v2] OMAP3: SERIAL: Initialize all omap-uarts for > zoom boards > > Iniatize all omap-uarts for zoom boards. > Remove serial_init from 3630sdp board init > as zoom_peripheral_init does the same. I think this description mismatches slightly to what is really being done in the patch. I mean, the way you say it, it's like the omap_serial_init was already in zoom_peripheral_init... which is not. Regards, Sergio > > Signed-off-by: Kevin Hilman <khilman@xxxxxx> > Signed-off-by: Anand Gadiyar <gadiyar@xxxxxx> > Signed-off-by: Govindraj.R <govindraj.raja@xxxxxx> > --- > arch/arm/mach-omap2/board-3630sdp.c | 1 - > arch/arm/mach-omap2/board-zoom-peripherals.c | 1 + > 2 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/mach-omap2/board-3630sdp.c b/arch/arm/mach- > omap2/board-3630sdp.c > index b359c3f..d510451 100644 > --- a/arch/arm/mach-omap2/board-3630sdp.c > +++ b/arch/arm/mach-omap2/board-3630sdp.c > @@ -208,7 +208,6 @@ static struct flash_partitions sdp_flash_partitions[] > = { > static void __init omap_sdp_init(void) > { > omap3_mux_init(board_mux, OMAP_PACKAGE_CBP); > - omap_serial_init(); > zoom_peripherals_init(); > board_smc91x_init(); > board_flash_init(sdp_flash_partitions, chip_sel_sdp); > diff --git a/arch/arm/mach-omap2/board-zoom-peripherals.c > b/arch/arm/mach-omap2/board-zoom-peripherals.c > index 6b39849..641765a 100644 > --- a/arch/arm/mach-omap2/board-zoom-peripherals.c > +++ b/arch/arm/mach-omap2/board-zoom-peripherals.c > @@ -282,4 +282,5 @@ void __init zoom_peripherals_init(void) > omap_i2c_init(); > usb_musb_init(&musb_board_data); > enable_board_wakeup_source(); > + omap_serial_init(); > } > -- > 1.6.3.3 > > > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html