Re: [PATCH V2] ARM: imx: imx8m: Adjust thermal trip points for Industrial parts

2022-05-07 Thread Adam Ford
On Sat, May 7, 2022 at 5:00 PM Marek Vasut wrote: > > On 5/7/22 14:04, Adam Ford wrote: > > If the thermal sensor is enabled in U-Boot, adjust the cpu-thermal > > trip points for industrial rated parts. This should apply to 8MQ, > > 8MM, 8MN, and 8MP. > > > > Signed-off-by: Adam Ford > > Reviewe

[PATCH] ARM: imx: mx5: Convert MX53 Menlo board to DM I2C and DM RTC

2022-05-07 Thread Marek Vasut
Convert the board to DM I2C and DM RTC. This leads to removal of board side iomuxc configuration, which is now done using pin control driver, and conversion of board side legacy I2C accessors to DM ones. Signed-off-by: Marek Vasut Cc: Stefano Babic --- board/menlo/m53menlo/m53menlo.c | 27 +

Re: [PATCH v7 6/7] fpga: zynqmp: support loading authenticated images

2022-05-07 Thread Oleksandr Suvorov
Hi Michal, On Tue, May 3, 2022 at 10:56 AM Michal Simek wrote: > > > > On 4/11/22 20:00, Adrian Fiergolski wrote: > > From: Oleksandr Suvorov > > > > Add supporting new compatible string "u-boot,zynqmp-fpga-ddrauth" to > > handle loading authenticated images (DDR). > > > > Based on solution by J

Re: [PATCH V2] ARM: imx: imx8m: Adjust thermal trip points for Industrial parts

2022-05-07 Thread Marek Vasut
On 5/7/22 14:04, Adam Ford wrote: If the thermal sensor is enabled in U-Boot, adjust the cpu-thermal trip points for industrial rated parts. This should apply to 8MQ, 8MM, 8MN, and 8MP. Signed-off-by: Adam Ford Reviewed-by: Tim Harvey --- V2: Switch the check from looking for industrial or c

[PATCH 1/1] Makefile: support CROSS_COMPILE=armv7a-*

2022-05-07 Thread Heinrich Schuchardt
Gentoo uses armv7a-hardfloat-linux-gnueabi- as cross compiler prefix. Signed-off-by: Heinrich Schuchardt --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index ea80f00716..d9081815fd 100644 --- a/Makefile +++ b/Makefile @@ -30,7 +30,7 @@ else i

[PATCH v2 1/1] dm: fix DM_EVENT dependencies

2022-05-07 Thread Heinrich Schuchardt
CONFIG_DM_EVENT without CONFIG_EVENT is non-functional. Let CONFIG_DM_EVENT depend on CONFIG_EVENT. Remove superfluous stub in include/event.h. Fixes: 5b896ed5856f ("event: Add events for device probe/remove") Reported-by: Jan Kiszka Signed-off-by: Heinrich Schuchardt --- v2: Let DM_EVE

Re: [PATCH] efi_loader: Select EVENT as well

2022-05-07 Thread Heinrich Schuchardt
On 4/27/22 07:47, Jan Kiszka wrote: From: Jan Kiszka Fixes WARNING: unmet direct dependencies detected for EVENT_DYNAMIC Depends on [n]: EVENT [=n] Selected by [y]: - EFI_LOADER [=y] && OF_LIBFDT [=y] && ... and the succeeding build breakage. Signed-off-by: Jan Kiszka Reviewed-by

Re: [PATCH] RFC: net: fec: get rid of CONFIG_FEC_MXC_MDIO_BASE

2022-05-07 Thread Oleh Kravchenko
Hello Tim, Your patch looks good, and I'm pleased to verify it. But I don't have access to the o4-imx6ull-nano board. 06.05.22 01:12, Tim Harvey пише: A number of IMX8Q boards using FEC configure the address of the ethernet controller via defconfig as there are two FEC controllers in the SoC. N

Re: [UNTESTED PATCH] Nokia RX-51: Convert to CONFIG_DM_SERIAL

2022-05-07 Thread Merlijn Wajer
Hi Pali, Tom, On 13/04/2022 21:34, Pali Rohár wrote: For CONFIG_DM_SERIAL it is required to increase CONFIG_SYS_MALLOC_F_LEN as default value is not enough for memory hungry CONFIG_DM_SERIAL code. Signed-off-by: Pali Rohár Tested-by: Merlijn Wajer Regards, Merlijn

Re: Nokia RX51 U-Boot USB support updates?

