public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Nikita V. Youshchenko <yoush@debian.org>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [PATCH] add missing 'console' var to default mpc8349itx config
Date: Wed, 23 May 2007 12:45:25 +0400	[thread overview]
Message-ID: <200705231245.25995@sercond.localdomain> (raw)

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 

             reply	other threads:[~2007-05-23  8:45 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-23  8:45 Nikita V. Youshchenko [this message]
2007-05-23 15:05 ` [U-Boot-Users] [PATCH] add missing 'console' var to default mpc8349itx config Timur Tabi
2007-06-01 20:18 ` Kim Phillips

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200705231245.25995@sercond.localdomain \
    --to=yoush@debian.org \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox