December 2021 - U-Boot

[PATCH v3] xilinx: Kconfig: add XILINX_OF_BOARD_DTB_ADDR default value for microblaze
by Ovidiu Panait 03 Dec '21

03 Dec '21

The xilinx board_fdt_blob_setup() implementation makes use of XILINX_OF_BOARD_DTB_ADDR, but no default value is currently defined for microblaze. Add one so that microblaze could also work with CONFIG_OF_SEPARATE. Signed-off-by: Ovidiu Panait <ovidiu.panait(a)windriver.com> --- board/xilinx/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/board/xilinx/Kconfig b/board/xilinx/Kconfig index 64507b5d84..1788066173 100644 --- a/board/xilinx/Kconfig +++ b/board/xilinx/Kconfig @@ -43,6 +43,7 @@ endif config XILINX_OF_BOARD_DTB_ADDR hex "Default DTB pickup address" default 0x1000 if ARCH_VERSAL + default 0x8000 if MICROBLAZE default 0x100000 if ARCH_ZYNQ || ARCH_ZYNQMP depends on OF_BOARD || OF_SEPARATE help -- 2.25.1

2 1

a question about falcon mode
by Chan Kim 03 Dec '21

03 Dec '21

Hello all, I'm trying to implement falcon mode for our board. Then should I first implement the normal mode(spl + proper)? It looks like so while I'm reading doc/README.falcon. (It says, after loading kernel, DT etc. I should give 'spl export' command). Please someone confirm this. Thank you. Chan Kim

3 7

[PATCH] RFC: gitlab: x86: Add a coreboot test
by Simon Glass 03 Dec '21

03 Dec '21

Coreboot supports U-Boot as a payload and this recently got a bit of a facelist. Add a test for this. For now this uses a binary build of coreboot (v4.15). Future work could potentially build it from source, but we need to figure out the toolchain problems first, since coreboot uses its own toolchain. This needs some changes to the hooks scripts as well. An example build is at https://source.denx.de/u-boot/custodians/u-boot-dm/-/jobs/359687 Signed-off-by: Simon Glass <sjg(a)chromium.org> --- .gitlab-ci.yml | 28 +++++++++++++++++++++++++++- 1 file changed, 27 insertions(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 4c89daeadcf..f5d7e0f77e2 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -14,7 +14,7 @@ stages: stage: test.py before_script: # Clone uboot-test-hooks - - git clone --depth=1 https://source.denx.de/u-boot/u-boot-test-hooks /tmp/uboot-test-hooks + - git clone -b try --depth=1 https://github.com/sjg20/uboot-test-hooks.git /tmp/uboot-test-hooks - ln -s travis-ci /tmp/uboot-test-hooks/bin/`hostname` - ln -s travis-ci /tmp/uboot-test-hooks/py/`hostname` - grub-mkimage --prefix="" -o ~/grub_x86.efi -O i386-efi normal echo lsefimmap lsefi lsefisystab efinet tftp minicmd @@ -52,6 +52,16 @@ stages: genimage --inputpath . --config board/sifive/unleashed/genimage_spi-nor.cfg; cp images/spi-nor.img ${UBOOT_TRAVIS_BUILD_DIR}/; fi + - if [[ "${TEST_PY_BD}" == "coreboot" ]]; then + wget -O - + "https://drive.google.com/uc?id=1x6nrtWIyIRPLS2cQBwYTnT2TbOI8UjmM&export=dow…" | + xz -dc >${UBOOT_TRAVIS_BUILD_DIR}/coreboot.rom; + wget -O - + "https://drive.google.com/uc?id=149Cz-5SZXHNKpi9xg6R_5XITWohu348y&export=dow…" > + cbfstool; + chmod a+x cbfstool; + ./cbfstool ${UBOOT_TRAVIS_BUILD_DIR}/coreboot.rom add-flat-binary -f ${UBOOT_TRAVIS_BUILD_DIR}/u-boot.bin -n fallback/payload -c LZMA -l 0x1110000 -e 0x1110000; + fi - virtualenv -p /usr/bin/python3 /tmp/venv - . /tmp/venv/bin/activate - pip install -r test/py/requirements.txt @@ -61,6 +71,10 @@ stages: ./test/py/test.py -ra --bd ${TEST_PY_BD} ${TEST_PY_ID} ${TEST_PY_TEST_SPEC:+"-k ${TEST_PY_TEST_SPEC}"} --build-dir "$UBOOT_TRAVIS_BUILD_DIR" + # It seems that the files in /tmp go away, so copy out what we need + - if [[ "${TEST_PY_BD}" == "coreboot" ]]; then + cp -v /tmp/coreboot/*.{html,css} .; + fi build all 32bit ARM platforms: stage: world build @@ -366,3 +380,15 @@ xtfpga test.py: TEST_PY_TEST_SPEC: "not sleep" TEST_PY_ID: "--id qemu" <<: *buildman_and_testpy_dfn + +coreboot test.py: + variables: + TEST_PY_BD: "coreboot" + TEST_PY_TEST_SPEC: "not sleep" + TEST_PY_ID: "--id qemu" + artifacts: + paths: + - "*.html" + - "*.css" + expire_in: 1 week + <<: *buildman_and_testpy_dfn -- 2.34.0.384.gca35af8252-goog

