Re: [PATCH] socfpga: enable EFI partition support for de0-nano-soc

2021-10-27 Thread Matthias Brugger
On 22/10/2021 19:59, Frank Kunz wrote: This fixes boot problems with distributions that use EFI boot like opensuse. Signed-off-by: Frank Kunz Reviewed-by: Matthias Brugger --- configs/socfpga_de0_nano_soc_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/c

Re: kirkwood stack size

2021-10-27 Thread Stefan Roese
Added Holger, the maintainer of the km_kirkwood boards to Cc... On 28.10.21 03:28, Marek Behún wrote: On Wed, 27 Oct 2021 16:06:58 -0700 Tony Dinh wrote: Hi Marek, In reference to: https://lists.denx.de/pipermail/u-boot/2021-October/465221.html I have several Kirkwood boards, and currently

Re: kirkwood stack size

2021-10-27 Thread Stefan Roese
Hi Marek, On 28.10.21 00:07, Marek Behún wrote: Hello Stefan, do you have some ARM Kirkwood board? No, I don't have of them here. Sorry. I am working on some env patches, because the code in the env/ directory is a pain in the ass of old relics, and Kirkwood is the only platform using the e

[PATCH v5 11/11] (RFC) efi_loader, dts: add public keys for capsules to device tree

2021-10-27 Thread AKASHI Takahiro
By specifying CONFIG_EFI_CAPSULE_KEY_PATH, the build process will automatically insert the given key into the device tree. Otherwise, users are required to do so manually, possibly, with the utility script, fdtsig.sh. Signed-off-by: AKASHI Takahiro --- doc/develop/uefi/uefi.rst | 4 dts/Ma

[PATCH v5 10/11] (RFC) tools: add fdtsig.sh

2021-10-27 Thread AKASHI Takahiro
With this script, a public key is added to a device tree blob as the default efi_get_public_key_data() expects. Signed-off-by: AKASHI Takahiro --- MAINTAINERS | 1 + tools/fdtsig.sh | 40 2 files changed, 41 insertions(+) create mode 100755 tools/fd

[PATCH v5 09/11] test/py: efi_capsule: check the results in case of CAPSULE_AUTHENTICATE

2021-10-27 Thread AKASHI Takahiro
Before the capsule authentication is supported, this test script works correctly, but with the feature enabled, most tests will fail due to unsigned capsules. So check the results depending on CAPSULE_AUTHENTICATE or not. Signed-off-by: AKASHI Takahiro --- .../test_efi_capsule/test_capsule_firmw

[PATCH v5 08/11] test/py: efi_capsule: add a test for "--guid" option

2021-10-27 Thread AKASHI Takahiro
This test scenario tests a new feature of mkeficapsule, "--guid" option, which allows us to specify FMP driver's guid explicitly at the command line. Signed-off-by: AKASHI Takahiro --- test/py/tests/test_efi_capsule/conftest.py| 3 + .../test_efi_capsule/test_capsule_firmware.py | 67 ++

[PATCH v5 07/11] test/py: efi_capsule: align with the syntax change of mkeficapsule

2021-10-27 Thread AKASHI Takahiro
Since the syntax of mkeficapsule was changed in the previous commit, we need to modify command line arguments in a pytest script. Signed-off-by: AKASHI Takahiro --- test/py/tests/test_efi_capsule/conftest.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/py/tests/te

[PATCH v5 05/11] test/py: efi_capsule: add image authentication test

2021-10-27 Thread AKASHI Takahiro
Add a couple of test cases against capsule image authentication for capsule-on-disk, where only a signed capsule file with the verified signature will be applied to the system. Due to the difficulty of embedding a public key (esl file) in U-Boot binary during pytest setup time, all the keys/certif

[PATCH v5 06/11] tools: mkeficapsule: allow for specifying GUID explicitly

2021-10-27 Thread AKASHI Takahiro
The existing options, "--fit" and "--raw," are only used to put a proper GUID in a capsule header, where GUID identifies a particular FMP (Firmware Management Protocol) driver which then would handle the firmware binary in a capsule. In fact, mkeficapsule does the exact same job in creating a capsu

[PATCH v5 04/11] doc: update UEFI document for usage of mkeficapsule

2021-10-27 Thread AKASHI Takahiro
Now we can use mkeficapsule command instead of EDK-II's script to create a signed capsule file. So update the instruction for capsule authentication. Signed-off-by: AKASHI Takahiro --- doc/develop/uefi/uefi.rst | 143 ++ 1 file changed, 67 insertions(+), 76 de

[PATCH v5 02/11] tools: mkeficapsule: add firmwware image signing

2021-10-27 Thread AKASHI Takahiro
With this enhancement, mkeficapsule will be able to sign a capsule file when it is created. A signature added will be used later in the verification at FMP's SetImage() call. To do that, We need specify additional command parameters: -monotonic-cout : monotonic count -private-key : private k

[PATCH v5 03/11] tools: mkeficapsule: add man page

2021-10-27 Thread AKASHI Takahiro
Add a man page for mkeficapsule command. Signed-off-by: AKASHI Takahiro --- MAINTAINERS| 1 + doc/mkeficapsule.1 | 95 ++ 2 files changed, 96 insertions(+) create mode 100644 doc/mkeficapsule.1 diff --git a/MAINTAINERS b/MAINTAINERS index 50

[PATCH v5 01/11] efi_loader: capsule: drop __weak from efi_get_public_key_data()

2021-10-27 Thread AKASHI Takahiro
As we discussed in ML, currently a device tree is the only place to store public keys for capsule authentication. So __weak is not necessary for now. Signed-off-by: AKASHI Takahiro --- lib/efi_loader/efi_capsule.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/efi_loader

