[PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes

[PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes

[Sergio Aguirre]

Hi,

This series contains fixes for omap2/3/4 serial code, and are
fixing:
 - Avoid doing ioremap of a zero-based physical address.
   (causing a kernel panic during early init on 3630boards)
 - Unproper omap_revision check during uart globals setup.
   (omap_revision is not yet filled at that point)
 - Don't try to enable all uarts generically for zoom2/3
   and 3630sdp.
 - Make zoom2/3 just register one serial port (ttyS0).

This is a subset of the original series found here: [1]

Thanks to:
 - Vikram Pandita
 - Paul Walmsley
 - Kevin Hilman
 - Manjunath Kondaiah
 - Felipe Balbi
 - Tony Lindgreen

For all the feedback recieved so far.

Regards,
Sergio

Detailed changelog:

Sergio Aguirre (7):
  OMAP3: serial: Check for zero-based physical addr
  OMAP3: serial: Use dev_* macros instead of printk
  omap2/3/4: serial: Remove condition for getting uart4_phys
  omap3: zoom2/3 / 3630sdp: Don't init always all uarts
  omap3: 3630sdp: Explicitly enable all UARTs
  omap3: zoom 2/3: Change debugboard serial port id
  omap3: zoom2/3: Register only 1 8250 port

 arch/arm/configs/omap_zoom2_defconfig        |    2 +-
 arch/arm/configs/omap_zoom3_defconfig        |    2 +-
 arch/arm/mach-omap2/board-3630sdp.c          |    1 +
 arch/arm/mach-omap2/board-zoom-debugboard.c  |    2 +-
 arch/arm/mach-omap2/board-zoom-peripherals.c |    1 -
 arch/arm/mach-omap2/serial.c                 |   35 +++++++++++++------------
 6 files changed, 22 insertions(+), 21 deletions(-)

[1] http://marc.info/?l=linux-omap&m=126826639903105&w=2

[PATCH 1/7] OMAP3: serial: Check for zero-based physical addr

[Sergio Aguirre]

This is for protecting a wrong mapping attempt of a zero-based
physical address.

The result is that, no serial port will be attempted to be mapped.

Also add an additional protection for NULL clocks before attempting
to enable them (if above condition applies)

Signed-off-by: Sergio Aguirre <saaguirre@ti.com>
---
 arch/arm/mach-omap2/serial.c |   13 +++++++++++++
 1 files changed, 13 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-omap2/serial.c b/arch/arm/mach-omap2/serial.c
index da77930..ef91fc0 100644
--- a/arch/arm/mach-omap2/serial.c
+++ b/arch/arm/mach-omap2/serial.c
@@ -664,6 +664,12 @@ void __init omap_serial_early_init(void)
 		struct device *dev = &pdev->dev;
 		struct plat_serial8250_port *p = dev->platform_data;
 
+		/* Don't map zero-based physical address */
+		if (p->mapbase == 0) {
+			printk(KERN_WARNING "omap serial: No physical address"
+			       " for uart#%d, so skipping early_init...\n", i);
+			continue;
+		}
 		/*
 		 * Module 4KB + L4 interconnect 4KB
 		 * Static mapping, never released
@@ -727,6 +733,13 @@ void __init omap_serial_init_port(int port)
 	pdev = &uart->pdev;
 	dev = &pdev->dev;
 
+	/* Don't proceed if there's no clocks available */
+	if (unlikely(!uart->ick || !uart->fck)) {
+		WARN(1, "%s: can't init uart%d, no clocks available\n",
+		     kobject_name(&dev->kobj), port);
+		return;
+	}
+
 	omap_uart_enable_clocks(uart);
 
 	omap_uart_reset(uart);
-- 
1.6.3.3

[PATCH 2/7] OMAP3: serial: Use dev_* macros instead of printk

[Sergio Aguirre]

As we have a struct device populated at the time we are
printing the errors, using dev_* macros makes more sense,
as could give a better idea where the error/warning came from.

Signed-off-by: Sergio Aguirre <saaguirre@ti.com>
---
 arch/arm/mach-omap2/serial.c |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/arch/arm/mach-omap2/serial.c b/arch/arm/mach-omap2/serial.c
index ef91fc0..a55e6ae 100644
--- a/arch/arm/mach-omap2/serial.c
+++ b/arch/arm/mach-omap2/serial.c
@@ -550,7 +550,7 @@ static ssize_t sleep_timeout_store(struct device *dev,
 	unsigned int value;
 
 	if (sscanf(buf, "%u", &value) != 1) {
-		printk(KERN_ERR "sleep_timeout_store: Invalid value\n");
+		dev_err(dev, "sleep_timeout_store: Invalid value\n");
 		return -EINVAL;
 	}
 
@@ -666,8 +666,8 @@ void __init omap_serial_early_init(void)
 
 		/* Don't map zero-based physical address */
 		if (p->mapbase == 0) {
-			printk(KERN_WARNING "omap serial: No physical address"
-			       " for uart#%d, so skipping early_init...\n", i);
+			dev_warn(dev, "no physical address for uart#%d,"
+				 " so skipping early_init...\n", i);
 			continue;
 		}
 		/*
@@ -676,21 +676,21 @@ void __init omap_serial_early_init(void)
 		 */
 		p->membase = ioremap(p->mapbase, SZ_8K);
 		if (!p->membase) {
-			printk(KERN_ERR "ioremap failed for uart%i\n", i + 1);
+			dev_err(dev, "ioremap failed for uart%i\n", i + 1);
 			continue;
 		}
 
 		sprintf(name, "uart%d_ick", i + 1);
 		uart->ick = clk_get(NULL, name);
 		if (IS_ERR(uart->ick)) {
-			printk(KERN_ERR "Could not get uart%d_ick\n", i + 1);
+			dev_err(dev, "Could not get uart%d_ick\n", i + 1);
 			uart->ick = NULL;
 		}
 
 		sprintf(name, "uart%d_fck", i+1);
 		uart->fck = clk_get(NULL, name);
 		if (IS_ERR(uart->fck)) {
-			printk(KERN_ERR "Could not get uart%d_fck\n", i + 1);
+			dev_err(dev, "Could not get uart%d_fck\n", i + 1);
 			uart->fck = NULL;
 		}
 
-- 
1.6.3.3

[PATCH 3/7] omap2/3/4: serial: Remove condition for getting uart4_phys

[Sergio Aguirre]

This check is invalid, since we haven't filled the
omap_revision var at this point.

Signed-off-by: Sergio Aguirre <saaguirre@ti.com>
---
 arch/arm/mach-omap2/serial.c |   14 +-------------
 1 files changed, 1 insertions(+), 13 deletions(-)

diff --git a/arch/arm/mach-omap2/serial.c b/arch/arm/mach-omap2/serial.c
index a55e6ae..3771254 100644
--- a/arch/arm/mach-omap2/serial.c
+++ b/arch/arm/mach-omap2/serial.c
@@ -115,7 +115,6 @@ static struct plat_serial8250_port serial_platform_data2[] = {
 	}
 };
 
-#if defined(CONFIG_ARCH_OMAP3) || defined(CONFIG_ARCH_OMAP4)
 static struct plat_serial8250_port serial_platform_data3[] = {
 	{
 		.irq		= 70,
@@ -128,23 +127,12 @@ static struct plat_serial8250_port serial_platform_data3[] = {
 	}
 };
 
-static inline void omap2_set_globals_uart4(struct omap_globals *omap2_globals)
-{
-	serial_platform_data3[0].mapbase = omap2_globals->uart4_phys;
-}
-#else
-static inline void omap2_set_globals_uart4(struct omap_globals *omap2_globals)
-{
-}
-#endif
-
 void __init omap2_set_globals_uart(struct omap_globals *omap2_globals)
 {
 	serial_platform_data0[0].mapbase = omap2_globals->uart1_phys;
 	serial_platform_data1[0].mapbase = omap2_globals->uart2_phys;
 	serial_platform_data2[0].mapbase = omap2_globals->uart3_phys;
-	if (cpu_is_omap3630() || cpu_is_omap44xx())
-		omap2_set_globals_uart4(omap2_globals);
+	serial_platform_data3[0].mapbase = omap2_globals->uart4_phys;
 }
 
 static inline unsigned int __serial_read_reg(struct uart_port *up,
-- 
1.6.3.3

[PATCH 4/7] omap3: zoom2/3 / 3630sdp: Don't init always all uarts

[Sergio Aguirre]

This is useless, since in Zoom2/3 boards, the ports aren't even
physically accessible.

They must be explicitly initted in the board-zoom2.c, board-zoom3.c
and board-3630sdp.c files instead.

Signed-off-by: Sergio Aguirre <saaguirre@ti.com>
---
 arch/arm/mach-omap2/board-zoom-peripherals.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/board-zoom-peripherals.c b/arch/arm/mach-omap2/board-zoom-peripherals.c
index ca95d8d..6b39849 100644
--- a/arch/arm/mach-omap2/board-zoom-peripherals.c
+++ b/arch/arm/mach-omap2/board-zoom-peripherals.c
@@ -280,7 +280,6 @@ static void enable_board_wakeup_source(void)
 void __init zoom_peripherals_init(void)
 {
 	omap_i2c_init();
-	omap_serial_init();
 	usb_musb_init(&musb_board_data);
 	enable_board_wakeup_source();
 }
-- 
1.6.3.3

[PATCH 5/7] omap3: 3630sdp: Explicitly enable all UARTs

[Sergio Aguirre]

All UARTs seem physically reachable, so, enable them all.

Signed-off-by: Sergio Aguirre <saaguirre@ti.com>
---
 arch/arm/mach-omap2/board-3630sdp.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-omap2/board-3630sdp.c b/arch/arm/mach-omap2/board-3630sdp.c
index a0a2a11..504d2bd 100644
--- a/arch/arm/mach-omap2/board-3630sdp.c
+++ b/arch/arm/mach-omap2/board-3630sdp.c
@@ -96,6 +96,7 @@ static struct omap_board_mux board_mux[] __initdata = {
 static void __init omap_sdp_init(void)
 {
 	omap3_mux_init(board_mux, OMAP_PACKAGE_CBP);
+	omap_serial_init();
 	zoom_peripherals_init();
 	board_smc91x_init();
 	enable_board_wakeup_source();
-- 
1.6.3.3

[PATCH 6/7] omap3: zoom 2/3: Change debugboard serial port id

[Sergio Aguirre]

This is now changed to PLAT8250_DEV_PLATFORM (= 0), because
it's the only port that's going to be initialized in
Zoom 2/3 boards.

So, it doesn't make sense to keep the hardcoded 3 value anymore.

Signed-off-by: Sergio Aguirre <saaguirre@ti.com>
---
 arch/arm/mach-omap2/board-zoom-debugboard.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/board-zoom-debugboard.c b/arch/arm/mach-omap2/board-zoom-debugboard.c
index bb4018b..e15d2e8 100644
--- a/arch/arm/mach-omap2/board-zoom-debugboard.c
+++ b/arch/arm/mach-omap2/board-zoom-debugboard.c
@@ -96,7 +96,7 @@ static struct plat_serial8250_port serial_platform_data[] = {
 
 static struct platform_device zoom_debugboard_serial_device = {
 	.name			= "serial8250",
-	.id			= 3,
+	.id			= PLAT8250_DEV_PLATFORM,
 	.dev			= {
 		.platform_data	= serial_platform_data,
 	},
-- 
1.6.3.3

[PATCH 7/7] omap3: zoom2/3: Register only 1 8250 port

[Sergio Aguirre]

There's no more serial ports available, so, doesn't make sense
to create 4 device nodes.

Signed-off-by: Sergio Aguirre <saaguirre@ti.com>
---
 arch/arm/configs/omap_zoom2_defconfig |    2 +-
 arch/arm/configs/omap_zoom3_defconfig |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/configs/omap_zoom2_defconfig b/arch/arm/configs/omap_zoom2_defconfig
index f5c6e11..881faea 100644
--- a/arch/arm/configs/omap_zoom2_defconfig
+++ b/arch/arm/configs/omap_zoom2_defconfig
@@ -661,7 +661,7 @@ CONFIG_DEVKMEM=y
 CONFIG_SERIAL_8250=y
 CONFIG_SERIAL_8250_CONSOLE=y
 CONFIG_SERIAL_8250_NR_UARTS=32
-CONFIG_SERIAL_8250_RUNTIME_UARTS=4
+CONFIG_SERIAL_8250_RUNTIME_UARTS=1
 CONFIG_SERIAL_8250_EXTENDED=y
 CONFIG_SERIAL_8250_MANY_PORTS=y
 CONFIG_SERIAL_8250_SHARE_IRQ=y
diff --git a/arch/arm/configs/omap_zoom3_defconfig b/arch/arm/configs/omap_zoom3_defconfig
index ea9a501..5e55b55 100644
--- a/arch/arm/configs/omap_zoom3_defconfig
+++ b/arch/arm/configs/omap_zoom3_defconfig
@@ -680,7 +680,7 @@ CONFIG_DEVKMEM=y
 CONFIG_SERIAL_8250=y
 CONFIG_SERIAL_8250_CONSOLE=y
 CONFIG_SERIAL_8250_NR_UARTS=32
-CONFIG_SERIAL_8250_RUNTIME_UARTS=4
+CONFIG_SERIAL_8250_RUNTIME_UARTS=1
 CONFIG_SERIAL_8250_EXTENDED=y
 CONFIG_SERIAL_8250_MANY_PORTS=y
 CONFIG_SERIAL_8250_SHARE_IRQ=y
-- 
1.6.3.3

Re: [PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes

[Tony Lindgren]

* Sergio Aguirre <saaguirre@ti.com> [100315 15:59]:
> Hi,
> 
> This series contains fixes for omap2/3/4 serial code, and are
> fixing:
>  - Avoid doing ioremap of a zero-based physical address.
>    (causing a kernel panic during early init on 3630boards)
>  - Unproper omap_revision check during uart globals setup.
>    (omap_revision is not yet filled at that point)
>  - Don't try to enable all uarts generically for zoom2/3
>    and 3630sdp.
>  - Make zoom2/3 just register one serial port (ttyS0).

Thanks, I've pulled this into omap-fixes-for-linus.

Regards,

Tony

 
> This is a subset of the original series found here: [1]
> 
> Thanks to:
>  - Vikram Pandita
>  - Paul Walmsley
>  - Kevin Hilman
>  - Manjunath Kondaiah
>  - Felipe Balbi
>  - Tony Lindgreen
> 
> For all the feedback recieved so far.
> 
> Regards,
> Sergio
> 
> Detailed changelog:
> 
> Sergio Aguirre (7):
>   OMAP3: serial: Check for zero-based physical addr
>   OMAP3: serial: Use dev_* macros instead of printk
>   omap2/3/4: serial: Remove condition for getting uart4_phys
>   omap3: zoom2/3 / 3630sdp: Don't init always all uarts
>   omap3: 3630sdp: Explicitly enable all UARTs
>   omap3: zoom 2/3: Change debugboard serial port id
>   omap3: zoom2/3: Register only 1 8250 port
> 
>  arch/arm/configs/omap_zoom2_defconfig        |    2 +-
>  arch/arm/configs/omap_zoom3_defconfig        |    2 +-
>  arch/arm/mach-omap2/board-3630sdp.c          |    1 +
>  arch/arm/mach-omap2/board-zoom-debugboard.c  |    2 +-
>  arch/arm/mach-omap2/board-zoom-peripherals.c |    1 -
>  arch/arm/mach-omap2/serial.c                 |   35 +++++++++++++------------
>  6 files changed, 22 insertions(+), 21 deletions(-)
> 
> [1] http://marc.info/?l=linux-omap&m=126826639903105&w=2
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes

[Sergio Aguirre]

Hi,

This series contains fixes for omap2/3/4 serial code, and are
fixing:
 - Avoid doing ioremapp of a zero-based physical address.
   (causing a kernel panic during early init on 3630boards)
 - Unproper omap_revision check during uart globals setup.
   (omap_revision is not yet filled at that point)
 - Don't try to enable all uarts generically for zoom2/3
   and 3630sdp.
 - Make zoom2/3 just register one serial port (ttyS0).

This is a subset of the original series found here: [1]

Thanks to:
 - Vikram Pandita
 - Paul Walmsley
 - Kevin Hilman
 - Manjunath Kondaiah
 - Felipe Balbi
 - Tony Lindgreen

For all the feedback recieved so far.

Regards,
Sergio

Sergio Aguirre (7):
  OMAP3: serial: Check for zero-based physical addr
  OMAP3: serial: Use dev_* macros instead of printk
  omap2/3/4: serial: Remove condition for getting uart4_phys
  omap3: zoom2/3 / 3630sdp: Don't init always all uarts
  omap3: 3630sdp: Explicitly enable all UARTs
  omap3: zoom 2/3: Change debugboard serial port id
  omap3: zoom2/3: Register only 1 8250 port

 arch/arm/configs/omap_zoom2_defconfig        |    2 +-
 arch/arm/configs/omap_zoom3_defconfig        |    2 +-
 arch/arm/mach-omap2/board-3630sdp.c          |    1 +
 arch/arm/mach-omap2/board-zoom-debugboard.c  |    2 +-
 arch/arm/mach-omap2/board-zoom-peripherals.c |    1 -
 arch/arm/mach-omap2/serial.c                 |   35 +++++++++++++------------
 6 files changed, 22 insertions(+), 21 deletions(-)

[1] http://marc.info/?l=linux-omap&m=126826639903105&w=2

RE: [PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes

[Aguirre, Sergio]

[-- Attachment #1: Type: text/plain, Size: 2048 bytes --]

Tony,

Please find attached pull request for your convenience.

Regards,
Sergio

> -----Original Message-----
> From: Aguirre, Sergio
> Sent: Friday, March 12, 2010 2:00 PM
> To: Tony Lindgren
> Cc: linux-omap@vger.kernel.org; Kevin Hilman; Pandita, Vikram; Paul
> Walmsley; Felipe Balbi; Aguirre, Sergio
> Subject: [PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes
> 
> Hi,
> 
> This series contains fixes for omap2/3/4 serial code, and are
> fixing:
>  - Avoid doing ioremapp of a zero-based physical address.
>    (causing a kernel panic during early init on 3630boards)
>  - Unproper omap_revision check during uart globals setup.
>    (omap_revision is not yet filled at that point)
>  - Don't try to enable all uarts generically for zoom2/3
>    and 3630sdp.
>  - Make zoom2/3 just register one serial port (ttyS0).
> 
> This is a subset of the original series found here: [1]
> 
> Thanks to:
>  - Vikram Pandita
>  - Paul Walmsley
>  - Kevin Hilman
>  - Manjunath Kondaiah
>  - Felipe Balbi
>  - Tony Lindgreen
> 
> For all the feedback recieved so far.
> 
> Regards,
> Sergio
> 
> Sergio Aguirre (7):
>   OMAP3: serial: Check for zero-based physical addr
>   OMAP3: serial: Use dev_* macros instead of printk
>   omap2/3/4: serial: Remove condition for getting uart4_phys
>   omap3: zoom2/3 / 3630sdp: Don't init always all uarts
>   omap3: 3630sdp: Explicitly enable all UARTs
>   omap3: zoom 2/3: Change debugboard serial port id
>   omap3: zoom2/3: Register only 1 8250 port
> 
>  arch/arm/configs/omap_zoom2_defconfig        |    2 +-
>  arch/arm/configs/omap_zoom3_defconfig        |    2 +-
>  arch/arm/mach-omap2/board-3630sdp.c          |    1 +
>  arch/arm/mach-omap2/board-zoom-debugboard.c  |    2 +-
>  arch/arm/mach-omap2/board-zoom-peripherals.c |    1 -
>  arch/arm/mach-omap2/serial.c                 |   35 +++++++++++++--------
> ----
>  6 files changed, 22 insertions(+), 21 deletions(-)
> 
> [1] http://marc.info/?l=linux-omap&m=126826639903105&w=2


[-- Attachment #2: pullreq_tony_20100315.txt --]
[-- Type: text/plain, Size: 1109 bytes --]

The following changes since commit a842b5f9ce70e1b738eabb4d719860070180ed1c:
  Tony Lindgren (1):
        Revert "omap: Add DSI regulator supply to OMAP3EVM board file"

are available in the git repository at:

  git://dev.omapzoom.org/pub/scm/saaguirre/linux-omap-camera.git omap-fixes

Sergio Aguirre (7):
      OMAP3: serial: Check for zero-based physical addr
      OMAP3: serial: Use dev_* macros instead of printk
      omap2/3/4: serial: Remove condition for getting uart4_phys
      omap3: zoom2/3 / 3630sdp: Don't init always all uarts
      omap3: 3630sdp: Explicitly enable all UARTs
      omap3: zoom 2/3: Change debugboard serial port id
      omap3: zoom2/3: Register only 1 8250 port

 arch/arm/configs/omap_zoom2_defconfig        |    2 +-
 arch/arm/configs/omap_zoom3_defconfig        |    2 +-
 arch/arm/mach-omap2/board-3630sdp.c          |    1 +
 arch/arm/mach-omap2/board-zoom-debugboard.c  |    2 +-
 arch/arm/mach-omap2/board-zoom-peripherals.c |    1 -
 arch/arm/mach-omap2/serial.c                 |   35 +++++++++++++------------
 6 files changed, 22 insertions(+), 21 deletions(-)

Re: [PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes

[Tony Lindgren]

* Aguirre, Sergio <saaguirre@ti.com> [100315 13:33]:
> Tony,
> 
> Please find attached pull request for your convenience.

<snip>

> The following changes since commit a842b5f9ce70e1b738eabb4d719860070180ed1c:
>   Tony Lindgren (1):
>         Revert "omap: Add DSI regulator supply to OMAP3EVM board file"
> 
> are available in the git repository at:
> 
>   git://dev.omapzoom.org/pub/scm/saaguirre/linux-omap-camera.git omap-fixes

Please base all the branches against the recent tag in Linus' tree,
in this case against v2.6.34-rc1. What we have in the linux-omap master
branch is a merge of various development branches, and we don't want
to merge that history to the mainline tree.

If the recent mainline tag won't work as the base, then please
rebase it on omap-fixes-for-linus or omap-for-linus. In this case,
omap-fixes-for-linus at commit 29b2ee5af5f3a02846bd38a1e2121d62ee5f6aca.
The omap-for-linus branch will contain things going into the next
merge window, currently it's empty.

In general, the commits ins omap-fixes-for-linus and omap-for-linus
should stay around unless something goes wrong. But please use
the recent mainline tag where possible, that's the most flexible
option for merging.

Regards,

Tony

RE: [PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes

[Aguirre, Sergio]

[-- Attachment #1: Type: text/plain, Size: 1788 bytes --]



From: Tony Lindgren [mailto:tony@atomide.com]
Sent: Monday, March 15, 2010 4:15 PM
> * Aguirre, Sergio <saaguirre@ti.com> [100315 13:33]:
> > Tony,
> >
> > Please find attached pull request for your convenience.
> 
> <snip>
> 
> > The following changes since commit
> a842b5f9ce70e1b738eabb4d719860070180ed1c:
> >   Tony Lindgren (1):
> >         Revert "omap: Add DSI regulator supply to OMAP3EVM board file"
> >
> > are available in the git repository at:
> >
> >   git://dev.omapzoom.org/pub/scm/saaguirre/linux-omap-camera.git omap-
> fixes
> 
> Please base all the branches against the recent tag in Linus' tree,
> in this case against v2.6.34-rc1. What we have in the linux-omap master
> branch is a merge of various development branches, and we don't want
> to merge that history to the mainline tree.
> 
> If the recent mainline tag won't work as the base, then please
> rebase it on omap-fixes-for-linus or omap-for-linus. In this case,
> omap-fixes-for-linus at commit 29b2ee5af5f3a02846bd38a1e2121d62ee5f6aca.
> The omap-for-linus branch will contain things going into the next
> merge window, currently it's empty.
> 
> In general, the commits ins omap-fixes-for-linus and omap-for-linus
> should stay around unless something goes wrong. But please use
> the recent mainline tag where possible, that's the most flexible
> option for merging.

Understood,

Please find updated pull request, rebased on omap-fixes-for-linux.

I tried it on master, but they don't apply clean, since "OMAP2: serial.c: Fix number of uarts in early_init" is not upstream, and there's a small cleanup change in it, which makes patch 0002 don't apply (printk to dev_* patch).

Thanks for your attention.

Regards,
Sergio

> 
> Regards,
> 
> Tony

[-- Attachment #2: pullreq_tony_20100315_v2.txt --]
[-- Type: text/plain, Size: 1114 bytes --]

The following changes since commit 29b2ee5af5f3a02846bd38a1e2121d62ee5f6aca:
  Andrew Clayton (1):
        ARM/OMAP: Remove the +x bit from a couple of source files

are available in the git repository at:

  git://dev.omapzoom.org/pub/scm/saaguirre/linux-omap-camera.git omap-fixes-for-tony

Sergio Aguirre (7):
      OMAP3: serial: Check for zero-based physical addr
      OMAP3: serial: Use dev_* macros instead of printk
      omap2/3/4: serial: Remove condition for getting uart4_phys
      omap3: zoom2/3 / 3630sdp: Don't init always all uarts
      omap3: 3630sdp: Explicitly enable all UARTs
      omap3: zoom 2/3: Change debugboard serial port id
      omap3: zoom2/3: Register only 1 8250 port

 arch/arm/configs/omap_zoom2_defconfig        |    2 +-
 arch/arm/configs/omap_zoom3_defconfig        |    2 +-
 arch/arm/mach-omap2/board-3630sdp.c          |    1 +
 arch/arm/mach-omap2/board-zoom-debugboard.c  |    2 +-
 arch/arm/mach-omap2/board-zoom-peripherals.c |    1 -
 arch/arm/mach-omap2/serial.c                 |   35 +++++++++++++------------
 6 files changed, 22 insertions(+), 21 deletions(-)

Re: [PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes

[Tony Lindgren]

* Aguirre, Sergio <saaguirre@ti.com> [100315 14:47]:
> 
> 
> From: Tony Lindgren [mailto:tony@atomide.com]
> Sent: Monday, March 15, 2010 4:15 PM
> > * Aguirre, Sergio <saaguirre@ti.com> [100315 13:33]:
> > > Tony,
> > >
> > > Please find attached pull request for your convenience.
> > 
> > <snip>
> > 
> > > The following changes since commit
> > a842b5f9ce70e1b738eabb4d719860070180ed1c:
> > >   Tony Lindgren (1):
> > >         Revert "omap: Add DSI regulator supply to OMAP3EVM board file"
> > >
> > > are available in the git repository at:
> > >
> > >   git://dev.omapzoom.org/pub/scm/saaguirre/linux-omap-camera.git omap-
> > fixes
> > 
> > Please base all the branches against the recent tag in Linus' tree,
> > in this case against v2.6.34-rc1. What we have in the linux-omap master
> > branch is a merge of various development branches, and we don't want
> > to merge that history to the mainline tree.
> > 
> > If the recent mainline tag won't work as the base, then please
> > rebase it on omap-fixes-for-linus or omap-for-linus. In this case,
> > omap-fixes-for-linus at commit 29b2ee5af5f3a02846bd38a1e2121d62ee5f6aca.
> > The omap-for-linus branch will contain things going into the next
> > merge window, currently it's empty.
> > 
> > In general, the commits ins omap-fixes-for-linus and omap-for-linus
> > should stay around unless something goes wrong. But please use
> > the recent mainline tag where possible, that's the most flexible
> > option for merging.
> 
> Understood,
> 
> Please find updated pull request, rebased on omap-fixes-for-linux.
> 
> I tried it on master, but they don't apply clean, since "OMAP2: serial.c: Fix number of uarts in early_init" is not upstream, and there's a small cleanup change in it, which makes patch 0002 don't apply (printk to dev_* patch).

OK, thanks. One more thing: Please repost your fixes one more
time with LAKML Cc'd for review. Then assuming no more comments, I'll
merge them into omap-fixes-for-linus.

Meanwhile, I'll merge it into l-o master branch for testing.

Tony

RE: [PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes

[Aguirre, Sergio]

From: Tony Lindgren [mailto:tony@atomide.com]
Sent: Monday, March 15, 2010 5:49 PM

<snip>

> 
> OK, thanks. One more thing: Please repost your fixes one more
> time with LAKML Cc'd for review. Then assuming no more comments, I'll
> merge them into omap-fixes-for-linus.

Done.

> 
> Meanwhile, I'll merge it into l-o master branch for testing.

Ok, thanks.

Regards,
Sergio

> 
> Tony