November 2020 - U-Boot

CONFIG_SPL_UNIT_TEST
by Heinrich Schuchardt 03 Nov '20

03 Nov '20

Hello Simon, CONFIG_SPL_UNIT_TEST can be selected for non-sandboard boards. How would the tests be executed here? Best regards Heinrich

2 1

[PATCH 0/2] rtc: initialize emulated RTC from environment variable
by Heinrich Schuchardt 03 Nov '20

03 Nov '20

Up to now the emulated RTC is initialized using the U-Boot build time. With this patch the environment variable 'rtc_emul_epoch' can be used to provide a better initial time. The variable is a decimal string with the number of seconds since 1970-01-01. Heinrich Schuchardt (2): rtc: use probe() to initialize emulated RTC rtc: initialize emulated RTC from environment variable drivers/rtc/Kconfig | 13 ++++++++----- drivers/rtc/emul_rtc.c | 29 +++++++++++++++++++++++------ 2 files changed, 31 insertions(+), 11 deletions(-) -- 2.28.0

2 4

[PATCH v3 00/23] log: Add commands for manipulating filters
by Sean Anderson 03 Nov '20

03 Nov '20

This series adds several commands for adding, listing, and removing log filters. It also adds getopt, since the filter-add command needs to have several optional arguments to be complete, and positional specification of those arguments would have been difficult. Changes in v3: - Convert log_test from python to C - Document assumption that erroneous results from log_get_cat_name begin with '<' - Fix heading level of Filters section - Remove a few more already-implemented features from the TODO list - Update copyright for log_filter.c Changes in v2: - Add % before constants in kerneldocs - Add a few informational commands - Add compiletime assert on size of log_cat_name - Add const qualifier to log_*_name - Add option to remove all filters to filter-remove - Clarify filter-* help text - Clarify wording of filter documentation - Converted log filter-* tests to C from python - Document log_level_t and log_category_t members - Expand documentation of getopt() to include examples - Expose log_has_cat and log_has_file for filter tests - Include enum definitions instead of re-documenting them - Print an error message if the log level is invalid. - Remove opt prefix from getopt_state members - Reorganize log documentation; related sections should now be more proximate Sean Anderson (23): log: Fix missing negation of ENOMEM log: Fix incorrect documentation of log_filter.cat_list log: Add additional const qualifier to arrays log: Add new category names to log_cat_name log: Use CONFIG_IS_ENABLED() for LOG_TEST log: Expose some helper functions log: Add function to create a filter with flags log: Add filter flag to deny on match test: log: Convert log_test from python to C test: log: Give tests names instead of numbers test: Add tests for LOGFF_DENY log: Add filter flag to match greater than a log level test: Add test for LOGFF_MIN cmd: log: Use sub-commands for log cmd: log: Split off log level parsing cmd: log: Add commands to list categories and drivers cmd: log: Make "log level" print all log levels lib: Add getopt test: Add a test for getopt cmd: log: Add commands to manipulate filters test: Add a test for log filter-* doc: Add log kerneldocs to documentation doc: Update logging documentation MAINTAINERS | 1 + cmd/Kconfig | 1 + cmd/log.c | 353 ++++++++++++++++++++++--- common/log.c | 66 ++--- doc/api/getopt.rst | 8 + doc/api/index.rst | 1 + doc/develop/logging.rst | 245 +++++++++-------- include/getopt.h | 130 +++++++++ include/log.h | 212 +++++++++++---- include/test/log.h | 3 + lib/Kconfig | 5 + lib/Makefile | 1 + lib/getopt.c | 125 +++++++++ test/lib/Makefile | 1 + test/lib/getopt.c | 123 +++++++++ test/log/Makefile | 1 + test/log/log_filter.c | 108 ++++++++ test/log/log_test.c | 536 +++++++++++++++++++++++++------------- test/log/syslog_test.h | 2 - test/py/tests/test_log.py | 104 -------- 20 files changed, 1482 insertions(+), 544 deletions(-) create mode 100644 doc/api/getopt.rst create mode 100644 include/getopt.h create mode 100644 lib/getopt.c create mode 100644 test/lib/getopt.c create mode 100644 test/log/log_filter.c -- 2.28.0

4 36

[PATCH 00/11] bootm: Support substitions in bootargs and add tests
by Simon Glass 03 Nov '20

