Re: Broken CONFIG_SPL_SATA_SUPPORT=y

2021-08-15 Thread Tom Rini
On Sun, Aug 15, 2021 at 01:48:26PM +0200, Pali Rohár wrote: > On Monday 02 August 2021 21:40:07 Tom Rini wrote: > > OK, thanks. Then the right fix is to introduce whatever SPL_xxx symbols > > you need so that you can turn off what you don't want in SPL but leave > > it enabled in full U-Boot, and

Pull request for efi-2021-10-rc2-2

2021-08-15 Thread Heinrich Schuchardt
Dear Tom, The following changes since commit 85ccbf666e549f0b06c29d565b9e4fdd87cf6600: Merge https://source.denx.de/u-boot/custodians/u-boot-x86 (2021-08-13 08:37:47 -0400) are available in the Git repository at: https://source.denx.de/u-boot/custodians/u-boot-efi.git

Re: [PATCH 13/14] lmb: Mark arch_lmb_reserve() as weak symbol

2021-08-15 Thread Tom Rini
On Sun, Aug 15, 2021 at 08:13:13PM +0200, Marek Vasut wrote: > Mark arch_lmb_reserve() weak on architecture level, so it can be > overridden if necessary. This might be necessary if there is some > sort of architectural exception where e.g. more LMB areas have to > be reserved. > > Note that

Re: [PATCH 0/3] common: Add fdt network helper

2021-08-15 Thread Tony Dinh
Hi Simon, On Sun, Aug 15, 2021 at 7:10 AM Simon Glass wrote: > > Hi Tony, > > On Thu, 5 Aug 2021 at 22:49, Tony Dinh wrote: > > > > > > At the moment, there is no common fdt helper function specific to decoding > > network related > > information from FDTs. This new helper functional group

Re: [bug?] SiFive Unmatched default boot script update?

2021-08-15 Thread Alexander von Gluck IV
August 15, 2021 9:59 AM, "Alexander von Gluck IV" wrote: > Good morning! > > Mainline Haiku is now getting into the kernel on the SiFive Unmatched > board! > > Our current boot process involves running the following from u-boot > to boot Haiku from a USB drive: > > pci enum ; usb start ; run

[PATCH 06/14] lmb: Switch to generic arch_lmb_reserve_generic()

2021-08-15 Thread Marek Vasut
Switch arc/arm/m68k/microblaze/mips/ppc arch_lmb_reserve() to arch_lmb_reserve_generic(). Signed-off-by: Marek Vasut Cc: Alexey Brodkin Cc: Angelo Dureghello Cc: Daniel Schwierzeck Cc: Eugeniy Paltsev Cc: Hai Pham Cc: Michal Simek Cc: Simon Goldschmidt Cc: Tom Rini Cc: Wolfgang Denk ---

[PATCH 08/14] lmb: nds32: Add arch_lmb_reserve()

2021-08-15 Thread Marek Vasut
Add arch_lmb_reserve() implemented using arch_lmb_reserve_generic(). It is rather likely this architecture also needs to cover U-Boot with LMB before booting Linux. Signed-off-by: Marek Vasut Cc: Rick Chen Cc: Simon Goldschmidt Cc: Tom Rini --- arch/nds32/lib/bootm.c | 13 + 1

[PATCH 07/14] lmb: nios2: Add arch_lmb_reserve()

2021-08-15 Thread Marek Vasut
Add arch_lmb_reserve() implemented using arch_lmb_reserve_generic(). It is rather likely this architecture also needs to cover U-Boot with LMB before booting Linux. Signed-off-by: Marek Vasut Cc: Simon Goldschmidt Cc: Thomas Chou Cc: Tom Rini --- arch/nios2/lib/bootm.c | 16

[PATCH 04/14] lmb: Always compile arch_lmb_reserve() into U-Boot on arc

2021-08-15 Thread Marek Vasut
The arch_lmb_reserve() is called by lib/lmb.c lmb_reserve_common() even if CMD_BOOTM is not enabled. However, the arc variant of arch_lmb_reserve() is only compiled in if CMD_BOOTM is enabled. This currently does not trigger build error, because there is an empty weak implementation of

