Re: [PATCH] arm: mach-snapdragon: pinctrl: Place pin_name in .data section

2021-07-15 Thread Jorge Ramirez-Ortiz, Gmail
On 06/07/21, Ramon Fried wrote: > On Mon, Jul 5, 2021 at 3:19 PM Stephan Gerhold wrote: > > > > According to arch/arm/lib/crt0_64.S, the BSS section is "UNAVAILABLE" > > and uninitialized before relocation. Also, it overlaps with the > > appended DTB before relocation, so writing data into a varia

Re: [PATCH] arm: mach-snapdragon: pinctrl: Place pin_name in .data section

2021-07-15 Thread Jorge Ramirez-Ortiz, Gmail
On 15/07/21, Jorge Ramirez-Ortiz, Gmail wrote: > On 06/07/21, Ramon Fried wrote: > > On Mon, Jul 5, 2021 at 3:19 PM Stephan Gerhold wrote: > > > > > > According to arch/arm/lib/crt0_64.S, the BSS section is "UNAVAILABLE" > > > and uninitialized before relocation. Also, it overlaps with the > > > a

Re: [PATCH 1/5] efi_loader: increase eventlog buffer size

2021-07-15 Thread Masahisa Kojima
On Thu, 15 Jul 2021 at 15:46, Ilias Apalodimas wrote: > > > On Thu, Jul 15, 2021 at 02:09:57PM +0900, Masahisa Kojima wrote: > > Hi Simon, Ilias, > > > > On Wed, 14 Jul 2021 at 23:50, Simon Glass wrote: > > > > > > Hi Masahisa, > > > > > > On Mon, 12 Jul 2021 at 02:40, Masahisa Kojima > > > wrot

RE: [EXT] Re: Problem on imx: add rom api support on imx8mn

2021-07-15 Thread Ye Li
Hi Michael, > -Original Message- > From: Michael Nazzareno Trimarchi > Sent: Thursday, July 15, 2021 2:52 PM > To: Peng Fan (OSS) > Cc: Peng Fan ; U-Boot-Denx ; > Stefano Babic ; Ye Li ; Fabio Estevam > ; Jagan Teki > Subject: [EXT] Re: Problem on imx: add rom api support on imx8mn > >

Re: [RFC 1/7] ARM: mvebu: a38x: Correct mismatched bound warnings

2021-07-15 Thread Stefan Roese
Hi Pali, Hi Kosta, On 14.07.21 22:52, Pali Rohár wrote: On Monday 03 May 2021 16:48:53 Tom Rini wrote: With gcc-11 we see: drivers/ddr/marvell/a38x/ddr3_debug.c:672:47: error: argument 2 of type 'u32[5]' {aka 'unsigned int[5]'} with mismatched bound [-Werror=array-parameter=] 672 | int ddr

Re: [PATCH 1/3] arm: mvebu: Espressobin: Fix setting $fdtfile env

2021-07-15 Thread Stefan Roese
On 14.07.21 16:37, Pali Rohár wrote: Ensure that 'env default -a' always set correct value to $fdtfile, even when custom user variable is already stored in non-volatile env storage (means that env_get("fdtfile") call returns non-NULL value). As default value is now correctly set like if specifie

Re: [PATCH 2/3] arm: mvebu: Espressobin: Use function mmc_get_op_cond() for detecting eMMC

2021-07-15 Thread Stefan Roese
On 14.07.21 16:37, Pali Rohár wrote: Use function mmc_get_op_cond() instead of mmc_init() for detecting presence of eMMC. Documentation for this function says that it could be used to detect the presence of SD/eMMC when no card detect logic is available. This function is also used by mx6cuboxi b

Re: [PATCH 3/3] mmc: mmc_get_op_cond: Allow quiet detection of eMMC

2021-07-15 Thread Stefan Roese
On 14.07.21 16:37, Pali Rohár wrote: Add a new 'quiet' argument to mmc_get_op_cond() function which avoids printing error message when SD/eMMC card is not detected. Espressobin and mx6cuboxi boards use this function for detecting presence of eMMC and therefore it is expected and normal that eMMC

Re: [PATCH v3 3/4] NSA310S : Use Ethernet PHY name from device tree

2021-07-15 Thread Stefan Roese
Hi Tony, On 15.07.21 03:11, Tony Dinh wrote: Hi Stefan, I will need to resend this patch (V3 3/4). Ok. Please add the patch version and history back into the patches next time again. Thanks, Stefan Thanks, Tony On Wed, Jul 7, 2021 at 2:07 AM Tony Dinh > wrote:

Re: [PATCH v3 3/4] NSA310S : Use Ethernet PHY name from device tree

2021-07-15 Thread Stefan Roese
On 15.07.21 10:05, Stefan Roese wrote: Hi Tony, On 15.07.21 03:11, Tony Dinh wrote: Hi Stefan, I will need to resend this patch (V3 3/4). Ok. Please add the patch version and history back into the patches next time again. Also, when re-sending please include the already collected tags into

Re: [PATCH V2 00/46] imx: add i.MX8ULP support

2021-07-15 Thread Stefano Babic
Hi Peng, On 15.07.21 05:29, Peng Fan (OSS) wrote: All, If no objection to this patchset, I'll send a PR to Stefano's repo in Friday this week. We will come out more update in rc[x] No need - I am already on the way to merge them. TODO for next weekend. I plan to merge this big chunk, too, b

Re: [PATCH u-boot-mvebu 1/2] arm: mvebu: turris_omnia: force 40 MHz speed on SPI NOR