2022-05-07 Thread Tom Rini
On Sat, May 07, 2022 at 05:18:02PM +0200, Merlijn Wajer wrote: > Hi Tom, Pali, > > On 25/07/2021 15:43, Tom Rini wrote: > > On Sun, Jul 25, 2021 at 12:40:13PM +0200, Merlijn Wajer wrote: > > > Hi Tom, > > > > > > On 13/07/2021 22:33, Tom Rini wrote: > > > > Hey all, > > > > > > > > So, the DM_US

Re: [UNTESTED PATCH] Nokia RX-51: Convert to CONFIG_DM_SERIAL

2022-05-07 Thread Tom Rini
On Sat, May 07, 2022 at 05:16:57PM +0200, Merlijn Wajer wrote: > Hi Pali, Tom, > > On 13/04/2022 21:34, Pali Rohár wrote: > > For CONFIG_DM_SERIAL it is required to increase CONFIG_SYS_MALLOC_F_LEN as > > default value is not enough for memory hungry CONFIG_DM_SERIAL code. > > > > Signed-off-by:

Re: Nokia RX51 U-Boot USB support updates?

2022-05-07 Thread Merlijn Wajer
Hi Tom, Pali, On 25/07/2021 15:43, Tom Rini wrote: On Sun, Jul 25, 2021 at 12:40:13PM +0200, Merlijn Wajer wrote: Hi Tom, On 13/07/2021 22:33, Tom Rini wrote: Hey all, So, the DM_USB migration deadline issue has been addressed by the series here: https://patchwork.ozlabs.org/project/uboot/li

Re: [UNTESTED PATCH] Nokia RX-51: Convert to CONFIG_DM_SERIAL

2022-05-07 Thread Merlijn Wajer
Hi Pali, Tom, On 13/04/2022 21:34, Pali Rohár wrote: For CONFIG_DM_SERIAL it is required to increase CONFIG_SYS_MALLOC_F_LEN as default value is not enough for memory hungry CONFIG_DM_SERIAL code. Signed-off-by: Pali Rohár --- Who has serial console on the real device, please test this patch i

Re: [PATCH 2/2] arm: mvebu: add support for Methode eDPU

2022-05-07 Thread Pali Rohár
On Friday 06 May 2022 20:01:40 Robert Marko wrote: > Methode eDPU is an Armada 3720 power board based on the Methode uDPU. > > They feature the same CPU, RAM, and storage as well as the form factor. > > However, eDPU only has one SFP slot plus a copper G.hn port which does not > work under U-boot

Re: [PATCH 1/2] pinctrl: probe pinctrl drivers during post-bind

2022-05-07 Thread Pali Rohár
On Friday 06 May 2022 20:01:39 Robert Marko wrote: > Currently, pinctrl drivers only get probed if pinconf is actually being > used, however on SoC-s like Armada 3720 pinctrl driver is a also the GPIO > driver. > > So, if the pinctrl driver doesn't get probed GPIO-s won't get registered > and thus

Re: [PATCH 0/7] arm64: binman: use binman symbols for imx

2022-05-07 Thread Tom Rini
On Sat, May 07, 2022 at 05:21:35PM +0800, Peng Fan (OSS) wrote: > From: Peng Fan > > binman symbol is a good feature, but only used on X86 for now. This patchset > is to use it for i.MX8M platform. > > The current imx8m ddr phy firmware consumes lots of space, because we pad > them to the large

Re: [PATCH 1/7] spl: guard u_boot_any with X86

2022-05-07 Thread Tom Rini
On Sat, May 07, 2022 at 05:21:36PM +0800, Peng Fan (OSS) wrote: > From: Peng Fan > > set the symbol as weak not work if LTO is enabled. Since u_boot_any is > only used on X86 for now, so guard it with X86, otherwise build break > if we use BINMAN_SYMBOLS on i.MX. > > Signed-off-by: Peng Fan >

[PATCH 1/1] cmd/sbi: add implementation ID 6 - Coffer

2022-05-07 Thread Heinrich Schuchardt
The sbi command displays the ID of the implementation of the RISC-V Supervisor Binary Interface Specification. A new ID for Coffer has recently been added. Signed-off-by: Heinrich Schuchardt --- cmd/riscv/sbi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/cmd/riscv/sbi.c b/cmd/riscv/sbi.c

[PATCH] boot: image-pre-load: drop unused CONFIG_SYS_BOOTM_LEN

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan CONFIG_SYS_BOOTM_LEN is not used in this file, drop it. Signed-off-by: Peng Fan --- boot/image-pre-load.c | 5 - 1 file changed, 5 deletions(-) diff --git a/boot/image-pre-load.c b/boot/image-pre-load.c index 78d89069a98..5ab9ae18746 100644 --- a/boot/image-pre-load.c +++ b

