February 2019 - U-Boot

[U-Boot] [PATCH] coreboot: add support fot CB_TAG_BOOT_MEDIA_PARAMS
by Christian Gmeiner 21 Feb '19

21 Feb '19

Change-Id: I7a2e320f2296bc20e1ac2f10cc2297697c50e097 Signed-off-by: Christian Gmeiner <christian.gmeiner(a)gmail.com> --- arch/x86/cpu/coreboot/tables.c | 13 +++++++++++++ arch/x86/include/asm/arch-coreboot/sysinfo.h | 6 +++++- arch/x86/include/asm/coreboot_tables.h | 11 +++++++++++ 3 files changed, 29 insertions(+), 1 deletion(-) diff --git a/arch/x86/cpu/coreboot/tables.c b/arch/x86/cpu/coreboot/tables.c index bc18b710c9..fa26b66f24 100644 --- a/arch/x86/cpu/coreboot/tables.c +++ b/arch/x86/cpu/coreboot/tables.c @@ -109,6 +109,16 @@ static void cb_parse_string(unsigned char *ptr, char **info) *info = (char *)((struct cb_string *)ptr)->string; } +static void cb_parse_boot_meda_params(unsigned char *ptr, struct sysinfo_t *info) +{ + struct cb_boot_media_params *params = (struct cb_boot_media_params *)ptr; + + info->fmap_offset = params->fmap_offset; + info->cbfs_offset = params->cbfs_offset; + info->cbfs_size = params->cbfs_size; + info->boot_media_size = params->boot_media_size; +} + static int cb_parse_header(void *addr, int len, struct sysinfo_t *info) { struct cb_header *header; @@ -211,6 +221,9 @@ static int cb_parse_header(void *addr, int len, struct sysinfo_t *info) case CB_TAG_VBNV: cb_parse_vbnv(ptr, info); break; + case CB_TAG_BOOT_MEDIA_PARAMS: + cb_parse_boot_meda_params(ptr, info); + break; } ptr += rec->size; diff --git a/arch/x86/include/asm/arch-coreboot/sysinfo.h b/arch/x86/include/asm/arch-coreboot/sysinfo.h index dd8d1cba92..0969bf946b 100644 --- a/arch/x86/include/asm/arch-coreboot/sysinfo.h +++ b/arch/x86/include/asm/arch-coreboot/sysinfo.h @@ -51,8 +51,12 @@ struct sysinfo_t { void *cbmem_cons; struct cb_serial *serial; -}; + u64 fmap_offset; + u64 cbfs_offset; + u64 cbfs_size; + u64 boot_media_size; +}; extern struct sysinfo_t lib_sysinfo; int get_coreboot_info(struct sysinfo_t *info); diff --git a/arch/x86/include/asm/coreboot_tables.h b/arch/x86/include/asm/coreboot_tables.h index c42175b94d..be752fc726 100644 --- a/arch/x86/include/asm/coreboot_tables.h +++ b/arch/x86/include/asm/coreboot_tables.h @@ -193,6 +193,17 @@ struct cb_vbnv { uint32_t vbnv_size; }; +#define CB_TAG_BOOT_MEDIA_PARAMS 0x0030 +struct cb_boot_media_params { + uint32_t tag; + uint32_t size; + /* offsets are relative to start of boot media */ + uint64_t fmap_offset; + uint64_t cbfs_offset; + uint64_t cbfs_size; + uint64_t boot_media_size; +}; + #define CB_TAG_CMOS_OPTION_TABLE 0x00c8 struct cb_cmos_option_table { -- 2.20.1

3 3

[U-Boot] [PATCH v2 00/13] net: Add Allwinner EMAC CLK, RESET support
by Jagan Teki 21 Feb '19

21 Feb '19

