Re: [U-Boot] [PATCH 2/3] mx23evk: Remove CONFIG_SYS_BAUDRATE_TABLE

2013-02-12 Thread Stefano Babic 
On 28/01/2013 12:41, Fabio Estevam wrote:
 The baudrate is already defined by CONFIG_BAUDRATE and there is no need
 to keep CONFIG_SYS_BAUDRATE_TABLE.
 
 Signed-off-by: Fabio Estevam fabio.este...@freescale.com
 ---

Applied to u-boot-imx, thanks.

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 2/3] mx23evk: Remove CONFIG_SYS_BAUDRATE_TABLE

2013-01-28 Thread Fabio Estevam 
The baudrate is already defined by CONFIG_BAUDRATE and there is no need
to keep CONFIG_SYS_BAUDRATE_TABLE.

Signed-off-by: Fabio Estevam fabio.este...@freescale.com
---
 include/configs/mx23evk.h |1 -
 1 file changed, 1 deletion(-)

diff --git a/include/configs/mx23evk.h b/include/configs/mx23evk.h
index 51aa9c4..e43b93f 100644
--- a/include/configs/mx23evk.h
+++ b/include/configs/mx23evk.h
@@ -110,7 +110,6 @@
 #define CONFIG_PL01x_PORTS { (void *)MXS_UARTDBG_BASE }
 #define CONFIG_CONS_INDEX  0
 #define CONFIG_BAUDRATE115200  /* Default baud rate */
-#define CONFIG_SYS_BAUDRATE_TABLE  { 9600, 19200, 38400, 57600, 115200 }
 
 /* DMA */
 #define CONFIG_APBH_DMA
-- 
1.7.9.5


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/3] mx23evk: Remove CONFIG_SYS_BAUDRATE_TABLE

2013-01-28 Thread Otavio Salvador 
On Mon, Jan 28, 2013 at 9:41 AM, Fabio Estevam
fabio.este...@freescale.com wrote:
 The baudrate is already defined by CONFIG_BAUDRATE and there is no need
 to keep CONFIG_SYS_BAUDRATE_TABLE.

 Signed-off-by: Fabio Estevam fabio.este...@freescale.com

Acked-by: Otavio Salvador ota...@ossystems.com.br

--
Otavio Salvador O.S. Systems
E-mail: ota...@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854  http://projetos.ossystems.com.br
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/3] mx23evk: Remove CONFIG_SYS_BAUDRATE_TABLE

2013-01-28 Thread Marek Vasut 
Dear Fabio Estevam,

 The baudrate is already defined by CONFIG_BAUDRATE and there is no need
 to keep CONFIG_SYS_BAUDRATE_TABLE.
 
 Signed-off-by: Fabio Estevam fabio.este...@freescale.com
 ---
  include/configs/mx23evk.h |1 -
  1 file changed, 1 deletion(-)
 
 diff --git a/include/configs/mx23evk.h b/include/configs/mx23evk.h
 index 51aa9c4..e43b93f 100644
 --- a/include/configs/mx23evk.h
 +++ b/include/configs/mx23evk.h
 @@ -110,7 +110,6 @@
  #define CONFIG_PL01x_PORTS   { (void *)MXS_UARTDBG_BASE }
  #define CONFIG_CONS_INDEX0
  #define CONFIG_BAUDRATE  115200  /* Default baud rate */
 -#define CONFIG_SYS_BAUDRATE_TABLE{ 9600, 19200, 38400, 57600, 115200 }

This will not allow different baudrates now, will it?

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/3] mx23evk: Remove CONFIG_SYS_BAUDRATE_TABLE

2013-01-28 Thread Fabio Estevam 
On Mon, Jan 28, 2013 at 10:46 AM, Marek Vasut ma...@denx.de wrote:

 This will not allow different baudrates now, will it?

You still may change the baudrate if you need to.

CONFIG_SYS_BAUDRATE_TABLE has been placed to a common place after the
commit below:

commit 26750c8aee2383a026e0cf89e9310628d3a5a6a0
Author: Tom Rini tr...@ti.com
Date:   Tue Jun 19 12:54:34 2012 +

CONFIG_SYS_BAUDRATE_TABLE: Add config_fallbacks.h, place there

We provide a default table of { 9600, 19200, 38400, 57600, 115200 }
in config_fallbacks.h which mkconfig places after configs/...h in
the generated config file.  This is used when a board has not set its
own table.

Signed-off-by: Tom Rini tr...@ti.com
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/3] mx23evk: Remove CONFIG_SYS_BAUDRATE_TABLE

2013-01-28 Thread Marek Vasut 
Dear Fabio Estevam,

 On Mon, Jan 28, 2013 at 10:46 AM, Marek Vasut ma...@denx.de wrote:
  This will not allow different baudrates now, will it?
 
 You still may change the baudrate if you need to.
 
 CONFIG_SYS_BAUDRATE_TABLE has been placed to a common place after the
 commit below:
 
 commit 26750c8aee2383a026e0cf89e9310628d3a5a6a0
 Author: Tom Rini tr...@ti.com
 Date:   Tue Jun 19 12:54:34 2012 +
 
 CONFIG_SYS_BAUDRATE_TABLE: Add config_fallbacks.h, place there
 
 We provide a default table of { 9600, 19200, 38400, 57600, 115200 }
 in config_fallbacks.h which mkconfig places after configs/...h in
 the generated config file.  This is used when a board has not set its
 own table.
 
 Signed-off-by: Tom Rini tr...@ti.com

Ok then I'd say -- kill 'em all!!

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot