July 2016 - U-Boot

Re: [U-Boot] (no subject)
by Joe Hershberger 12 Jul '16

12 Jul '16

Hi , https://patchwork.ozlabs.org/patch/2307ea4/ was applied to u-boot-net.git. Thanks! -Joe

1 0

[U-Boot] Avoiding overwriting FDT from FIT image
by Chris Packham 12 Jul '16

12 Jul '16

Hi, We have several boards making use of FIT images for the kernel/initrd/fdt. With v2015.04 we can successfully boot our application image. With v2016.05 we run into a problem where the FDT is overwritten when the ramdisk is relocated. Here's a snippet of boot output that illustrates the problem v2015.04 ------------- Booting image 01000000#x930-52GPX ... WARNING: adjusting available memory to 30000000 ## Loading kernel from FIT Image at 01000000 ... ## Loading ramdisk from FIT Image at 01000000 ... ## Loading fdt from FIT Image at 01000000 ... Booting using the fdt blob at 0x34ab4d4 Uncompressing Kernel Image ... OK Loading Ramdisk to 2df6b000, end 30000000 ... OK Loading Device Tree to 03fe6000, end 03fff862 ... OK v2016.04 ------------- Booting image 01000000#x930-52GPX ... WARNING: adjusting available memory to 30000000 ## Loading kernel from FIT Image at 01000000 ... ## Loading ramdisk from FIT Image at 01000000 ... ## Loading fdt from FIT Image at 01000000 ... Booting using the fdt blob at 0x34ab4d4 Uncompressing Kernel Image ... OK Loading Ramdisk to 01f6a000, end 03fff000 ... OK ERROR: image is not a fdt - must RESET the board to recover. Error: -9 Reverting commit 20e072f37402 ("image: check "bootm_low" and "bootm_size" if "initrd_high" is missing") avoids the problem as does setting initrd_high to 0xfffffffff. But I'm wondering if a better fix would be to tell boot_ramdisk_high() to avoid overwriting anything in the range $loadaddr -- +$filesize. Any suggestion as to how to go about this?

2 1

[U-Boot] [U-boot] Add Memset and Memcpy support to standalone Application
by Amit Mahadik 12 Jul '16

12 Jul '16

Hi, I have a Cortex-A9 processor based development board with me. I have run the standalone Hello World Application successfully. I want to add support for memcpy and memset functions so I have modified the hello_world code and written basic implementations of memcpy and memset functions. The code gets compiled and I can see the mem functions code via objdump into the hello_world elf. But when I try to run the code on the board it hangs after the following message. Please let me know what I am missing. Even The Basic print statements are not printed. I have attached the modified hello world code. /***********************************************************/ Starting application at 0x0C100000 ... /***********************************************************/ Regards, Amit.

1 0

[U-Boot] Support for GPIO in U-boot for Intel Edison
by SANRIO ALVARES 12 Jul '16

12 Jul '16

Hi, Is there any support for GPIO in U-boot for Intel Edison Boards? Thanks salvares

2 1

[U-Boot] [PATCH 1/3] COSMETIC: mmc: sdhci: Add CONFIG_ prefix to SDHCI_READ_STATUS_TIMEOUT
by Lukasz Majewski 12 Jul '16

12 Jul '16

