diff mbox

[U-Boot] am33xx: Update serial pdata

Message ID 1449418236-13343-1-git-send-email-trini@konsulko.com
State Accepted
Delegated to: Tom Rini
Headers show

Commit Message

Tom Rini Dec. 6, 2015, 4:10 p.m. UTC
We now want to make sure that we have the platform data for NS16550 when
we do not have OF_CONTROL set.

Signed-off-by: Tom Rini <trini@konsulko.com>
---
 arch/arm/cpu/armv7/am33xx/board.c |   18 +++++++-----------
 1 file changed, 7 insertions(+), 11 deletions(-)

Comments

Simon Glass Dec. 6, 2015, 4:43 p.m. UTC | #1
Hi Tom,

On 6 December 2015 at 08:10, Tom Rini <trini@konsulko.com> wrote:
> We now want to make sure that we have the platform data for NS16550 when
> we do not have OF_CONTROL set.
>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
>  arch/arm/cpu/armv7/am33xx/board.c |   18 +++++++-----------
>  1 file changed, 7 insertions(+), 11 deletions(-)
>

Reviewed-by: Simon Glass <sjg@chromium.org>

...so I won't include this patch. Thanks!

Regards,
Simon
Tom Rini Dec. 6, 2015, 10:09 p.m. UTC | #2
On Sun, Dec 06, 2015 at 08:43:36AM -0800, Simon Glass wrote:

> Hi Tom,
> 
> On 6 December 2015 at 08:10, Tom Rini <trini@konsulko.com> wrote:
> > We now want to make sure that we have the platform data for NS16550 when
> > we do not have OF_CONTROL set.
> >
> > Signed-off-by: Tom Rini <trini@konsulko.com>
> > ---
> >  arch/arm/cpu/armv7/am33xx/board.c |   18 +++++++-----------
> >  1 file changed, 7 insertions(+), 11 deletions(-)
> >
> 
> Reviewed-by: Simon Glass <sjg@chromium.org>

Applied to u-boot/master, thanks!
diff mbox

Patch

diff --git a/arch/arm/cpu/armv7/am33xx/board.c b/arch/arm/cpu/armv7/am33xx/board.c
index 1633ddc..466348f 100644
--- a/arch/arm/cpu/armv7/am33xx/board.c
+++ b/arch/arm/cpu/armv7/am33xx/board.c
@@ -38,11 +38,7 @@ 
 
 DECLARE_GLOBAL_DATA_PTR;
 
-#if defined(CONFIG_DM_SERIAL) && !defined(CONFIG_OF_CONTROL)
-/*
- * TODO(sjg@chromium.org): When we can move SPL serial to DM, we can remove
- * the CONFIGs. At the same time, we should move this to the board files.
- */
+#if !CONFIG_IS_ENABLED(OF_CONTROL)
 static const struct ns16550_platdata am33xx_serial[] = {
 	{ CONFIG_SYS_NS16550_COM1, 2, CONFIG_SYS_NS16550_CLK },
 # ifdef CONFIG_SYS_NS16550_COM2
@@ -57,14 +53,14 @@  static const struct ns16550_platdata am33xx_serial[] = {
 };
 
 U_BOOT_DEVICES(am33xx_uarts) = {
-	{ "serial_omap", &am33xx_serial[0] },
+	{ "ns16550_serial", &am33xx_serial[0] },
 #  ifdef CONFIG_SYS_NS16550_COM2
-	{ "serial_omap", &am33xx_serial[1] },
+	{ "ns16550_serial", &am33xx_serial[1] },
 #   ifdef CONFIG_SYS_NS16550_COM3
-	{ "serial_omap", &am33xx_serial[2] },
-	{ "serial_omap", &am33xx_serial[3] },
-	{ "serial_omap", &am33xx_serial[4] },
-	{ "serial_omap", &am33xx_serial[5] },
+	{ "ns16550_serial", &am33xx_serial[2] },
+	{ "ns16550_serial", &am33xx_serial[3] },
+	{ "ns16550_serial", &am33xx_serial[4] },
+	{ "ns16550_serial", &am33xx_serial[5] },
 #   endif
 #  endif
 };