Hi Tero,
On 5/6/21 2:55 AM, Tero Kristo wrote:
> From: Tero Kristo
>
> Normally, power domains are handled via TI-SCI in K3 SoCs. However,
> SPL is not going to have access to sysfw resources, so it must control
> them directly. Add driver for supporting this.
>
> Signed-off-by: Tero Kristo
>
On 5/5/21 11:40 PM, Tianrui Wei wrote:
From: Tianrui Wei
Date: Thu, 6 May 2021 11:30:20 +0800
Subject: [PATCH V4 1/2] mmc: add OpenPiton mmc support
This patch adds mmc support for OpenPiton.
Specifically, some dts bindings were not used because
our mmc controller doens't have those
Hi Rasmus,
On Tue, 4 May 2021 at 01:17, Rasmus Villemoes
wrote:
>
> On 04/05/2021 01.10, Simon Glass wrote:
> > Rather than adding an #ifdef and open-coding this calculation, add a
> > helper function to handle it. Use this in the image code.
> >
> > Signed-off-by: Simon Glass
> > ---
> >
> >
On 5/6/21 11:28 PM, Tianrui Wei wrote:
On 5/7/2021 11:15 AM, Sean Anderson wrote:
On 5/6/21 11:06 PM, Tianrui Wei wrote:
Hi Sean,
Many thanks again for reviewing our code! We really appreciate it. Will fix the
things you're suggesting ;p Though I have a few questions in line in the
The RTC module contains a kicker mechanism to prevent any spurious
writes from changing the register values. To set the time, you must
first unlock the TC registers, update them and then lock.
Signed-off-by: Dario Binacchi
---
drivers/rtc/davinci.c | 14 ++
1 file changed, 14
Update the driver to support the device tree and the driver model.
The read / write helpers in rtc_ops allow access to scratch registers
only. The offset parameter is added to the address of the scratch0
register.
Signed-off-by: Dario Binacchi
---
drivers/rtc/davinci.c | 373
There have been some changes to the am335x- DTs related to the
rtc node, so let's re-syncs them with Linux.
Signed-off-by: Dario Binacchi
---
arch/arm/dts/am335x-bone-common.dtsi| 5 +
arch/arm/dts/am335x-evm.dts | 5 +
arch/arm/dts/am335x-evmsk.dts | 5 +
On reset, the RTC loads the 2000-01-01 date with a wrong day of the
week (Sunday instead of Saturday).
Signed-off-by: Dario Binacchi
---
drivers/rtc/davinci.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/drivers/rtc/davinci.c b/drivers/rtc/davinci.c
index
To write correct data to the TC registers, the STATUS register must be
read until the BUSY bit is equal to zero. Once the BUSY flag is zero,
there is a 15 μs access period in which the TC registers can be
programmed.
The rtc_wait_not_busy() has been inspired by the Kernel.
Signed-off-by: Dario
Heinrich,
On Mon, Apr 26, 2021 at 11:44:59AM +0900, AKASHI Takahiro wrote:
> Heinrich,
>
> Do you have any comments?
> # not only on this issue, but also other issues that I pointed out
> # in the initial RFC.
Ping?
-Takahiro Akashi
> On Fri, Apr 23, 2021 at 02:38:09PM +0530, Sughosh Ganu
On 5/6/21 10:50 PM, Tianrui Wei wrote:
Hi Sean,
Many thanks for taking your valuable time to review our patch and give
suggestions. I'll improve both our patches accordingly. Just for the
record, what would be the exact command you'd recommend to run
checkpatch.pl? Because it only showed one
On Fri, May 07, 2021 at 09:06:33AM +0800, Leo Liang wrote:
> Hi Tom,
>
> CI result:
> https://source.denx.de/u-boot/custodians/u-boot-riscv/-/pipelines/7400
>
> The following changes since commit 8ddaf943589756442bba21e5be645cd47526d82b:
>
> Merge tag 'dm-pull-29apr21' of
>
On 5/6/21 9:41 PM, Leo Liang wrote:
On Fri, May 07, 2021 at 09:09:43AM +0800, Tom Rini wrote:
On Fri, May 07, 2021 at 09:06:33AM +0800, Leo Liang wrote:
Hi Tom,
CI result:
https://source.denx.de/u-boot/custodians/u-boot-riscv/-/pipelines/7400
The following changes since commit
On 5/5/21 11:42 PM, Tianrui Wei wrote:
From: Tianrui Wei
Date: Thu, 6 May 2021 11:30:20 +0800
Subject: [PATCH V4 2/2] riscv: board: Support OpenPiton SoC
This patch add board support for OpenPiton.
Please add some information to the commit message describing the board.
Signed-off-by:
Hi Sean,
Thanks for reviewing the patches, I'll fix the initialization thing and
the tegra_mmc credit :P
On 5/7/2021 10:55 AM, Sean Anderson wrote:
On 5/6/21 10:50 PM, Tianrui Wei wrote:
Hi Sean,
Many thanks for taking your valuable time to review our patch and give
suggestions. I'll
On 5/6/21 10:57 PM, Tianrui Wei wrote:
Hi Sean,
Thanks for reviewing the patches, I'll fix the initialization thing and the
tegra_mmc credit :P
Please also add a changelog to your next revision. It helps reviewers
know what you have fixed.
--Sean
On 5/7/2021 10:55 AM, Sean Anderson
On 5/7/2021 11:15 AM, Sean Anderson wrote:
On 5/6/21 11:06 PM, Tianrui Wei wrote:
Hi Sean,
Many thanks again for reviewing our code! We really appreciate it.
Will fix the things you're suggesting ;p Though I have a few
questions in line in the comment. Also, checkpatch didn't catch any
On 5/7/2021 11:41 AM, Sean Anderson wrote:
On 5/6/21 11:28 PM, Tianrui Wei wrote:
On 5/7/2021 11:15 AM, Sean Anderson wrote:
On 5/6/21 11:06 PM, Tianrui Wei wrote:
Hi Sean,
Many thanks again for reviewing our code! We really appreciate it.
Will fix the things you're suggesting ;p Though
Use 32-bit access only where it is needed. Most of the RTC registers
contain useful information in the 8 least significant bits, the others
are reserved.
Signed-off-by: Dario Binacchi
---
drivers/rtc/davinci.c | 32
1 file changed, 16 insertions(+), 16
The Davinci's onchip RTC is also present on TI OMAP1, AM33XX, AM43XX and
DRA7XX SOCs. So, let's enable compilation for these architectures too.
Signed-off-by: Dario Binacchi
---
drivers/rtc/Kconfig | 7 +++
drivers/rtc/davinci.c | 11 ---
2 files changed, 15 insertions(+), 3
The series adds driver model support for omap RTC plus some fixes.
Dario Binacchi (7):
rtc: davinci: enable compilation for omap architectures
rtc: davinci: replace 32bit access with 8bit access
rtc: davinci: check BUSY bit before set TC registers
rtc: davinci: use unlock/lock
Change the USB erratum number A-050106 to A-050204 as A-050106 is
a duplicate and never be published.
Fixes 0cfa00cdb94 (“armv8: Add workaround for USB erratum A-050106”)
Signed-off-by: Ran Wang
---
Change in v2:
- Update copyright information.
arch/arm/cpu/armv8/fsl-layerscape/Kconfig | 8
Hi Tom,
CI result:
https://source.denx.de/u-boot/custodians/u-boot-riscv/-/pipelines/7400
The following changes since commit 8ddaf943589756442bba21e5be645cd47526d82b:
Merge tag 'dm-pull-29apr21' of
https://source.denx.de/u-boot/custodians/u-boot-dm (2021-04-29 21:03:38 -0400)
are available
Hi Leo,
Are you maintaining RISC-V now? Should I be CC-ing you on my series?
Can you update MAINTAINERS with this information? Thanks.
--Sean
On 5/6/21 9:06 PM, Leo Liang wrote:
Hi Tom,
CI result:
https://source.denx.de/u-boot/custodians/u-boot-riscv/-/pipelines/7400
The following changes
Versions of U-Boot before 2014.01 created a symlink from include/asm to
the architecture-specific header directory.
If an ARM board is build with that old version, then sandbox is built on
a more recent version (both with in-tree builds), the include/asm symlink
confuses the build system. It
Hi Sean,
Many thanks for taking your valuable time to review our patch and give
suggestions. I'll improve both our patches accordingly. Just for the
record, what would be the exact command you'd recommend to run
checkpatch.pl? Because it only showed one warning about me not putting
my name
On Fri, May 07, 2021 at 09:09:43AM +0800, Tom Rini wrote:
> On Fri, May 07, 2021 at 09:06:33AM +0800, Leo Liang wrote:
>
> > Hi Tom,
> >
> > CI result:
> > https://source.denx.de/u-boot/custodians/u-boot-riscv/-/pipelines/7400
> >
> > The following changes since commit
On 5/6/21 11:06 PM, Tianrui Wei wrote:
Hi Sean,
Many thanks again for reviewing our code! We really appreciate it. Will fix the
things you're suggesting ;p Though I have a few questions in line in the
comment. Also, checkpatch didn't catch any of the identation issues. I was
wondering if
From: Chaitanya Sakinam
Read Linux PPFE firmware from flash partition and pass it to Linux through
FDT entry. So that we can avoid placing PPFE firmware in Linux rootfs.
(FDT may increase at max by 64KB)
Signed-off-by: Chaitanya Sakinam
Signed-off-by: Anji J
Signed-off-by: Biwen Li
---
Hi Adarsh,
On 06/05/21 11:28 am, Adarsh Babu Kalepalli wrote:
> Hi,
>
> Error output "*Command 'gpio' failed: Error -19*" is occurring with "*gpio
> status -a*"
> command on Beaglebone Black.
Error -19 is ENODEV. Can you check if you have gpio devices being probed?
>
> The "gpio" command is
The CMD_UNZIP provides the 'gzwrite' command, which is convenient
for writing e.g. gz-compressed images to eMMC from U-Boot.
Signed-off-by: Patrick Delaunay
---
configs/stm32mp15_basic_defconfig | 1 +
configs/stm32mp15_trusted_defconfig | 1 +
2 files changed, 2 insertions(+)
diff --git
Hi Lokesh,
> Error -19 is ENODEV. Can you check if you have gpio devices being probed?
The device is not being probed.Output of "dm tree" is captured in
log_uboot-2021.07-rc1.txt
Where as in uboot-2021.04-rc2.txt,the gpio is probed.
The output of "dm tree" is in log_uboot-2021.04-rc2.txt
>>
>>
Chipid will be needed for SoC detection for all stages of U-Boot.
So make it u-boot,dm-spl
Signed-off-by: Lokesh Vutla
---
arch/arm/dts/k3-am642-evm-u-boot.dtsi | 7 +++
1 file changed, 7 insertions(+)
diff --git a/arch/arm/dts/k3-am642-evm-u-boot.dtsi
Now that single defconfig can be used for booting AM64 EVM and SK,
default device tree will not work for selecting dtb for kernel.
Update the env to select right dtb based on eeprom.
Signed-off-by: Lokesh Vutla
---
include/configs/am64x_evm.h | 11 +++
1 file changed, 7 insertions(+), 4
Current BSS allocation of SPL is as below:
size spl/u-boot-spl
textdata bss dec hex filename
14457254841752 151808 25100 spl/u-boot-spl
But 20KB is allocated currently for BSS. Reduce it to 4KB and
save some space for stack.
Signed-off-by: Lokesh Vutla
---
Add I2C nodes for AM64 and enable pinmux for i2c0 for reading eeprom data.
Signed-off-by: Lokesh Vutla
---
arch/arm/dts/k3-am64-main.dtsi| 44 +++
arch/arm/dts/k3-am64.dtsi | 2 ++
arch/arm/dts/k3-am642-evm-u-boot.dtsi | 18 +++
3 files
Avoid R5 SPL stack writing into ROM index table. Re-use the same space
for storing EEPROM data.
Signed-off-by: Lokesh Vutla
---
include/configs/am64x_evm.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/include/configs/am64x_evm.h b/include/configs/am64x_evm.h
index
I2C EEPROM will be probed before SYSFW is available.
So drop the power-domains property for I2C.
Signed-off-by: Lokesh Vutla
---
arch/arm/dts/k3-am642-r5-evm.dts | 5 +
1 file changed, 5 insertions(+)
diff --git a/arch/arm/dts/k3-am642-r5-evm.dts b/arch/arm/dts/k3-am642-r5-evm.dts
index
On 23/04/21 9:57 pm, Dave Gerlach wrote:
> Hi,
>
> This series adds initial support for the latest new SoC, AM642,
> from Texas Instruments.
>
> Additional detail can be found in the patch descriptions, also
> see AM64X Technical Reference Manual (SPRUIM2, Revised Jan 2021)
> for further
On 4/15/21 9:59 AM, Reto Schneider wrote:
> From: Reto Schneider
>
> This commit updates the default config with the values that are actually
> used "in the wild" and which are close to what is used on the MediaTek
> MT7688 based, 2nd generation of the GARDENA smart gateway:
> - Reduce startup
Enable support for selecting DTB within SPL based on EEPROM.
This will help to use single defconfig for both EVM and SK
Signed-off-by: Lokesh Vutla
---
arch/arm/mach-k3/am642_init.c | 26 ++
1 file changed, 26 insertions(+)
diff --git a/arch/arm/mach-k3/am642_init.c
Include the relevant configs to enable support for reading EEPROM in
R5SPL.
Signed-off-by: Lokesh Vutla
---
board/ti/am64x/Kconfig | 3 +++
1 file changed, 3 insertions(+)
diff --git a/board/ti/am64x/Kconfig b/board/ti/am64x/Kconfig
index 3dd5b54c94..7150dbb3b0 100644
---
I2C EEPROM data contains the board name and its revision.
Add support for:
- Reading EEPROM data and store a copy at end of SRAM
- Updating env variable with relevant board info
- Printing board info during boot.
Signed-off-by: Lokesh Vutla
---
arch/arm/mach-k3/include/mach/am64_hardware.h | 3
Update the board_fit_config_name_match() to choose the right dtb.
Signed-off-by: Lokesh Vutla
---
board/ti/am64x/evm.c | 13 +
1 file changed, 9 insertions(+), 4 deletions(-)
diff --git a/board/ti/am64x/evm.c b/board/ti/am64x/evm.c
index 18e49112ac..a1ffcb929c 100644
---
AM642 StarterKit (SK) board is a low cost, small form factor board
designed for TI’s AM642 SoC. It supports the following interfaces:
* 2 GB LPDDR4 RAM
* x2 Gigabit Ethernet interfaces capable of working in switch and MAC mode
* x1 USB 3.0 Type-A port
* x1 UHS-1 capable µSD card slot
* 2.4/5 GHz
Hi,
On 4/29/21 6:10 PM, Simon Glass wrote:
Hi Patrick,
On Wed, 28 Oct 2020 at 03:06, Patrick Delaunay wrote:
Allow pin name parameter for pimux staus command,
as gpio command to get status of one pin.
The possible usage of the command is:
pinmux dev pinctrl
pinmux status
pinmux status -a
Add support for AM64 SoC identification.
Signed-off-by: Lokesh Vutla
---
drivers/soc/soc_ti_k3.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/soc/soc_ti_k3.c b/drivers/soc/soc_ti_k3.c
index ecc3278cb4..9abed7d490 100644
--- a/drivers/soc/soc_ti_k3.c
+++
Add R5 specific dts for AM64 SK
Signed-off-by: Lokesh Vutla
Signed-off-by: Sinthu Raja
---
arch/arm/dts/Makefile|3 +-
arch/arm/dts/k3-am64-sk-lp4-1333MTs.dtsi | 2190 ++
arch/arm/dts/k3-am642-r5-sk.dts | 145 ++
3 files changed, 2337
Enable all relevant configs for printing CPU info.
Signed-off-by: Lokesh Vutla
---
configs/am64x_evm_a53_defconfig | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/configs/am64x_evm_a53_defconfig b/configs/am64x_evm_a53_defconfig
index 5aa4ad3022..43885827a5 100644
---
AM642 StarterKit (SK) board is a low cost, small form factor board
designed for TI’s AM642 SoC. It supports the following interfaces:
* 2 GB LPDDR4 RAM
* x2 Gigabit Ethernet interfaces capable of working in switch and MAC mode
* x1 USB 3.0 Type-A port
* x1 UHS-1 capable µSD card slot
* 2.4/5 GHz
Enable relevant configs for reading eeprom data and updating env
variables.
Signed-off-by: Lokesh Vutla
---
board/ti/am64x/Kconfig | 3 +++
configs/am64x_evm_a53_defconfig | 4
2 files changed, 7 insertions(+)
diff --git a/board/ti/am64x/Kconfig b/board/ti/am64x/Kconfig
index
Enable all relevant configs for building multiple dtbs into a single fit
image and load the right dtb for next stage.
Signed-off-by: Lokesh Vutla
---
configs/am64x_evm_a53_defconfig | 3 +++
1 file changed, 3 insertions(+)
diff --git a/configs/am64x_evm_a53_defconfig
Enable relevant configs that checks for the size of image and stack:
BSS: 4KB
Initial MALLOC: 512KB
Initial Stack: 8K
SPL Image size can be: ~960KB
Signed-off-by: Lokesh Vutla
---
configs/am64x_evm_r5_defconfig | 5 +
1 file changed, 5 insertions(+)
diff --git
Enable defconfigs for building multiple device trees into a single FIT
image.
Signed-off-by: Lokesh Vutla
---
configs/am64x_evm_r5_defconfig | 3 +++
1 file changed, 3 insertions(+)
diff --git a/configs/am64x_evm_r5_defconfig b/configs/am64x_evm_r5_defconfig
index 3d5b22b4a0..31d2238320 100644
Tidy up the warnings.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/image-board.c | 142 ++-
1 file changed, 72 insertions(+), 70 deletions(-)
diff --git a/common/image-board.c b/common/image-board.c
index 0c4e32e14e4..5d3eafb5245 100644
To avoid having #ifdefs in a few functions which are completely different
in the board and host code, create a new image-host.c file.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/image-board.c | 17 +
common/image-host.c | 27 +++
Add a macro to handle manually relocating a pointer. Update the iamge code
to use this to avoid needing #ifdefs.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/image-sig.c | 43 ---
include/relocate.h | 6 ++
2 files changed, 30
Adjust this function so that preprocessor macros are not needed. With
this, the host build uses more of the same header files as the target
build.
Rather than definining CONFIG_SYS_MALLOC_LEN, add a CONSERVE_MEMORY
define, since that is the purpose of the value.
This appears to have no impact on
Rather than adding an #ifdef and open-coding this calculation, add a
helper function to handle it. Use this in the image code.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/image.c | 33 +++--
include/relocate.h | 24 +++-
2
Remove ifdefs in this file, so far as possible without too much
refactoring.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/image-board.c | 111 +++
include/image.h | 7 ++-
2 files changed, 55 insertions(+), 63 deletions(-)
diff
Use a simple return to drop the unwanted code.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/image-fit.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/common/image-fit.c b/common/image-fit.c
index 3ee306143b3..f8aa61fc99d 100644
---
Add a host Kconfig for MD5. With this we can use CONFIG_IS_ENABLED(MD5)
directly in the host build, so drop the unnecessary indirection.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/image-fit.c | 2 +-
include/image.h| 8
tools/Kconfig | 5 +
3 files
These option are named inconsistently with other SPL options, thus making
them incompatible with the CONFIG_IS_ENABLED() macro. Rename them.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/spl/Kconfig | 8
include/image.h| 8
2 files changed, 8 insertions(+),
Make use of the host Kconfig for FIT. With this we can use
CONFIG_IS_ENABLED(FIT) directly in the host build, so drop the unnecessary
indirection.
Signed-off-by: Simon Glass
---
(no changes since v1)
arch/arm/mach-imx/hab.c | 2 +-
common/bootm.c | 10 +-
Add a host Kconfig for CRC32. With this we can use CONFIG_IS_ENABLED(CRC32)
directly in the host build, so drop the unnecessary indirection.
Add a few more conditions to SPL_CRC32 to avoid build failures as well as
TPL_CRC32. Also update hash.c to make crc32 optional.
Signed-off-by: Simon Glass
Drop the _SUPPORT suffix so we can use CONFIG_IS_ENABLED() with this
option.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/spl/Kconfig | 2 +-
include/image.h| 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/common/spl/Kconfig b/common/spl/Kconfig
index
On Wed, Apr 21, 2021 at 11:06:55AM +0200, Rasmus Villemoes wrote:
> Check that a variable defined in /config/environment is found in the
> run-time environment, and that clearing fdt_env_path from within that
> node works.
>
> Reviewed-by: Simon Glass
> Signed-off-by: Rasmus Villemoes
>
On Tue, Apr 20, 2021 at 10:50:58AM -0400, Sean Anderson wrote:
> This adds a test for the gpio-sysinfo driver.
>
> Signed-off-by: Sean Anderson
> Reviewed-by: Simon Glass
Applied to u-boot/master, thanks!
--
Tom
signature.asc
Description: PGP signature
Hi Patrick,
On Thu, 6 May 2021 at 02:38, Patrick DELAUNAY
wrote:
>
> Hi,
>
> On 4/29/21 6:10 PM, Simon Glass wrote:
> > Hi Patrick,
> >
> > On Wed, 28 Oct 2020 at 03:06, Patrick Delaunay
> > wrote:
> >> Allow pin name parameter for pimux staus command,
> >> as gpio command to get status of one
On Wed, Apr 21, 2021 at 11:06:54AM +0200, Rasmus Villemoes wrote:
> It can be useful to use the same U-Boot binary for multiple purposes,
> say the normal one, one for developers that allow breaking into the
> U-Boot shell, and one for use during bootstrapping which runs a
> special-purpose
On 06/05/21 1:32 pm, Lokesh Vutla wrote:
>
>
> On 23/04/21 9:57 pm, Dave Gerlach wrote:
>> Hi,
>>
>> This series adds initial support for the latest new SoC, AM642,
>> from Texas Instruments.
>>
>> Additional detail can be found in the patch descriptions, also
>> see AM64X Technical Reference
Dave,
On 5/4/21 8:22 AM, Dave Gerlach wrote:
> static void k3_lpddr4_ack_freq_upd_req(void)
> {
> - if (infotype == LPDDR4_DRV_SOC_PLL_UPDATE) {
> - j721e_lpddr4_ack_freq_upd_req();
> + u32 dram_class;
> +
> + debug("--->>> LPDDR4 Initialization is in progress ...
On 5/5/21 12:23 PM, Christoph Niedermaier wrote:
Adding new DH electronics mailing list.
Signed-off-by: Christoph Niedermaier
---
board/dhelectronics/dh_stm32mp1/MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/board/dhelectronics/dh_stm32mp1/MAINTAINERS
Hi,
On 5/5/21 6:23 PM, Christoph Niedermaier wrote:
Adding new DH electronics mailing list.
Signed-off-by: Christoph Niedermaier
---
V2: Correct message tag
---
board/dhelectronics/dh_stm32mp1/MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git
When passing a data buffer back from a function, it is not always clear
who owns the buffer, i.e. who is responsible for freeing the memory used.
An example of this is where multiple files are decompressed from the
firmware image, using a temporary buffer for reading (since the
compressed data has
The gzip API uses the u64 type in it, which is not available in the host
build. This makes it impossible to include the header file.
We could make this type available, but it seems unnecessary. Limiting the
compression size to that of the 'unsigned long' type seems good enough. On
32-bit machines
The zstd implementation prints the error in image_decomp() which is
incorrect and does not match other algorithms. Drop this and let the
caller report the error.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/image.c | 8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
At present this function is full of preprocessor macros. Adjust it to
check for an unsupported algorithm after the switch(). This will allow
us to drop the macros.
Fix up the return-value path and an extra blank line while we are here.
Signed-off-by: Simon Glass
---
(no changes since v1)
Drop the ENABLE and SUPPORT parts of this, which are redundant.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/Kconfig.boot | 2 +-
common/image-sig.c | 4 ++--
configs/bcm963158_ram_defconfig | 2 +-
configs/sandbox_defconfig | 2 +-
This is not needed with Kconfig, since we can use IS_ENABLED() easily
enough. Drop it.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/image-fdt.c | 4 ++--
include/image.h| 6 --
2 files changed, 2 insertions(+), 8 deletions(-)
diff --git a/common/image-fdt.c
We can use the __maybe_unused attribute to avoid some of the #ifdefs in
this file. Update the functions accordingly.
Note: The actual hashing interface is still a mess, with four separate
combinations and lots of #ifdefs. This should really use a driver
approach, e.g. as is done with partition
This is not needed with Kconfig, since we can use IS_ENABLED() easily
enough. Drop it.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/image-fit.c | 2 +-
include/image.h| 5 -
2 files changed, 1 insertion(+), 6 deletions(-)
diff --git a/common/image-fit.c
This is not needed with Kconfig, since we can use IS_ENABLED() easily
enough. Drop it.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/image-fdt.c | 2 +-
include/image.h| 6 --
2 files changed, 1 insertion(+), 7 deletions(-)
diff --git a/common/image-fdt.c
The ENABLE part of this name is redundant, since all boolean Kconfig
options serve to enable something. The SUPPORT part is also redundant
since Kconfigs can be assumed to enable support for something. Together
they just serve to make these options overly long and inconsistent
with other options.
Add a host Kconfig for FIT_VERBOSE. With this we can use
CONFIG_IS_ENABLED(FIT_VERBOSE) directly in the host build, so drop the
forcing of this in the image.h header.
Signed-off-by: Simon Glass
---
(no changes since v1)
include/image.h | 5 ++---
tools/Kconfig | 5 +
2 files changed, 7
We already have a host Kconfig for SHA1. Use CONFIG_IS_ENABLED(SHA1)
directly in the code shared with the host build, so we can drop the
unnecessary indirection.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/image-fit.c | 2 +-
include/image.h| 11 ---
2 files
From: Igor Opaniuk
s/psci_features/request_psci_features/g for the case when both
ARCH_SUPPORT_PSCI=y and ARM_PSCI_FW=y, that leads to these
compilation issues:
drivers/firmware/psci.c:69:12: error: conflicting types for 'psci_features'
69 | static int psci_features(u32 psci_func_id)
|
We can use the new host_build() function for this, so drop it.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/image-fit.c | 2 +-
include/image.h| 3 ---
2 files changed, 1 insertion(+), 4 deletions(-)
diff --git a/common/image-fit.c b/common/image-fit.c
index
Move the variables declarations to where they are needed, to reduce the
number of #ifdefs needed.
Signed-off-by: Simon Glass
---
(no changes since v1)
common/image-board.c | 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/common/image-board.c
On Tue, Apr 20, 2021 at 10:50:54AM -0400, Sean Anderson wrote:
> of_parse_phandle_with_args (called by dev_read_phandle_with_args) does not
> support getting the length of a phandle list by using the index -1.
> Instead, use dev_count_phandle_with_args which supports exactly this
> use-case.
>
>
On Tue, Apr 20, 2021 at 10:50:55AM -0400, Sean Anderson wrote:
> Since 07c9e683a4 ("smbios: Allow a few values to come from sysinfo")
> there are common global sysinfo IDs. This patch moved existing IDs above
> SYSINFO_ID_USER.
>
> Signed-off-by: Sean Anderson
> Reviewed-by: Simon Glass
On Fri, Feb 26, 2021 at 10:17:33AM -0800, Farhan Ali wrote:
> This change would enhance the existing 'gpt read' command to allow
> (optionally) writing of the read GPT partitions to an environment
> variable in the UBOOT partitions layout format. This would allow users
> to easily change the
On Mon, Feb 22, 2021 at 07:18:51PM +, Niko Mauno wrote:
> By declaring board-specific board_fdt_chosen_bootargs() the kernel
> command line arguments can be adjusted before injecting to flat dt
> chosen node.
>
> Signed-off-by: Niko Mauno
Applied to u-boot/master, thanks!
--
Tom
On Tue, Apr 20, 2021 at 10:50:56AM -0400, Sean Anderson wrote:
> This has the uclass enforce calling detect() before other methods. This
> allows drivers to cache information in detect() and perform (cheaper)
> retrieval in the other accessors. This also modifies the only instance
> where this
On Tue, Apr 20, 2021 at 10:50:57AM -0400, Sean Anderson wrote:
> This uses the newly-added dm_gpio_get_values_as_int_base3 function to
> implement a sysinfo device. The revision map is stored in the device tree.
>
> Signed-off-by: Sean Anderson
> Reviewed-by: Simon Glass
Applied to
Hi Kishon,
On Thu, 6 May 2021 at 07:09, Kishon Vijay Abraham I wrote:
>
> Hi Simon,
>
> On 06/05/21 5:07 am, Simon Glass wrote:
> > Hi Kishon,
> >
> > On Tue, 4 May 2021 at 21:25, Kishon Vijay Abraham I wrote:
> >>
> >> Hi Simon,
> >>
> >> On 04/05/21 10:28 pm, Simon Glass wrote:
> >>> Hi
Use the common function to avoid code duplication.
Acked-by: Qu Wenruo
Signed-off-by: Simon Glass
---
(no changes since v1)
fs/btrfs/compression.c | 51 +-
1 file changed, 5 insertions(+), 46 deletions(-)
diff --git a/fs/btrfs/compression.c
Much of the image-handling code predates the introduction of Kconfig and
has quite a few #ifdefs in it. It also uses its own IMAGE_... defines to
help reduce the #ifdefs, which is unnecessary now that we can use
IS_ENABLED() et al.
The image code is also where quite a bit of code is shared with
The existing zstd API requires the same sequence of calls to perform its
task. Create a helper for U-Boot, to avoid code duplication, as is done
with other compression algorithms. Make use of of this from the image
code.
Note that the zstd code lacks a test in test/compression.c and this should
Add a function to duplicate a memory region, a little like strdup().
Signed-off-by: Simon Glass
---
Changes in v2:
- Add a patch to introduce a memdup() function
include/linux/string.h | 13 +
lib/string.c | 13 +
test/lib/string.c | 32
1 - 100 of 132 matches
Mail list logo