03 Nov '20

This series adds tests to the fixup_silent-linux() function and extends the 'zimage' command to use it. It also adds a new string-substition feature to allow bootargs to be a template, rather than having to build it up piece by piece with information obtained in a build script. It also updates zimage to use the same command-line processing. With these additions it is possible to boot Chrome OS from a U-Boot script on most Chromebooks. Simon Glass (11): env: Allow returning errors from hdelete_r() bootm: Add tests for fixup_silent_linux() bootm: Update fixup_silent_linux() to return an error bootm: Rename fixup_silent_linux() bootm: Add a bool parameter to bootm_process_cmdline_env() bootm: Use size rather than length for CONSOLE_ARG bootm: Split out bootargs environment reading / writing bootm: Update bootm_process_cmdline_env() to use flags bootm: Allow updating the bootargs in a buffer x86: zimage: Add silent-console processing bootm: Support string substitution in bootargs README | 16 +++ arch/Kconfig | 2 + arch/x86/lib/zimage.c | 14 +++ cmd/nvedit.c | 6 +- common/Kconfig.boot | 20 ++++ common/bootm.c | 206 +++++++++++++++++++++++++++++------ include/bootm.h | 40 +++++++ include/search.h | 11 +- include/test/suites.h | 1 + lib/hashtable.c | 12 +-- test/Makefile | 1 + test/bootm.c | 243 ++++++++++++++++++++++++++++++++++++++++++ test/cmd_ut.c | 1 + test/env/hashtable.c | 2 +- 14 files changed, 534 insertions(+), 41 deletions(-) create mode 100644 test/bootm.c -- 2.29.0.rc1.297.gfa9743e501-goog

5 36

[PATCH v2 00/16] riscv: k210: Enable use of AI ram bank
by Sean Anderson 03 Nov '20

03 Nov '20

This ram bank was previously unusable because PLL1 was not started correctly. This series fixes that bug, and allows U-Boot to relocate into the AI ram. This provides an extra 2M of space in which to load payloads. Second time around and I realized that the bypass clock is no longer necessary (yay). This also necessitates a few more small fixups. I've also added some minor patches to bring the device tree and clock driver closer to what Linux has (or will have). This should be the last round of additions to this series. Changes in v2: - Don't re-enable the PLL - Remove bypass clock, which is no longer necessary - Simplify PLL instantiation - Modify clock tree so clint is a child of aclk - Sync memory dts node with Linux - Use correct aisram clock Sean Anderson (16): clk: k210: Fix PLLs not being enabled clk: k210: Fix PLL enable always getting taken clk: k210: Remove bypass clock clk: k210: Remove k210_register_pll clk: k210: Move the clint clock to under aclk clk: Add support for the k210 clock driver pre-relocation riscv: Enable some devices pre-relocation lib: fdt: Add fdtdec_setup_mem_size_base_highest test: Add a test for fdtdec_setup_mem_size_base et al. ram: Add driver for K210 SRAM ram: sifive: Default to y only if compiling for fu540 riscv: Probe ram in dram_init riscv: Enable AI ram on K210 riscv: k210: Rename airam to aisram riscv: k210: Use AI as the parent clock of aisram, not PLL1 riscv: Don't reserve AI ram in k210 dts MAINTAINERS | 1 + arch/riscv/cpu/generic/dram.c | 26 +++ arch/riscv/dts/k210.dtsi | 22 +-- arch/sandbox/dts/test.dts | 12 ++ board/sipeed/maix/Kconfig | 2 + board/sipeed/maix/maix.c | 26 --- configs/sandbox64_defconfig | 2 +- configs/sandbox_defconfig | 2 +- configs/sandbox_flattree_defconfig | 2 +- configs/sipeed_maix_bitm_defconfig | 1 + drivers/clk/kendryte/Makefile | 2 +- drivers/clk/kendryte/bypass.c | 273 ----------------------------- drivers/clk/kendryte/clk.c | 61 +++---- drivers/clk/kendryte/pll.c | 26 +-- drivers/ram/Kconfig | 7 + drivers/ram/Makefile | 1 + drivers/ram/kendryte.c | 56 ++++++ drivers/ram/sifive/Kconfig | 2 +- include/configs/sipeed-maix.h | 4 - include/fdtdec.h | 19 +- include/kendryte/bypass.h | 31 ---- include/kendryte/pll.h | 4 - lib/fdtdec.c | 34 +++- test/dm/fdtdec.c | 38 ++++ 24 files changed, 230 insertions(+), 424 deletions(-) delete mode 100644 drivers/clk/kendryte/bypass.c create mode 100644 drivers/ram/kendryte.c delete mode 100644 include/kendryte/bypass.h -- 2.28.0

