January 2016 - U-Boot

[U-Boot] [PATCH] mailmap: fix Ricardo's entry
by Masahiro Yamada 02 Feb '16

02 Feb '16

Commit df48b23428e9 (".mailmap: Add all the mail alias for Ricardo Ribalda") assigned two different proper names for the email address "ricardo.ribalda(a)uam.es". This is a completely wrong usage as the mailmap feature exists for coalescing together commits by the same person whose name is sometimes spelled differently. Signed-off-by: Masahiro Yamada <yamada.masahiro(a)socionext.com> --- .mailmap | 1 - 1 file changed, 1 deletion(-) diff --git a/.mailmap b/.mailmap index f72fef1..5ea9f93 100644 --- a/.mailmap +++ b/.mailmap @@ -21,7 +21,6 @@ Jagan Teki <jagannadha.sutradharudu-teki(a)xilinx.com> Markus Klotzbuecher <mk(a)denx.de> Prabhakar Kushwaha <prabhakar(a)freescale.com> Rajeshwari Shinde <rajeshwari.s(a)samsung.com> -Ricardo Ribalda Delgado <ricardo.ribalda(a)uam.es> Ricardo Ribalda <ricardo.ribalda(a)uam.es> Ricardo Ribalda <ricardo.ribalda(a)gmail.com> Sandeep Paulraj <s-paulraj(a)ti.com> -- 1.9.1

2 2

[U-Boot] [PATCH] mmc: add missing prototype for mmc_get_env_dev
by Clemens Gruber 02 Feb '16

02 Feb '16

This is a follow-up patch to e92029c0f4 and adds a prototype for the weak mmc_get_env_dev function. Cc: Tom Rini <trini(a)konsulko.com> Cc: Stephen Warren <swarren(a)nvidia.com> Signed-off-by: Clemens Gruber <clemens.gruber(a)pqgruber.com> --- include/mmc.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/mmc.h b/include/mmc.h index 465daeb..d652c14 100644 --- a/include/mmc.h +++ b/include/mmc.h @@ -479,6 +479,7 @@ void board_mmc_power_init(void); int board_mmc_init(bd_t *bis); int cpu_mmc_init(bd_t *bis); int mmc_get_env_addr(struct mmc *mmc, int copy, u32 *env_addr); +int mmc_get_env_dev(void); struct pci_device_id; -- 2.7.0

3 3

[U-Boot] [PATCH] malloc: work around some memalign fragmentation issues
by Stephen Warren 02 Feb '16

02 Feb '16