[PATCH v5 00/11] efi_loader: capsule: improve capsule authentication support

2021-10-27 Thread AKASHI Takahiro
As I proposed and discussed in [1] and [2], I have made a couple of improvements on the current implementation of capsule update in this patch set. * add signing feature to mkeficapsule * add "--guid" option to mkeficapsule * add man page of mkeficapsule * update uefi document regarding capsule up

Re: [PATCH u-boot-marvell 00/13] Yet another kwboot improvements

2021-10-27 Thread Stefan Roese
On 27.10.21 23:03, Pali Rohár wrote: On Wednesday 27 October 2021 17:27:41 Stefan Roese wrote: Still I see no speed change. But this is a different story... Could you try to upload some file in u-boot via xmodem at different speeds? E.g. via loadx command: loadx 921600 and loadx 115200

Re: [PATCH v3 05/35] efi: Create a 64-bit app

2021-10-27 Thread Heinrich Schuchardt
On 10/26/21 5:30 AM, Simon Glass wrote: Most modern platforms use 64-bit EFI so it is useful to have a U-Boot app that runs under that. Add a (non-functional) build for this. Note that --whole-archive causes the gcc 9.2 linker to crash, so disable this for now. Once this is resolved, things shou

Re: [PATCH v3 03/35] efi: Drop the OF_EMBED warning for EFI

2021-10-27 Thread Heinrich Schuchardt
On 10/26/21 5:30 AM, Simon Glass wrote: For the EFI app, we must embed the devicetree in the ELF file since that is the only thing that is run by UEFI. Drop the warning to avoid confusion. Signed-off-by: Simon Glass Reviewed-by: Heinrich Schuchardt --- (no changes since v2) Changes in v2

Re: [PATCH v3 02/35] efi: Enable DM_ETH for the app

2021-10-27 Thread Heinrich Schuchardt
On 10/26/21 5:30 AM, Simon Glass wrote: There is no need to avoid driver model for networking. Drop this. Signed-off-by: Simon Glass This seems to be an unrelated problem in building efi-x86_app_defconfig: LD u-boot ld.bfd: arch/x86/lib/bios_asm.o: warning: relocation against `idt_ptr

Re: [PATCH v3 01/35] efi: Add a script for building and testing U-Boot on UEFI

2021-10-27 Thread Heinrich Schuchardt
On 10/26/21 5:30 AM, Simon Glass wrote: It is quite complicated to run U-Boot on QEMU since we have four different builds and they must use different versions of qemu and the UEFI binaries. Add a script to help. Signed-off-by: Simon Glass It seems that you missed some of my review comments:

Re: [PATCH 1/1] video: fix positioning in TrueType console

2021-10-27 Thread Heinrich Schuchardt
On 10/27/21 9:52 PM, Anatolij Gustschin wrote: Hi Heinrich, On Wed, 27 Oct 2021 20:10:36 +0200 Heinrich Schuchardt xypron.g...@gmx.de wrote: Dear Anatolij, somehow you lost track of this patch since March: https://patchwork.ozlabs.org/project/uboot/patch/20210203161254.970-1-xypron.g...@gmx.

[PATCH v2 1/1] video: fix positioning in TrueType console

2021-10-27 Thread Heinrich Schuchardt
With the patch accurate positioning is possible for mono-typed fonts: Fix the return value of console_truetype_putc_xy(). The current position is passed as parameter x. Some part of x represents a fractional pixel. The return value represents how much the character position must be advanced. This

[BUG] Spurious error in Gitlab CI for sandbox_spl test.py

2021-10-27 Thread Heinrich Schuchardt
Hello Tom, we have some trouble with Gitlab CI: In https://source.denx.de/u-boot/custodians/u-boot-efi/-/jobs/342220 sandbox: + sandbox_spl +/usr/bin/ld: /tmp/u-boot-spl.Z4D5y0.ltrans0.ltrans.o: in function `dm_setup_inst.lto_priv.0': +drivers/core/root.c:134: undefined reference to `_u_b

Re: [PATCH 1/3] net: dsa: Use true instead of 1 in the set_promisc() call

2021-10-27 Thread Ramon Fried
On Wed, Oct 27, 2021 at 5:19 AM Bin Meng wrote: > > On Sun, Oct 17, 2021 at 2:26 AM Ramon Fried wrote: > > > > On Wed, Sep 29, 2021 at 4:32 PM Vladimir Oltean > > wrote: > > > > > > On Wed, Sep 29, 2021 at 01:50:44PM +0800, Bin Meng wrote: > > > > set_promisc() call accepts the parameter of a b

[PATCH 5/5] arm: mvebu: Espressobin: Use new API for setting default env at runtime

2021-10-27 Thread Marek Behún
From: Marek Behún ESPRESSObin's board code uses an ad-hoc solution for ensuring that ethaddrs are not overwritten by `env default -a` command and that the `fdtfile` is set to correct value when `env default -a` is called. This ad-hoc solution is overwriting the default_environment[] buffer in bo

[PATCH 4/5] env: Add support for board specific special default environment

2021-10-27 Thread Marek Behún
From: Marek Behún The default_environment[] buffer is built at compile time, but sometimes it makes sense for some default environment variables to be determined at runtime, for example: - one board code may support different boards, and needs that fdtfile, board, board_name are set appropr

[PATCH 3/5] env: Simplify env_get_default()

2021-10-27 Thread Marek Behún
From: Marek Behún Instead of pretending that we don't have environment to force searching default environment in env_get_default(), get the data from the default_environment[] buffer directly. Signed-off-by: Marek Behún --- env/common.c | 45 - 1 fil

[PATCH 1/5] env: Don't set ready flag if import failed in env_set_default()