[PATCH 10/14] lmb: sh: Add arch_lmb_reserve()

2021-08-15 Thread Marek Vasut
Add arch_lmb_reserve() implemented using arch_lmb_reserve_generic(). This architecture also needs to cover U-Boot with LMB before booting Linux. Signed-off-by: Marek Vasut Cc: Simon Goldschmidt Cc: Tom Rini --- arch/sh/lib/bootm.c | 16 1 file changed, 16 insertions(+) diff

[PATCH 09/14] lmb: riscv: Add arch_lmb_reserve()

2021-08-15 Thread Marek Vasut
Add arch_lmb_reserve() implemented using arch_lmb_reserve_generic(). It is rather likely this architecture also needs to cover U-Boot with LMB before booting Linux. Signed-off-by: Marek Vasut Cc: Atish Patra Cc: Leo Cc: Rick Chen Cc: Simon Goldschmidt Cc: Tom Rini ---

Re: [PATCH 14/14] lmb: Switch imx board_lmb_reserve() to arch_lmb_reserve()

2021-08-15 Thread Tom Rini
On Sun, Aug 15, 2021 at 08:13:14PM +0200, Marek Vasut wrote: > This function is clearly architecture specific code, not board specific > code. The only difference from the generic arm arch_lmb_reserve() is the > extra reservation of 16k of memory below the stack bottom, rather than > the default

Re: [PATCH 02/14] lmb: Use CONFIG_LMB_*_REGIONS only if they are defined

2021-08-15 Thread Tom Rini
On Sun, Aug 15, 2021 at 08:13:02PM +0200, Marek Vasut wrote: > The CONFIG_LMB_*_REGIONS are defined only if CONFIG_LMB is enabled, > protect access to those two config options to avoid undefined macro > errors. > > Signed-off-by: Marek Vasut > Cc: Simon Glass > Cc: Simon Goldschmidt > Cc: Tom

Re: [PATCH 03/14] lmb: Always compile arch_lmb_reserve() into U-Boot on arm

2021-08-15 Thread Tom Rini
On Sun, Aug 15, 2021 at 08:13:03PM +0200, Marek Vasut wrote: > The arch_lmb_reserve() is called by lib/lmb.c lmb_reserve_common() even > if CMD_BOOT{I,M,Z} is not enabled. However, the arm32/arm64 variant of > arch_lmb_reserve() is only compiled in if CMD_BOOT{I,M,Z} is enabled. > > This

Re: [PATCH 06/14] lmb: Switch to generic arch_lmb_reserve_generic()

2021-08-15 Thread Tom Rini
On Sun, Aug 15, 2021 at 08:13:06PM +0200, Marek Vasut wrote: > Switch arc/arm/m68k/microblaze/mips/ppc arch_lmb_reserve() to > arch_lmb_reserve_generic(). > > Signed-off-by: Marek Vasut > Cc: Alexey Brodkin > Cc: Angelo Dureghello > Cc: Daniel Schwierzeck > Cc: Eugeniy Paltsev > Cc: Hai

[PATCH] config/sifive_unmatched: add preboot commands

2021-08-15 Thread Alexander von Gluck IV
* Opens up u-boot to properly search through USB media for a valid boot media * The MMC on the unmatched is slow, so a model of running the OS from USB or NVMe is a compelling one. (putting just u-boot on the sd card) --- configs/sifive_unmatched_defconfig | 1 + 1 file changed, 1

[PATCH 13/14] lmb: Mark arch_lmb_reserve() as weak symbol

2021-08-15 Thread Marek Vasut
Mark arch_lmb_reserve() weak on architecture level, so it can be overridden if necessary. This might be necessary if there is some sort of architectural exception where e.g. more LMB areas have to be reserved. Note that sandbox now needs an empty implementation of arch_lmb_reserve().

[PATCH 11/14] lmb: xtensa: Add arch_lmb_reserve()