This change gives common prefix for SDHCI_READ_STATUS_TIMEOUT. Signed-off-by: Lukasz Majewski <l.majewski(a)samsung.com> --- drivers/mmc/sdhci.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c index 604f18d..aa4cd4f 100644 --- a/drivers/mmc/sdhci.c +++ b/drivers/mmc/sdhci.c @@ -127,7 +127,7 @@ static int sdhci_transfer_data(struct sdhci_host *host, struct mmc_data *data, #define CONFIG_SDHCI_CMD_MAX_TIMEOUT 3200 #endif #define CONFIG_SDHCI_CMD_DEFAULT_TIMEOUT 100 -#define SDHCI_READ_STATUS_TIMEOUT 1000 +#define CONFIG_SDHCI_READ_STATUS_TIMEOUT 1000 static int sdhci_send_command(struct mmc *mmc, struct mmc_cmd *cmd, struct mmc_data *data) @@ -244,9 +244,9 @@ static int sdhci_send_command(struct mmc *mmc, struct mmc_cmd *cmd, if (stat & SDHCI_INT_ERROR) break; } while (((stat & mask) != mask) && - (get_timer(start) < SDHCI_READ_STATUS_TIMEOUT)); + (get_timer(start) < CONFIG_SDHCI_READ_STATUS_TIMEOUT)); - if (get_timer(start) >= SDHCI_READ_STATUS_TIMEOUT) { + if (get_timer(start) >= CONFIG_SDHCI_READ_STATUS_TIMEOUT) { if (host->quirks & SDHCI_QUIRK_BROKEN_R1B) return 0; else { -- 2.0.0.rc2

3 5

[U-Boot] [PATCH] storage device support for ub_dev_write
by Ihar Filipau 12 Jul '16

12 Jul '16

Hi Simon, > Can you please resend this inline with patman / git send-email, rather > than as an attachment? I'm behind a corporate firewall - I can't SMTP/IMAP - on which git send-email relies. (And I'm sure you do not want me to send patches with Outlook.) At most, without attachments, I can post in-line what patman wanted to send: P.S. But here a pastebin - http://pastebin.com/JirJw9NX - just in case. P.P.S. Do not blame me for the formatting, inconsistent with the rest of the file: patman has insisted. >From 8d05ccdea2fa533d6427acfe5b78300984344d5e Mon Sep 17 00:00:00 2001 From: Ihar Filipau <ihar.filipau(a)cadcon.de> Date: Tue, 12 Jul 2016 13:27:02 +0200 Subject: [PATCH] storage device support for ub_dev_write I have stumbled upon the lack of support for storage devices in API's ub_dev_write()/API_dev_write() functions. Currently the function supports only the network devices. I have implemented the support for the storage by adapting the code from API_dev_read() function, and added the ub_dev_write() to the glue.c. Interface for the network devices is unchanged. Signed-off-by: Ihar Filipau <ihar.filipau(a)cadcon.de> --- api/api.c | 90 +++++++++++++++++++++++++++++++++++------------------ api/api_private.h | 1 + api/api_storage.c | 21 +++++++++++++ examples/api/glue.c | 20 ++++++++++++ 4 files changed, 102 insertions(+), 30 deletions(-) diff --git a/api/api.c b/api/api.c index 8a1433a..2ce5d07 100644 --- a/api/api.c +++ b/api/api.c @@ -295,28 +295,44 @@ static int API_dev_close(va_list ap) /* - * Notice: this is for sending network packets only, as U-Boot does not - * support writing to storage at the moment (12.2007) + * pseudo signature, storage: * - * pseudo signature: + * int API_dev_write( + * struct device_info *di, + * void *buf, + * lbasize_t *len, + * lbastart_t *start, + * lbasize_t *act_len + * ) * - * int API_dev_write( - * struct device_info *di, - * void *buf, - * int *len - * ) + * pseudo signature, net: + * + * int API_dev_write( + * struct device_info *di, + * void *buf, + * int *len + * ) + * + * buf: ptr to buffer with data to write * - * buf: ptr to buffer from where to get the data to send + * len: length to be written + * - network: len of packet to write (in bytes) + * - storage: # of blocks to write (can vary in size depending on define) * - * len: length of packet to be sent (in bytes) + * start: start block (only used for storage devices, ignored for + * network) + * + * act_len: ptr to where to put the len actually written * */ static int API_dev_write(va_list ap) { struct device_info *di; void *buf; - int *len; - int err = 0; + lbasize_t *len_stor, *act_len_stor; + lbastart_t *start; + int *len_net; + int ret = 0; /* 1. arg is ptr to the device_info struct */ di = (struct device_info *)va_arg(ap, uintptr_t); @@ -328,31 +344,45 @@ static int API_dev_write(va_list ap) if (di->cookie == NULL) return API_ENODEV; - /* 2. arg is ptr to buffer from where to get data to write */ + /* 2. arg is ptr to buffer from where to put the read data */ buf = (void *)va_arg(ap, uintptr_t); if (buf == NULL) return API_EINVAL; - /* 3. arg is length of buffer */ - len = (int *)va_arg(ap, uintptr_t); - if (len == NULL) - return API_EINVAL; - if (*len <= 0) - return API_EINVAL; + if (di->type & DEV_TYP_STOR) { + /* 3. arg - ptr to var with # of blocks to read */ + len_stor = (lbasize_t *)va_arg(ap, uintptr_t); + if (!len_stor) + return API_EINVAL; + if (*len_stor <= 0) + return API_EINVAL; - if (di->type & DEV_TYP_STOR) - /* - * write to storage is currently not supported by U-Boot: - * no storage device implements block_write() method - */ - return API_ENODEV; + /* 4. arg - ptr to var with start block */ + start = (lbastart_t *)va_arg(ap, uintptr_t); - else if (di->type & DEV_TYP_NET) - err = dev_write_net(di->cookie, buf, *len); - else - err = API_ENODEV; + /* 5. arg - ptr to var where to put the len actually read */ + act_len_stor = (lbasize_t *)va_arg(ap, uintptr_t); + if (!act_len_stor) + return API_EINVAL; - return err; + *act_len_stor = dev_write_stor(di->cookie, buf, *len_stor, + *start); + + } else if (di->type & DEV_TYP_NET) { + /* 3. arg points to the var with length of packet to read */ + len_net = (int *)va_arg(ap, uintptr_t); + if (!len_net) + return API_EINVAL; + if (*len_net <= 0) + return API_EINVAL; + + ret = dev_write_net(di->cookie, buf, *len_net); + + } else { + return API_ENODEV; + } + + return ret; } diff --git a/api/api_private.h b/api/api_private.h index a8866ef..86d45bc 100644 --- a/api/api_private.h +++ b/api/api_private.h @@ -23,6 +23,7 @@ int dev_close_stor(void *); int dev_close_net(void *); lbasize_t dev_read_stor(void *, void *, lbasize_t, lbastart_t); +lbasize_t dev_write_stor(void *, void *, lbasize_t, lbastart_t); int dev_read_net(void *, void *, int); int dev_write_net(void *, void *, int); diff --git a/api/api_storage.c b/api/api_storage.c index d425a9a..efb23c1 100644 --- a/api/api_storage.c +++ b/api/api_storage.c @@ -365,3 +365,24 @@ lbasize_t dev_read_stor(void *cookie, void *buf, lbasize_t len, lbastart_t start return dd->block_read(dd, start, len, buf); } + +lbasize_t dev_write_stor(void *cookie, void *buf, lbasize_t len, + lbastart_t start) +{ + int type; + block_dev_desc_t *dd = (block_dev_desc_t *)cookie; + + type = dev_stor_type(dd); + if (type == ENUM_MAX) + return 0; + + if (!dev_stor_is_valid(type, dd)) + return 0; + + if ((dd->block_write) == NULL) { + debugf("no block_write() for device 0x%08x\n", cookie); + return 0; + } + + return dd->block_write(dev_stor_index(dd), start, len, buf); +} diff --git a/examples/api/glue.c b/examples/api/glue.c index 8aabf32..1a1b69b 100644 --- a/examples/api/glue.c +++ b/examples/api/glue.c @@ -290,6 +290,26 @@ int ub_dev_read(int handle, void *buf, lbasize_t len, lbastart_t start, return err; } +int ub_dev_write(int handle, void *buf, lbasize_t len, lbastart_t start, + lbasize_t *rlen) +{ + struct device_info *di; + lbasize_t act_len; + int err = 0; + + if (!dev_stor_valid(handle)) + return API_ENODEV; + + di = &devices[handle]; + if (!syscall(API_DEV_WRITE, &err, di, buf, &len, &start, &act_len)) + return API_ESYSC; + + if (!err && rlen) + *rlen = act_len; + + return err; +} + static int dev_net_valid(int handle) { if (!dev_valid(handle)) -- 1.9.1

1 0

[U-Boot] EL3 to EL1 for u-boot-2016.05
by wilson002 12 Jul '16

12 Jul '16

The linux kernel runs under EL1 model for MMU(msr ttbr0_el1, x25/msr ttbr1_el1, x26). But uboot woks at EL3. How to switch from EL3 to EL1 inside UBOOT for linux kernel loading? -- View this message in context: http://u-boot.10912.n7.nabble.com/EL3-to-EL1-for-u-boot-2016-05-tp260647.ht… Sent from the U-Boot mailing list archive at Nabble.com.

1 0

[U-Boot] [PATCH] mmc: exynos_dw_mmc: support the Driver mode for Exynos
by Jaehoon Chung 12 Jul '16

12 Jul '16

This patch support the driver mode for exynos dwmmc controller. To support the legacy model, maintained the existing code. Signed-off-by: Jaehoon Chung <jh80.chung(a)samsung.com> --- drivers/mmc/exynos_dw_mmc.c | 72 +++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 72 insertions(+) diff --git a/drivers/mmc/exynos_dw_mmc.c b/drivers/mmc/exynos_dw_mmc.c index ab0df46..283befc 100644 --- a/drivers/mmc/exynos_dw_mmc.c +++ b/drivers/mmc/exynos_dw_mmc.c @@ -23,8 +23,21 @@ #define DWMMC_MMC0_SDR_TIMING_VAL 0x03030001 #define DWMMC_MMC2_SDR_TIMING_VAL 0x03020001 +#ifdef CONFIG_DM_MMC +#include <dm.h> +DECLARE_GLOBAL_DATA_PTR; + +struct exynos_mmc_plat { + struct mmc_config cfg; + struct mmc mmc; +}; +#endif + /* Exynos implmentation specific drver private data */ struct dwmci_exynos_priv_data { +#ifdef CONFIG_DM_MMC + struct dwmci_host host; +#endif u32 sdr_timing; }; @@ -105,11 +118,15 @@ static int exynos_dwmci_core_init(struct dwmci_host *host) host->caps = MMC_MODE_DDR_52MHz; host->clksel = exynos_dwmci_clksel; host->get_mmc_clk = exynos_dwmci_get_clk; + +#ifndef CONFIG_DM_MMC /* Add the mmc channel to be registered with mmc core */ if (add_dwmci(host, DWMMC_MAX_FREQ, DWMMC_MIN_FREQ)) { printf("DWMMC%d registration failed\n", host->dev_index); return -1; } +#endif + return 0; } @@ -237,3 +254,58 @@ int exynos_dwmmc_init(const void *blob) return err; } + +#ifdef CONFIG_DM_MMC +static int exynos_dwmmc_probe(struct udevice *dev) +{ + struct exynos_mmc_plat *plat = dev_get_platdata(dev); + struct mmc_uclass_priv *upriv = dev_get_uclass_priv(dev); + struct dwmci_exynos_priv_data *priv = dev_get_priv(dev); + struct dwmci_host *host = &priv->host; + int err; + + err = exynos_dwmci_get_config(gd->fdt_blob, dev->of_offset, host); + if (err) + return err; + err = do_dwmci_init(host); + if (err) + return err; + + dwmci_setup_cfg(&plat->cfg, host->name, host->buswidth, host->caps, + DWMMC_MAX_FREQ, DWMMC_MIN_FREQ); + host->mmc = &plat->mmc; + host->mmc->priv = &priv->host; + host->priv = dev; + upriv->mmc = host->mmc; + + return dwmci_probe(dev); +} + +static int exynos_dwmmc_bind(struct udevice *dev) +{ + struct exynos_mmc_plat *plat = dev_get_platdata(dev); + int ret; + + ret = dwmci_bind(dev, &plat->mmc, &plat->cfg); + if (ret) + return ret; + + return 0; +} + +static const struct udevice_id exynos_dwmmc_ids[] = { + { .compatible = "samsung,exynos4412-dw-mshc" }, + { } +}; + +U_BOOT_DRIVER(exynos_dwmmc_drv) = { + .name = "exynos_dwmmc", + .id = UCLASS_MMC, + .of_match = exynos_dwmmc_ids, + .bind = exynos_dwmmc_bind, + .ops = &dm_dwmci_ops, + .probe = exynos_dwmmc_probe, + .priv_auto_alloc_size = sizeof(struct dwmci_exynos_priv_data), + .platdata_auto_alloc_size = sizeof(struct exynos_mmc_plat), +}; +#endif -- 1.9.1

3 2

[U-Boot] [PATCH 0/5] mmc: exynos_dw_mmc: clean the unnecessary codes
by Jaehoon Chung 12 Jul '16

12 Jul '16

This patch-set is for maintaining more easier than now. Now, exynos_dw_mmc codes are too huge. Some codes can be reused and removed. Basic goal is the using DM. Before applied DM, it needs to clean more. In future, exynos_dw_mmc controller should support DM. Jaehoon Chung (5): mmc: exynos_dw_mmc: remove the unused function mmc: exynos_dw_mmc: remove #ifdef for OF_CONTROL mmc: exynos_dw_mmc: add the error control for checking index mmc: exynos_dw_mmc: clean the unused and unnecessary codes mmc: exynos_dw_mmc: use the 4bit bus-width by default arch/arm/mach-exynos/include/mach/dwmmc.h | 1 - drivers/mmc/exynos_dw_mmc.c | 78 ++++++------------------------- 2 files changed, 14 insertions(+), 65 deletions(-) -- 1.9.1

3 11

[U-Boot] [PATCH] ARM: exynos4: dts: add the prefix '/' for aliases nodes
by Jaehoon Chung 12 Jul '16

12 Jul '16

It's correct to use '/' as prefix for aliases nodes. Signed-off-by: Jaehoon Chung <jh80.chung(a)samsung.com> --- arch/arm/dts/exynos4210-origen.dts | 2 +- arch/arm/dts/exynos4210-trats.dts | 4 ++-- arch/arm/dts/exynos4210-universal_c210.dts | 4 ++-- arch/arm/dts/exynos4412-odroid.dts | 4 ++-- arch/arm/dts/exynos4412-trats2.dts | 6 +++--- 5 files changed, 10 insertions(+), 10 deletions(-) diff --git a/arch/arm/dts/exynos4210-origen.dts b/arch/arm/dts/exynos4210-origen.dts index 3f87761..26c4d7f 100644 --- a/arch/arm/dts/exynos4210-origen.dts +++ b/arch/arm/dts/exynos4210-origen.dts @@ -22,7 +22,7 @@ aliases { serial0 = "/serial@13800000"; console = "/serial@13820000"; - mmc2 = "sdhci@12530000"; + mmc2 = "/sdhci@12530000"; }; sdhci@12510000 { diff --git a/arch/arm/dts/exynos4210-trats.dts b/arch/arm/dts/exynos4210-trats.dts index f3fac80..2ed38f3 100644 --- a/arch/arm/dts/exynos4210-trats.dts +++ b/arch/arm/dts/exynos4210-trats.dts @@ -29,8 +29,8 @@ i2c7 = "/i2c@138d0000"; serial0 = "/serial@13800000"; console = "/serial@13820000"; - mmc0 = "sdhci@12510000"; - mmc2 = "sdhci@12530000"; + mmc0 = "/sdhci@12510000"; + mmc2 = "/sdhci@12530000"; }; fimd@11c00000 { diff --git a/arch/arm/dts/exynos4210-universal_c210.dts b/arch/arm/dts/exynos4210-universal_c210.dts index ad3527e..8cac7dd 100644 --- a/arch/arm/dts/exynos4210-universal_c210.dts +++ b/arch/arm/dts/exynos4210-universal_c210.dts @@ -17,8 +17,8 @@ aliases { serial0 = "/serial@13800000"; console = "/serial@13820000"; - mmc0 = "sdhci@12510000"; - mmc2 = "sdhci@12530000"; + mmc0 = "/sdhci@12510000"; + mmc2 = "/sdhci@12530000"; }; sdhci@12510000 { diff --git a/arch/arm/dts/exynos4412-odroid.dts b/arch/arm/dts/exynos4412-odroid.dts index a63e8ab..188cb93 100644 --- a/arch/arm/dts/exynos4412-odroid.dts +++ b/arch/arm/dts/exynos4412-odroid.dts @@ -25,8 +25,8 @@ i2c7 = "/i2c@138d0000"; serial0 = "/serial@13800000"; console = "/serial@13810000"; - mmc2 = "sdhci@12530000"; - mmc4 = "dwmmc@12550000"; + mmc2 = "/sdhci@12530000"; + mmc4 = "/dwmmc@12550000"; }; i2c@13860000 { diff --git a/arch/arm/dts/exynos4412-trats2.dts b/arch/arm/dts/exynos4412-trats2.dts index 2d4e522..1fbcf89 100644 --- a/arch/arm/dts/exynos4412-trats2.dts +++ b/arch/arm/dts/exynos4412-trats2.dts @@ -29,9 +29,9 @@ i2c7 = "/i2c@138d0000"; serial0 = "/serial@13800000"; console = "/serial@13820000"; - mmc0 = "sdhci@12510000"; - mmc2 = "sdhci@12530000"; - mmc4 = "dwmmc@12550000"; + mmc0 = "/sdhci@12510000"; + mmc2 = "/sdhci@12530000"; + mshc0 = "/dwmmc@12550000"; }; i2c@138d0000 { -- 1.9.1

2 1