2021-10-27 Thread Marek Behún
From: Marek Behún Do not set GD_FLG_ENV_READY nor GD_FLG_ENV_DEFAULT if failed importing in env_set_default(). Signed-off-by: Marek Behún --- env/common.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/env/common.c b/env/common.c index 99729ca002..2aa23545ba 100644 ---

[PATCH 2/5] env: Fix env_get() when returning empty string using env_get_f()

2021-10-27 Thread Marek Behún
From: Marek Behún The env_get_f() function returns -1 on failure. Returning 0 means that the variable exists, and is empty string. Signed-off-by: Marek Behún --- env/common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/env/common.c b/env/common.c index 2aa23545ba..757c5

[PATCH 0/5] Board specific runtime determined default env

2021-10-27 Thread Marek Behún
From: Marek Behún Hello Simon, Stefan, Pali, this series adds support for board specific runtime determined default environment variables. IMPORTANT: This series depends on the series http://patchwork.ozlabs.org/project/uboot/list/?series=268452 Currently the env default [-a] command uses

Re: kirkwood stack size

2021-10-27 Thread Tony Dinh
Hi Marek, Ah. That makes sense! I don't have the km_kirkwood_xxx boards. Thanks, Tony On Wed, Oct 27, 2021 at 6:28 PM Marek Behún wrote: > > On Wed, 27 Oct 2021 16:06:58 -0700 > Tony Dinh wrote: > > > Hi Marek, > > > > In reference to: > > https://lists.denx.de/pipermail/u-boot/2021-October/46

Re: kirkwood stack size

2021-10-27 Thread Marek Behún
On Wed, 27 Oct 2021 16:06:58 -0700 Tony Dinh wrote: > Hi Marek, > > In reference to: > https://lists.denx.de/pipermail/u-boot/2021-October/465221.html > > I have several Kirkwood boards, and currently support these boards in > mainline: Sheevaplug, Zyxel NSA310S, GoFlex Home, Dockstar. I also >

Re: [PATCH 2/2] cmd: sbi: show SBI implementation version

2021-10-27 Thread Rick Chen
> From: Heinrich Schuchardt > Sent: Monday, October 25, 2021 9:10 PM > To: Rick Jian-Zhi Chen(陳建志) ; Leo Yu-Chi Liang(梁育齊) > > Cc: u-boot@lists.denx.de; Heinrich Schuchardt > > Subject: [PATCH 2/2] cmd: sbi: show SBI implementation version > > Let the sbi command show the SBI implementation ve

Re: [PATCH 1/2] riscv: function to retrieve SBI implementation version

2021-10-27 Thread Rick Chen
> From: Heinrich Schuchardt > Sent: Monday, October 25, 2021 9:10 PM > To: Rick Jian-Zhi Chen(陳建志) ; Leo Yu-Chi Liang(梁育齊) > > Cc: u-boot@lists.denx.de; Heinrich Schuchardt > > Subject: [PATCH 1/2] riscv: function to retrieve SBI implementation version > > Provide function sbi_get_impl_version

Re: kirkwood stack size

2021-10-27 Thread Tony Dinh
Hi Marek, In reference to: https://lists.denx.de/pipermail/u-boot/2021-October/465221.html I have several Kirkwood boards, and currently support these boards in mainline: Sheevaplug, Zyxel NSA310S, GoFlex Home, Dockstar. I also have a few out-of-tree Kirkwood u-boots that have not been submitted

Re: [PATCH v5 00/26] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread Mark Kettenis
> From: Simon Glass > Date: Wed, 27 Oct 2021 12:23:21 -0600 > > Hi François, > > On Wed, 27 Oct 2021 at 09:14, François Ozog wrote: > > > > > > > > On Wed, 27 Oct 2021 at 16:08, Simon Glass wrote: > >> > >> Hi François, > >> > >> On Tue, 26 Oct 2021 at 00:07, François Ozog > >> wrote: > >> >

[PATCH] Fix MMC numbering issue for Raspberry Pi 3

2021-10-27 Thread karels
From: Mike Karels Using mmc.dtbo from rpi-firmware to switch the controller for the SD card slot from sdhci to sdhost causes the numbering to change; the SD card is then not recognized at boot. Add to the range checked. --- include/configs/rpi.h | 3 ++- 1 file changed, 2 insertions(+), 1 delet

kirkwood stack size

2021-10-27 Thread Marek Behún
Hello Stefan, do you have some ARM Kirkwood board? I am working on some env patches, because the code in the env/ directory is a pain in the ass of old relics, and Kirkwood is the only platform using the eeprom environment driver. I would like to ask about Kirkwood stack size. In arch/arm/mach-k

Re: [PATCH 00/16] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread François Ozog
Hi Tom Le mer. 27 oct. 2021 à 21:06, Tom Rini a écrit : > On Wed, Oct 27, 2021 at 06:02:19PM +0200, François Ozog wrote: > > Hi Mark, > > > > On Wed, 27 Oct 2021 at 17:10, Mark Kettenis > wrote: > > > > > > From: François Ozog > > > > Date: Wed, 27 Oct 2021 15:15:01 +0200 > > > > > > > > Hi, >