2021-07-15 Thread Stefan Roese
On 09.07.21 16:56, Marek Behún wrote: Commit e2e95e5e2542 ("spi: Update speed/mode on change") changed the boot time on Turris Omnia from ~2.3s to over 8s, due to SPL loading main U-Boot from SPI NOR at 1 MHz instead of 40 MHz. This is because the SPL code passes the CONFIG_SF_DEFAULT_SPEED opti

Re: [PATCH u-boot-mvebu 2/2] arm: mvebu: turris_{omnia,mox}: enable MTD command

2021-07-15 Thread Stefan Roese
On 09.07.21 16:56, Marek Behún wrote: Now that the MTD subsystem properly supports OF partitions of a SPI NOR, we can enable the MTD command and start using it instead of the deprecated sf command. Signed-off-by: Marek Behún Cc: Stefan Roese Cc: Pali Rohár Reviewed-by: Stefan Roese Thanks

Re: [PATCH v2 3/6] efi_loader: add boot variable measurement

2021-07-15 Thread Masahisa Kojima
On Thu, 15 Jul 2021 at 15:59, Heinrich Schuchardt wrote: > > > > On 7/14/21 3:00 PM, Masahisa Kojima wrote: > > TCG PC Client PFP spec requires to measure "Boot" > > and "BootOrder" variables, EV_SEPARATOR event prior > > to the Ready to Boot invocation. > > Since u-boot does not implement Rea

Re: [PATCH v3 07/10] watchdog: wdt-uclass.c: handle all DM watchdogs in watchdog_reset()

2021-07-15 Thread Stefan Roese
Hi Rasmus, On 05.07.21 17:30, Simon Glass wrote: Hi Rasmus, On Fri, 2 Jul 2021 at 06:45, Rasmus Villemoes wrote: A board can have and make use of more than one watchdog device, say one built into the SOC and an external gpio-petted one. Having wdt-uclass only handle the first is both a littl

Re: [EXT] Re: Problem on imx: add rom api support on imx8mn

2021-07-15 Thread Michael Nazzareno Trimarchi
Hi On Thu, Jul 15, 2021 at 9:56 AM Ye Li wrote: > > Hi Michael, > > > -Original Message- > > From: Michael Nazzareno Trimarchi > > Sent: Thursday, July 15, 2021 2:52 PM > > To: Peng Fan (OSS) > > Cc: Peng Fan ; U-Boot-Denx ; > > Stefano Babic ; Ye Li ; Fabio Estevam > > ; Jagan Teki >

Re: [PATCH 3/3] mmc: mmc_get_op_cond: Allow quiet detection of eMMC

2021-07-15 Thread Stefan Roese
(added Jaehoon as MMC co-maintainer) On 15.07.21 10:03, Stefan Roese wrote: On 14.07.21 16:37, Pali Rohár wrote: Add a new 'quiet' argument to mmc_get_op_cond() function which avoids printing error message when SD/eMMC card is not detected. Espressobin and mx6cuboxi boards use this function fo

Re: [PATCH 17/27] ppc: Remove T1023RBD boards and T1024RDB_SECURE_BOOT

2021-07-15 Thread gianluca
Hello list! I know this mail will be OT, but I am worried about what will happen to a new project using the PowerPC T2080,e6500 64-bit PPC on this web site: https://www.powerpc-notebook.org I suspect the same architecture will be move outside the official Linux release when the same device-mo

Re: [PATCH u-boot-mvebu 01/31] tools: kwbimage: Fix compilation without CONFIG_SYS_U_BOOT_OFFS

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár The CONFIG_SYS_U_BOOT_OFFS option may be defined as empty string. In this case it causes compilation error: tools/kwbimage.c: In function ‘image_headersz_v1’: tools/kwbimage.c:1002:39: error: expected expression before ‘)’ token

Re: [PATCH u-boot-mvebu 02/31] tools: kwbimage: Simplify aligning and calculating checksum

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár The return value of kwbimage_generate() is used for aligning the data part of kwbimage. Use it for calculating proper 4 byte alignment as is required by BootROM and also use it for allocating additional 4 bytes for the 32-bit data checksum.

Re: [PATCH u-boot-mvebu 03/31] tools: kwbimage: Align SPI and NAND images to 256 bytes

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár Writing into SPI NOR and NAND memory can be done only in 256 bytes long blocks. Align final image size so that when it is burned into SPI NOR or NAND memory via U-Boot's commands (sf or mtd), we can use the $filesize variable directly as th

Re: [PATCH u-boot-mvebu 04/31] tools: kwbimage: Add constant for SDIO bootfrom

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: Add constant for SDIO value of the bootfrom header field. Signed-off-by: Marek Behún Reviewed-by: Stefan Roese Thanks, Stefan --- tools/kwbimage.h | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/kwbimage.h b/tools/kwbimage.h index 0b6d05b

Re: [PATCH u-boot-mvebu 05/31] tools: kwbimage: Fix generation of SATA, SDIO and PCIe images

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár SATA and SDIO images must be aligned to sector size (which in most cases is 512 bytes) and Source Address in main header is stored in number of sectors from the beginning of the drive. SATA image must be stored at sector 1 and SDIO image at

Re: [PATCH u-boot-mvebu 06/31] tools: kwbimage: Don't crash when binary file name does not contain '/'

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár In the case when the file name is specified relative to the current working directory, it does not contain '/' character and strrchr() returns NULL. The following strcmp() function then crashes on NULL pointer dereference. Signed-off-by:

Re: [PATCH u-boot-mvebu 07/31] tools: kwbimage: Fix check for v0 extended header checksum

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár Extended header checksum for v0 image is present only in the case when extended header is present. Skip checksum validation if extended header is not present. Signed-off-by: Pali Rohár Reviewed-by: Marek Behún Reviewed-by: Stefan Roese