2 28

Re: [PATCH 1/3] riscv: Add timer_get_us() for tracing
by Rick Chen 03 Nov '20

03 Nov '20

Hi Pragnesh > From: Bin Meng [mailto:bmeng.cn@gmail.com] > Sent: Friday, September 11, 2020 2:48 PM > To: Pragnesh Patel > Cc: U-Boot Mailing List; Atish Patra; Anup Patel; Sagar Kadam; Rick Jian-Zhi Chen(陳建志); Paul Walmsley; Bin Meng; Lukas Auer; Sean Anderson > Subject: Re: [PATCH 1/3] riscv: Add timer_get_us() for tracing > > Hi Pragnesh, > > On Mon, Aug 24, 2020 at 10:45 PM Pragnesh Patel > <pragnesh.patel(a)sifive.com> wrote: > > > > timer_get_us() will use timer_ops->get_count() function for timer counter. > > For S-mode U-Boot, CSR_TIMEH and CSR_TIME will provide a timer counter and > > For M-mode U-Boot, mtime register will provide the same. > > > > Signed-off-by: Pragnesh Patel <pragnesh.patel(a)sifive.com> > > --- > > arch/riscv/lib/Makefile | 1 + > > arch/riscv/lib/timer.c | 50 +++++++++++++++++++++++++++++++++++++++++ > > 2 files changed, 51 insertions(+) > > create mode 100644 arch/riscv/lib/timer.c > > > > diff --git a/arch/riscv/lib/Makefile b/arch/riscv/lib/Makefile > > index 10ac5b06d3..fbb68e583b 100644 > > --- a/arch/riscv/lib/Makefile > > +++ b/arch/riscv/lib/Makefile > > @@ -26,6 +26,7 @@ obj-y += setjmp.o > > obj-$(CONFIG_$(SPL_)SMP) += smp.o > > obj-$(CONFIG_SPL_BUILD) += spl.o > > obj-y += fdt_fixup.o > > +obj-$(CONFIG_TIMER) += timer.o > > > > # For building EFI apps > > CFLAGS_$(EFI_CRT0) := $(CFLAGS_EFI) > > diff --git a/arch/riscv/lib/timer.c b/arch/riscv/lib/timer.c > > new file mode 100644 > > index 0000000000..3e423f2805 > > --- /dev/null > > +++ b/arch/riscv/lib/timer.c > > @@ -0,0 +1,50 @@ > > +// SPDX-License-Identifier: GPL-2.0+ > > +/* > > + * Copyright (C) 2020 SiFive, Inc > > + * > > + * Authors: > > + * Pragnesh Patel <pragnesh.patel(a)sifive.com> > > + */ > > + > > +#include <common.h> > > +#include <dm.h> > > +#include <timer.h> > > + > > +DECLARE_GLOBAL_DATA_PTR; > > + > > +static struct udevice *timer; > > + > > +ulong notrace timer_get_us(void) > > Does the weak one in lib/time.c not work on RISC-V? Do you have any comments about Bin's reply ? Thanks, Rick > > > +{ > > + u64 count; > > + ulong rate; > > + int ret; > > + > > + /** > > + * gd->timer will become NULL in initr_dm(), so assign gd->timer > > + * to other static global timer, so that timer_get_us() can use it. > > + */ > > + if (!timer && gd->timer) > > + timer = (struct udevice *)gd->timer; > > + > > + if (timer) { > > + ret = timer_get_count(timer, &count); > > + if (ret) > > + return ret; > > + > > + rate = timer_get_rate(timer); > > + } > > + > > + return (ulong)count / rate; > > +} > > + > > +int timer_init(void) > > Why is this function necessary? > > > +{ > > + int ret; > > + > > + ret = dm_timer_init(); > > Does enabling CONFIG_TIMER_EARLY help? > > > + if (ret) > > + return ret; > > + > > + return 0; > > +} > > Regards, > Bin