From: Stephen Warren <swarren(a)nvidia.com> Use of memalign can trigger fragmentation issues such as: // Internally, this needs to find a free block quite bit larger than s. // Once the free region is found, any unaligned "padding" immediately // before and after the block is marked free, so that the allocation // takes only s bytes (plus malloc header overhead). p = memalign(a, s); // If there's little fragmentation so far, this allocation is likely // located immediately after p. p2 = malloc(x); free(p); // In theory, this should return the same value for p. However, the hole // left by the free() call is only s in size (plus malloc header overhead) // whereas memalign searches for a larger block in order to guarantee it // can adjust the returned pointer to the alignment requirements. Hence, // the pointer returned, if any, won't be p. If there's little or no space // left after p2, this allocation will fail. p = memalign(a, s); In practice, this issue occurs when running the "dfu" command repeatedly on NVIDIA Tegra boards, since DFU allocates a large 32M data buffer, and then initializes the USB controller. If this is the first time USB has been used in the U-Boot session, this causes a probe of the USB driver, which causes various allocations, including a strdup() of a GPIO name when requesting the VBUS GPIO. When DFU is torn down, the USB driver is left probed, and hence its memory is left allocated. If "dfu" is executed again, allocation of the 32M data buffer fails as described above. In practice, there is a memory hole exactly large enough to hold the 32M data buffer than DFU needs. However, memalign() can't know that in a general way. Given that, it's particularly annoying that the allocation fails! The issue is that memalign() tries to allocate something larger to guarantee the ability to align the returned pointer. This patch modifies memalign() so that if the "general case" over-sized allocation fails, another allocation is attempted, of the exact size the user desired. If that allocation just happens to be aligned in the way the user wants, (and in the case described above, it will be, since the free memory region is located where a previous identical allocation was located), the pointer can be returned. This patch is somewhat related to 806bd245b1ab "dfu: don't keep freeing/reallocating". That patch worked around the issue by removing repeated free/memalign within a single execution of "dfu". However, the same technique can't be applied across multiple invocations, since there's no reason to keep the DFU buffer allocated while DFU isn't running. This patch addresses the root-cause a bit more directly. This problem highlights some of the disadvantages of dynamic allocation and deferred probing of devices. This patch isn't checkpatch-clean, since it conforms to the existing coding style in dlmalloc.c, which is different to the rest of U-Boot. Signed-off-by: Stephen Warren <swarren(a)nvidia.com> --- common/dlmalloc.c | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/common/dlmalloc.c b/common/dlmalloc.c index b5bb05191c24..2b964d16b11e 100644 --- a/common/dlmalloc.c +++ b/common/dlmalloc.c @@ -2829,6 +2829,28 @@ Void_t* mEMALIGn(alignment, bytes) size_t alignment; size_t bytes; nb = request2size(bytes); m = (char*)(mALLOc(nb + alignment + MINSIZE)); + /* + * The attempt to over-allocate (with a size large enough to guarantee the + * ability to find an aligned region within allocated memory) failed. + * + * Try again, this time only allocating exactly the size the user wants. If + * the allocation now succeeds and just happens to be aligned, we can still + * fulfill the user's request. + */ + if (m == NULL) { + /* + * Use bytes not nb, since mALLOc internally calls request2size too, and + * each call increases the size to allocate, to account for the header. + */ + m = (char*)(mALLOc(bytes)); + /* Aligned -> return it */ + if ((((unsigned long)(m)) % alignment) == 0) + return m; + /* Otherwise, fail */ + fREe(m); + return NULL; + } + if (m == NULL) return NULL; /* propagate failure */ p = mem2chunk(m); -- 2.7.0

3 5

[U-Boot] [PATCH 00/14] EFI payload / application support v2
by Alexander Graf 02 Feb '16

02 Feb '16

