From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Date: Sun, 07 Feb 2010 17:08:51 -0600 Subject: [U-Boot] [PATCH 3/3] edb93xx: enable the uart in devicecfg register In-Reply-To: <20100206222402.GC15617@darwin> References: <75fc423bb48364ffb440aaa0f6e72d09e8406d37.1265484838.git.rubini@unipv.it> <20100206222402.GC15617@darwin> Message-ID: <4B6F4803.6080201@windriver.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Matthias Kaehlcke wrote: > Hi Alessandro, > > El Sat, Feb 06, 2010 at 08:54:05PM +0100 Alessandro Rubini ha dit: > >> printf goes to uart1, but it will block forever waiting for >> busy to go off unless the uart is enabled first. >> >> Signed-off-by: Alessandro Rubini >> --- >> board/edb93xx/edb93xx.c | 6 ++++++ >> 1 files changed, 6 insertions(+), 0 deletions(-) >> >> diff --git a/board/edb93xx/edb93xx.c b/board/edb93xx/edb93xx.c >> index 4df2246..dde30ff 100644 >> --- a/board/edb93xx/edb93xx.c >> +++ b/board/edb93xx/edb93xx.c >> @@ -64,6 +64,12 @@ int board_init(void) >> value |= SYSCON_PWRCNT_UART_BAUD; >> writel(value, &syscon->pwrcnt); >> >> + /* Enable the uart in devicecfg */ >> + value = readl(&syscon->devicecfg); >> + value |= 1<<18 /* U1EN */; > > using a constant like DEVCFG_U1EN would be preferrable, as the patch > is correct at the functional and coding style level i'll leave it to > Tom to decide if we fix this now or later > It is ok to fix this later. The patch set have been applied. Thanks Tom