This is v2 version for Allwinner EMAC CLK, RESET support, which was initially be a part of previous series[1]. Changes for v2: - rebase on master - add dm tests for new clk and reset functions. Any inputs? Jagan. [1] https://patchwork.ozlabs.org/patch/959351/ Jagan Teki (13): clk: sunxi: Implement A10 EMAC clocks net: sunxi_emac: Add CLK support net: sun8i_emac: Retrieve GMAC clock via 'syscon' phandle clk: Get the CLK by index without device clk: Use clk_get_by_index_tail() test/dm: clk: Add clk_get_by_index[_nodev] test reset: Get the RESET by index without device test/dm: reset: Add reset_get_by_index[_nodev] test clk: sunxi: Implement EMAC, GMAC clocks, resets net: sun8i_emac: Add CLK and RESET support clk: sunxi: h3: Implement EPHY CLK and RESET net: sun8i_emac: Add EPHY CLK and RESET support board: sunxi: gmac: Remove Ethernet clock and reset board/sunxi/gmac.c | 8 -- drivers/clk/clk-uclass.c | 84 +++++++++++----- drivers/clk/sunxi/clk_a10.c | 1 + drivers/clk/sunxi/clk_a10s.c | 1 + drivers/clk/sunxi/clk_a31.c | 2 + drivers/clk/sunxi/clk_a64.c | 2 + drivers/clk/sunxi/clk_a83t.c | 2 + drivers/clk/sunxi/clk_h3.c | 6 ++ drivers/clk/sunxi/clk_h6.c | 4 + drivers/clk/sunxi/clk_r40.c | 3 + drivers/net/sun8i_emac.c | 182 +++++++++++++++++++++++------------ drivers/net/sunxi_emac.c | 28 ++++-- drivers/reset/reset-uclass.c | 53 ++++++---- image.map | 4 + include/clk.h | 15 +++ include/reset.h | 16 +++ test/dm/clk.c | 21 ++++ test/dm/reset.c | 22 +++++ 18 files changed, 338 insertions(+), 116 deletions(-) create mode 100644 image.map -- 2.18.0.321.gffc6fa0e3

2 18

[U-Boot] v2 Tegra cleanups and Kconfig migrations
by Peter Robinson 20 Feb '19

20 Feb '19

The following patches are the remaining bits from my previous series that is rebased on your PR from yesterday. Peter

3 8

[U-Boot] [PATCH v2 0/1] Pass board revision and serial number to the
by Anton Gerasimov 20 Feb '19

20 Feb '19

Raspberry Pi proprietary bootloader adds this information to the device tree to finally land in /proc/cpuinfo. It is then used by some userspace tools. In particular `gpio` tool from WiringPi suite would use the wrong register mapping when this information is missing from /proc/cpuinfo. Anton Gerasimov (1): board: raspberrypi: add serial and revision to the device tree board/raspberrypi/rpi/rpi.c | 35 +++++++++++++++++++++++++++++++++-- 1 file changed, 33 insertions(+), 2 deletions(-) -- 2.19.1

3 6

[U-Boot] [PATCH] ARM: dts: i.MX6QDL: add missing spba-bus to aips-bus
by Hannes Schmelzer 20 Feb '19

20 Feb '19