Re: [PATCH 00/16] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread Mark Kettenis
> From: Simon Glass > Date: Wed, 27 Oct 2021 09:24:25 -0600 > > Hi Mark, > > On Wed, 27 Oct 2021 at 09:11, Mark Kettenis wrote: > > > > > From: François Ozog > > > Date: Wed, 27 Oct 2021 15:15:01 +0200 > > > > > > In my view U-Boot shall be able to leverage device tree format > > > (source and

Re: [PATCH v5 26/26] fdt: Don't call board_fdt_blob_setup() without OF_BOARD

2021-10-27 Thread Tom Rini
On Wed, Oct 27, 2021 at 11:33:53PM +0200, François Ozog wrote: > Hi Tom > > Le mer. 27 oct. 2021 à 21:12, Tom Rini a écrit : > > > On Wed, Oct 27, 2021 at 10:17:05AM +0300, Ilias Apalodimas wrote: > > > Hi Simon, > > > > > > On Tue, 26 Oct 2021 at 18:27, Simon Glass wrote: > > > > > > > > Hi Il

Re: [PATCH v5 26/26] fdt: Don't call board_fdt_blob_setup() without OF_BOARD

2021-10-27 Thread François Ozog
Hi Tom Le mer. 27 oct. 2021 à 21:12, Tom Rini a écrit : > On Wed, Oct 27, 2021 at 10:17:05AM +0300, Ilias Apalodimas wrote: > > Hi Simon, > > > > On Tue, 26 Oct 2021 at 18:27, Simon Glass wrote: > > > > > > Hi Ilias, > > > > > > On Tue, 26 Oct 2021 at 07:56, Ilias Apalodimas > > > wrote: > > >

Re: [PATCH u-boot-marvell 00/13] Yet another kwboot improvements

2021-10-27 Thread Pali Rohár
On Wednesday 27 October 2021 17:27:41 Stefan Roese wrote: > Still I see no speed change. But this is a different story... Could you try to upload some file in u-boot via xmodem at different speeds? E.g. via loadx command: loadx 921600 and loadx 115200 And compare if xmodem transfer is in U-B

Re: [PATCH v4 2/2] arm: mvebu: add Globalscale MOCHAbin support

2021-10-27 Thread Tom Rini
On Wed, Oct 27, 2021 at 10:11:36PM +0200, Robert Marko wrote: > On Wed, Oct 27, 2021 at 10:07 PM Tom Rini wrote: > > > > On Wed, Oct 27, 2021 at 09:53:24PM +0200, Robert Marko wrote: > > > On Wed, Oct 27, 2021 at 9:48 PM Marek Behún wrote: > > > > > > > > On Wed, 27 Oct 2021 21:40:11 +0200 > > >

Re: [PATCH v5 02/26] doc: Add documentation about devicetree usage

2021-10-27 Thread François Ozog
Hi Tom Le mer. 27 oct. 2021 à 21:48, Tom Rini a écrit : > On Wed, Oct 27, 2021 at 05:38:28PM +0200, François Ozog wrote: > > Hi Simon, > > > > On Wed, 27 Oct 2021 at 16:13, Simon Glass wrote: > > > > > Hi François, > > > > > > On Tue, 26 Oct 2021 at 09:57, François Ozog > > > wrote: > > > > >

Re: [PATCH v4 2/2] arm: mvebu: add Globalscale MOCHAbin support

2021-10-27 Thread Robert Marko
On Wed, Oct 27, 2021 at 10:07 PM Tom Rini wrote: > > On Wed, Oct 27, 2021 at 09:53:24PM +0200, Robert Marko wrote: > > On Wed, Oct 27, 2021 at 9:48 PM Marek Behún wrote: > > > > > > On Wed, 27 Oct 2021 21:40:11 +0200 > > > Robert Marko wrote: > > > > > > > To me, it also doesn't make sense to do

Re: [PATCH v4 1/4] tools: Separate image types which depend on OpenSSL

2021-10-27 Thread Peter Robinson
> > >> I also have been doing some packaging of u-boot for GNU Guix, where I > > >> suspect the stance wouldn't be as willing to accept such a compromise... > > >> > > >> So... I would *love* an option to be able to build a board-only config > > >> without any of the tools; > > > > > > Why is this

Re: [PATCH v5 00/26] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread François Ozog
Hi Simon Le mer. 27 oct. 2021 à 20:23, Simon Glass a écrit : > Hi François, > > On Wed, 27 Oct 2021 at 09:14, François Ozog > wrote: > > > > > > > > On Wed, 27 Oct 2021 at 16:08, Simon Glass wrote: > >> > >> Hi François, > >> > >> On Tue, 26 Oct 2021 at 00:07, François Ozog > wrote: > >> > >

Re: [PATCH v4 2/2] arm: mvebu: add Globalscale MOCHAbin support

2021-10-27 Thread Tom Rini
On Wed, Oct 27, 2021 at 09:53:24PM +0200, Robert Marko wrote: > On Wed, Oct 27, 2021 at 9:48 PM Marek Behún wrote: > > > > On Wed, 27 Oct 2021 21:40:11 +0200 > > Robert Marko wrote: > > > > > To me, it also doesn't make sense to do it like that as what's the > > > purpose of having stuff that is

Re: [PATCH v4 2/2] arm: mvebu: add Globalscale MOCHAbin support

2021-10-27 Thread Robert Marko
On Wed, Oct 27, 2021 at 9:48 PM Marek Behún wrote: > > On Wed, 27 Oct 2021 21:40:11 +0200 > Robert Marko wrote: > > > To me, it also doesn't make sense to do it like that as what's the > > purpose of having stuff that is completely unsupported in U-boot in > > DTS? > > It makes sense because the

Re: [PATCH v5 02/26] doc: Add documentation about devicetree usage

2021-10-27 Thread Simon Glass
Hi Ilias, On Wed, 27 Oct 2021 at 13:13, Ilias Apalodimas wrote: > > On Wed, 27 Oct 2021 at 21:33, Simon Glass wrote: > > > > Hi François, > > > > On Wed, 27 Oct 2021 at 09:38, François Ozog > > wrote: > > > > > > Hi Simon, > > > > > > On Wed, 27 Oct 2021 at 16:13, Simon Glass wrote: > > >> >

Re: [PATCH 1/1] video: fix positioning in TrueType console

2021-10-27 Thread Anatolij Gustschin
Hi Heinrich, On Wed, 27 Oct 2021 20:10:36 +0200 Heinrich Schuchardt xypron.g...@gmx.de wrote: > Dear Anatolij, > > somehow you lost track of this patch since March: > > https://patchwork.ozlabs.org/project/uboot/patch/20210203161254.970-1-xypron.g...@gmx.de/ > > Could you, please, review it an

Re: [PATCH v4 2/2] arm: mvebu: add Globalscale MOCHAbin support

2021-10-27 Thread Marek Behún
On Wed, 27 Oct 2021 21:40:11 +0200 Robert Marko wrote: > To me, it also doesn't make sense to do it like that as what's the > purpose of having stuff that is completely unsupported in U-boot in > DTS? It makes sense because the device tree should compeltely describe the device. It doesn't matter

Re: [PATCH v5 02/26] doc: Add documentation about devicetree usage

2021-10-27 Thread Tom Rini
On Wed, Oct 27, 2021 at 05:38:28PM +0200, François Ozog wrote: > Hi Simon, > > On Wed, 27 Oct 2021 at 16:13, Simon Glass wrote: > > > Hi François, > > > > On Tue, 26 Oct 2021 at 09:57, François Ozog > > wrote: > > > > > > > > > > > > On Tue, 26 Oct 2021 at 17:27, Simon Glass wrote: > > >> > >

Re: [PATCH v5 02/26] doc: Add documentation about devicetree usage

2021-10-27 Thread François Ozog
Hi Simon, Le mer. 27 oct. 2021 à 20:33, Simon Glass a écrit : > Hi François, > > On Wed, 27 Oct 2021 at 09:38, François Ozog > wrote: > > > > Hi Simon, > > > > On Wed, 27 Oct 2021 at 16:13, Simon Glass wrote: > >> > >> Hi François, > >> > >> On Tue, 26 Oct 2021 at 09:57, François Ozog > wrote

Re: [PATCH 05/16] arm: qemu: Add a devicetree file for qemu_arm64

2021-10-27 Thread Alex Bennée
There are QEMU targets that have fixed hardware which do rely on the firmware to have a fixed view of the hardware. sbsa-ref is one such board although there will be some versioning required for later levels. On Wed, 27 Oct 2021, 19:39 Tom Rini, wrote: > On Wed, Oct 27, 2021 at 12:34:26PM -0600,

Re: [PATCH v4 2/2] arm: mvebu: add Globalscale MOCHAbin support

2021-10-27 Thread Robert Marko
On Wed, Oct 27, 2021 at 8:32 PM Pali Rohár wrote: > > On Wednesday 27 October 2021 20:01:33 Robert Marko wrote: > > Globalscale MOCHAbin is a Armada 7040 based development board. > > > > Specifications: > > * Armada 7040 Quad core ARMv8 Cortex A-72 @ 1.4GHz > > * 2 / 4 / 8 GB of DDR4 DRAM > > * 16

Re: [PATCH v5 00/26] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread Tom Rini
On Wed, Oct 27, 2021 at 12:23:21PM -0600, Simon Glass wrote: > Hi François, > > On Wed, 27 Oct 2021 at 09:14, François Ozog wrote: > > > > > > > > On Wed, 27 Oct 2021 at 16:08, Simon Glass wrote: > >> > >> Hi François, > >> > >> On Tue, 26 Oct 2021 at 00:07, François Ozog > >> wrote: > >> > >

Re: [PATCH v5 00/26] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread Tom Rini
On Wed, Oct 27, 2021 at 06:36:12PM +0300, Tuomas Tynkkynen wrote: > Hi, > > On 27.10.2021 17.08, Simon Glass wrote: > > Hi François, > > > > On Tue, 26 Oct 2021 at 00:07, François Ozog > > wrote: > > > > > > Hi Simon > > > > > > Position unchanged on this series: adding fake dts for boards th

Re: [PATCH v5 02/26] doc: Add documentation about devicetree usage

2021-10-27 Thread Ilias Apalodimas
On Wed, 27 Oct 2021 at 21:33, Simon Glass wrote: > > Hi François, > > On Wed, 27 Oct 2021 at 09:38, François Ozog wrote: > > > > Hi Simon, > > > > On Wed, 27 Oct 2021 at 16:13, Simon Glass wrote: > >> > >> Hi François, > >> > >> On Tue, 26 Oct 2021 at 09:57, François Ozog > >> wrote: > >> > >

Re: [PATCH v5 26/26] fdt: Don't call board_fdt_blob_setup() without OF_BOARD

2021-10-27 Thread Tom Rini
On Wed, Oct 27, 2021 at 10:17:05AM +0300, Ilias Apalodimas wrote: > Hi Simon, > > On Tue, 26 Oct 2021 at 18:27, Simon Glass wrote: > > > > Hi Ilias, > > > > On Tue, 26 Oct 2021 at 07:56, Ilias Apalodimas > > wrote: > > > > > > Hi Simon, > > > > > > As I said here [1], this is moving on an entire

Re: [PATCH 00/16] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread Tom Rini
On Wed, Oct 27, 2021 at 06:02:19PM +0200, François Ozog wrote: > Hi Mark, > > On Wed, 27 Oct 2021 at 17:10, Mark Kettenis wrote: > > > > From: François Ozog > > > Date: Wed, 27 Oct 2021 15:15:01 +0200 > > > > > > Hi, > > > > > > On Wed, 27 Oct 2021 at 14:48, Tom Rini wrote: > > > > > > > On Fr

[PATCH u-boot-marvell 5/5] tools: kwboot: Do not use stack when setting baudrate back to default value

2021-10-27 Thread Marek Behún
From: Pali Rohár The ARM code we inject into the image to change baudrate back to the default value of 115200 Baud, which is run after successful UART transfer of the whole image, cannot use stack as at this stage stack pointer is not initialized yet. Stack can only be used when BootROM is execu

[PATCH u-boot-marvell 4/5] tools: kwboot: Replace ARM mov + movt instruction pair by mov + orr

2021-10-27 Thread Marek Behún
From: Pali Rohár Older Armada SoCs have custom ARMv5te compatible core which does not support movt instruction. So replace mov + movt instruction pair used for immediate move construction by mov + orr instructions which are supported also by ARMv5te. After this change kwboot ARM code should be c

[PATCH u-boot-marvell 2/5] tools: kwboot: Do not call tcdrain() after each sent packet

2021-10-27 Thread Marek Behún
From: Pali Rohár Kwboot puts each xmodem packet to kernel queue, then waits until all bytes of that packet are transmitted over UART and then waits for xmodem reply until it is received into kernel queue. If some reply is received during the time we are waiting until all bytes are transmitted, t

[PATCH u-boot-marvell 3/5] tools: kwboot: Increase delay after changing baudrate in ARM code

2021-10-27 Thread Marek Behún
From: Pali Rohár Increase loop cycles from 60 to 2998272, which should increase delay from 1ms to about 5ms on 1200 MHz CPU. The Number 2998272 was chosen as the nearest value around 300 which can be encoded into one ARM mov instruction. It avoids usage of movt instruction which is not s

[PATCH u-boot-marvell 1/5] tools: kwboot: Fix sending retry of last header packet

2021-10-27 Thread Marek Behún
From: Pali Rohár After the trasfer of last header packet, it is possible that baudrate change pattern is received, and also that NAK byte is received so that the packet should be sent again. Thus we should not clear the baudrate change state when sending retry of that packet. Move code for init

[PATCH u-boot-marvell 0/5] kwboot fix for AXP + some others

2021-10-27 Thread Marek Behún
From: Marek Behún Hi Stefan, here are some more Pali's changes for kwboot, reviewed by me. The last one should fix boot on AXP. Marek Pali Rohár (5): tools: kwboot: Fix sending retry of last header packet tools: kwboot: Do not call tcdrain() after each sent packet tools: kwboot: Increase

Re: [PATCH 05/16] arm: qemu: Add a devicetree file for qemu_arm64

2021-10-27 Thread Tom Rini
On Wed, Oct 27, 2021 at 12:34:26PM -0600, Simon Glass wrote: > Hi all, > > On Wed, 27 Oct 2021 at 08:56, Tom Rini wrote: > > > > On Wed, Oct 27, 2021 at 03:44:08PM +0100, Alex Bennée wrote: > > > > > > François Ozog writes: > > > > > > > Hi Simon > > > > > > > > The only place I could agree with

Re: [PATCH v2 31/41] bootstd: Add an implementation of EFI boot

2021-10-27 Thread Simon Glass
Hi Ilias, On Wed, 27 Oct 2021 at 08:48, Ilias Apalodimas wrote: > > Hi Simon, > > On Wed, Oct 27, 2021 at 08:09:04AM -0600, Simon Glass wrote: > > Hi Ilias, > > > > On Wed, 27 Oct 2021 at 02:36, Ilias Apalodimas > > wrote: > > > > > > Hi Simon, > > > > > > On Sun, 24 Oct 2021 at 02:27, Simon Gla

Re: [PATCH v1 1/2] x86: tangier: pinmux: Move is_protected assignment closer to its user

2021-10-27 Thread Simon Glass
On Wed, 27 Oct 2021 at 08:23, Andy Shevchenko wrote: > > Move is_protected assignment closer to its user. > This increases readability and makes maintenance easier. > > Signed-off-by: Andy Shevchenko > --- > arch/x86/cpu/tangier/pinmux.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-)