1 0

[PATCH 1/2] board/sunxi/dram_sun4i_auto: use DRAM_MEMORY_TYPE_DDR3 instead of magic number 3
by Giulio Benetti 03 Dec '21

03 Dec '21

Since DRAM_MEMORY_TYPE_DDR3 is defined let's use it instead of magic number 3. Signed-off-by: Giulio Benetti <giulio.benetti(a)benettiengineering.com> --- board/sunxi/dram_sun4i_auto.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/board/sunxi/dram_sun4i_auto.c b/board/sunxi/dram_sun4i_auto.c index e8bbee4ee5..547d1c0cb4 100644 --- a/board/sunxi/dram_sun4i_auto.c +++ b/board/sunxi/dram_sun4i_auto.c @@ -4,7 +4,7 @@ static struct dram_para dram_para = { .clock = CONFIG_DRAM_CLK, - .type = 3, + .type = DRAM_MEMORY_TYPE_DDR3, .rank_num = 1, .density = 0, .io_width = 0, -- 2.25.1

1 1

[PATCH 1/1] sandbox: replace putchar(ch) by fputc(ch, stdout)
by Heinrich Schuchardt 02 Dec '21

02 Dec '21

When compiled with -Og for better debugability u-boot ends up in a stack overflow using gcc (Ubuntu 11.2.0-7ubuntu2) 11.2.0 GNU Binutils for Ubuntu 2.37 putchar(ch) is defined as a macro which ends up calling U-Boot's putc() implementation instead of the glibc one, which calls os_putc() ... Let's use fputc(ch, stdout) instead as fputc() does not exist in U-Boot. Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt(a)canonical.com> --- arch/sandbox/cpu/os.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/sandbox/cpu/os.c b/arch/sandbox/cpu/os.c index b72dafca2b..25a04a650a 100644 --- a/arch/sandbox/cpu/os.c +++ b/arch/sandbox/cpu/os.c @@ -628,7 +628,7 @@ int os_get_filesize(const char *fname, loff_t *size) void os_putc(int ch) { - putchar(ch); + fputc(ch, stdout); } void os_puts(const char *str) -- 2.32.0

2 2

[PATCH] drivers: core: lists: fix for loop index type
by fpga-garage＠preissner-muc.de 02 Dec '21

02 Dec '21