This is my Christmas present for my openSUSE friends :). U-Boot is a great project for embedded devices. However, convincing everyone involved that only for "a few oddball ARM devices" we need to support different configuration formats from grub2 when all other platforms (PPC, System Z, x86) are standardized on a single format is a nightmare. So we started to explore alternatives. At first, people tried to get grub2 running using the u-boot api interface. However, FWIW that one doesn't support relocations, so you need to know where to link grub2 to at compile time. It also seems to be broken more often than not. And on top of it all, it's a one-off interface, so yet another thing to maintain. That led to a nifty idea. What if we can just implement the EFI application protocol on top of U-Boot? Then we could compile a single grub2 binary for uEFI based systems and U-Boot based systems and as soon as that one's loaded, everything looks and feels (almost) the same. This patch set is the result of pursuing this endeavor. - I am successfully able to run grub2 and Linux EFI binaries with this code. - When enabled, the resulting U-Boot binary only grows by ~10kb, so it's very light weight. - It works on 32bit ARM and AArch64. - All storage devices are directly accessible - No EFI variables - Removable media booting (search for /efi/boot/boota{a64,arm}.efi) Of course, there are still a few things one could do on top: - Improve disk media detection (don't scan, use what information we have) - Add EFI variable support using NVRAM - Add GFX support - Make EFI Shell work ;) - Network device support - Support for payload exit But so far, I'm very happy with the state of the patches. They completely eliminate potential arguments against U-Boot internally and give users the chance to run with the same level of comfort on all firmware types. Version 2 was successfully tested to boot grub2 and Linux from there on a HiKey (AArch64, dcache disabled) and on a BeagleBone Black. If you read this far and want to try out the patches, feel free to grab the source from git at: https://github.com/agraf/u-boot efi-v2 or just use the readily compiled version from https://build.opensuse.org/project/show/home:algraf:branches:Base:System v1 -> v2: - move memory allocation to separate patch - limit 32/64 to hosts that support it - check 32bit optional nt header magic - switch to GPLv2+ - Fix typo s/does now/does not/ - Add #ifdefs around header to allow inclusion when efi_loader is disabled - Add stub efi_restore_gd() function when efi_loader is disabled - Disable debug - Mark runtime region as such - Fix up memory map - Allow efi_restore_gd to be called before first efi entry - Add 32bit arm cache workaround - Move memory map to separate patch - Change BTS version to 2.5 - Fix return values for a few callbacks to more EFI compliant ones - Change vendor to "Das U-Boot" - Add warning when truncating timer trigger - Add runtime detach - Enable runtime relocations - Add get_time - Fix relocation - Fix 32bit - Add am335x support - Move section definition to header - Add systab to runtime section - Add self-relocation hook table - Relocate efi_runtime section early during bootup - Fix return values for a number of callbacks to be more UEFI compliant - Move to block_drvr array - Fix header order - Document efi block object struct - Use calloc rather than malloc & memset - Default to y - New patches: - disk/part.c: Expose list of available block drivers - arm64: Allow exceptions to return - arm64: Allow EFI payload code to take exceptions - efi_loader: Add DCACHE_OFF support for arm64 - efi_loader: Add distro boot script for removable media Alex Alexander Graf (14): disk/part.c: Expose list of available block drivers include/efi_api.h: Add more detailed API definitions efi_loader: Add PE image loader efi_loader: Add boot time services efi_loader: Add console interface efi_loader: Add runtime services efi_loader: Add disk interfaces efi_loader: Add "bootefi" command efi_loader: Implement memory allocation and map arm64: Allow exceptions to return arm64: Allow EFI payload code to take exceptions efi_loader: Add DCACHE_OFF support for arm64 efi_loader: hook up in build environment efi_loader: Add distro boot script for removable media arch/arm/config.mk | 4 + arch/arm/cpu/armv8/exceptions.S | 44 ++ arch/arm/cpu/armv8/u-boot.lds | 16 + arch/arm/cpu/u-boot.lds | 30 ++ arch/arm/lib/Makefile | 3 + arch/arm/lib/interrupts_64.c | 28 ++ arch/arm/lib/sections.c | 4 + arch/arm/lib/unaligned_64.c | 284 +++++++++++++ board/ti/am335x/u-boot.lds | 30 ++ common/Makefile | 1 + common/board_r.c | 4 + common/cmd_bootefi.c | 159 +++++++ disk/part.c | 7 +- include/config_distro_bootcmd.h | 47 ++- include/efi_api.h | 197 +++++++-- include/efi_loader.h | 90 ++++ include/part.h | 8 + include/pe.h | 263 ++++++++++++ lib/Kconfig | 1 + lib/Makefile | 1 + lib/efi_loader/Kconfig | 9 + lib/efi_loader/Makefile | 11 + lib/efi_loader/efi_boottime.c | 856 ++++++++++++++++++++++++++++++++++++++ lib/efi_loader/efi_console.c | 357 ++++++++++++++++ lib/efi_loader/efi_disk.c | 218 ++++++++++ lib/efi_loader/efi_image_loader.c | 202 +++++++++ lib/efi_loader/efi_runtime.c | 300 +++++++++++++ 27 files changed, 3130 insertions(+), 44 deletions(-) create mode 100644 arch/arm/lib/unaligned_64.c create mode 100644 common/cmd_bootefi.c create mode 100644 include/efi_loader.h create mode 100644 include/pe.h create mode 100644 lib/efi_loader/Kconfig create mode 100644 lib/efi_loader/Makefile create mode 100644 lib/efi_loader/efi_boottime.c create mode 100644 lib/efi_loader/efi_console.c create mode 100644 lib/efi_loader/efi_disk.c create mode 100644 lib/efi_loader/efi_image_loader.c create mode 100644 lib/efi_loader/efi_runtime.c -- 2.1.4

3 44

[U-Boot] [PATCH] arm: use common instructions applicable to armv7m & other arm archs
by Vikas Manocha 02 Feb '16

02 Feb '16

BIC instruction to clear the SP is not allowed in armv7m & is deprecated in ARMv6T2 & above. This patch cleans the code by using instructions allowed for armv7m as well as other Arm archs. Signed-off-by: Vikas Manocha <vikas.manocha(a)st.com> --- arch/arm/lib/crt0.S | 25 +++++++------------------ 1 file changed, 7 insertions(+), 18 deletions(-) diff --git a/arch/arm/lib/crt0.S b/arch/arm/lib/crt0.S index 2f4c14e..3731d0c 100644 --- a/arch/arm/lib/crt0.S +++ b/arch/arm/lib/crt0.S @@ -71,18 +71,12 @@ ENTRY(_main) */ #if defined(CONFIG_SPL_BUILD) && defined(CONFIG_SPL_STACK) - ldr sp, =(CONFIG_SPL_STACK) + ldr r0, =(CONFIG_SPL_STACK) #else - ldr sp, =(CONFIG_SYS_INIT_SP_ADDR) + ldr r0, =(CONFIG_SYS_INIT_SP_ADDR) #endif -#if defined(CONFIG_CPU_V7M) /* v7M forbids using SP as BIC destination */ - mov r3, sp - bic r3, r3, #7 - mov sp, r3 -#else - bic sp, sp, #7 /* 8-byte alignment for ABI compliance */ -#endif - mov r0, sp + bic r0, r0, #7 /* 8-byte alignment for ABI compliance */ + mov sp, r0 bl board_init_f_alloc_reserve mov sp, r0 /* set up gd here, outside any C code */ @@ -100,14 +94,9 @@ ENTRY(_main) * 'here' but relocated. */ - ldr sp, [r9, #GD_START_ADDR_SP] /* sp = gd->start_addr_sp */ -#if defined(CONFIG_CPU_V7M) /* v7M forbids using SP as BIC destination */ - mov r3, sp - bic r3, r3, #7 - mov sp, r3 -#else - bic sp, sp, #7 /* 8-byte alignment for ABI compliance */ -#endif + ldr r0, [r9, #GD_START_ADDR_SP] /* sp = gd->start_addr_sp */ + bic r0, r0, #7 + mov sp, r0 /* 8-byte alignment for ABI compliance */ ldr r9, [r9, #GD_BD] /* r9 = gd->bd */ sub r9, r9, #GD_SIZE /* new GD is below bd */ -- 1.9.1

4 5

[U-Boot] [PATCH] ARMv8:ls2080: Move the core errata defines out of board specific file
by Ashish Kumar 01 Feb '16

01 Feb '16

* Valid for core A57 Signed-off-by: Ashish Kumar <Ashish.Kumar(a)nxp.com> --- arch/arm/include/asm/arch-fsl-layerscape/config.h | 4 ++++ include/configs/ls2080a_common.h | 3 --- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/arch/arm/include/asm/arch-fsl-layerscape/config.h b/arch/arm/include/asm/arch-fsl-layerscape/config.h index f1b164f..5d68a8a 100644 --- a/arch/arm/include/asm/arch-fsl-layerscape/config.h +++ b/arch/arm/include/asm/arch-fsl-layerscape/config.h @@ -121,6 +121,10 @@ #define CONFIG_SYS_FSL_ERRATUM_A009663 #define CONFIG_SYS_FSL_ERRATUM_A009942 +/* ARM core A57 Errata fixes */ +#define CONFIG_ARM_ERRATA_828024 +#define CONFIG_ARM_ERRATA_826974 + #elif defined(CONFIG_LS1043A) #define CONFIG_MAX_CPUS 4 #define CONFIG_SYS_CACHELINE_SIZE 64 diff --git a/include/configs/ls2080a_common.h b/include/configs/ls2080a_common.h index 7323e10..def0a6f 100644 --- a/include/configs/ls2080a_common.h +++ b/include/configs/ls2080a_common.h @@ -15,9 +15,6 @@ #define CONFIG_GICV3 #define CONFIG_FSL_TZPC_BP147 -/* Errata fixes */ -#define CONFIG_ARM_ERRATA_828024 -#define CONFIG_ARM_ERRATA_826974 #include <asm/arch/ls2080a_stream_id.h> #include <asm/arch/config.h> -- 1.7.6.GIT

2 1

[U-Boot] [PATCH][v2] ARMv8:ls2-2080a: Implement core ERRATA 829520, 833471
by Ashish Kumar 01 Feb '16

01 Feb '16

* 829520: Code bounded by indirect conditional branch might corrupt instruction stream, Workaround: Set CPUACTLR_EL1[4] = 1'b1 to disable the Indirect Predictor * 833471: VMSR FPSCR functional failure or deadlock Workaround: Set CPUACTLR[38] to 1, which forces FPSCR write flush Signed-off-by: Ashish Kumar <Ashish.Kumar(a)nxp.com> --- v2: Incorporated review comments arch/arm/cpu/armv8/start.S | 19 +++++++++++++++++++ arch/arm/include/asm/arch-fsl-layerscape/config.h | 4 ++++ 2 files changed, 23 insertions(+), 0 deletions(-) diff --git a/arch/arm/cpu/armv8/start.S b/arch/arm/cpu/armv8/start.S index 2ee60d6..67b166c 100644 --- a/arch/arm/cpu/armv8/start.S +++ b/arch/arm/cpu/armv8/start.S @@ -168,6 +168,25 @@ apply_a57_core_errata: msr S3_1_c15_c2_0, x0 /* cpuactlr_el1 */ #endif +#ifdef CONFIG_ARM_ERRATA_833471 + mrs x0, S3_1_c15_c2_0 /* cpuactlr_el1 */ + /* FPSCR write flush. + * Note that in some cases where a flush is unnecessary this + could impact performance. */ + orr x0, x0, #1 << 38 + msr S3_1_c15_c2_0, x0 /* cpuactlr_el1 */ +#endif + +#ifdef CONFIG_ARM_ERRATA_829520 + mrs x0, S3_1_c15_c2_0 /* cpuactlr_el1 */ + /* Disable Indirect Predictor bit will prevent this erratum + from occurring + * Note that in some cases where a flush is unnecessary this + could impact performance. */ + orr x0, x0, #1 << 4 + msr S3_1_c15_c2_0, x0 /* cpuactlr_el1 */ +#endif + #ifdef CONFIG_ARM_ERRATA_833069 mrs x0, S3_1_c15_c2_0 /* cpuactlr_el1 */ /* Disable Enable Invalidates of BTB bit */ diff --git a/arch/arm/include/asm/arch-fsl-layerscape/config.h b/arch/arm/include/asm/arch-fsl-layerscape/config.h index 49b113d..9267082 100644 --- a/arch/arm/include/asm/arch-fsl-layerscape/config.h +++ b/arch/arm/include/asm/arch-fsl-layerscape/config.h @@ -118,6 +118,10 @@ #define CONFIG_SYS_FSL_ERRATUM_A008585 #define CONFIG_SYS_FSL_ERRATUM_A008751 #define CONFIG_SYS_FSL_ERRATUM_A009635 +/* ARM A57 CORE ERRATA */ +#define CONFIG_ARM_ERRATA_833471 +#define CONFIG_ARM_ERRATA_829520 + #elif defined(CONFIG_LS1043A) #define CONFIG_MAX_CPUS 4 #define CONFIG_SYS_CACHELINE_SIZE 64 -- 1.7.6.GIT

2 1

[U-Boot] [PATCH] driver: net: fsl-mc: Update print to reflect correct string
by Prabhakar Kushwaha 01 Feb '16

01 Feb '16

From: Prabhakar Kushwaha <prabhakar(a)freescale.com> Update printf with dpbp_exit to match with previous function call. Signed-off-by: Itai Katz <itai.katz(a)nxp.com> Signed-off-by: Prabhakar Kushwaha <prabhakar.kushwaha(a)nxp.com> --- drivers/net/fsl-mc/mc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/fsl-mc/mc.c b/drivers/net/fsl-mc/mc.c index 6269dab..b5eafd2 100644 --- a/drivers/net/fsl-mc/mc.c +++ b/drivers/net/fsl-mc/mc.c @@ -1099,7 +1099,7 @@ int fsl_mc_ldpaa_exit(bd_t *bd) err = dpbp_exit(); if (err < 0) { - printf("dpni_exit() failed: %d\n", err); + printf("dpbp_exit() failed: %d\n", err); goto err; } -- 1.9.1

2 1

[U-Boot] [PATCH] driver: net: fsl-mc: Memset dprc_cfg before configuring
by Prabhakar Kushwaha 01 Feb '16

01 Feb '16

From: Prabhakar Kushwaha <prabhakar(a)freescale.com> All fields of struct dprc_cfg are not being configured while creating child container. "Not" configured fields are assumed to be 0. So memset dprc_cfg before configuring the fields. Signed-off-by: Itai Katz <itai.katz(a)nxp.com> Signed-off-by: Prabhakar Kushwaha <prabhakar.kushwaha(a)nxp.com> --- drivers/net/fsl-mc/mc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/fsl-mc/mc.c b/drivers/net/fsl-mc/mc.c index bac4610..6269dab 100644 --- a/drivers/net/fsl-mc/mc.c +++ b/drivers/net/fsl-mc/mc.c @@ -784,6 +784,7 @@ static int dprc_init(void) goto err_root_open; } + memset(&cfg, 0, sizeof(struct dprc_cfg)); cfg.options = DPRC_CFG_OPT_TOPOLOGY_CHANGES_ALLOWED | DPRC_CFG_OPT_OBJ_CREATE_ALLOWED | DPRC_CFG_OPT_ALLOC_ALLOWED; -- 1.9.1

