[U-Boot] [PATCH 3/4] lpc32xx: devkit3250: update of board configuration
Vladimir Zapolskiy
vz at mleia.com
Thu Jul 16 14:05:25 CEST 2015
Hello Albert,
On 16.07.2015 10:27, Albert ARIBAUD wrote:
> Hello Vladimir,
>
> On Thu, 16 Jul 2015 02:33:47 +0300, Vladimir Zapolskiy <vz at mleia.com>
> wrote:
>> This change adds more peripherals to Timll DevKit3250 board, namely
>> MAC and SMSC phy, SLC NAND, GPIO, SPI and I2C.
>>
>> Also the default serial console is changed to UART5, added an option
>> to pass device tree blob by means of bootm, predefined environment
>> variables are slightly extended and reserved space on NAND to store
>> user defined U-boot environment.
>>
>> Signed-off-by: Vladimir Zapolskiy <vz at mleia.com>
>> ---
>> board/timll/devkit3250/devkit3250.c | 31 ++++++++++++-
>> configs/devkit3250_defconfig | 3 ++
>> include/configs/devkit3250.h | 92 +++++++++++++++++++++++++++++++++++--
>> 3 files changed, 120 insertions(+), 6 deletions(-)
>>
>> diff --git a/board/timll/devkit3250/devkit3250.c b/board/timll/devkit3250/devkit3250.c
>> index 6acc416..4b3c94e 100644
>> --- a/board/timll/devkit3250/devkit3250.c
>> +++ b/board/timll/devkit3250/devkit3250.c
>> @@ -1,23 +1,52 @@
>> /*
>> * Embest/Timll DevKit3250 board support
>> *
>> - * Copyright (C) 2011 Vladimir Zapolskiy <vz at mleia.com>
>> + * Copyright (C) 2011-2015 Vladimir Zapolskiy <vz at mleia.com>
>> *
>> * SPDX-License-Identifier: GPL-2.0+
>> */
>>
>> #include <common.h>
>> #include <asm/arch/sys_proto.h>
>> +#include <asm/arch/clk.h>
>> #include <asm/arch/cpu.h>
>> #include <asm/arch/emc.h>
>> +#include <asm/arch/wdt.h>
>> +#include <asm/io.h>
>>
>> DECLARE_GLOBAL_DATA_PTR;
>>
>> static struct emc_regs *emc = (struct emc_regs *)EMC_BASE;
>> +static struct clk_pm_regs *clk = (struct clk_pm_regs *)CLK_PM_BASE;
>> +static struct wdt_regs *wdt = (struct wdt_regs *)WDT_BASE;
>> +
>> +void reset_periph(void)
>> +{
>> + /* This function resets peripherals by triggering RESOUT_N */
>> + setbits_le32(&clk->timclk_ctrl, CLK_TIMCLK_WATCHDOG);
>> + writel(WDTIM_MCTRL_RESFRC1, &wdt->mctrl);
>> + udelay(300);
>> +
>> + writel(0, &wdt->mctrl);
>> + clrbits_le32(&clk->timclk_ctrl, CLK_TIMCLK_WATCHDOG);
>> +
>> + /* Such a long delay is needed to initialize SMSC phy */
>> + udelay(10000);
>> +}
>>
>> int board_early_init_f(void)
>> {
>> lpc32xx_uart_init(CONFIG_SYS_LPC32XX_UART);
>> + lpc32xx_i2c_init(1);
>> + lpc32xx_i2c_init(2);
>> + lpc32xx_ssp_init();
>> + lpc32xx_mac_init();
>> +
>> + /*
>> + * nWP may be controlled by GPO19, but unpopulated by default R23
>> + * makes no sense to configure this GPIO level, nWP is always high
>> + */
>> + lpc32xx_slc_nand_init();
>>
>> return 0;
>> }
>> diff --git a/configs/devkit3250_defconfig b/configs/devkit3250_defconfig
>> index f0c4ee1..56d719f 100644
>> --- a/configs/devkit3250_defconfig
>> +++ b/configs/devkit3250_defconfig
>> @@ -1,3 +1,6 @@
>> CONFIG_ARM=y
>> CONFIG_TARGET_DEVKIT3250=y
>> +# CONFIG_CMD_FPGA is not set
>> # CONFIG_CMD_SETEXPR is not set
>> +CONFIG_DM=y
>> +CONFIG_DM_GPIO=y
>> diff --git a/include/configs/devkit3250.h b/include/configs/devkit3250.h
>> index 4f35234..b8218b5 100644
>> --- a/include/configs/devkit3250.h
>> +++ b/include/configs/devkit3250.h
>> @@ -1,7 +1,7 @@
>> /*
>> * Embest/Timll DevKit3250 board configuration file
>> *
>> - * Copyright (C) 2011 Vladimir Zapolskiy <vz at mleia.com>
>> + * Copyright (C) 2011-2015 Vladimir Zapolskiy <vz at mleia.com>
>> *
>> * SPDX-License-Identifier: GPL-2.0+
>> */
>> @@ -43,10 +43,44 @@
>> /*
>> * Serial Driver
>> */
>> -#define CONFIG_SYS_LPC32XX_UART 2 /* UART2 */
>> +#define CONFIG_SYS_LPC32XX_UART 5 /* UART5 */
>
> This will affect existing users of this board, who expect it to use
> UART2 as console. Was this expectation wrong, or can both UART2 and
> UART5 be used as console? IOW, is this change required or is it just
> easier for you secifically?
The board has 8 UARTs (provided by LPC32xx and SC16IS752IPW), 4 of them
can be used as a console interchangeably, namely LPC32xx UART2, UART3,
UART4 and UART5.
Previously UART2 was selected as a default UART, because it is a faster
14-clock UART, this plays a role if e.g. an image is downloaded over
ymodem. Now with MAC support in Uboot this is not needed, and preferably
to use UART5, which is mapped as a default serial console in Linux.
>> #define CONFIG_BAUDRATE 115200
>>
>> /*
>> + * I2C
>> + */
>> +#define CONFIG_SYS_I2C
>> +#define CONFIG_SYS_I2C_LPC32XX
>> +#define CONFIG_SYS_I2C_SPEED 100000
>> +#define CONFIG_CMD_I2C
>> +
>> +/*
>> + * GPIO
>> + */
>> +#define CONFIG_LPC32XX_GPIO
>> +#define CONFIG_CMD_GPIO
>> +
>> +/*
>> + * SSP/SPI
>> + */
>> +#define CONFIG_LPC32XX_SSP
>> +#define CONFIG_LPC32XX_SSP_TIMEOUT 100000
>> +#define CONFIG_CMD_SPI
>> +
>> +/*
>> + * Ethernet
>> + */
>> +#define CONFIG_RMII
>> +#define CONFIG_PHY_SMSC
>> +#define CONFIG_LPC32XX_ETH
>> +#define CONFIG_PHYLIB
>> +#define CONFIG_PHY_ADDR 0x1F
>> +#define CONFIG_SYS_FAULT_ECHO_LINK_DOWN
>> +#define CONFIG_CMD_MII
>> +#define CONFIG_CMD_PING
>> +#define CONFIG_CMD_DHCP
>> +
>> +/*
>> * NOR Flash
>> */
>> #define CONFIG_SYS_MAX_FLASH_BANKS 1
>> @@ -56,6 +90,29 @@
>> #define CONFIG_SYS_FLASH_CFI
>>
>> /*
>> + * NAND controller
>> + */
>> +#define CONFIG_NAND_LPC32XX_SLC
>> +#define CONFIG_SYS_NAND_BASE SLC_NAND_BASE
>> +#define CONFIG_SYS_MAX_NAND_DEVICE 1
>> +#define CONFIG_SYS_NAND_BASE_LIST { CONFIG_SYS_NAND_BASE }
>> +
>> +/*
>> + * NAND chip timings
>> + */
>> +#define CONFIG_LPC32XX_NAND_SLC_WDR_CLKS 14
>> +#define CONFIG_LPC32XX_NAND_SLC_WWIDTH 66666666
>> +#define CONFIG_LPC32XX_NAND_SLC_WHOLD 200000000
>> +#define CONFIG_LPC32XX_NAND_SLC_WSETUP 50000000
>> +#define CONFIG_LPC32XX_NAND_SLC_RDR_CLKS 14
>> +#define CONFIG_LPC32XX_NAND_SLC_RWIDTH 66666666
>> +#define CONFIG_LPC32XX_NAND_SLC_RHOLD 200000000
>> +#define CONFIG_LPC32XX_NAND_SLC_RSETUP 50000000
>> +
>> +#define CONFIG_SYS_NAND_USE_FLASH_BBT
>> +#define CONFIG_CMD_NAND
>> +
>> +/*
>> * U-Boot General Configurations
>> */
>> #define CONFIG_SYS_LONGHELP
>> @@ -71,8 +128,33 @@
>> #define CONFIG_DISPLAY_CPUINFO
>> #define CONFIG_DOS_PARTITION
>>
>> -#define CONFIG_ENV_IS_NOWHERE
>> +/*
>> + * Pass open firmware flat tree
>> + */
>> +#define CONFIG_OF_LIBFDT
>> +
>> +/*
>> + * Environment
>> + */
>> +#define CONFIG_ENV_IS_IN_NAND 1
>> #define CONFIG_ENV_SIZE SZ_128K
>> +#define CONFIG_ENV_OFFSET 0x000A0000
>
> This in itself is an unanticipated change too, but I think it does not
> affect current users of the boards much, except for a warning at boot
> that the (NAND) env is invalid -- but the env does not contain much
> anyway.
Correct, new users will get a warning until their partition layout is
updated.
> But how does this affect potential mapping of the NAND? Do Linux images
> hard-code partitions on the NAND and is the address and size chosen
> compatible with that, or do the images just not care about NAND?
The default manufacturer's NAND layout is slightly changed, but kernel
and rootfs partitions are untouched, for users who care, kernel will be
updated to reflect this change.
Previously only S1L partition occupied pages 0x00000 - 0xc0000, new
layout is:
0x00000 - 0x40000 - S1L / SPL
0x40000 - 0xa0000 - U-boot
0xa0000 - 0xc0000 - U-boot environment
>> +#define CONFIG_BOOTCOMMAND \
>> + "dhcp; " \
>> + "tftp ${loadaddr} ${serverip}:${tftpdir}/${bootfile}; " \
>> + "tftp ${dtbaddr} ${serverip}:${tftpdir}/devkit3250.dtb; " \
>> + "setenv nfsargs ip=dhcp root=/dev/nfs nfsroot=${serverip}:${nfsroot},tcp; " \
>> + "setenv bootargs ${bootargs} ${nfsargs} ${userargs}; " \
>> + "bootm ${loadaddr} - ${dtbaddr}"
>> +
>> +#define CONFIG_EXTRA_ENV_SETTINGS \
>> + "autoload=no\0" \
>> + "ethaddr=00:01:90:00:C0:81\0" \
>> + "dtbaddr=0x81000000\0" \
>> + "nfsroot=/opt/projects/images/vladimir/oe/devkit3250/rootfs\0" \
>> + "tftpdir=vladimir/oe/devkit3250\0" \
>> + "userargs=oops=panic\0"
>>
>> /*
>> * U-Boot Commands
>> @@ -85,10 +167,10 @@
>> #define CONFIG_CMDLINE_TAG
>> #define CONFIG_SETUP_MEMORY_TAGS
>> #define CONFIG_ZERO_BOOTDELAY_CHECK
>> -#define CONFIG_BOOTDELAY 3
>> +#define CONFIG_BOOTDELAY 1
>>
>> #define CONFIG_BOOTFILE "uImage"
>> -#define CONFIG_BOOTARGS "console=ttyS2,115200n8"
>> +#define CONFIG_BOOTARGS "console=ttyS0,115200n8"
>> #define CONFIG_LOADADDR 0x80008000
>>
>> /*
>> --
>> 2.1.4
>>
--
With best wishes,
Vladimir
More information about the U-Boot
mailing list