From: Alexander Preißner <fpga-garage(a)preissner-muc.de> * fixes the bug in function bind_drivers_pass that for CONFIG_CC_OPTIMIZE_FOR_SIZE=n and no entries in the driver_info list, i.e. n_ents == 0, the processor steps into the first loop iteration despite the loop condition being false. * the Xilinx Zynq-7000 device would eventually hang due to an attempted access to an invalid memory address * the bug is fixed by changing the type of idx from uint to int Board: zynq-zybo Target: ARM Compiler: arm-none-eabi-gcc 9.2.1 Signed-off-by: Alexander Preissner <fpga-garage(a)preissner-muc.de> --- drivers/core/lists.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/core/lists.c b/drivers/core/lists.c index 5d4f2ea0e3..d2e9dc5719 100644 --- a/drivers/core/lists.c +++ b/drivers/core/lists.c @@ -58,7 +58,7 @@ static int bind_drivers_pass(struct udevice *parent, bool pre_reloc_only) const int n_ents = ll_entry_count(struct driver_info, driver_info); bool missing_parent = false; int result = 0; - uint idx; + int idx; /* * Do one iteration through the driver_info records. For of-platdata, -- 2.25.1

2 2

[PATCH 00/17] binman: Various tidy-ups and refactors
by Simon Glass 02 Dec '21

02 Dec '21

This series improves the implementation of the entry_Section class so that it is easier to subclass it. It also updates the documentation in this area and adds support for a few more types in the Fdt class. Using this, the CBFS implementation is tidied up a bit, but is not yet updated to use entry_Section as a base class. That needs a bit more work. Still, this provides a better basis for new file formats. Simon Glass (17): dtoc: Bring in the libfdt module automatically dtoc: Add support for reading 64-bit ints dtoc: Add support for reading fixed-length bytes properties binman: Tidy up style in cmdline binman: Add a way to obtain the version binman: Correct init of entry in Entry class binman: Correct comments for ReadChildData() binman: Drop the underscore in _ReadEntries() binman: Drop the filename property in entry_Section binman: Allow overriding BuildSectionData() binman: Allow control of which entries to read binman: Update the section documentation binman: Move cbfs.ObtainContents() down a bit binman: Use normal entries in cbfs binman: cbfs: Refactor the init process binman: cfbs: Refactor ObtainContents() for consistency binman: Rename testCbfsNoCOntents() scripts/pylint.base | 6 +- tools/binman/cmdline.py | 74 ++++++++--- tools/binman/entries.rst | 149 +++++++++++++++++---- tools/binman/entry.py | 7 +- tools/binman/etype/blob_phase.py | 2 +- tools/binman/etype/cbfs.py | 90 +++++++------ tools/binman/etype/files.py | 2 +- tools/binman/etype/section.py | 190 ++++++++++++++++++++------- tools/binman/ftest.py | 24 +++- tools/binman/state.py | 18 +++ tools/dtoc/fdt_util.py | 55 ++++++++ tools/dtoc/test/dtoc_test_simple.dts | 1 + tools/dtoc/test_dtoc.py | 2 + tools/dtoc/test_fdt.py | 44 ++++++- 14 files changed, 520 insertions(+), 144 deletions(-) -- 2.34.0.rc2.393.gf8c9666880-goog

1 34

Please pull u-boot-net/master
by Ramon Fried 02 Dec '21

02 Dec '21

Hi Tom, The below contains two fixes from Marek for designware and mdio. The following changes since commit 4a14bfffd42f968ed9d72a780a8d44a9053c5b95: Merge https://source.denx.de/u-boot/custodians/u-boot-marvell (2021-11-30 08:59:22 -0500) are available in the Git repository at: https://source.denx.de/u-boot/custodians/u-boot-net.git/ for you to fetch changes up to 3fbd17aadf792735846a5f2052b3f68f7075201f: net: dwc_eth_qos: Enable clock in probe (2021-12-02 08:35:44 +0200) ---------------------------------------------------------------- Marek Vasut (2): net: eth-phy: Handle gpio_request_by_name() return value net: dwc_eth_qos: Enable clock in probe drivers/net/dwc_eth_qos.c | 22 +++++++++++----------- drivers/net/eth-phy-uclass.c | 2 +- 2 files changed, 12 insertions(+), 12 deletions(-) Thanks, Ramon.

2 1

[PATCH] mtd: sf: Set SF parameters as env variables
by Marek Vasut 02 Dec '21

02 Dec '21

Set the SF page size, erase block size and total size as an environment variable after "sf probe". This lets us discern boards with multiple distinct SPI flash options and also e.g. set mtdparts accordingly. Signed-off-by: Marek Vasut <marex(a)denx.de> Cc: Jagan Teki <jagan(a)amarulasolutions.com> Cc: Vignesh R <vigneshr(a)ti.com> --- drivers/mtd/spi/spi-nor-core.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/mtd/spi/spi-nor-core.c b/drivers/mtd/spi/spi-nor-core.c index d5d905fa5a1..448653b9931 100644 --- a/drivers/mtd/spi/spi-nor-core.c +++ b/drivers/mtd/spi/spi-nor-core.c @@ -15,6 +15,7 @@ #include <dm.h> #include <dm/device_compat.h> #include <dm/devres.h> +#include <env.h> #include <linux/bitops.h> #include <linux/err.h> #include <linux/errno.h> @@ -3829,8 +3830,11 @@ int spi_nor_scan(struct spi_nor *nor) #ifndef CONFIG_SPL_BUILD printf("SF: Detected %s with page size ", nor->name); print_size(nor->page_size, ", erase size "); + env_set_hex("sf_pagesize", nor->page_size); print_size(nor->erase_size, ", total "); + env_set_hex("sf_erasesize", nor->erase_size); print_size(nor->size, ""); + env_set_hex("sf_size", nor->size); puts("\n"); #endif -- 2.33.0

4 11

[PATCH 0/5] clk: Clean up optional helpers, and add API docs to HTML
by Sean Anderson 02 Dec '21

02 Dec '21

This cleans up the various optional helpers for clocks, and adds a new one. While we're at it, also convert the existing API docs to our HTML documentation. Sean Anderson (5): clk: Rename clk_get_optional_nodev clk: Inline clk_get_*_optional clk: Add client API to HTML docs clk: Add driver API to HTML docs clk: Add clk_get_by_name_optional doc/api/clk.rst | 19 +++ doc/api/index.rst | 1 + drivers/clk/clk-uclass.c | 21 --- drivers/clk/clk_zynq.c | 5 +- drivers/phy/phy-mtk-tphy.c | 8 +- drivers/rng/meson-rng.c | 4 +- include/clk-uclass.h | 187 +++++++++++++---------- include/clk.h | 306 +++++++++++++++++++++---------------- 8 files changed, 310 insertions(+), 241 deletions(-) create mode 100644 doc/api/clk.rst -- 2.33.0

4 11