- Nov 02, 2023
-
-
Simon Glass authored
It isn't clear why we need to have two different paths for closing down the pipe. Unify them and use the Python to avoid this warning: subprocess.py:1127: ResourceWarning: subprocess 83531 is still running Note that this code appears to originally have come from [1] and was committed into the ChromeOS chromiumos/platform/crosutils repo in the bin/cros_image_to_target.py file. The addition of the extra code path came later, so that is chosen for the fixes tag. [1] https://codereview.chromium.org/3391008 Signed-off-by:
Simon Glass <sjg@chromium.org> Fixes: a10fd93c patman: Make command methods return a CommandResult
-
Simon Glass authored
Buildman assumes that branch names do not have a slash in them, since slash is used to delimit remotes, etc. This means that a branch called 'WIP/tryme' in remote dm ends up being 'tryme'. Adjust the logic a little, to try to accommodate this. For now, no tests are added for this behaviour. Signed-off-by:
Simon Glass <sjg@chromium.org> Reviewed-by:
Tom Rini <trini@konsulko.com>
-
The sandbox should closely mimic other architectures. Place each function or data in a separate section and let the linker eliminate unused ones. This will reduce the binary size. In the linker script mark that u_boot_sandbox_getopt are to be kept. Signed-off-by:
Heinrich Schuchardt <heinrich.schuchardt@canonical.com> Reviewed-by:
Tom Rini <trini@konsulko.com>
-
According to the TRMs of K3 platform of devices, the ROM boot image format specifies a "Core Options Field" that provides the capability to set the boot core in lockstep when set to 0 or to split mode when set to 2. Add support for providing the same from the binman DTS. Also modify existing test case for ensuring future coverage. Signed-off-by:
Neha Malcom Francis <n-francis@ti.com> Reviewed-by:
Simon Glass <sjg@chromium.org>
-
Simon Glass authored
When symbols switch between the inited data section and the read-only data section their visbility changes, at present, with the -B option. This is confusing, since adding 'const' to a variable declaration can make it look like a significant improvement in bloat. But in fact nothing has changed. Add 'r' to the list of symbols types that are recorded, to correct this problem. Add a constant to make it easier to find this code next time. Signed-off-by:
Simon Glass <sjg@chromium.org> Reported-by:
Tom Rini <trini@konsulko.com> Reviewed-by:
Tom Rini <trini@konsulko.com>
-
on Peach-pi console every key press is echoed with message 'cros_ec_command: Returned status 1' this is not proper fix, just hack to disable this message Signed-off-by:
Milan P. Stanić <mps@arvanta.net> Reviewed-by:
Simon Glass <sjg@chromium.org>
-
Simon Glass authored
cbfsutil completely changed the way that stages are formatted in CBFS. Adjust the binman implementation to do the same. This mirrors commit 81dc20e744 in coreboot. Signed-off-by:
Simon Glass <sjg@chromium.org>
-
Simon Glass authored
In preparation for changing how stages are stored, rename the existing stage tag. Signed-off-by:
Simon Glass <sjg@chromium.org>
-
Simon Glass authored
cbfsutil changed to 4-byte alignment for filenames instead of 16. Adjust the binman implementation to do the same. This mirrors commit 5779ca718c in coreboot. Signed-off-by:
Simon Glass <sjg@chromium.org>
-
Simon Glass authored
cbfsutil changed to write zero bytes instead of 0xff when a small padding must be added. Adjust the binman implementation to do the same. Drop the code which looks for an unused attribute tag, since it is not used. A future patch moves the attributes to the end of the header in any case, so no data will follow the attributes. This mirrors commit f0cc7adb2f in coreboot. Signed-off-by:
Simon Glass <sjg@chromium.org>
-
Simon Glass authored
cbfsutil changed to skip adding a compression attribute if there is no compression. Adjust the binman implementation to do the same. This mirrors commit 105cdf5625 in coreboot. Signed-off-by:
Simon Glass <sjg@chromium.org> Reviewed-by: Neha Malcom Francis
-
Simon Glass authored
For tests which fake bintools being missing, we need to reset the list afterwards, to ensure that future tests do not also see the bintools as missing. Reset the list when processing is complete. Signed-off-by:
Simon Glass <sjg@chromium.org> Reviewed-by:
Neha Malcom Francis <n-francis@ti.com>
-
A Change-Id can be useful for traceability purposes, and some projects may wish to have them preserved. This change makes it configurable via a new 'keep_change_id' setting. Signed-off-by:
Maxim Cournoyer <maxim.cournoyer@gmail.com> Reviewed-by:
Simon Glass <sjg@chromium.org>
-
https://source.denx.de/u-boot/custodians/u-boot-i2cTom Rini authored
i2c updates for v2024.01-rc2 - nuvoton: support standard/fast/fast plus mode - bootcount: remove legacy i2c driver and implement DM based version Bugfixes: - designware_i2c: adjust timing calculation SPL probing failed on the StarFive VisionFive 2 board Heinrich fixed this, by syncing timing calculation with linux implementation.
-
https://source.denx.de/u-boot/custodians/u-boot-riscvTom Rini authored
+ CI: Use OpenSBI 1.3.1 release for testing + riscv: Support resume after exception + rng: Support RNG provided by RISC-V Zkr ISA extension + board: starfive VF2: Support jtag + board: starfive VF2: Support TRNG driver + board: sifive unmatched: Move kernel load address
-
Chanho Park authored
Enables JH7110 RNG driver to visionfive2 board. Signed-off-by:
Chanho Park <chanho61.park@samsung.com> Reviewed-by:
Heinrich Schuchardt <xypron.glpk@gmx.de>
-
Chanho Park authored
Adds jh7110 trng device tree node. Signed-off-by:
Chanho Park <chanho61.park@samsung.com> Reviewed-by:
Leo Yu-Chi Liang <ycliang@andestech.com>
-
Chanho Park authored
Adds to support JH7110 TRNG driver which is based on linux kernel's jh7110-trng.c. This can support to generate 256-bit random numbers and 128-bit but this makes 256-bit default for convenience. Signed-off-by:
Chanho Park <chanho61.park@samsung.com> Reviewed-by:
Leo Yu-Chi Liang <ycliang@andestech.com>
-
Chanho Park authored
Add STGCLK_SEC_HCLK and STGCLK_SEC_MISCAHB clocks for JH7110 TRNG device. Signed-off-by:
Chanho Park <chanho61.park@samsung.com> Reviewed-by:
Leo Yu-Chi Liang <ycliang@andestech.com>
-
Chanho Park authored
This imports mmio functions from Linux's arch/riscv/include/asm/mmio.h to use read/write[b|w|l|q]_relaxed functions. Signed-off-by:
Chanho Park <chanho61.park@samsung.com> Reviewed-by:
Leo Yu-Chi Liang <ycliang@andestech.com>
-
Heinrich Schuchardt authored
The Zkr ISA extension (ratified Nov 2021) introduced the seed CSR. It provides an interface to a physical entropy source. A RNG driver based on the seed CSR is provided. It depends on mseccfg.sseed being set in the SBI firmware. Signed-off-by:
Heinrich Schuchardt <heinrich.schuchardt@canonical.com> Reviewed-by:
Leo Yu-Chi Liang <ycliang@andestech.com>
-
Heinrich Schuchardt authored
If CSRs like seed are readable by S-mode, may not be determinable by S-mode. For safe driver probing allow to resume via a longjmp after an exception. Signed-off-by:
Heinrich Schuchardt <heinrich.schuchardt@canonical.com> Reviewed-by:
Leo Yu-Chi Liang <ycliang@andestech.com>
-
Chanho Park authored
JTAG pins are mapped as below. To access the JTAG pins, we need to control the GPIO pins from SPL which seems to be the earliest stage for JTAG. - JTAG nTRST: GPIO36 / Input - JTAG TDI: GPIO61 / Input - JTAG TMS: GPIO63 / Input - JTAG TCK: GPIO60 / Input - JTAG TDO: GPIO44 / Output Signed-off-by:
Chanho Park <chanho61.park@samsung.com> Reviewed-by:
Leo Yu-Chi Liang <ycliang@andestech.com>
-
Chanho Park authored
Add gpio.h header file that includes JH7110 helper macros. The file is imported from StarFive github[1] with small changes such as alignment. [1]: https://github.com/starfive-tech/u-boot Signed-off-by:
Chanho Park <chanho61.park@samsung.com> Reviewed-by:
Leo Yu-Chi Liang <ycliang@andestech.com>
-
Samuel Holland authored
Some RISC-V CPUs, such as the T-HEAD XuanTie series, have a vendor-specific way to invalidate a portion of the instruction cache. Allow them to override invalidate_icache_range(). Signed-off-by:
Samuel Holland <samuel@sholland.org> Reviewed-by:
Leo Yu-Chi Liang <ycliang@andestech.com>
-
Samuel Holland authored
This is required on CPUs which always operate in CLIC mode, such as the T-HEAD E906 and E907. Per the CLIC specification: "In this mode, the trap vector base address held in mtvec is constrained to be aligned on a 64-byte or larger power-of-two boundary." Reported-by:
Madushan Nishantha <jlmadushan@gmail.com> Signed-off-by:
Samuel Holland <samuel@sholland.org> Reviewed-by:
Leo Yu-Chi Liang <ycliang@andestech.com>
-
Samuel Holland authored
Clean things up for the next time somebody adds a target. Signed-off-by:
Samuel Holland <samuel@sholland.org> Reviewed-by:
Leo Yu-Chi Liang <ycliang@andestech.com>
-
Yong-Xuan Wang authored
U-boot initially loads the kernel image to the kernel_addr_r, and subsequently relocates it to memory address 0x80200000. Setting kernel_addr_r to 0x80200000 can eliminate one copy operation. Signed-off-by:
Yong-Xuan Wang <yongxuan.wang@sifive.com> Reviewed-by:
Tom Rini <trini@konsulko.com>
-
Heinrich Schuchardt authored
Use the most recent upstream release of OpenSBI for CI testing. Signed-off-by:
Heinrich Schuchardt <heinrich.schuchardt@canonical.com> Reviewed-by:
Leo Yu-Chi Liang <ycliang@andestech.com>
-
- Nov 01, 2023
-
-
https://source.denx.de/u-boot/custodians/u-boot-clkTom Rini authored
Clock changes for 2024.01-rc2 This contains several fixes for the clock core.
-
If the device does not specify any clocks in device tree, these functions will return PTR_ERR(-ENOENT). This is not the intended behavior and does not comply with linux kernel CCF. Fix that by returning NULL under such circumstances instead. Signed-off-by:
Yang Xiwen <forbidden405@outlook.com> Reviewed-by:
Sean Anderson <seanga2@gmail.com> Link: https://lore.kernel.org/r/20230818-clk-fix-v1-3-49ec18f820bf@outlook.com
-
In clk_enable()/clk_disable() functions, when CCF is activated, we must pass a private clk struct to enable()/disable() ops functions. Otherwise, the use of a container_of() construction within these ops should be banned. Because passing a non-private clk struct to container_of() results in an out of range error. At the moment, clk-mux, clk-fixed-factor, clk-gate and possibly other clocks use container_of() in their enable()/disable() functions. Therefore, for these functions to work correclty, private clk struct must be passed. Signed-off-by:
Maksim Kiselev <bigunclemax@gmail.com> Reviewed-by:
Sean Anderson <seanga2@gmail.com> Link: https://lore.kernel.org/r/20230905221649.3577929-1-bigunclemax@gmail.com
-
The second parameter for clk_release_all is used as an unsigned (which makes sense) but the function prototype declares it as an int. This causes warnings/error like such below: include/clk.h:422:48: error: conversion to ‘int’ from ‘unsigned int’ may change the sign of the result [-Werror=sign-conversion] 422 | return clk_release_all(bulk->clks, bulk->count); To fix this, changed the type of the count to `unsigned int` Fixes: 82a8a669 ("clk: add clk_release_all()") Signed-off-by:
Eugen Hristev <eugen.hristev@collabora.com> Reviewed-by:
Xavier Drudis Ferran <xdrudis@tinet.cat> Reviewed-by:
Sean Anderson <seanga2@gmail.com> Link: https://lore.kernel.org/r/20230619104752.278500-1-eugen.hristev@collabora.com
-
In commit 5c5992cb ("clk: Add debugging for return values"), a temporary storage variable was added around the ops->get_rate() call inside clk_get_rate(), so that the result could be passed through log_ret. This temporary variable was declared as an int, yet when we look in struct clk_ops, we can see this needs to be a ulong: ulong (*get_rate)(struct clk *clk); This was resulting in a signed to unsigned casting error on our builds, where a clock value of 0xABCDABCD was being incorrectly cast to 0xFFFFFFFFABCDABCD. Signed-off-by:
Nathan Barrett-Morrison <nathan.morrison@timesys.com> Reviewed-by:
Sean Anderson <seanga2@gmail.com> Link: https://lore.kernel.org/r/20230515195005.1961495-1-nathan.morrison@timesys.com
-
Tom Rini authored
- Four patches to address issues with bootstd flows in some cases
-
This has a typo which makes the method inoperable. Correct it so that 'bootflow read' works correctly for ChromeOS. Signed-off-by:
Simon Glass <sjg@chromium.org>
-
Two bugs have appeared: - arguments can have an equals sign embedded in them, which must be considered part of the value - arguments must fully match the name; partial matches should be ignored Fix these and add a test to cover both. Signed-off-by:
Simon Glass <sjg@chromium.org>
-
Enable the bootflow based on this bootmeth if the BootOrder EFI variable is set. Signed-off-by:
Mark Kettenis <kettenis@openbsd.org> Reviewed-by:
Simon Glass <sjg@chromium.org>
-
Compiling sandbox_defconfig with CONFIG_BOOTSTD=n fails: /usr/bin/ld: drivers/mtd/spi/sf_bootdev.o: in function `sf_get_bootflow': /drivers/mtd/spi/sf_bootdev.c:43:(.text+0x96): undefined reference to `bootmeth_set_bootflow' Add the missing Kconfig dependency. Fixes: Fixes: 0c1f4a9f ("bootstd: Add a SPI flash bootdev") Signed-off-by:
Heinrich Schuchardt <heinrich.schuchardt@canonical.com> Reviewed-by:
Simon Glass <sjg@chromium.org>
-
-