[U-Boot-Users] [PATCH] add missing 'console' var to default mpc8349itx config

[U-Boot-Users] [PATCH] add missing 'console' var to default mpc8349itx config

[Nikita V. Youshchenko]

Default environment defined inside include/configs/MPC8349ITX.h file,
references 'console' variable, but does not define it.

This patch fixes this.

Also it adds usage of new CONFIG_CONSOLE macro in default bootargs,
for consistency.

Signed-Off-By: Nikita Youshchenko <yoush@debian.org>

--- a/include/configs/MPC8349ITX.h
+++ b/include/configs/MPC8349ITX.h
@@ -289,6 +289,7 @@ boards, we say we have two, but don't di
 #define CFG_BAUDRATE_TABLE  \
        {300, 600, 1200, 2400, 4800, 9600, 19200, 38400, 115200}

+#define CONFIG_CONSOLE         ttyS0
 #define CONFIG_BAUDRATE                115200

 #define CFG_NS16550_COM1       (CFG_IMMR + 0x4500)
@@ -670,9 +679,10 @@ boards, we say we have two, but don't di
        " ip=" MK_STR(CONFIG_IPADDR) ":" MK_STR(CONFIG_SERVERIP) ":"    \
                MK_STR(CONFIG_GATEWAYIP) ":" MK_STR(CONFIG_NETMASK) ":" \
                MK_STR(CONFIG_HOSTNAME) ":" MK_STR(CONFIG_NETDEV) ":off" \
-       " console=ttyS0," MK_STR(CONFIG_BAUDRATE)
+       " console=" MK_STR(CONFIG_CONSOLE) "," MK_STR(CONFIG_BAUDRATE)

 #define CONFIG_EXTRA_ENV_SETTINGS \
+       "console=" MK_STR(CONFIG_CONSOLE) "\0"                          \
        "netdev=" MK_STR(CONFIG_NETDEV) "\0"                            \
        "uboot=" MK_STR(CONFIG_UBOOTPATH) "\0"                          \
        "tftpflash=tftpboot $loadaddr $uboot; "                         \
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://lists.denx.de/pipermail/u-boot/attachments/20070523/41eb9b8c/attachment.pgp 

[U-Boot-Users] [PATCH] add missing 'console' var to default mpc8349itx config

[Timur Tabi]

Nikita V. Youshchenko wrote:
> Default environment defined inside include/configs/MPC8349ITX.h file,
> references 'console' variable, but does not define it.
> 
> This patch fixes this.
> 
> Also it adds usage of new CONFIG_CONSOLE macro in default bootargs,
> for consistency.
> 
> Signed-Off-By: Nikita Youshchenko <yoush@debian.org>

Acked-by: Timur Tabi <timur@freescale.com>

Kim, please apply.

-- 
Timur Tabi
Linux Kernel Developer @ Freescale

[U-Boot-Users] [PATCH] add missing 'console' var to default mpc8349itx config

[Kim Phillips]

On Wed, 23 May 2007 12:45:25 +0400
"Nikita V. Youshchenko" <yoush@debian.org> wrote:

> Default environment defined inside include/configs/MPC8349ITX.h file,
> references 'console' variable, but does not define it.
> 
> This patch fixes this.
> 
> Also it adds usage of new CONFIG_CONSOLE macro in default bootargs,
> for consistency.
> 
> Signed-Off-By: Nikita Youshchenko <yoush@debian.org>
> 

applied, thanks.

please don't send patches in nested multipart messages in the future.

Kim