2 1

[PATCH] edid: investigate dtd from cea861 extension if necessary
by Jerome Brunet 03 Nov '20

03 Nov '20

If no valid detailed timing can be found in the edid base block, check the detailed timing provided in the cea861 extension block, if any. Reported-by: Da Xue <da(a)libre.computer> Tested-by: Da Xue <da(a)libre.computer> Signed-off-by: Jerome Brunet <jbrunet(a)baylibre.com> --- common/edid.c | 72 +++++++++++++++++++++++++++++++++++---------------- 1 file changed, 49 insertions(+), 23 deletions(-) diff --git a/common/edid.c b/common/edid.c index 553ab8fd01a1..7dd0c924c24a 100644 --- a/common/edid.c +++ b/common/edid.c @@ -169,6 +169,33 @@ static bool cea_is_hdmi_vsdb_present(struct edid_cea861_info *info) return false; } +bool edid_get_dtd_timing_validate(struct edid_monitor_descriptor *desc, + unsigned int dtd_count, + struct display_timing *timing, + bool (*mode_valid)(void *priv, + const struct display_timing *timing), + void *mode_valid_priv) +{ + bool timing_done = false; + int i; + + for (i = 0; i < dtd_count; i++, desc++) { + if (desc->zero_flag_1 != 0) { + decode_timing((u8 *)desc, timing); + if (mode_valid) + timing_done = mode_valid(mode_valid_priv, + timing); + else + timing_done = true; + + if (timing_done) + break; + } + } + + return timing_done; +} + int edid_get_timing_validate(u8 *buf, int buf_size, struct display_timing *timing, int *panel_bits_per_colourp, @@ -177,8 +204,9 @@ int edid_get_timing_validate(u8 *buf, int buf_size, void *mode_valid_priv) { struct edid1_info *edid = (struct edid1_info *)buf; + struct edid_cea861_info *info = NULL; + struct edid_monitor_descriptor *desc; bool timing_done; - int i; if (buf_size < sizeof(*edid) || edid_check_info(edid)) { debug("%s: Invalid buffer\n", __func__); @@ -190,24 +218,27 @@ int edid_get_timing_validate(u8 *buf, int buf_size, return -ENOENT; } - /* Look for detailed timing */ - timing_done = false; - for (i = 0; i < 4; i++) { - struct edid_monitor_descriptor *desc; + desc = edid->monitor_details.descriptor; + timing_done = edid_get_dtd_timing_validate(desc, 4, timing, + mode_valid, mode_valid_priv); - desc = &edid->monitor_details.descriptor[i]; - if (desc->zero_flag_1 != 0) { - decode_timing((u8 *)desc, timing); - if (mode_valid) - timing_done = mode_valid(mode_valid_priv, - timing); - else - timing_done = true; + if (edid->extension_flag && (buf_size >= EDID_EXT_SIZE)) { + info = (struct edid_cea861_info *)(buf + sizeof(*edid)); - if (timing_done) - break; - } + if (info->extension_tag != EDID_CEA861_EXTENSION_TAG) + info = NULL; + } + + /* Check CEA861 info block for timing if don't have one yet */ + if (info && !timing_done && info->dtd_offset) { + unsigned int dtd_count = EDID_CEA861_DTD_COUNT(*info); + + desc = (struct edid_monitor_descriptor *)((u8 *)info + + info->dtd_offset); + timing_done = edid_get_dtd_timing_validate(desc, dtd_count, timing, + mode_valid, mode_valid_priv); } + if (!timing_done) return -EINVAL; @@ -225,13 +256,8 @@ int edid_get_timing_validate(u8 *buf, int buf_size, } timing->hdmi_monitor = false; - if (edid->extension_flag && (buf_size >= EDID_EXT_SIZE)) { - struct edid_cea861_info *info = - (struct edid_cea861_info *)(buf + sizeof(*edid)); - - if (info->extension_tag == EDID_CEA861_EXTENSION_TAG) - timing->hdmi_monitor = cea_is_hdmi_vsdb_present(info); - } + if (info) + timing->hdmi_monitor = cea_is_hdmi_vsdb_present(info); return 0; } -- 2.28.0

1 0

[PATCH v6 0/7] Add HiHope RZ/G2[HMN] board support
by Biju Das 03 Nov '20

03 Nov '20

This patch series adds the required SoC/Board support to boot HopeRun HiHope RZ/G2[HMN] boards. It uses SoC driver for unique cpu identification for RZ/G and R-Car SoC's, since they share same PRR ID. SDHI quirks added using soc_device_match api. This patch series depend upon [1] [1] https://lists.denx.de/pipermail/u-boot/2020-November/431539.html Note: 1) This patch series tested on RZ/G2[HMN] boards. 2) SOC_DEVICE_RENESAS option enabled only RZ/G2[HMN] boards. After testing this needs to be enabled for other boards. Biju Das (7): arm: rmobile: Add RZ/G2[HMNE] SoC support mmc: renesas-sdhi: Add SDHI quirks for R-Car M3-W and RZ/G2M arm: rmobile: Add HopeRun HiHope RZ/G2M board support mmc: renesas-sdhi: Add SDHI quirks for R-Car M3-N and RZ/G2N arm: rmobile: Add HopeRun HiHope RZ/G2N board support mmc: renesas-sdhi: Add SDHI quirks for R-Car H3 and RZ/G2H arm: rmobile: Add HopeRun HiHope RZ/G2H board support arch/arm/dts/Makefile | 3 + arch/arm/dts/hihope-common.dtsi | 377 ++++++++++++++++++ arch/arm/dts/hihope-rev4.dtsi | 124 ++++++ arch/arm/dts/hihope-rzg2-ex.dtsi | 92 +++++ arch/arm/dts/r8a774a1-hihope-rzg2m-ex.dts | 21 + arch/arm/dts/r8a774a1-hihope-rzg2m-u-boot.dts | 27 ++ arch/arm/dts/r8a774a1-hihope-rzg2m.dts | 37 ++ arch/arm/dts/r8a774a1-u-boot.dtsi | 55 +++ arch/arm/dts/r8a774b1-hihope-rzg2n-ex.dts | 16 + arch/arm/dts/r8a774b1-hihope-rzg2n-u-boot.dts | 27 ++ arch/arm/dts/r8a774b1-hihope-rzg2n.dts | 41 ++ arch/arm/dts/r8a774b1-u-boot.dtsi | 53 +++ arch/arm/dts/r8a774e1-hihope-rzg2h-ex.dts | 15 + arch/arm/dts/r8a774e1-hihope-rzg2h-u-boot.dts | 27 ++ arch/arm/dts/r8a774e1-hihope-rzg2h.dts | 26 ++ arch/arm/dts/r8a774e1-u-boot.dtsi | 25 ++ arch/arm/mach-rmobile/Kconfig.64 | 16 + arch/arm/mach-rmobile/cpu_info-rcar.c | 22 +- arch/arm/mach-rmobile/cpu_info.c | 10 +- arch/arm/mach-rmobile/include/mach/rmobile.h | 60 ++- board/hoperun/hihope-rzg2/Kconfig | 15 + board/hoperun/hihope-rzg2/MAINTAINERS | 6 + board/hoperun/hihope-rzg2/Makefile | 9 + board/hoperun/hihope-rzg2/hihope-rzg2.c | 99 +++++ configs/hihope_rzg2_defconfig | 77 ++++ drivers/mmc/renesas-sdhi.c | 157 ++++++++ include/configs/hihope-rzg2.h | 20 + 27 files changed, 1438 insertions(+), 19 deletions(-) create mode 100644 arch/arm/dts/hihope-common.dtsi create mode 100644 arch/arm/dts/hihope-rev4.dtsi create mode 100644 arch/arm/dts/hihope-rzg2-ex.dtsi create mode 100644 arch/arm/dts/r8a774a1-hihope-rzg2m-ex.dts create mode 100644 arch/arm/dts/r8a774a1-hihope-rzg2m-u-boot.dts create mode 100644 arch/arm/dts/r8a774a1-hihope-rzg2m.dts create mode 100644 arch/arm/dts/r8a774a1-u-boot.dtsi create mode 100644 arch/arm/dts/r8a774b1-hihope-rzg2n-ex.dts create mode 100644 arch/arm/dts/r8a774b1-hihope-rzg2n-u-boot.dts create mode 100644 arch/arm/dts/r8a774b1-hihope-rzg2n.dts create mode 100644 arch/arm/dts/r8a774b1-u-boot.dtsi create mode 100644 arch/arm/dts/r8a774e1-hihope-rzg2h-ex.dts create mode 100644 arch/arm/dts/r8a774e1-hihope-rzg2h-u-boot.dts create mode 100644 arch/arm/dts/r8a774e1-hihope-rzg2h.dts create mode 100644 arch/arm/dts/r8a774e1-u-boot.dtsi create mode 100644 board/hoperun/hihope-rzg2/Kconfig create mode 100644 board/hoperun/hihope-rzg2/MAINTAINERS create mode 100644 board/hoperun/hihope-rzg2/Makefile create mode 100644 board/hoperun/hihope-rzg2/hihope-rzg2.c create mode 100644 configs/hihope_rzg2_defconfig create mode 100644 include/configs/hihope-rzg2.h -- 2.17.1

3 15

[PATCH v2 0/3] Various minor patches
by Simon Glass 03 Nov '20

03 Nov '20

This series contains a number of minor patches collected while completing the ACPI updates for U-Boot. Changes in v2: - Drop patches already applied - Rebase ot master Simon Glass (3): x86: zimage: Add a little more logging x86: zimage: Sanity-check the kernel version before printing it x86: zimage: Quieten down the zimage boot process arch/x86/lib/zimage.c | 26 +++++++++++++++++++++----- common/log.c | 1 + include/log.h | 1 + 3 files changed, 23 insertions(+), 5 deletions(-) -- 2.29.1.341.ge80a0c044ae-goog

2 6

[PATCH 1/2] cros_ec: Fix up driver names to avoid dtoc warnings
by Simon Glass 03 Nov '20

03 Nov '20

Fix the dtoc warning in these file by using a driver name that matches the compatible string. Signed-off-by: Simon Glass <sjg(a)chromium.org> --- drivers/misc/cros_ec_i2c.c | 4 ++-- drivers/misc/cros_ec_lpc.c | 4 ++-- drivers/misc/cros_ec_spi.c | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/misc/cros_ec_i2c.c b/drivers/misc/cros_ec_i2c.c index c00f5f764a0..664bd2b9385 100644 --- a/drivers/misc/cros_ec_i2c.c +++ b/drivers/misc/cros_ec_i2c.c @@ -231,8 +231,8 @@ static const struct udevice_id cros_ec_ids[] = { { } }; -U_BOOT_DRIVER(cros_ec_i2c) = { - .name = "cros_ec_i2c", +U_BOOT_DRIVER(google_cros_ec_i2c) = { + .name = "google_cros_ec_i2c", .id = UCLASS_CROS_EC, .of_match = cros_ec_ids, .probe = cros_ec_probe, diff --git a/drivers/misc/cros_ec_lpc.c b/drivers/misc/cros_ec_lpc.c index 4ad6c8ca66d..63702f90fbc 100644 --- a/drivers/misc/cros_ec_lpc.c +++ b/drivers/misc/cros_ec_lpc.c @@ -243,8 +243,8 @@ static const struct udevice_id cros_ec_ids[] = { { } }; -U_BOOT_DRIVER(cros_ec_lpc) = { - .name = "cros_ec_lpc", +U_BOOT_DRIVER(google_cros_ec_lpc) = { + .name = "google_cros_ec_lpc", .id = UCLASS_CROS_EC, .of_match = cros_ec_ids, .probe = cros_ec_probe, diff --git a/drivers/misc/cros_ec_spi.c b/drivers/misc/cros_ec_spi.c index 153f971bdeb..bbc96301aeb 100644 --- a/drivers/misc/cros_ec_spi.c +++ b/drivers/misc/cros_ec_spi.c @@ -184,8 +184,8 @@ static const struct udevice_id cros_ec_ids[] = { { } }; -U_BOOT_DRIVER(cros_ec_spi) = { - .name = "cros_ec_spi", +U_BOOT_DRIVER(google_cros_ec_spi) = { + .name = "google_cros_ec_spi", .id = UCLASS_CROS_EC, .of_match = cros_ec_ids, .probe = cros_ec_probe, -- 2.28.0.806.g8561365e88-goog

2 5