Re: [PATCH v1 2/2] x86: tangier: pinmux: Move error message to the caller

2021-10-27 Thread Simon Glass
On Wed, 27 Oct 2021 at 08:23, Andy Shevchenko wrote: > > Move error message to the caller of mrfld_pinconfig*() in order > to unify them in the future. > > Signed-off-by: Andy Shevchenko > --- > arch/x86/cpu/tangier/pinmux.c | 10 +++--- > 1 file changed, 3 insertions(+), 7 deletions(-) Rev

Re: [PATCH 05/16] arm: qemu: Add a devicetree file for qemu_arm64

2021-10-27 Thread Simon Glass
Hi all, On Wed, 27 Oct 2021 at 08:56, Tom Rini wrote: > > On Wed, Oct 27, 2021 at 03:44:08PM +0100, Alex Bennée wrote: > > > > François Ozog writes: > > > > > Hi Simon > > > > > > The only place I could agree with this file presence is in the > > > documentation directory, not in dts. It create

Re: [PATCH v2 00/41] Initial implementation of standard boot

2021-10-27 Thread Simon Glass
Hi Heinrich, On Wed, 27 Oct 2021 at 05:38, Heinrich Schuchardt wrote: > > > > On 10/24/21 01:25, Simon Glass wrote: > > > > The bootflow feature provide a built-in way for U-Boot to automatically > > boot an Operating System without custom scripting and other customisation. > > This is called 'st