2021-08-15 Thread Marek Vasut
Add arch_lmb_reserve() implemented using arch_lmb_reserve_generic(). It is rather likely this architecture also needs to cover U-Boot with LMB before booting Linux. Signed-off-by: Marek Vasut Cc: Chris Zankel Cc: Simon Goldschmidt Cc: Tom Rini --- arch/xtensa/lib/bootm.c | 12 1

[PATCH 14/14] lmb: Switch imx board_lmb_reserve() to arch_lmb_reserve()

2021-08-15 Thread Marek Vasut
This function is clearly architecture specific code, not board specific code. The only difference from the generic arm arch_lmb_reserve() is the extra reservation of 16k of memory below the stack bottom, rather than the default 4k. Switch this from board_lmb_reserve() to arch_lmb_reserve() and use

[PATCH 12/14] lmb: x86: Add arch_lmb_reserve()

2021-08-15 Thread Marek Vasut
Add arch_lmb_reserve() implemented using arch_lmb_reserve_generic(). It is rather likely this architecture also needs to cover U-Boot with LMB before booting Linux. Signed-off-by: Marek Vasut Cc: Simon Glass Cc: Simon Goldschmidt Cc: Tom Rini --- arch/x86/lib/bootm.c | 18 ++

[PATCH 2/2] ata: ahci-pci: Add new option CONFIG_SPL_AHCI_PCI

2021-08-15 Thread Pali Rohár
This new option allows to disable ahci-pci driver in SPL. Disabling it is needed when SPL_PCI is not enabled as ahci-pci depends on PCI. This change fixes following compile error when CONFIG_SPL_SATA_SUPPORT is enabled and SPL_PCI is disabled. LD spl/u-boot-spl

[PATCH 1/2] ata: ahci-pci: Fix dependency on DM_PCI

2021-08-15 Thread Pali Rohár
File drivers/ata/ahci-pci.c calls function ahci_probe_scsi_pci() which is compiled only when DM_PCI is enabled. So add missing dependency into Kconfig. Signed-off-by: Pali Rohár --- drivers/ata/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/ata/Kconfig b/drivers/ata/Kconfig

[PATCH] Allow colon in PXE bootfile URLs

2021-08-15 Thread Lyle Franklin
From: Lyle Franklin Date: Sun, 15 Aug 2021 07:17:14 -0400 Subject: [PATCH] Allow colon in PXE bootfile URLs - U-boot's PXE flow supports prefixing your bootfile name with an IP address to fetch from a server other than the DHCP server, e.g. `hostIPaddr:bootfilename`:

Re: [PATCH 0/3] common: Add fdt network helper

2021-08-15 Thread Simon Glass
Hi Tony, On Thu, 5 Aug 2021 at 22:49, Tony Dinh wrote: > > > At the moment, there is no common fdt helper function specific to decoding > network related > information from FDTs. This new helper functional group fdt_support_net is > intended to be used > by board-specific code within U-Boot

[bug?] SiFive Unmatched default boot script update?

2021-08-15 Thread Alexander von Gluck IV
Good morning! Mainline Haiku is now getting into the kernel on the SiFive Unmatched board! Our current boot process involves running the following from u-boot to boot Haiku from a USB drive: pci enum ; usb start ; run bootcmd_usb0 I feel like "pci enum; usb start;" should be default things

[PATCH 03/14] lmb: Always compile arch_lmb_reserve() into U-Boot on arm

2021-08-15 Thread Marek Vasut
The arch_lmb_reserve() is called by lib/lmb.c lmb_reserve_common() even if CMD_BOOT{I,M,Z} is not enabled. However, the arm32/arm64 variant of arch_lmb_reserve() is only compiled in if CMD_BOOT{I,M,Z} is enabled. This currently does not trigger build error, because there is an empty weak

[PATCH 02/14] lmb: Use CONFIG_LMB_*_REGIONS only if they are defined

2021-08-15 Thread Marek Vasut
The CONFIG_LMB_*_REGIONS are defined only if CONFIG_LMB is enabled, protect access to those two config options to avoid undefined macro errors. Signed-off-by: Marek Vasut Cc: Simon Glass Cc: Simon Goldschmidt Cc: Tom Rini --- include/lmb.h | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH 05/14] lmb: Add generic arch_lmb_reserve_generic()