Add this spba-bus@02000000 sub-bus to the aips-bus@02000000, because below there are essential boot-devices (ecspi1-4) which are needed in SPL for booting from SPI. Signed-off-by: Hannes Schmelzer <hannes.schmelzer(a)br-automation.com> --- arch/arm/dts/imx6qdl-u-boot.dtsi | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/arm/dts/imx6qdl-u-boot.dtsi b/arch/arm/dts/imx6qdl-u-boot.dtsi index dffc21b..fff5f93 100644 --- a/arch/arm/dts/imx6qdl-u-boot.dtsi +++ b/arch/arm/dts/imx6qdl-u-boot.dtsi @@ -9,6 +9,9 @@ aips-bus@02000000 { u-boot,dm-spl; + spba-bus@02000000 { + u-boot,dm-spl; + }; }; aips-bus@02100000 { -- 2.7.4

3 2

[U-Boot] [PATCH] arm: dts: imx6qdl-u-boot: Enable spba-bus@2000000 simple bus
by Adam Ford 20 Feb '19

20 Feb '19

spba-bus has a few nodes under it including the UART1 and some ESPI buses. In order to use them in SPL, the u-boot,dm-spl flag needs to be added to the spba-bus@2000000 container. Signed-off-by: Adam Ford <aford173(a)gmail.com> diff --git a/arch/arm/dts/imx6qdl-u-boot.dtsi b/arch/arm/dts/imx6qdl-u-boot.dtsi index 45ae2fad0b..0aa29e38b8 100644 --- a/arch/arm/dts/imx6qdl-u-boot.dtsi +++ b/arch/arm/dts/imx6qdl-u-boot.dtsi @@ -9,6 +9,9 @@ aips-bus@2000000 { u-boot,dm-spl; + spba-bus@2000000 { + u-boot,dm-spl; + }; }; aips-bus@2100000 { -- 2.17.1

2 1

[U-Boot] [PATCH 0/2] mtd: rawnand: add STM32 FMC2 NAND flash controller driver
by Christophe Kerello 20 Feb '19

20 Feb '19

This patchset adds the support for the STMicroelectronics FMC2 NAND flash controller found on STM32MP SOCs. This patchset supports: - the polling mode feature - a maximum 8k page size - following ECC strength and step size - nand-ecc-strength = <8>, nand-ecc-step-size = <512> (BCH8) - nand-ecc-strength = <4>, nand-ecc-step-size = <512> (BCH4) - nand-ecc-strength = <1>, nand-ecc-step-size = <512> (Extended ecc based on HAMMING) This patchset has been tested on Micron MT29F8G08ABACAH4 (8-bit SLC NAND). Christophe Kerello (2): dt-bindings: mtd: stm32_fmc2: add STM32 FMC2 NAND controller documentation mtd: rawnand: stm32_fmc2: add STM32 FMC2 NAND flash controller driver doc/device-tree-bindings/mtd/stm32-fmc2-nand.txt | 59 ++ drivers/mtd/nand/raw/Kconfig | 11 + drivers/mtd/nand/raw/Makefile | 1 + drivers/mtd/nand/raw/stm32_fmc2_nand.c | 1092 ++++++++++++++++++++++ 4 files changed, 1163 insertions(+) create mode 100644 doc/device-tree-bindings/mtd/stm32-fmc2-nand.txt create mode 100644 drivers/mtd/nand/raw/stm32_fmc2_nand.c -- 1.9.1

1 2

[U-Boot] [PATCH] armv8: Disable exception vectors in SPL by default
by Alexander Graf 20 Feb '19

20 Feb '19

Commit 1416e2d2253 ("armv8: make SPL exception vectors optional") had a typo in it which effectively disabled exception handling in SPL code always. Since nobody complained, I guess we may as well disable exception handling in SPL always by default. So fix the bug to make the config option effective, but disable exception handling in SPL by default. This gets us to the same functionality as before by default, but with much less code included in the binary. Signed-off-by: Alexander Graf <agraf(a)suse.de> --- arch/arm/cpu/armv8/Kconfig | 2 +- arch/arm/cpu/armv8/start.S | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/arm/cpu/armv8/Kconfig b/arch/arm/cpu/armv8/Kconfig index 1c12bbde75..fb061c8d73 100644 --- a/arch/arm/cpu/armv8/Kconfig +++ b/arch/arm/cpu/armv8/Kconfig @@ -3,7 +3,7 @@ if ARM64 config ARMV8_SPL_EXCEPTION_VECTORS bool "Install crash dump exception vectors" depends on SPL - default y + default n help The default exception vector table is only used for the crash dump, but still takes quite a lot of space in the image size. diff --git a/arch/arm/cpu/armv8/start.S b/arch/arm/cpu/armv8/start.S index 12a78ee38b..87bff9408b 100644 --- a/arch/arm/cpu/armv8/start.S +++ b/arch/arm/cpu/armv8/start.S @@ -88,11 +88,11 @@ pie_fixup_done: bl reset_sctrl #endif -#if defined(CONFIG_ARMV8__SPL_EXCEPTION_VECTORS) || !defined(CONFIG_SPL_BUILD) +#if defined(CONFIG_ARMV8_SPL_EXCEPTION_VECTORS) || !defined(CONFIG_SPL_BUILD) .macro set_vbar, regname, reg msr \regname, \reg .endm - adr x0, vectors + adr x0, _start #else .macro set_vbar, regname, reg .endm @@ -354,7 +354,7 @@ ENDPROC(smp_kick_all_cpus) /*-----------------------------------------------------------------------*/ ENTRY(c_runtime_cpu_setup) -#if defined(CONFIG_ARMV8__SPL_EXCEPTION_VECTORS) || !defined(CONFIG_SPL_BUILD) +#if defined(CONFIG_ARMV8_SPL_EXCEPTION_VECTORS) || !defined(CONFIG_SPL_BUILD) /* Relocate vBAR */ adr x0, vectors switch_el x1, 3f, 2f, 1f -- 2.12.3

2 2

[U-Boot] [PATCH v3 1/1] gpio: altera_pio: fix get_value
by Julien Béraud 20 Feb '19

20 Feb '19

gpio_get_value should return 0 or 1, not the value of bit & (1 << pin) Acked-by: Marek Vasut <marex(a)denx.de> Signed-off-by: Julien Beraud <julien.beraud(a)orolia.com> --- drivers/gpio/altera_pio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpio/altera_pio.c b/drivers/gpio/altera_pio.c index 59e30979f0..324f9c29a8 100644 --- a/drivers/gpio/altera_pio.c +++ b/drivers/gpio/altera_pio.c @@ -56,7 +56,7 @@ static int altera_pio_get_value(struct udevice *dev, unsigned pin) struct altera_pio_platdata *plat = dev_get_platdata(dev); struct altera_pio_regs *const regs = plat->regs; - return readl(&regs->data) & (1 << pin); + return !!(readl(&regs->data) & (1 << pin)); } -- 2.20.1

2 4

[U-Boot] ARMv8 linking: how to properly accommodate the exception vector table?
by Andre Przywara 20 Feb '19

20 Feb '19

Hi, I was looking for a solution to some linker problem and was hoping that some linker cracks here on the list have some idea: The generic ARMv8 ("arm64") code provides an exception vector table, which architecturally requires a 2KB alignment. At the moment we don't take any special measures about it, so we end up with a random padding between 0 and 2044 bytes before the table: 0 bytes n * 2KB | | <code before table><padding>|<vector table> <code after table> The vector table takes up a bit more than 1KB, so we happily use the space after that and can't afford to just waste the rest of the 2KB slot (by putting it at the end of the code, for instance). Depending on the link order some code ends up before the table, other behind it, with seemingly random effects on the resulting code size, due to the padding. This is quite annoying with the Allwinner SPLs, which are quite tight. On the H6 it seems like most code additions lead to a build break at the moment due to this, which needs fixing. Actually it should affect all ARMv8 boards, but it might just not be critical for most of them. I was wondering if we can tell the linker in the (SPL) linker script to drop code into *two* possible regions, so that we can use a fixed location for the vector table somewhere in the middle, and tell the linker to fill up the space before *and* after that at will, without requiring us to specify the exact link order of code. I can *manually* arrange the code to make best use of the space, but this is very fragile and breaks every so often, and is thus not sustainable. Another idea would be to put the exception table at the end of the image (at least for sunxi), then fill the remaining <1KB with some data. But I didn't find a low hanging fruit data block which fills the gap nicely. If anyone has any ideas on how to overcome this, I would be very grateful to hear about it. Cheers, Andre.

2 4