Re: [PATCH v5 02/26] doc: Add documentation about devicetree usage

2021-10-27 Thread Simon Glass
Hi François, On Wed, 27 Oct 2021 at 09:38, François Ozog wrote: > > Hi Simon, > > On Wed, 27 Oct 2021 at 16:13, Simon Glass wrote: >> >> Hi François, >> >> On Tue, 26 Oct 2021 at 09:57, François Ozog wrote: >> > >> > >> > >> > On Tue, 26 Oct 2021 at 17:27, Simon Glass wrote: >> >> >> >> Hi Fra

Re: [PATCH v4 2/2] arm: mvebu: add Globalscale MOCHAbin support

2021-10-27 Thread Pali Rohár
On Wednesday 27 October 2021 20:01:33 Robert Marko wrote: > Globalscale MOCHAbin is a Armada 7040 based development board. > > Specifications: > * Armada 7040 Quad core ARMv8 Cortex A-72 @ 1.4GHz > * 2 / 4 / 8 GB of DDR4 DRAM > * 16 GB eMMC > * 4MB SPI-NOR (Bootloader) > * 1x M.2-2280 B-key socket

Re: [PATCH 00/16] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread François Ozog
Hi Tom Le mer. 27 oct. 2021 à 20:06, Tom Rini a écrit : > On Wed, Oct 27, 2021 at 09:24:25AM -0600, Simon Glass wrote: > > Hi Mark, > > > > On Wed, 27 Oct 2021 at 09:11, Mark Kettenis > wrote: > > > > > > > From: François Ozog > > > > Date: Wed, 27 Oct 2021 15:15:01 +0200 > > > > > > > > Hi, >