2021-08-15 Thread Marek Vasut
The arc/arm/m68k/microblaze/mips/ppc arch_lmb_reserve() implementations are all mostly the same, except for a couple of details. Implement a generic arch_lmb_reserve_generic() function which can be parametrized enough to cater for those differences between architectures. This can also be

[PATCH 01/14] configs: Disable LMB and BDI for tools-only

2021-08-15 Thread Marek Vasut
These two options are useless for tools-only build, since they pull in LMB support which is only useful in a running U-Boot. Signed-off-by: Marek Vasut Cc: Simon Glass Cc: Simon Goldschmidt Cc: Tom Rini --- configs/tools-only_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git

Re: [PATCH 1/2] imx8mm-cl-iot-gate: Do not build fip.bin by default

2021-08-15 Thread Paul Liu
Hi Fabio, No. I think they actually use fip.bin but just not upstreamed yet. Please see the "Firmware version" in the list. For Compulab we are using mainline U-boot. But for imx8mm-evk it is NXP released firmware. And yes we do use mainline ATF and mainline OPTEE. I'll explain how to build all

Re: [PATCH 05/14] lmb: Add generic arch_lmb_reserve_generic()

2021-08-15 Thread Tom Rini
On Sun, Aug 15, 2021 at 08:13:05PM +0200, Marek Vasut wrote: > The arc/arm/m68k/microblaze/mips/ppc arch_lmb_reserve() implementations > are all mostly the same, except for a couple of details. Implement a > generic arch_lmb_reserve_generic() function which can be parametrized > enough to cater

Re: [PATCH v2 3/3] doc: Add a note about why devicetree is used

2021-08-15 Thread AKASHI Takahiro
Heinrich, On Sat, Aug 14, 2021 at 09:21:25AM +0200, Heinrich Schuchardt wrote: > On 8/2/21 2:57 AM, Simon Glass wrote: > > This question comes up every now and then with people coming from Linux. > > Add some notes about it so we can point to it in the mailing list. > > > > Signed-off-by: Simon

[PATCH v1] imx: spl: fix imx8m secure boot

2021-08-15 Thread Heiko Schocher
inspired from NXP code: 719d665a87c6: ("MLK-20467 imx8m: Fix issue for booting signed image through uuu") fix loading FIT buffer address to fix position CONFIG_SYS_TEXT_BASE - CONFIG_FIT_EXTERNAL_OFFSET and fix secure boot on imx8m based boards. Problem was that FIT header and so IVT header

[PATCH 1/1] doc: move i.MX7D/i.MX8MM A/B booting to board specific