[PATCH V2] ARM: imx: imx8m: Adjust thermal trip points for Industrial parts

2022-05-07 Thread Adam Ford
If the thermal sensor is enabled in U-Boot, adjust the cpu-thermal trip points for industrial rated parts. This should apply to 8MQ, 8MM, 8MN, and 8MP. Signed-off-by: Adam Ford Reviewed-by: Tim Harvey --- V2: Switch the check from looking for industrial or checkoing for anything but comme

Re: [PATCH] ARM: imx: imx8m: Adjust thermal trip points for Industrial parts

2022-05-07 Thread Adam Ford
On Thu, May 5, 2022 at 2:42 PM Francesco Dolcini wrote: > > On Thu, May 05, 2022 at 01:10:27PM -0500, Adam Ford wrote: > > If the thermal sensor is enabled in U-Boot, adjust the cpu-thermal > > trip points for industrial rated parts. > Is this supposed to work fine with any i.MX8M? Mini/Plus/... ?

[PATCH V2 7/7] binman_sym: guard with CONFIG_SPL_BINMAN_SYMBOLS

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan There is case that CONFIG_BINMAN is defined, but CONFIG_SPL_BINMAN_SYMBOLS is not defined. In that case, there will be build failure. So use CONFIG_SPL_BINMAN_SYMBOLS to guard the macros, and define CONFIG_SPL_BINMAN_SYMBOLS in binman syms test. Signed-off-by: Peng Fan --- inclu

[PATCH V2 6/7] arm: dts: imx8m: shrink ddr firmware size to actual file size

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan After we switch to use BINMAN_SYMBOLS, there is no need to pad the file size to 0x8000 and 0x4000. After we use BINMAN_SYMBOLS, the u-boot-spl-ddr.bin shrink about 36KB with i.MX8MP-EVK. Signed-off-by: Peng Fan --- arch/arm/dts/imx8mm-u-boot.dtsi | 8 a

[PATCH V2 5/7] ddr: imx8m: helper: load ddr firmware according to binman symbols

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan By reading binman symbols, we no need hard coded IMEM_LEN/DMEM_LEN after we update the binman dtsi to drop 0x8000/0x4000 length for the firmware. And that could save binary size for many KBs. Signed-off-by: Peng Fan --- drivers/ddr/imx/imx8m/helper.c | 53 ++

[PATCH V2 3/7] armv8: u-boot-spl.lds: mark __image_copy_start as symbol

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan In arch/arm/lib/sections.c there is below code: char __image_copy_start[0] __section(".__image_copy_start"); But actually 'objdump -t spl/u-boot-spl' not able to find out symbol '__image_copy_start' for binman update image-pos/size. So update link file Signed-off-by: Peng Fan --

[PATCH V2 4/7] tools: binman: section: replace @ with -

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan In arch/arm/dts/imx8mp-u-boot.dtsi, there are blob-ext@1, blob-ext@2 and etc which is for packing ddr phy firmware. However we could not declare symbol name such as 'binman_sym_declare(ulong, blob_ext@1, image_pos)', because '@' is not allowed, so we choose to declare the symbol 'b

[PATCH V2 2/7] arm: dts: imx8m: update binman ddr firmware node name

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan We are migrating to use BINMAN SYMBOLS, the current name is not a valid binman type, so update to use blob-ext@[1,2,3,4]. Signed-off-by: Peng Fan --- arch/arm/dts/imx8mm-u-boot.dtsi | 8 arch/arm/dts/imx8mn-bsh-smm-s2-u-boot-common.dtsi | 4 ++-- arch/

[PATCH V2 1/7] spl: guard u_boot_any with X86

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan set the symbol as weak not work if LTO is enabled. Since u_boot_any is only used on X86 for now, so guard it with X86, otherwise build break if we use BINMAN_SYMBOLS on i.MX. Signed-off-by: Peng Fan --- common/spl/spl.c | 8 ++-- common/spl/spl_ram.c | 4 2 files ch

[PATCH V2 0/7] arm64: binman: use binman symbols for imx

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan V2: resolve some CI failure include patch 7 binman symbol is a good feature, but only used on X86 for now. This patchset is to use it for i.MX8M platform. The current imx8m ddr phy firmware consumes lots of space, because we pad them to the largest 32KB and 16KB for IMEM and DM