Re: [PATCH v5 00/26] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread Tom Rini
On Wed, Oct 27, 2021 at 08:08:19AM -0600, Simon Glass wrote: [snip] > But trying to do any driver / core work for a board where you don't > have the devicetree is currently not possible. The devicetree is a > core component and being unable to modify it is simply not practical. > We are talking he

Re: [PATCH v5 00/26] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread Simon Glass
Hi François, On Wed, 27 Oct 2021 at 09:14, François Ozog wrote: > > > > On Wed, 27 Oct 2021 at 16:08, Simon Glass wrote: >> >> Hi François, >> >> On Tue, 26 Oct 2021 at 00:07, François Ozog wrote: >> > >> > Hi Simon >> > >> > Position unchanged on this series: adding fake dts for boards that >

Re: [PATCH v5 00/26] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread Tom Rini
On Wed, Oct 27, 2021 at 08:08:19AM -0600, Simon Glass wrote: > Hi François, > > On Tue, 26 Oct 2021 at 00:07, François Ozog wrote: > > > > Hi Simon > > > > Position unchanged on this series: adding fake dts for boards that generate > > their device tree in the dts directory is not good. If you

Re: [PATCH 1/1] video: fix positioning in TrueType console

2021-10-27 Thread Heinrich Schuchardt
Dear Anatolij, somehow you lost track of this patch since March: https://patchwork.ozlabs.org/project/uboot/patch/20210203161254.970-1-xypron.g...@gmx.de/ Could you, please, review it and consider it for merging. Best regards Heinrich On 2/3/21 17:12, Heinrich Schuchardt wrote: With the pa

Re: [PATCH 00/16] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread Tom Rini
On Wed, Oct 27, 2021 at 09:24:25AM -0600, Simon Glass wrote: > Hi Mark, > > On Wed, 27 Oct 2021 at 09:11, Mark Kettenis wrote: > > > > > From: François Ozog > > > Date: Wed, 27 Oct 2021 15:15:01 +0200 > > > > > > Hi, > > > > > > On Wed, 27 Oct 2021 at 14:48, Tom Rini wrote: > > > > > > > On Fri

Re: [PATCH 00/16] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread Tom Rini
On Wed, Oct 27, 2021 at 05:02:39PM +0200, Heinrich Schuchardt wrote: > On 10/27/21 16:55, Tom Rini wrote: > > On Wed, Oct 27, 2021 at 03:23:01PM +0200, Heinrich Schuchardt wrote: > > > > [snip] > > > One passed to U-Boot for fixups and further passed to the OS. This > > > devicetree may originate

Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-27 Thread Marek Behún
On Wed, 27 Oct 2021 10:58:35 -0700 Roman Bacik wrote: > Thank you very much for your suggestion. To simplify, we will decouple > this issue from the current bnxt driver submission. Very well, that seems reasonable. I am willing to help you with this afterwards. Marek

[PATCH v4 2/2] arm: mvebu: add Globalscale MOCHAbin support