2 1

[U-Boot] [PATCH] armv7: ls102xa: Increase the supported kernel size
by Alison Wang 01 Feb '16

01 Feb '16

Increase the kernel size supported for LS1021A platform. Define CONFIG_SYS_BOOTM_LEN to 64M. Signed-off-by: Alison Wang <alison.wang(a)nxp.com> --- include/configs/ls1021aqds.h | 2 ++ include/configs/ls1021atwr.h | 2 ++ 2 files changed, 4 insertions(+) diff --git a/include/configs/ls1021aqds.h b/include/configs/ls1021aqds.h index 2e8dbc7..bbbb057 100644 --- a/include/configs/ls1021aqds.h +++ b/include/configs/ls1021aqds.h @@ -668,4 +668,6 @@ unsigned long get_board_ddr_clk(void); #include <asm/fsl_secure_boot.h> #endif +#define CONFIG_SYS_BOOTM_LEN (64 << 20) /* Increase max gunzip size */ + #endif diff --git a/include/configs/ls1021atwr.h b/include/configs/ls1021atwr.h index c12ba3a..dd1270b 100644 --- a/include/configs/ls1021atwr.h +++ b/include/configs/ls1021atwr.h @@ -524,4 +524,6 @@ #include <asm/fsl_secure_boot.h> #endif +#define CONFIG_SYS_BOOTM_LEN (64 << 20) /* Increase max gunzip size */ + #endif -- 2.1.0.27.g96db324

2 1