Re: [PATCH 0/7] arm64: binman: use binman symbols for imx

2022-05-07 Thread Michael Nazzareno Trimarchi
Hi Peng On Sat, May 7, 2022 at 10:39 AM Peng Fan (OSS) wrote: > > From: Peng Fan > > binman symbol is a good feature, but only used on X86 for now. This patchset > is to use it for i.MX8M platform. > > The current imx8m ddr phy firmware consumes lots of space, because we pad > them to the larges

[PATCH] binman_sym: guard with CONFIG_SPL_BINMAN_SYMBOLS

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan There is case that CONFIG_BINMAN is defined, but CONFIG_SPL_BINMAN_SYMBOLS is not defined. In that case, there will be build failure. So use CONFIG_SPL_BINMAN_SYMBOLS to guard the macros, and define CONFIG_SPL_BINMAN_SYMBOLS in binman syms test. Signed-off-by: Peng Fan --- inclu

[PATCH 7/7] arm: dts: imx8m: shrink ddr firmware size to actual file size

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan After we switch to use BINMAN_SYMBOLS, there is no need to pad the file size to 0x8000 and 0x4000. After we use BINMAN_SYMBOLS, the u-boot-spl-ddr.bin shrink about 36KB with i.MX8MP-EVK. Signed-off-by: Peng Fan --- arch/arm/dts/imx8mm-u-boot.dtsi | 8 a

[PATCH 6/7] ddr: imx8m: helper: load ddr firmware according to binman symbols

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan By reading binman symbols, we no need hard coded IMEM_LEN/DMEM_LEN after we update the binman dtsi to drop 0x8000/0x4000 length for the firmware. And that could save binary size for many KBs. Signed-off-by: Peng Fan --- drivers/ddr/imx/imx8m/helper.c | 45 ++

[PATCH 5/7] tools: binman: section: replace @ with -

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan In arch/arm/dts/imx8mp-u-boot.dtsi, there are blob-ext@1, blob-ext@2 and etc which is for packing ddr phy firmware. However we could not declare symbol name such as 'binman_sym_declare(ulong, blob_ext@1, image_pos)', because '@' is not allowed, so we choose to declare the symbol 'b

[PATCH 4/7] binman_sym: update symbol alignment to 8 bytes

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan To ARM64 SPL which normally not enable MMU and Dcache, there will be data abort if the symbol is not 8 bytes aligned, because symbol type is ulong which is 8 bytes. So update to 8 bytes aligned. Signed-off-by: Peng Fan --- include/binman_sym.h | 6 +++--- 1 file changed, 3 inser

[PATCH 3/7] armv8: u-boot-spl.lds: mark __image_copy_start as symbol

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan In arch/arm/lib/sections.c there is below code: char __image_copy_start[0] __section(".__image_copy_start"); But actually 'objdump -t spl/u-boot-spl' not able to find out symbol '__image_copy_start' for binman update image-pos/size. So update link file Signed-off-by: Peng Fan --

[PATCH 2/7] arm: dts: imx8m: update binman ddr firmware node name

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan We are migrating to use BINMAN SYMBOLS, the current name is not a valid binman type, so update to use blob-ext@[1,2,3,4]. Signed-off-by: Peng Fan --- arch/arm/dts/imx8mm-u-boot.dtsi | 8 arch/arm/dts/imx8mn-bsh-smm-s2-u-boot-common.dtsi | 4 ++-- arch/

[PATCH 1/7] spl: guard u_boot_any with X86

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan set the symbol as weak not work if LTO is enabled. Since u_boot_any is only used on X86 for now, so guard it with X86, otherwise build break if we use BINMAN_SYMBOLS on i.MX. Signed-off-by: Peng Fan --- common/spl/spl.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(

[PATCH 0/7] arm64: binman: use binman symbols for imx

2022-05-07 Thread Peng Fan (OSS)
From: Peng Fan binman symbol is a good feature, but only used on X86 for now. This patchset is to use it for i.MX8M platform. The current imx8m ddr phy firmware consumes lots of space, because we pad them to the largest 32KB and 16KB for IMEM and DMEM. With this patchset we use binman symbols t

Re: [PATCH 1/2 v4] efi_loader: add sha384/512 on certificate revocation

2022-05-07 Thread Ilias Apalodimas
Hi Heinrich, [...] > > /* > > @@ -500,7 +528,9 @@ bool efi_signature_verify(struct efi_image_regions > > *regs, > >*/ > > if (!msg->data && > > !efi_hash_regions(regs->reg, regs->num, > > - (v