2021-10-27 Thread Robert Marko
Globalscale MOCHAbin is a Armada 7040 based development board. Specifications: * Armada 7040 Quad core ARMv8 Cortex A-72 @ 1.4GHz * 2 / 4 / 8 GB of DDR4 DRAM * 16 GB eMMC * 4MB SPI-NOR (Bootloader) * 1x M.2-2280 B-key socket (for SSD expansion, SATA3 only) * 1x M.2-2250 B-key socket (for modems, U

[PATCH v4 1/2] arm: mvebu: Implement the mac command (Marvell hw_info)

2021-10-27 Thread Robert Marko
From: Luka Kovacic The mac command is implemented to enable parsing Marvell hw_info formatted environments. This format is often used on Marvell Armada devices to store parameters like the board serial number, factory MAC addresses and some other information. These parameters are usually written

RE: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-27 Thread Roman Bacik
Hi Marek, > -Original Message- > From: Marek Behún > Sent: Wednesday, October 27, 2021 10:40 AM > To: Roman Bacik > Cc: Simon Glass ; U-Boot Mailing List b...@lists.denx.de>; Bharat Gooty ; > Aswath Govindraju ; Bin Meng > ; Franck LENORMAND > ; Heinrich Schuchardt > ; Kory Maincent ; >

Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-27 Thread Marek Behún
On Wed, 27 Oct 2021 10:02:41 -0700 Roman Bacik wrote: > Marek, Simon, > > Thank you very much for your comments. We will remove bnxt commands and will > probe bnxt driver each boot in the next version. > Thanks, Roman I think that the idea of not loading fw or initializing the controller durin

Re: [PATCH v4 1/4] tools: Separate image types which depend on OpenSSL

2021-10-27 Thread Tom Rini
On Fri, Oct 22, 2021 at 12:46:59PM -0700, Vagrant Cascadian wrote: > On 2021-10-22, Andre Przywara wrote: > > On Fri, 22 Oct 2021 09:47:35 -0700 > > Vagrant Cascadian wrote: > >> On 2021-10-22, Tom Rini wrote: > >> > On Fri, Oct 22, 2021 at 04:56:09PM +0100, Andre Przywara wrote: > >> >> On Fri,

RE: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-27 Thread Roman Bacik
Marek, Simon, Thank you very much for your comments. We will remove bnxt commands and will probe bnxt driver each boot in the next version. Thanks, Roman > -Original Message- > From: Roman Bacik > Sent: Wednesday, October 27, 2021 9:47 AM > To: Marek Behún > Cc: Simon Glass ; U-Boot M

[PATCH 2/2] Revert "arch: arm: use dt and UCLASS_SYSCON to get gic lpi details"

2021-10-27 Thread Michael Walle
Stop using the device tree as a source for ad-hoc information. This reverts commit 2ae7adc659f7fca9ea65df4318e5bca2b8274310. Signed-off-by: Michael Walle --- arch/arm/Kconfig| 2 - arch/arm/cpu/armv8/fsl-layerscape/soc.c | 27 +- arch/arm/include/asm/gic-v3.h

[PATCH 1/2] Revert "arm64: Layerscape: Survive LPI one-way reset workaround"

2021-10-27 Thread Michael Walle
From: Tom Rini Ad-hoc bindings that are not part of the upstream device tree / bindings are not allowed in-tree. Only bindings that are in-progress with upstream and then re-synced once agreed upon are. This reverts commit af288cb291da3abef6be0875527729296f7de7a0. Cc: Hou Zhiqiang Cc: Priyank

[PATCH 0/2] arch: arm: gic-v3-its: stop abusing the device tree

2021-10-27 Thread Michael Walle
Please stop throwing every ad-hoc information in the device tree. Use the official bindings (or maybe some bindings which will get approved soon). On the quest of syncing the device tree used in u-boot with the one used in linux, there is this nice piece: gic_lpi_base: syscon@0x8000 {

RE: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-27 Thread Roman Bacik
Hi Marek > -Original Message- > From: Marek Behún > Sent: Wednesday, October 27, 2021 9:36 AM > To: Roman Bacik > Cc: Simon Glass ; U-Boot Mailing List b...@lists.denx.de>; Bharat Gooty ; > Aswath Govindraju ; Bin Meng > ; Franck LENORMAND > ; Heinrich Schuchardt > ; Kory Maincent ; > M

Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-27 Thread Marek Behún
On Wed, 27 Oct 2021 08:05:11 -0700 Roman Bacik wrote: > chimp_ld_secure #this command loads FW, which is necessary for PCIe to > enumerate it > pci enum #this command is necessary to call bnxt_bind Wait, so what is this firmware for? Is it firmware for the netXtreme controller or for PCIe contro

Re: [PATCH 00/16] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread François Ozog
Hi Mark, On Wed, 27 Oct 2021 at 17:10, Mark Kettenis wrote: > > From: François Ozog > > Date: Wed, 27 Oct 2021 15:15:01 +0200 > > > > Hi, > > > > On Wed, 27 Oct 2021 at 14:48, Tom Rini wrote: > > > > > On Fri, Oct 15, 2021 at 12:03:44PM -0600, Simon Glass wrote: > > > > Hi all, > > > > > > > >

RE: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-27 Thread Roman Bacik
Hi Simon, > -Original Message- > From: Simon Glass > Sent: Wednesday, October 27, 2021 8:41 AM > To: Roman Bacik > Cc: Marek Behún ; U-Boot Mailing List b...@lists.denx.de>; Bharat Gooty ; > Aswath Govindraju ; Bin Meng > ; Franck LENORMAND > ; Heinrich Schuchardt > ; Kory Maincent ; >

Re: [PATCH 00/16] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread Simon Glass
Hi Mark, On Wed, 27 Oct 2021 at 09:11, Mark Kettenis wrote: > > > From: François Ozog > > Date: Wed, 27 Oct 2021 15:15:01 +0200 > > > > Hi, > > > > On Wed, 27 Oct 2021 at 14:48, Tom Rini wrote: > > > > > On Fri, Oct 15, 2021 at 12:03:44PM -0600, Simon Glass wrote: > > > > Hi all, > > > > > > >

Re: [PATCH 00/16] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread Mark Kettenis
> From: François Ozog > Date: Wed, 27 Oct 2021 15:15:01 +0200 > > Hi, > > On Wed, 27 Oct 2021 at 14:48, Tom Rini wrote: > > > On Fri, Oct 15, 2021 at 12:03:44PM -0600, Simon Glass wrote: > > > Hi all, > > > > > > On Thu, 14 Oct 2021 at 09:28, Tom Rini wrote: > > > > > > > > On Thu, Oct 14, 20

Re: [PATCH 00/16] fdt: Make OF_BOARD a boolean option

2021-10-27 Thread Heinrich Schuchardt
On 10/27/21 16:55, Tom Rini wrote: On Wed, Oct 27, 2021 at 03:23:01PM +0200, Heinrich Schuchardt wrote: [snip] One passed to U-Boot for fixups and further passed to the OS. This devicetree may originate from a prior boot stage, from a file loaded by U-Boot, or from a later bootstage, e.g system

  1   2   >