2021-08-15 Thread Heinrich Schuchardt
Having "i.MX7D/i.MX8MM SRC_GPR10 PERSIST_SECONDARY_BOOT for bootloader A/B switching" at the top level of the documentation tree does not make sense. Move it to board specific information. Signed-off-by: Heinrich Schuchardt --- doc/board/freescale/index.rst | 1 + doc/{imx/misc =>

[PATCH] riscv: Add missing sentinel in ocores_i2c.c

2021-08-15 Thread Thomas Skibo
The ocores_i2c.c driver is missing a sentinel at the end of the compatible strings list. This causes the "dm compat" command to spew garbage. Signed-off-by: Thomas Skibo --- drivers/i2c/ocores_i2c.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/i2c/ocores_i2c.c

Re: Pull request for efi-2021-10-rc2-2

2021-08-15 Thread Tom Rini
On Sun, Aug 15, 2021 at 06:28:54PM +0200, Heinrich Schuchardt wrote: > Dear Tom, > > The following changes since commit 85ccbf666e549f0b06c29d565b9e4fdd87cf6600: > > Merge https://source.denx.de/u-boot/custodians/u-boot-x86 (2021-08-13 > 08:37:47 -0400) > > are available in the Git

[PATCH 7/7] doc: imx8mq_evk: update doc after using binman

2021-08-15 Thread Peng Fan (OSS)
From: Peng Fan Update doc after using binman to pack images Signed-off-by: Peng Fan --- doc/board/freescale/imx8mq_evk.rst | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/doc/board/freescale/imx8mq_evk.rst b/doc/board/freescale/imx8mq_evk.rst index

[PATCH 5/7] imx: makefile: drop the use of imx8mimage.sh

2021-08-15 Thread Peng Fan (OSS)
From: Peng Fan After switch to use binman, no need to use the bash script to check file exsiting or not. And there is bug that the script will be executed everytime Makefile is used which is confusing people. Signed-off-by: Peng Fan --- arch/arm/mach-imx/Makefile | 13 +++-- 1 file

[PATCH] ARM: Prevent the compiler from using NEON registers

2021-08-15 Thread Samuel Holland
For ARMv8-A, NEON is standard, so the compiler can use it even when no special target flags are provided. For example, it can use stores from NEON registers to zero-initialize large structures. GCC 11 decides to do this inside the DRAM init code for the Allwinner H6, which breaks boot on that

[PATCH 3/7] imx: imx8mq_phanbell: switch to binman

2021-08-15 Thread Peng Fan (OSS)
From: Peng Fan Switch to binman to pack images Signed-off-by: Peng Fan --- arch/arm/dts/imx8mq-phanbell-u-boot.dtsi | 2 ++ arch/arm/mach-imx/imx8m/Kconfig | 7 --- board/google/imx8mq_phanbell/imximage.cfg | 11 +++ configs/imx8mq_phanbell_defconfig | 4 ++--

[PATCH 2/7] imx: imx8mq_evk: switch to binman

2021-08-15 Thread Peng Fan (OSS)
From: Peng Fan Switch to use binman to pack images Signed-off-by: Peng Fan --- arch/arm/dts/imx8mq-evk-u-boot.dtsi | 2 + arch/arm/dts/imx8mq-u-boot.dtsi | 122 arch/arm/mach-imx/imx8m/Kconfig | 1 + board/freescale/imx8mq_evk/imximage.cfg |

[PATCH 4/7] imx: pico-imx8mq: switch to use binman

2021-08-15 Thread Peng Fan (OSS)
From: Peng Fan Switch to use binman to pack images Signed-off-by: Peng Fan --- arch/arm/dts/imx8mq-pico-pi.dts | 1 + arch/arm/mach-imx/imx8m/Kconfig | 1 + board/technexion/pico-imx8mq/imximage.cfg | 11 +++ configs/pico-imx8mq_defconfig | 4 ++-- 4

[PATCH 6/7] imx: imx8mq use common imximage.cfg

2021-08-15 Thread Peng Fan (OSS)
From: Peng Fan After all these board switch to binman, we could use common imximage.cfg Signed-off-by: Peng Fan --- arch/arm/mach-imx/imx8m/imximage.cfg | 12 +++- board/freescale/imx8mq_evk/imximage.cfg | 11 --- board/google/imx8mq_phanbell/imximage.cfg | 11

[PATCH 1/7] tools: imx8mimage: not abort when mmap fail

2021-08-15 Thread Peng Fan (OSS)
From: Peng Fan When creating flash.bin, the hdmi firmware might not be copied to U-Boot source tree. Then mkimage will fail. However we are switching to binman, binman will show the message if the file not there, and create empty file per i.MX8MQ binman node. So we not fail mkimage here

Re: [PATCH v4 3/5] efi_loader: add ExitBootServices() measurement

2021-08-15 Thread Masahisa Kojima
On Sat, 14 Aug 2021 at 18:06, Heinrich Schuchardt wrote: > > On 8/13/21 9:12 AM, Masahisa Kojima wrote: > > TCG PC Client PFP spec requires to measure > > "Exit Boot Services Invocation" if ExitBootServices() is invoked. > > Depending upon the return code from the ExitBootServices() call, > >

Re: Broken CONFIG_SPL_SATA_SUPPORT=y

2021-08-15 Thread Pali Rohár
On Monday 02 August 2021 21:40:07 Tom Rini wrote: > OK, thanks. Then the right fix is to introduce whatever SPL_xxx symbols > you need so that you can turn off what you don't want in SPL but leave > it enabled in full U-Boot, and update the code to use > CONFIG_IS_ENABLED(xxx). If you can do