Re: [PATCH u-boot-mvebu 08/31] tools: kwbimage: Validate extended headers of v1 images

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár Add basic checks for extended headers of v1 images. Signed-off-by: Pali Rohár Reviewed-by: Marek Behún Reviewed-by: Stefan Roese Thanks, Stefan --- tools/kwbimage.c | 28 1 file changed, 28 insertion

Re: [PATCH u-boot-mvebu 09/31] tools: kwbimage: Validate data checksum of v1 images

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár The data part of v1 images contains 32-bit checksum after the data. Validate whether this checksum is correct. Signed-off-by: Pali Rohár Reviewed-by: Marek Behún Reviewed-by: Stefan Roese Thanks, Stefan --- tools/kwbimage.c | 46

Re: [PATCH u-boot-mvebu 10/31] tools: kwbimage: Print size of binary header in kwbimage_print_header()

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár The binary header in kwbimage contains executable SPL code. Print information about this binary header and not only information about it's data part. Signed-off-by: Pali Rohár Reviewed-by: Marek Behún Reviewed-by: Stefan Roese Thank

Re: [PATCH u-boot-mvebu 11/31] tools: kwboot: Fix wrong parameter passed to read()

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár The 'buf' is a pointer and '_buf' is the array itself. Therefore we should pass sizeof(_buf) instead of sizeof(buf) to read(). Signed-off-by: Pali Rohár Reviewed-by: Marek Behún Reviewed-by: Stefan Roese Thanks, Stefan --- tools/

Re: [PATCH u-boot-mvebu 12/31] tools: kwboot: Fix restoring terminal

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár Call tcsetattr() only if the file descriptor is valid. It may be invalidated by previous lines (if it is not a tty descriptor). Signed-off-by: Pali Rohár Reviewed-by: Marek Behún Reviewed-by: Stefan Roese Thanks, Stefan --- tools

Re: [PATCH u-boot-mvebu 13/31] tools: kwboot: Print trailing newline after terminal is terminated

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár Print trailing newline as the last printed byte can be something different. Signed-off-by: Pali Rohár Reviewed-by: Marek Behún Reviewed-by: Stefan Roese Thanks, Stefan --- tools/kwboot.c | 1 + 1 file changed, 1 insertion(+) di

Re: [PATCH u-boot-mvebu 14/31] tools: kwboot: Cosmetic fix - add missing curly brackets

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár Add missing curly brackets for this else statement. Signed-off-by: Pali Rohár Reviewed-by: Marek Behún Reviewed-by: Stefan Roese Thanks, Stefan --- tools/kwboot.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --g

Re: [PATCH u-boot-mvebu 15/31] tools: kwboot: Check for v1 header size

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár Too small invalid headers may cause kwboot to crash. Check for header size of v1 images. Signed-off-by: Pali Rohár Reviewed-by: Marek Behún Reviewed-by: Stefan Roese Thanks, Stefan --- tools/kwboot.c | 5 + 1 file changed, 5

Re: [PATCH 3/3] mmc: mmc_get_op_cond: Allow quiet detection of eMMC

2021-07-15 Thread Jaehoon Chung
Hi Stefan, On 7/15/21 5:20 PM, Stefan Roese wrote: > (added Jaehoon as MMC co-maintainer) > > On 15.07.21 10:03, Stefan Roese wrote: >> On 14.07.21 16:37, Pali Rohár wrote: >>> Add a new 'quiet' argument to mmc_get_op_cond() function which avoids >>> printing error message when SD/eMMC card is no

Re: [PATCH u-boot-mvebu 16/31] tools: kwbimage: Cosmetic fix - remove redundant space character

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: Remove this space, since the constants are indented by tabs. Signed-off-by: Marek Behún Reviewed-by: Stefan Roese Thanks, Stefan --- tools/kwbimage.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/kwbimage.h b/tools/kwbim

Re: [PATCH u-boot-mvebu 17/31] tools: kwbimage: Use -a parameter (load address) for v1 images

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár The data part of v1 kwbimage currently contains U-Boot binary prepended by 64 bytes long Legacy U-Boot image header. This means that the load address is currently substracted by 64 bytes to ensure that U-Boot's entry point is at specified e

Re: [PATCH u-boot-mvebu 18/31] arm: mvebu: Fix return_to_bootrom()

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár Register r0 should be set to return value 0x0 - NO_ERR. Set r0 with return value after all registers are restored from the stack, so that the return value is always correct. Signed-off-by: Pali Rohár Fixes: 944c7a317675 ("arm: mvebu: Add

Re: [PATCH u-boot-mvebu 19/31] arm: mvebu: Mark return_to_bootrom() as a noreturn function

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár This function does not return, so add the appropriate compiler flag. Signed-off-by: Pali Rohár Reviewed-by: Marek Behún Reviewed-by: Stefan Roese Thanks, Stefan --- arch/arm/mach-mvebu/include/mach/cpu.h | 2 +- 1 file changed,

Re: [PATCH u-boot-mvebu 20/31] arm: mvebu: Implement return_to_bootrom() via U-Boot's SPL framework

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár U-Boot's SPL framework already has an API for loading U-Boot via BootROM. Implement the function board_return_to_bootrom() for mvebu SPL code. Signed-off-by: Pali Rohár Reviewed-by: Marek Behún Reviewed-by: Stefan Roese Thanks, Stef

Re: [PATCH u-boot-mvebu 21/31] arm: mvebu: Use U-Boot's SPL BootROM framework for booting from NAND/UART

2021-07-15 Thread Stefan Roese
On 08.07.21 19:30, Marek Behún wrote: From: Pali Rohár Current code uses hack in board_init_f() which calls return_to_bootrom() to skip U-Boot SPL code and return back to BootROM to load U-Boot via UART or from NAND. This change migrates that hack from the board_init_f() function and changes i

Re: [PATCH] riscv: booti: do not force relocation if force_reloc is not set

2021-07-15 Thread Leo Liang
On Tue, Apr 06, 2021 at 10:50:16AM +0300, Vitaly Wool wrote: > If force_reloc flag is not set and booti is called for an address > ouside RAM (i. e. QSPI NOR flash), we should honor that and not try > to force relocation in a bogus fashion. > > Signed-off-by: Vitaly Wool > --- > arch/riscv/lib/i

[PATCH 0/2] arm: allwinner: r40: add devicetree for Forlinx FETA40i-C & OKA40i-C

2021-07-15 Thread Ivan Uvarov
This patchset adds a devicetree for the Forlinx FETA40i-C SoM and OKA40i-C carrier board/devboard, mirroring changes merged into Linux 5.14-rc1. Signed-off-by: Ivan Uvarov Ivan Uvarov (2): arm: allwinner: r40: add pinmux settings for MMC3 and UARTs 2,4,5&7 arm: allwinner: r40: add devicetree

u-boot@lists.denx.de

2021-07-15 Thread Ivan Uvarov
The Forlinx OKA40i-C devboard makes use of UARTs 0,2,3,4,5 and 7 of the R40 SoC, of which UART 0 is connected to an RS232 converter, UART 5 routed to an RS485 converter, and the rest broken out directly via labeled headers. The board also contains a micro-SD slot connected to SDC3. This patch adds

[PATCH 2/2] arm: allwinner: r40: add devicetree for Forlinx FETA40i-C & OKA40i-C

2021-07-15 Thread Ivan Uvarov
The FETA40i-C is a SoM by Forlinx based on the Allwinner R40/A40i. SoM specifications: - SoC: R40 or A40i - PMIC: AXP221S - RAM: 1GiB/2GiB DDR3 (dual-rank) - eMMC: 8GB, - Mates with carrier board via four 80-pin connectors (AXK6F80337YG). OKA40i-C is a carrier board by the same manufacturer for

Re: [PATCH v2 2/2] arm: imx8m: add support for Advantech RSB-3720

2021-07-15 Thread Peng Fan (OSS)
Hi Paul, On 2021/7/15 8:50, Ying-Chun Liu wrote: From: "Ying-Chun Liu (PaulLiu)" Add initial support for Advantech RSB-3720 board. The initial support includes: - MMC - eMMC - I2C - FEC - Serial console Signed-off-by: Darren Huang Signed-off-by: Kevin12.Chen Signed-off-by: Phill.L

Re: [PATCH 1/3] arm: mvebu: Espressobin: Fix setting $fdtfile env

2021-07-15 Thread Stefan Roese
On 14.07.21 16:37, Pali Rohár wrote: Ensure that 'env default -a' always set correct value to $fdtfile, even when custom user variable is already stored in non-volatile env storage (means that env_get("fdtfile") call returns non-NULL value). As default value is now correctly set like if specifie

Re: [PATCH u-boot-mvebu 1/2] arm: mvebu: turris_omnia: force 40 MHz speed on SPI NOR

2021-07-15 Thread Stefan Roese
On 09.07.21 16:56, Marek Behún wrote: Commit e2e95e5e2542 ("spi: Update speed/mode on change") changed the boot time on Turris Omnia from ~2.3s to over 8s, due to SPL loading main U-Boot from SPI NOR at 1 MHz instead of 40 MHz. This is because the SPL code passes the CONFIG_SF_DEFAULT_SPEED opti

Re: [PATCH 1/3] arm: a37xx: pci: Extend validation for PCIe resources and oubound windows

2021-07-15 Thread Stefan Roese
On 08.07.21 20:18, Pali Rohár wrote: Remapped address of PCIe outbound window may have set only bits from the mask. Add additional check that remapped address which is calculated from PCIe bus address specified in DTS file is valid. Remove also useless clearing of low 16 bits in win_mask. As win

Please pull u-boot-marvell/master

2021-07-15 Thread Stefan Roese
Hi Tom, please pull the next batch of Marvell MVEBU related patches. Here the summary log: - Turris_omnia/mox: SPI NOR and MTD related changes / fixes (Marek) - a37xx: pci: Misc fixes / optimizations (Pali) - Espressobin: Fix setti

[PATCH v2] drivers: watchdog: wdt-uclass: Use IS_ENABLED for WATCHDOG_AUTOSTART

2021-07-15 Thread Teresa Remmet
There is no separate SPL/TPL config for WATCHDOG_AUTOSTART. So use IS_ENABLED instead of CONFIG_IS_ENABLED to make watchdog working in SPL again. Fixes: 830d29ac3721 ("watchdog: Allow to use CONFIG_WDT without starting watchdog") Signed-off-by: Teresa Remmet Reviewed-by: Stefan Roese --- Change

Re: [PATCH v3 03/16] x86: Allow coreboot serial driver to guess the UART

2021-07-15 Thread Bin Meng
Hi Simon, On Mon, Jun 28, 2021 at 7:51 AM Simon Glass wrote: > > At present this driver relies on coreboot to provide information about > the console UART. However if coreboot is not compiled with the UART > enabled, the information is left out. This configuration is quite > common, e.g. with shi

Re: [PATCH v3 00/16] misc: Some more misc patches

2021-07-15 Thread Bin Meng
Hi Simon, On Wed, Jul 14, 2021 at 11:47 PM Simon Glass wrote: > > Hi Bin, > > On Sun, 27 Jun 2021 at 17:51, Simon Glass wrote: > > > > Various issues were discovered in getting Chromium OS verified boot > > running on top of coreboot, booting into U-Boot. > > > > Improvements include: > > - enab

Re: [PATCH v2 0/5] arm: dts: Add PMIC node for J7200

2021-07-15 Thread Lokesh Vutla
On Wed, 23 Jun 2021 16:14:49 -0500, Gowtham Tammana wrote: > The J7200 EVM has PMIC LP876441 for supporting CPU AVS. This patchset > adds dt nodes, compatible string, and configs to enable the > corresponding driver. > > v2: > - rebased the changes are reordered patches 3/4 > > [...] Applied t

Re: [PATCH v3 0/5] Add PMIC node for J7200

2021-07-15 Thread Lokesh Vutla
On Wed, 14 Jul 2021 15:52:55 -0500, Gowtham Tammana wrote: > The J7200 EVM has PMIC LP876441 for supporting CPU AVS. This patchset > adds dt nodes, compatible string, and configs to enable the > corresponding driver. > > v3: > - rebased to resolve minor conflicts against master > > [...] Appli

Re: [PATCH 1/2] Nokia RX-51: Add support for booting kernel in zImage format

2021-07-15 Thread Lokesh Vutla
On Fri, 18 Jun 2021 15:27:03 +0200, Pali Rohár wrote: > Enable U-Boot bootz command and update env scripts to try loading also > zImage file and to try booting via bootz command. > > Update also lowlevel_init.S code for checking validity of zImage magic to > correctly relocate kernel in zImage for

Re: [PATCH] Nokia RX-51: Add check for /lib/ld-linux.so.2 in test script

2021-07-15 Thread Lokesh Vutla
On Fri, 18 Jun 2021 15:31:08 +0200, Pali Rohár wrote: > Unfortunately for testing is required qflasher which works only in 32-bit > x86 mode. Apparently 64-bit x86 Azure CI has no problems as it has > preinstalled 32-bit libraries and can execute also 32-bit x86 executables. > > This change just s

Re: [PATCH] x86: Ensure the e820 map is installed in all cases

2021-07-15 Thread Bin Meng
On Sun, Jul 11, 2021 at 11:15 AM Simon Glass wrote: > > This is a revert of a recent logic change in setup_zimage(). We do > actually need to install this information always. Change it to install > from the Coreboot tables if available, else the normal source. > > Fixes: e7bae8283fe ("x86: Allow i

Re: [PATCH] x86: Ensure the e820 map is installed in all cases

2021-07-15 Thread Bin Meng
On Thu, Jul 15, 2021 at 7:54 PM Bin Meng wrote: > > On Sun, Jul 11, 2021 at 11:15 AM Simon Glass wrote: > > > > This is a revert of a recent logic change in setup_zimage(). We do > > actually need to install this information always. Change it to install > > from the Coreboot tables if available,

Re: [PATCH 1/5] efi_loader: increase eventlog buffer size

2021-07-15 Thread Simon Glass
Hi Ilias, On Thu, 15 Jul 2021 at 00:20, Ilias Apalodimas wrote: > > On Wed, Jul 14, 2021 at 08:52:07AM -0600, Simon Glass wrote: > > Hi Ilias, > > > > On Mon, 12 Jul 2021 at 03:28, Ilias Apalodimas > > wrote: > > > > > > On Mon, 12 Jul 2021 at 11:40, Masahisa Kojima > > > wrote: > > > > > > > >

RE: [EXT] Re: [RFC 1/7] ARM: mvebu: a38x: Correct mismatched bound warnings

2021-07-15 Thread Kostya Porotchkin
Hi Pali and Stefan, > -Original Message- > From: Stefan Roese > Sent: Thursday, July 15, 2021 11:00 > To: Pali Rohár ; Tom Rini > Cc: u-boot@lists.denx.de; Marek Behún ; Kostya > Porotchkin > Subject: [EXT] Re: [RFC 1/7] ARM: mvebu: a38x: Correct mismatched bound > warnings > > Externa

Re: [PATCH 0/5] stm32mp1: handle TF-A boot with FIP

2021-07-15 Thread Patrick DELAUNAY
Hi Simon On 7/11/21 2:01 AM, Simon Glass wrote: Hi Patrick, On Thu, 8 Jul 2021 at 03:17, Patrick Delaunay wrote: In next TF-A version the stm32mp1 platform will support the Firmware Image Package (FIP) [1], a container filled with: - the U-Boot binary = u-boot-nodtb.bin - the U-Boot device t

Please pull u-boot-x86

2021-07-15 Thread Bin Meng
Hi Tom, This PR includes the following changes for v2021.10: - x86: various improvements made in getting Chromium OS verified boot running on top of coreboot, booting into U-Boot. Azure results: PASS https://dev.azure.com/bmeng/GitHub/_build/results?buildId=383&view=results The following chan

Re: [TF-A] Proposal: TF-A to adopt hand-off blocks (HOBs) for information passing between boot stages

2021-07-15 Thread François Ozog
On Sat, 10 Jul 2021 at 01:07, Julius Werner wrote: > > - "bloblist" is a general term for concept of linked list and it's not > exactly U-boot implementation. The proposed solution will cause some degree > of changes in all the participating projects. For backward compatibility > issue, we have a

Re: [PATCH 1/5] efi_loader: increase eventlog buffer size

2021-07-15 Thread Heinrich Schuchardt
On 7/15/21 2:57 PM, Simon Glass wrote: Hi Ilias, On Thu, 15 Jul 2021 at 00:20, Ilias Apalodimas wrote: On Wed, Jul 14, 2021 at 08:52:07AM -0600, Simon Glass wrote: Hi Ilias, On Mon, 12 Jul 2021 at 03:28, Ilias Apalodimas wrote: On Mon, 12 Jul 2021 at 11:40, Masahisa Kojima wrote: Hi S

Re: [PATCH] Disable timer check in file loading

2021-07-15 Thread Heinrich Schuchardt
On 7/8/21 10:22 AM, Michael Chang wrote: The u-boot efi console service registers a timer to poll the keyboard input in every 50ns. In the efi block io service, this timer is The event is triggered every 5000 ns not 50 ns. See lib/efi_loader/efi_console.c:1309. The SetTimer() parameter is in mu

Re: [TF-A] Proposal: TF-A to adopt hand-off blocks (HOBs) for information passing between boot stages

2021-07-15 Thread Simon Glass
Hi François, On Thu, 15 Jul 2021 at 08:25, François Ozog wrote: > > > On Sat, 10 Jul 2021 at 01:07, Julius Werner wrote: > >> > - "bloblist" is a general term for concept of linked list and it's not >> exactly U-boot implementation. The proposed solution will cause some degree >> of changes in

Re: [PATCH 0/5] stm32mp1: handle TF-A boot with FIP

2021-07-15 Thread Simon Glass
Hi Patrick, On Thu, 15 Jul 2021 at 07:50, Patrick DELAUNAY wrote: > > Hi Simon > > On 7/11/21 2:01 AM, Simon Glass wrote: > > Hi Patrick, > > > > On Thu, 8 Jul 2021 at 03:17, Patrick Delaunay > > wrote: > >> > >> In next TF-A version the stm32mp1 platform will support the Firmware > >> Image Pac

Re: [PATCH v3 03/16] x86: Allow coreboot serial driver to guess the UART

2021-07-15 Thread Simon Glass
Hi Bin, On Thu, 15 Jul 2021 at 05:44, Bin Meng wrote: > > Hi Simon, > > On Mon, Jun 28, 2021 at 7:51 AM Simon Glass wrote: > > > > At present this driver relies on coreboot to provide information about > > the console UART. However if coreboot is not compiled with the UART > > enabled, the infor

Re: [PATCH 1/5] efi_loader: increase eventlog buffer size

2021-07-15 Thread Simon Glass
Hi Heinrich, On Thu, 15 Jul 2021 at 08:38, Heinrich Schuchardt wrote: > > On 7/15/21 2:57 PM, Simon Glass wrote: > > Hi Ilias, > > > > On Thu, 15 Jul 2021 at 00:20, Ilias Apalodimas > > wrote: > >> > >> On Wed, Jul 14, 2021 at 08:52:07AM -0600, Simon Glass wrote: > >>> Hi Ilias, > >>> > >>> On M

[PATCH v2 0/5] stm32mp1: handle TF-A boot with FIP

2021-07-15 Thread Patrick Delaunay
In next TF-A version the stm32mp1 platform will support the Firmware Image Package (FIP) [1], a container filled with: - the U-Boot binary = u-boot-nodtb.bin - the U-Boot device tree = u-boot.dtb - the Secure OS (OP-TEE) or the secure monitor (SP_MIN) Upstream is in progress on TF-A side ([3] bu

[PATCH v2 2/5] arm: stm32mp: handle the OP-TEE nodes in DT with FIP support

2021-07-15 Thread Patrick Delaunay
With FIP support in TF-A (when CONFIG_STM32MP15x_STM32IMAGE is not activated), the DT nodes needed by OP-TEE are added by OP-TEE firmware in U-Boot device tree, present in FIP. These nodes are only required in trusted boot, when TF-A load the file u-boot.stm32, including the U-Boot device tree wit

[PATCH v2 1/5] arm: stm32mp: add config for STM32IMAGE support

2021-07-15 Thread Patrick Delaunay
By default for trusted boot with TF-A, U-Boot (u-boot-nodtb) is located in FIP container with its device tree and with the secure monitor (provided by TF-A or OP-TEE). The FIP file is loaded by TF-A BL2 and each components is extracted at the final location. This patch add CONFIG_STM32MP15x_STM32I

[PATCH v2 5/5] stm32mp1: stm32prog: remove stm32prog_get_tee_partitions with FIP

2021-07-15 Thread Patrick Delaunay
The MTD tee partitions used to save the OP-TEE binary are needed when TF-A doesn't use the FIP container to load binaries. This patch puts under CONFIG_STM32MP15x_STM32IMAGE flag the associated code in U-Boot binary and prepare the code cleanup when CONFIG_STM32MP15x_STM32IMAGE support will be rem

[PATCH v2 4/5] doc: st: stm32mp1: Add FIP support for trusted boot

2021-07-15 Thread Patrick Delaunay
TF-A for STM32MP15 now supports the FIP: it is a packaging format which includes the secure monitor, u-boot-nodtb.bin and u-boot.dtb This FIP file is loaded by FSBL = TF-A BL2. This patch updates the board documentation to use this FIP file and no more u-boot.stm32 (with STM32 image header) which

[PATCH v2 3/5] arm: stm32mp: add defconfig for trusted boot with FIP

2021-07-15 Thread Patrick Delaunay
Add TF-A FIP support for trusted boot on STM32MP15x, when STM32MP15x_STM32IMAGE is not activated. With FIP support the SSBL partition is named "fip" and its size is 4MB, so the ENV partition name in device tree (for SD card or eMMC) or offset in defconfig (CONFIG_ENV_OFFSET / CONFIG_ENV_OFFSET_RE

Re: [PATCH 1/5] efi_loader: increase eventlog buffer size

2021-07-15 Thread Heinrich Schuchardt
On 15.07.21 17:18, Simon Glass wrote: Hi Heinrich, On Thu, 15 Jul 2021 at 08:38, Heinrich Schuchardt wrote: On 7/15/21 2:57 PM, Simon Glass wrote: Hi Ilias, On Thu, 15 Jul 2021 at 00:20, Ilias Apalodimas wrote: On Wed, Jul 14, 2021 at 08:52:07AM -0600, Simon Glass wrote: Hi Ilias, O

[PATCHv2 1/2] i2c: add dm_i2c_reg_clrset

2021-07-15 Thread Sebastian Reichel
Add function to apply a bitmask to an i2c register, so that specific bits can be cleared and/or set. Suggested-by: Simon Glass Signed-off-by: Sebastian Reichel --- drivers/i2c/i2c-uclass.c | 15 +++ include/i2c.h| 14 ++ test/dm/i2c.c| 29

[PATCHv2 2/2] gpio: mcp230xx: Introduce new driver

2021-07-15 Thread Sebastian Reichel
Introduce driver for I2C based MCP230xx GPIO chips, which are quite common and already well supported by the Linux kernel. Reviewed-by: Simon Glass Signed-off-by: Sebastian Reichel --- drivers/gpio/Kconfig | 10 ++ drivers/gpio/Makefile| 1 + drivers/gpio/mcp230xx_gpio.c | 23

[PATCHv2 0/2] Introduce mcp230xx support

2021-07-15 Thread Sebastian Reichel
Hi, Add driver for mcp230xx devices, which are quite common. Changes since PATCHv1: * add new patch adding dm_i2c_reg_clrset() as suggested by Simon Glass * Use | instead of + to build device register (resulting number is the same) * Properly handle GPIOD_IS_OUT_ACTIVE flag * Use bool f

Re: [PATCH 1/5] efi_loader: increase eventlog buffer size

2021-07-15 Thread Simon Glass
Hi Heinrich, On Thu, 15 Jul 2021 at 09:35, Heinrich Schuchardt wrote: > > > > On 15.07.21 17:18, Simon Glass wrote: > > Hi Heinrich, > > > > On Thu, 15 Jul 2021 at 08:38, Heinrich Schuchardt > > wrote: > >> > >> On 7/15/21 2:57 PM, Simon Glass wrote: > >>> Hi Ilias, > >>> > >>> On Thu, 15 Jul 2

Re: [PATCH v3 2/5] arm/dts: k3-j7200-r5-common: Add pmic lp876441 node

2021-07-15 Thread Grygorii Strashko
On 14/07/2021 23:52, Gowtham Tammana wrote: Add pmic lp876411 node needed for CPU AVS support. Signed-off-by: Gowtham Tammana --- .../arm/dts/k3-j7200-r5-common-proc-board.dts | 26 +++ 1 file changed, 26 insertions(+) diff --git a/arch/arm/dts/k3-j7200-r5-common-proc-boa

[PATCH 1/3] efi_capsule: Move signature from DTB to .rodata

2021-07-15 Thread Ilias Apalodimas
The capsule signature is now part of our DTB. This is problematic when a user is allowed to change/fixup that DTB from U-Boots command line since he can overwrite the signature as well. So Instead of adding the key on the DTB, embed it in the u-boot binary it self as part of it's .rodata. This as

[PATCH 2/3] mkeficapsule: Remove dtb related options

2021-07-15 Thread Ilias Apalodimas
commit 322c813f4bec ("mkeficapsule: Add support for embedding public key in a dtb") added a bunch of options enabling the addition of the capsule public key in a dtb. Since now we embeded the key in U-Boot's .rodata we don't this this functionality anymore Signed-off-by: Ilias Apalodimas --- t

[PATCH 3/3] doc: Update CapsuleUpdate READMEs

2021-07-15 Thread Ilias Apalodimas
Since we removed embeddingg the capsule key into a .dtb and fixed authenticated capsule updates for all boards, move the relevant documentation in the efi file and update it accordingly Signed-off-by: Ilias Apalodimas --- doc/board/emulation/qemu_capsule_update.rst | 203 do

Re: [Uboot-stm32] [PATCH v2 2/5] arm: stm32mp: handle the OP-TEE nodes in DT with FIP support

2021-07-15 Thread Ahmad Fatoum
Hello Patrick, On 15.07.21 17:22, Patrick Delaunay wrote: > With FIP support in TF-A (when CONFIG_STM32MP15x_STM32IMAGE > is not activated), the DT nodes needed by OP-TEE are added by OP-TEE > firmware in U-Boot device tree, present in FIP. What about the SCMI nodes. Who will fix up those? > The

[PATCH u-boot-mvebu 1/2] ARM: dts: armada-385-turris-omnia: add `u-boot-env` NOR partition

2021-07-15 Thread Marek Behún
Specify a separate partition `u-boot-env` for U-Boot's env settings for the Turris Omnia board. Do this only in U-Boot's specific DTS. We do not want to do this in Linux' official DTS, because Omnia's stock U-Boot stores env at a different address, and there are still boards with stock U-Boot. In

[PATCH u-boot-mvebu 2/2] arm: mvebu: turris_omnia: fixup MTD partitions in Linux' DTB

2021-07-15 Thread Marek Behún
Fixup SPI NOR partition nodes in Linux' device tree prior booting Linux. Linux' devicetree does not contain "u-boot-env" partition and we do not want to add it there because the address is different between stock U-Boot and current upstream U-Boot. Instead we add code that recreates partition nod

Re: [PATCH 4/9] sandbox: tpm: Correct handling of get-capability

2021-07-15 Thread Ilias Apalodimas
Hi Simon, On Mon, Jul 05, 2021 at 09:48:44AM -0600, Simon Glass wrote: > This function current handles the kernel case incorrectly. Fix it, and > use the shorter TPM_HDR_LEN while we are here. > > Signed-off-by: Simon Glass > --- > > drivers/tpm/tpm_tis_sandbox.c | 14 -- > 1 file

Re: [PATCH 5/9] sandbox: tpm: Finish comments for struct sandbox_tpm2

2021-07-15 Thread Ilias Apalodimas
On Mon, Jul 05, 2021 at 09:48:45AM -0600, Simon Glass wrote: > Tidy up the missing comments for this struct. > > Signed-off-by: Simon Glass > --- > > drivers/tpm/tpm2_tis_sandbox.c | 20 > 1 file changed, 16 insertions(+), 4 deletions(-) > > diff --git a/drivers/tpm/tpm2_t

Re: [PATCHv2 1/2] i2c: add dm_i2c_reg_clrset

2021-07-15 Thread Simon Glass
On Thu, 15 Jul 2021 at 09:40, Sebastian Reichel wrote: > > Add function to apply a bitmask to an i2c register, so > that specific bits can be cleared and/or set. > > Suggested-by: Simon Glass > Signed-off-by: Sebastian Reichel > --- > drivers/i2c/i2c-uclass.c | 15 +++ > include/i2c

Re: [PATCHv2 2/2] gpio: mcp230xx: Introduce new driver

2021-07-15 Thread Simon Glass
Hi Sebastian, On Thu, 15 Jul 2021 at 09:40, Sebastian Reichel wrote: > > Introduce driver for I2C based MCP230xx GPIO chips, which are > quite common and already well supported by the Linux kernel. > > Reviewed-by: Simon Glass > Signed-off-by: Sebastian Reichel > --- > drivers/gpio/Kconfig

u-boot and mtdparts

2021-07-15 Thread Marek Behún
Hello Tom and others, many boards still use the obsolete mtdparts command. What is the plan with this command? Do we still want to support it? Recently as discovered by Masami, my patch mtd: spi-nor: allow registering multiple MTDs when DM is enabled caused that mtds may have different name th

[PATCH] CI: Update to LLVM-12

2021-07-15 Thread Tom Rini
The current stable release of LLVM is 12, update to that. While at it, fix that we had not correctly upgraded to LLVM 11 previously. Signed-off-by: Tom Rini --- .azure-pipelines.yml| 4 ++-- .gitlab-ci.yml | 4 ++-- tools/docker/Dockerfile | 6 +++--- 3 files changed, 7 insertions(

Re: u-boot and mtdparts

2021-07-15 Thread Simon Glass
Hi Marek, On Thu, 15 Jul 2021 at 12:13, Marek Behún wrote: > > Hello Tom and others, > > many boards still use the obsolete mtdparts command. > What is the plan with this command? Do we still want to support it? > > Recently as discovered by Masami, my patch > mtd: spi-nor: allow registering m

Re: [PATCH v4 1/5] spl: mmc: Support OP-TEE payloads in Falcon mode

2021-07-15 Thread Patrick DELAUNAY
Hi, On 5/31/21 7:43 PM, Alexandru Gagniuc wrote: In general, Falcon mode means we're booting a linux kernel directly. With FIT images, however, an OP-TEE secure kernel can be booted before linux. Thus, if the next stage is an IH_OS_TEE, this isn't necessarily a problem. Of course, a general sol

Re: Please pull u-boot-marvell/master

2021-07-15 Thread Tom Rini
On Thu, Jul 15, 2021 at 12:36:58PM +0200, Stefan Roese wrote: > Hi Tom, > > please pull the next batch of Marvell MVEBU related patches. Here the > summary log: > Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: Please pull u-boot-x86

2021-07-15 Thread Tom Rini
On Thu, Jul 15, 2021 at 09:54:08PM +0800, Bin Meng wrote: > Hi Tom, > > This PR includes the following changes for v2021.10: > > - x86: various improvements made in getting Chromium OS verified boot > running on top of coreboot, booting into U-Boot. > > Azure results: PASS > https://dev.azure

[PATCH v2 0/6] Convert MIPS Malta boards to PCI DM

2021-07-15 Thread Daniel Schwierzeck
This series converts the PCI host controller drivers used by MIPS Malta and the board-specific PCI setup code to PCI driver model. Because the AMD PCNET driver is already converted to ETH driver model, simply enable CONFIG_DM_ETH as well. A patch in PCI uclass core is currently required for MIPS

[PATCH v2 1/6] dm: pci: add option to map virtual system memory base address

2021-07-15 Thread Daniel Schwierzeck
On MIPS the DRAM start address respectively CONFIG_SYS_SDRAM_BASE is still used as a virtual, CPU-mapped address instead of being used as physical address. Converting all MIPS boards and generic MIPS code to fix that is not trivial. Due to the approaching deadline for PCI DM conversion, this workar

[PATCH v2 2/6] pci: gt64120: convert to driver model

2021-07-15 Thread Daniel Schwierzeck
This driver is currently only used on MIPS Malta boards. Signed-off-by: Daniel Schwierzeck Reviewed-by: Simon Glass --- Changes in v2: - add empty line before return statements drivers/pci/pci_gt64120.c | 74 ++- 1 file changed, 73 insertions(+), 1 deleti

  1   2   >