Re: [PATCH] pci: pci_mvebu: Add support for Kirkwood PCIe controllers

2022-01-14 Thread Tony Dinh
Hi Pali, On Thu, Jan 13, 2022 at 3:35 PM Tony Dinh wrote: > > Hi Pali, > > On Thu, Jan 13, 2022 at 5:28 AM Pali Rohár wrote: > > > > Kirkwood uses macros KW_DEFADR_PCI_MEM and KW_DEFADR_PCI_IO for base > > address of PCIe mappings. Size of PCIe windows is not defined in any macro > > yet, so

[PATCH] MAINTAINERS: remove Alexander Graf from EFI PAYLOAD

2022-01-14 Thread Heinrich Schuchardt
The last review by Alex was in 2019. Signed-off-by: Heinrich Schuchardt --- MAINTAINERS | 1 - 1 file changed, 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 6ae81c5659..38c68ee87d 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -737,7 +737,6 @@ F: test/dm/efi_media.c EFI PAYLOAD

[PATCH] efidebug: avoid 'dfu_alt_info not defined' message

2022-01-14 Thread Heinrich Schuchardt
If variable dfu_alt_info is not defined duplicate messages are displayed. => efidebug boot dump Scanning disk mmc2.blk... Scanning disk mmc1.blk... Scanning disk mmc0.blk... Found 3 disks No EFI system partition "dfu_alt_info" env variable not defined! Probably

[RFC 1/1] efi_loader: rename 'efidebug boot' 'bootefi'

2022-01-14 Thread Heinrich Schuchardt
The efidebug command was conceived for testing purposes. The manipulation of boot options does better fit to the bootefi command that is used to invoke the boot manager. Signed-off-by: Heinrich Schuchardt --- cmd/bootefi.c | 672 +- cmd/efidebug.c

Re: binman generator nodes break over read-back

2022-01-14 Thread Simon Glass
Hi Jan, On Fri, 14 Jan 2022 at 06:40, Simon Glass wrote: > > Hi Jan, > > On Fri, 14 Jan 2022 at 02:40, Jan Kiszka wrote: > > > > Hi Simon, > > > > something is fishy with generator nodes: We are playing with replacing > > fdts and configs in [1] with generator nodes, and that works so far - > >

[PATCH 1/1] configs: disable UEFI for Colibri VF610

2022-01-14 Thread Heinrich Schuchardt
The size of the board file is limited to 520192 bytes. This conflicts with the size requirement for the UEFI code. Signed-off-by: Heinrich Schuchardt --- configs/colibri_vf_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configs/colibri_vf_defconfig

[PATCH 1/1] efi_loader: fix SectionAlignment, FileAlignment

2022-01-14 Thread Heinrich Schuchardt
The alignment of sections in the EFI binaries generated by U-Boot is incorrect. According to the PE-COFF specification [1] the minimum value for FileAlignment is 512. If the value of SectionAlignment is less then the page size, it must equal FileAlignment. Let's set both values to 512 for the

Re: imx8mm-evk does not boot kernel 5.16

2022-01-14 Thread Marcel Ziswiler
Hi Fabio On Fri, 2022-01-14 at 16:44 -0300, Fabio Estevam wrote: > Hi Marcel, > > On Fri, Jan 14, 2022 at 3:10 PM Marcel Ziswiler > wrote: > > > Strange, remember, I am working on Verdin iMX8M Mini support [1] and run > > that stuff more or less daily. > > However, as you can see in that

[PATCH v3 1/3] imx: Don't define __ASSEMBLY__ in C files

2022-01-14 Thread Simon Glass
This is supposed to be a build-system flag. Move it there so we can define it before linux/kconfig.h is included. Signed-off-by: Simon Glass --- (no changes since v1) arch/arm/mach-imx/Makefile| 2 +- arch/arm/mach-imx/imx8m/imximage-8mm-lpddr4.cfg |

[PATCH v3 3/3] bloblist: Update to use conditional value

2022-01-14 Thread Simon Glass
Use the new IF_ENABLED_INT() feature to avoid needing our own inline function to handle this case. Tidy up the logic to ensure that the value is only used when present. Update the 'expected' comment also. Signed-off-by: Simon Glass --- Changes in v3: - Add new patch to update bloblist to use

[PATCH v3 2/3] kconfig: Add support for conditional values

2022-01-14 Thread Simon Glass
At present if an optional Kconfig value needs to be used it must be bracketed by #ifdef. For example, with this Kconfig setup: config WIBBLE bool "Support wibbles, the world needs more wibbles" config WIBBLE_ADDR hex "Address of the wibble" depends on WIBBLE then the

Re: [PATCH] phy: nop-phy: Enable reset-gpios support

2022-01-14 Thread Adam Ford
On Fri, Jan 14, 2022 at 1:36 PM Tom Rini wrote: > > On Wed, Dec 22, 2021 at 08:04:30AM -0600, Adam Ford wrote: > > > Some usb-nop-xceiv devices use a gpio to put them in and > > out of reset. Add a reset function to put them into that > > state. This is similar to how Linux handles the > >

Re: [PATCH] cmd: Add Kconfig option for multiprocessor cmds

2022-01-14 Thread Tom Rini
On Fri, Jan 14, 2022 at 01:07:34PM +0100, Michal Simek wrote: > From: Ashok Reddy Soma > > Add Kconfig option(CONFIG_CMD_MP) to enable or disable multiprocessor > commands. Compile cmd/mp.c based on CONFIG_CMD_MP. > > Signed-off-by: Ashok Reddy Soma > Signed-off-by: Michal Simek > --- > >

Re: imx8mm-evk does not boot kernel 5.16

2022-01-14 Thread Fabio Estevam
Hi Marcel, On Fri, Jan 14, 2022 at 3:10 PM Marcel Ziswiler wrote: > Strange, remember, I am working on Verdin iMX8M Mini support [1] and run that > stuff more or less daily. > However, as you can see in that patch series, I build the imx-sdma driver now > as a module just like already >

Re: [PATCH] phy: nop-phy: Enable reset-gpios support

2022-01-14 Thread Tom Rini
On Wed, Dec 22, 2021 at 08:04:30AM -0600, Adam Ford wrote: > Some usb-nop-xceiv devices use a gpio to put them in and > out of reset. Add a reset function to put them into that > state. This is similar to how Linux handles the > usb-nop-xceiv driver. > > Signed-off-by: Adam Ford > > diff

Re: [PATCH] arm: mach-k3: am642_init: Unlock MCU PADCFG regs

2022-01-14 Thread Nishanth Menon
On 16:14-20211215, Christian Gmeiner wrote: > From: Michael Liebert > > Currently only the PADCFG registers of the main domain are unlocked. > Also unlock PADCFG registers of MCU domain, so MCU pin muxing can be > configured by u-boot or Linux. > > Signed-off-by: Michael Liebert > Tested-by:

Re: [PATCH] pinctrl: single: add support for pinctrl-single, pins when #pinctrl-cells = 2

2022-01-14 Thread Tom Rini
On Fri, Dec 03, 2021 at 03:18:53PM +, AJ Bagwell wrote: > Changes to the am33xx device (33e9021a) trees have been merged in from > the upstream linux kernel which now means the device tree uses the new > pins format (as of 5.10) where the confinguration can be stores as a > separate

Re: [PATCH v3 1/1] rsa: adds rsa3072 algorithm

2022-01-14 Thread Tom Rini
On Fri, Dec 10, 2021 at 02:00:55PM +0800, Jamin Lin wrote: > Add to support rsa 3072 bits algorithm in tools > for image sign at host side and adds rsa 3072 bits > verification in the image binary. > > Add test case in vboot for sha384 with rsa3072 algorithm testing. > > Signed-off-by: Jamin

Re: imx8mm-evk does not boot kernel 5.16

2022-01-14 Thread Marcel Ziswiler
Hi Fabio On Fri, 2022-01-14 at 15:00 -0300, Fabio Estevam wrote: > Hi, > > I am able to boot Linux 5.15.14 on an imx8mm-evk running U-Boot > 2022.01 just fine. > > However, if I try to boot a 5.16 kernel, the boot fails in most of the > attempts (like in 80% of the boots. It does boot

imx8mm-evk does not boot kernel 5.16

2022-01-14 Thread Fabio Estevam
Hi, I am able to boot Linux 5.15.14 on an imx8mm-evk running U-Boot 2022.01 just fine. However, if I try to boot a 5.16 kernel, the boot fails in most of the attempts (like in 80% of the boots. It does boot completely in some of the attempts). Starting kernel ... (hangs here) Using earlycon I

[PATCH v2] tools: mkimage: Call verify_header after writing image to disk

2022-01-14 Thread Pali Rohár
If image backend provides verify_header callback then call it after writing image to disk. This ensures that written image is correct. Signed-off-by: Pali Rohár --- tools/mkimage.c | 41 + 1 file changed, 41 insertions(+) diff --git a/tools/mkimage.c

Re: Please pull u-boot-marvell/master

2022-01-14 Thread Tom Rini
On Fri, Jan 14, 2022 at 04:42:35PM +0100, Stefan Roese wrote: > Hi Tom, > > please pull the following Marvell MVEBU related patches: > Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH] introduce CONFIG_DEVICE_TREE_INCLUDES

2022-01-14 Thread Simon Glass
Hi Rasmus, On Tue, 26 Oct 2021 at 02:08, Rasmus Villemoes wrote: > > On 26/10/2021 03.28, Simon Glass wrote: > > Hi Rasmus, > > > > On Tue, 28 Sept 2021 at 02:57, Rasmus Villemoes > > wrote: > >> > >> The build system already automatically looks for and includes an > >> in-tree *-u-boot.dtsi

Re: [PATCH v2] introduce CONFIG_DEVICE_TREE_INCLUDES

2022-01-14 Thread Simon Glass
Hi Tom, On Fri, 14 Jan 2022 at 08:41, Tom Rini wrote: > > On Fri, Jan 14, 2022 at 09:30:29AM +0100, Rasmus Villemoes wrote: > > Ping > > > > On 21/11/2021 14.52, Rasmus Villemoes wrote: > > > The build system already automatically looks for and includes an > > > in-tree *-u-boot.dtsi when

[PATCH 3/3] riscv: qemu: Implement is_flash_available() for MTD NOR

2022-01-14 Thread Anup Patel
Currently, if MTD NOR is enabled then U-Boot tries to issue flash commands even when CFI flash DT node is not present. This causes access fault on RISC-V emulators or ISS which do not emulate CFI flash. To handle this issue, we implement is_flash_available() for qemu-riscv board which will return

[PATCH 2/3] riscv: qemu: Enable HTIF console support

2022-01-14 Thread Anup Patel
Enable support for HTIF console so that we can use QEMU RISC-V U-Boot on RISC-V emulators and ISS having it. Signed-off-by: Anup Patel Reviewed-by: Philipp Tomsich --- board/emulation/qemu-riscv/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/board/emulation/qemu-riscv/Kconfig

[PATCH 1/3] serial: Add RISC-V HTIF console driver

2022-01-14 Thread Anup Patel
Quite a few RISC-V emulators and ISS (including Spike) have host transfer interface (HTIF) based console. This patch adds HTIF based console driver for RISC-V platforms which depends totally on DT node for HTIF register base address. Signed-off-by: Anup Patel Reviewed-by: Philipp Tomsich ---

[PATCH 0/3] QEMU spike machine support for U-Boot

2022-01-14 Thread Anup Patel
We can use same U-Boot binary compiled using qemu-riscv64_smode_defconfig on QEMU virt machine and QEMU spike machine. To achieve this, we need HTIF console support for U-Boot QEMU RISC-V board hence this series. To test this series with latest OpenSBI, we can use the following command:

Please pull u-boot-marvell/master

2022-01-14 Thread Stefan Roese
Hi Tom, please pull the following Marvell MVEBU related patches: - mvebu: Move PCIe code from serdes to PCIe driver (Pali) - mtd: nand: pxa3xx: use marvell, prefix for custom DT properties (Pierre) - Add PCIe support for Iomega

Re: [PATCH] arm: mvebu: Replace hardcoded values 0x0030/0x4030 by proper calculation

2022-01-14 Thread Stefan Roese
On 1/12/22 18:32, Pali Rohár wrote: These hardcoded values were calculated from CONFIG_SPL_TEXT_BASE macro. Now this macro is configurable via Kconfig, so calculate values 0x0030/0x4030 at compile time via CONFIG_SPL_TEXT_BASE option. Values 0x0030/0x4030 represents offset of

Re: [PATCH v2] introduce CONFIG_DEVICE_TREE_INCLUDES

2022-01-14 Thread Tom Rini
On Fri, Jan 14, 2022 at 09:30:29AM +0100, Rasmus Villemoes wrote: > Ping > > On 21/11/2021 14.52, Rasmus Villemoes wrote: > > The build system already automatically looks for and includes an > > in-tree *-u-boot.dtsi when building the control .dtb. However, there > > are some things that are

Re: [PATCH] arm: mvebu: db-88f6720: Fix CONFIG_SPL_TEXT_BASE and remove wrong memory layout

2022-01-14 Thread Stefan Roese
On 1/12/22 18:30, Pali Rohár wrote: Memory layout in the comment is from Armada XP platform which uses load address 0x40004030. DB-88f6720 is Armada 375 platform which uses same load address as Armada 38x which is 0x4030. Currently SPL support for Armada 375 is unfinished and does not work.

Re: [PATCH u-boot-marvell v2 00/20] tools: kwbimage: Load address fixes

2022-01-14 Thread Stefan Roese
On 1/12/22 18:20, Pali Rohár wrote: This patch series fixes generating images in kwbimage format, main fix is setting correct load address of U-Boot SPL. Also it adds support for generating kwbimage config file from existing kwbimage file via dumpimage tool. Changes in v2: * Fix base address

Re: [PATCH u-boot-marvell] ddr: marvell: a38x: fix SPLIT_OUT_MIX state decision

2022-01-14 Thread Stefan Roese
On 1/12/22 17:06, Marek Behún wrote: From: Marek Behún This is a cleaned up and fixed version of a patch mv_ddr: a380: fix SPLIT_OUT_MIX state decision in each pattern cycle the bus state can be changed in order to avoide it, need to back to the same bus state on each pattern

Re: [PATCH u-boot-marvell] board: gdsys: Drop Dirk Eibach from MAINTAINERS

2022-01-14 Thread Stefan Roese
On 1/4/22 16:14, Marek Behún wrote: From: Marek Behún I got an : host mxlb.ispgateway.de[80.67.18.126] said: 554 Sorry, no mailbox here by that name. (in reply to RCPT TO command) when sending e-mail to dirk.eib...@gdsys.cc. Drop Dirk Eibach from MAINTAINERS of board/gdsys/a38x and

Re: [PATCH u-boot-marvell] ddr: marvell: a38x: Fix Synchronous vs Asynchronous mode determination

2022-01-14 Thread Stefan Roese
On 1/4/22 15:57, Marek Behún wrote: From: Marek Behún Before commit 4c289425752f ("mv_ddr: a38x: add support for ddr async mode"), Asynchornous Mode was only used when the CPU Subsystem Clock Options[4:0] field in the SAR1 register was set to value 0x13: CPU at 2 GHz and DDR at 933 MHz. Then

Re: [PATCH 0/3] Add PCIe support for Iomega iConnect board

2022-01-14 Thread Stefan Roese
On 1/2/22 05:57, Tony Dinh wrote: The Iomega iConnect board has an internal mPCIe slot. - Add PCIe support to enable the mPCIe slot to accept devices such as Wifi card or mSATA. - Add SYS_THUMB_BUILD to keep u-boot image size within 512K - Add myself as maintainer. Luka no longer has this

Re: [PATCH] mtd: nand: pxa3xx: use marvell, prefix for custom DT properties

2022-01-14 Thread Stefan Roese
On 12/25/21 05:46, Pierre Bourdon wrote: The DT properties for the "enable-arbiter" and "keep-config" config knobs were previously named inconsistently: - The u-boot driver used "nand-enable-arbiter" and "nand-keep-config" names, without Marvell prefixes. - The Linux driver uses

Re: [PATCH u-boot-marvell 0/9] mvebu: Move PCIe code from serdes to PCIe driver

2022-01-14 Thread Stefan Roese
On 12/21/21 12:20, Pali Rohár wrote: This patch series removes gdsys's board_pex_config() function by converting it to spl_board_init(), adds a new mvebu-reset driver for enabling / disabling PCIe ports and finally moves PCIe code from serdes driver to pci_mvebu.c driver. After all these

Re: [PATCH 2/2] cmd: fuse: Add a command to read fuses to memory

2022-01-14 Thread Angus Ainslie
Are there any problems with this patch ? Thanks Angus On 2021-11-28 08:02, Angus Ainslie wrote: With the fuse values in memory we can use some of the other u-boot shell conditonal operators to do tests. Signed-off-by: Angus Ainslie --- cmd/fuse.c | 27 +++ 1 file

Re: [PATCH 1/2] cmd: fuse: add a fuse comparison function

2022-01-14 Thread Angus Ainslie
Are there any problems with this patch ? Thanks Angus On 2021-11-28 08:02, Angus Ainslie wrote: Compare a hexval to the fuse value and return pass or fail. Signed-off-by: Angus Ainslie --- cmd/fuse.c | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git

Re: converting IMX6 board to DM_ETH and DM_USB - usb_ether gadget support

2022-01-14 Thread Heiko Schocher
Hello Tim, On 25.02.21 02:21, Tim Harvey wrote: > Greetings, > > I'm trying to convert the gwventana board support to DM_ETH and DM_USB > and one item I have not resolved yet is USB Ethernet gadget support. > > For non-dm a call to 'usb_eth_initialize' creates a usb_ether gadget > that can be

Re: [PATCH] board: stm32mp1: solve compilation issue when ENV_IS_IN_MMC is deactivated

2022-01-14 Thread Patrice CHOTARD
Hi Patrick On 1/11/22 4:37 PM, Patrick Delaunay wrote: > Solve compilation issue on undefined CONFIG_SYS_MMC_ENV_DEV when > CONFIG_ENV_IS_IN_MMC is deactivated on STMicroelectronics boards > defconfig > > Fixes: 9f97193616f1 ("board: stm32mp1: use CONFIG_SYS_MMC_ENV_DEV when > available") >

Re: [PATCH 1/1] stm32mp: fix board_get_alt_info_mmc()

2022-01-14 Thread Patrice CHOTARD
Hi Heinrich On 1/11/22 3:58 PM, Heinrich Schuchardt wrote: > MAX_SEARCH_PARTITIONS is the highest possible partition number. > Do not skip the last partition in board_get_alt_info_mmc(). > > Signed-off-by: Heinrich Schuchardt > --- > board/st/common/stm32mp_dfu.c | 2 +- > 1 file changed, 1

Re: [PATCH 2/2] stm32mp: correct the dependency for bootcount configs

2022-01-14 Thread Patrice CHOTARD
Hi Patrick On 12/7/21 10:05 AM, Patrick Delaunay wrote: > Default value for CONFIG_SYS_BOOTCOUNT_SINGLEWORD and > CONFIG_SYS_BOOTCOUNT_ADDR are only needed when > CONFIG_BOOTCOUNT_GENERIC is used. > > This patch avoids to define these configs when an other bootcount backend > is activated, for

Re: [PATCH 1/2] stm32mp: remove the bootcount activation

2022-01-14 Thread Patrice CHOTARD
Hi Patrick On 12/7/21 10:05 AM, Patrick Delaunay wrote: > Today the bootcount is not managed by the Linux kernel for STM32MP15 as > we don't have driver to update the used backup register in TAMP and the > recovery command still executes the normal bootcmd with > 'altbootcmd=run bootcmd'. > > So

Re: [PATCH v2 0/3] fs/erofs: new filesystem

2022-01-14 Thread Huang Jianan
在 2022/1/5 19:57, Gao Xiang 写道: Hi Jianan, On Wed, Aug 25, 2021 at 06:40:42PM -0400, Tom Rini wrote: On Mon, Aug 23, 2021 at 08:36:43PM +0800, Huang Jianan wrote: From: Huang Jianan Add erofs filesystem support. The code is adapted from erofs-utils in order to reduce maintenance burden

Re: [PATCH v2] drivers: spi-nor: Add JEDEC id for W25Q16JV

2022-01-14 Thread Angus Ainslie
Hi Dhananjay , Jagan, On 2022-01-13 10:30, Dhananjay Phadke wrote: On 1/13/2022 9:17 AM, Angus Ainslie wrote: Add a JEDEC id for the Winbond W25Q16JV 16M-BIT serial flash memory with DUAL/QUAD SPI Changes since v1: Updated the name for more suffixes Signed-off-by: Angus Ainslie ---

Re: binman generator nodes break over read-back

2022-01-14 Thread Simon Glass
Hi Jan, On Fri, 14 Jan 2022 at 02:40, Jan Kiszka wrote: > > Hi Simon, > > something is fishy with generator nodes: We are playing with replacing > fdts and configs in [1] with generator nodes, and that works so far - > until we want to read-back the image: > > $ binman ls -i flash.bin > binman:

[PATCH u-boot-marvell v4 7/9] arm: mvebu: spl: Use preferred types u8/u16/u32 instead of uintN_t

2022-01-14 Thread Pali Rohár
From: Marek Behún Checkpatch warns about using uint32/16/8_t instead of u32/16/8. Use the preferred types. Signed-off-by: Marek Behún Reviewed-by: Stefan Roese --- arch/arm/mach-mvebu/spl.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff

[PATCH u-boot-marvell v4 8/9] arm: mvebu: spl: Use IS_ENABLED() instead of #ifdef where possible

2022-01-14 Thread Pali Rohár
From: Marek Behún Use the preferred if (IS_ENABLED(X)) instead of #ifdef X where possible. There are still places where this is not possible or is more complicated to convert in this file. Leave those be for now. Signed-off-by: Marek Behún Reviewed-by: Stefan Roese ---

[PATCH u-boot-marvell v4 9/9] arm: mvebu: spl: Fix 100 column exceeds

2022-01-14 Thread Pali Rohár
From: Marek Behún Fix 100 column exceeds in arch/arm/mach-mvebu/spl.c. Signed-off-by: Marek Behún Reviewed-by: Stefan Roese --- arch/arm/mach-mvebu/spl.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/arch/arm/mach-mvebu/spl.c b/arch/arm/mach-mvebu/spl.c index

[PATCH u-boot-marvell v4 6/9] arm: mvebu: spl: Print srcaddr in error message

2022-01-14 Thread Pali Rohár
From: Marek Behún Print the wrong srcaddr (spl_image->offset) in error message also for SATA case. Signed-off-by: Marek Behún Reviewed-by: Stefan Roese --- arch/arm/mach-mvebu/spl.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/arch/arm/mach-mvebu/spl.c

[PATCH u-boot-marvell v4 5/9] arm: mvebu: Check for kwbimage data checksum

2022-01-14 Thread Pali Rohár
Last 4 bytes of kwbimage boot image is checksum. Verify it via the new spl_check_board_image() function which is called by U-Boot SPL after loading kwbimage. Signed-off-by: Pali Rohár Signed-off-by: Marek Behún Reviewed-by: Stefan Roese --- arch/arm/mach-mvebu/spl.c | 27

[PATCH u-boot-marvell v4 2/9] SPL: Add struct spl_boot_device parameter into spl_parse_board_header()

2022-01-14 Thread Pali Rohár
Add parameter spl_boot_device to spl_parse_board_header(), which allows the implementations to see from which device we are booting and do boot-device-specific checks of the image header. Signed-off-by: Pali Rohár Signed-off-by: Marek Behún Reviewed-by: Stefan Roese ---

[PATCH u-boot-marvell v4 4/9] SPL: Add support for checking board / BootROM specific image types

2022-01-14 Thread Pali Rohár
Commit 9baab60b8054 ("SPL: Add support for parsing board / BootROM specific image types") added support for loading board specific image types. This commit adds support for a new weak function spl_parse_board_header() which is called after loading boot image. Board may implement this function for

[PATCH u-boot-marvell v4 1/9] arm: mvebu: Check that kwbimage offset and blocksize are valid

2022-01-14 Thread Pali Rohár
There are certain restrictions for kwbimage offset and blocksize. Validate them. Signed-off-by: Pali Rohár Signed-off-by: Marek Behún Reviewed-by: Stefan Roese --- arch/arm/mach-mvebu/spl.c | 12 1 file changed, 12 insertions(+) diff --git a/arch/arm/mach-mvebu/spl.c

[PATCH u-boot-marvell v4 3/9] arm: mvebu: Check that kwbimage blockid matches boot mode

2022-01-14 Thread Pali Rohár
Each boot mode has its own kwbimage specified by blockid. So check that kwbimage is valid by blockid. Signed-off-by: Pali Rohár Signed-off-by: Marek Behún Reviewed-by: Stefan Roese --- arch/arm/mach-mvebu/spl.c | 35 ++- 1 file changed, 26 insertions(+), 9

[PATCH u-boot-marvell v4 0/9] More verifications for kwbimage in SPL

2022-01-14 Thread Pali Rohár
Hello Stefan, this is v4 of series that adds more checks for kwbimage validity and consistency to SPL, mainly checking image data checksum. Patches now passed Github CI testing: https://github.com/u-boot/u-boot/pull/106 Changes since v3: - fix compile erros detected by CI (patch 2) - print

Re: [ANN] U-Boot v2022.01 released

2022-01-14 Thread Wolfgang Denk
Dear Tom, In message <20220110184720.GB2773246@bill-the-cat> you wrote: > > It is release day and here is v2022.01. With this release > we are now at the fourth of our 2 years past a number of DM migration > deadlines. We're now at the point where conversion to CONFIG_DM itself > must have

[PATCH 4/4] arm64: zynqmp: Add command for disabling loading other overlays

2022-01-14 Thread Michal Simek
Add command "zynqmp pmufw node close" to disable permission to load additional pmufw config overlays. This command will make sure that any other sw will ask for changing permission. Signed-off-by: Michal Simek --- board/xilinx/zynqmp/cmds.c | 16

[PATCH 3/4] firmware: zynqmp: Do not report error if node is already configured

2022-01-14 Thread Michal Simek
Power domain driver sends PM fragment to PMUFW. It is sent for every node which is listed in DT. But some nodes could be already enabled but driver is not capable to find it out. That's why it blinly sents request for every listed IP. When PMUFW response by XST_PM_ALREADY_CONFIGURED error code

[PATCH 2/4] firmware: zynqmp: Move loading message to debug

2022-01-14 Thread Michal Simek
Power domain driver is using this function for every IP which is PD listed. This can end up with a lot of messages which end up in boot log. That's why show it only in EL3 as was used in past. Signed-off-by: Michal Simek --- drivers/firmware/firmware-zynqmp.c | 3 ++- 1 file changed, 2

[PATCH 1/4] xilinx: firmware: Introduce zynqmp_pmufw_node() for loading PMU fragments

2022-01-14 Thread Michal Simek
Introduce zynqmp_pmufw_node() for loading PMU configuration fragment for enabling IPs. Firmware driver has small overlay where NODE id is added and config fragment is sent to PMUFW. There is a need to build PMUFW with fragment support. Signed-off-by: Michal Simek ---

[PATCH 0/4] zynqmp: Add support for sending pmufw config object overlay

2022-01-14 Thread Michal Simek
Hi, this small series is extending firmware interface for sending pmufw configuration overlay with asking for access to certain device. This infrastructure can be used via command or via power domain driver. Thanks, Michal Michal Simek (4): xilinx: firmware: Introduce zynqmp_pmufw_node() for

[PATCH] zynqmp: Use the same style for macro definitions

2022-01-14 Thread Michal Simek
Use the same coding style for all macros. #defineNAMEVALUE Signed-off-by: Michal Simek --- include/zynqmp_firmware.h | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/include/zynqmp_firmware.h b/include/zynqmp_firmware.h index

[PATCH] Revert "net: gem: Disable broadcast setting"

2022-01-14 Thread Michal Simek
This reverts commit eafdcda4a854932c0319656de7bf3f017f17ae67. The main reason is that QEMU is using BOOTP protocol which is sending DHCP Offer to a broadcast address that's why it can't be disabled. DHCP protocol has no issue because it returns directly to client MAC address. Both of these

[PATCH 2/2] zynqmp: Run board_get_usable_ram_top() only on main U-Boot

2022-01-14 Thread Michal Simek
From: Ashok Reddy Soma With commit ce39ee28ec31 ("zynqmp: Do not place u-boot to reserved memory location"), the function board_get_usable_ram_top() is allocating MMU_SECTION_SIZE of about 2MB using lmb_alloc(). But we dont have this much memory in case of mini U-Boot. Keep these functions

[PATCH 1/2] lmb: Fix lmb property's defination under struct lmb

2022-01-14 Thread Michal Simek
From: Ashok Reddy Soma Under struct lmb {} the lmb property's should be defined only if CONFIG_LMB_MEMORY_REGIONS is defined. Signed-off-by: Ashok Reddy Soma Signed-off-by: Michal Simek --- include/lmb.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/lmb.h

[PATCH] arm64: zynqmp: Change firmware dependency

2022-01-14 Thread Michal Simek
In case of mini U-Boot configurations there is no need to enable firmware driver which just consume space for nothing. That's why add an option to disable it. Signed-off-by: Michal Simek --- arch/arm/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH] net: gem: Workaround gmii2rgmii bridge DT node issue

2022-01-14 Thread Michal Simek
From: Ashok Reddy Soma For configurations with gmii2rgmii and external phy the DT nodes link should be gem->gmii2rgmii->phy. But due to limitation in Linux driver the DT is mentioned as gem->phy and gmii2rgmii->phy as shown in below DT. ethernet@ff0c { compatible =

[PATCH] cmd: Add Kconfig option for multiprocessor cmds

2022-01-14 Thread Michal Simek
From: Ashok Reddy Soma Add Kconfig option(CONFIG_CMD_MP) to enable or disable multiprocessor commands. Compile cmd/mp.c based on CONFIG_CMD_MP. Signed-off-by: Ashok Reddy Soma Signed-off-by: Michal Simek --- cmd/Kconfig | 8 cmd/Makefile | 2 +- 2 files changed, 9 insertions(+), 1

[PATCH] arm64: xilinx: dts: Add dma properties to fix dtbs_check warnings

2022-01-14 Thread Michal Simek
From: Shravya Kumbham Update dma name and add #dma-cells properties to fix dtbs_check warnings. Signed-off-by: Shravya Kumbham Signed-off-by: Michal Simek --- arch/arm/dts/zynqmp.dtsi | 48 ++-- 1 file changed, 32 insertions(+), 16 deletions(-) diff

[PATCH] arm64: zynqmp: Update USB node handle from dwc3 to usb

2022-01-14 Thread Michal Simek
From: Manish Narani The DWC3 bindings require all USB node handles to be '^usb@[0-9a-f]+$'. Update the same in ZynqMP device tree. Signed-off-by: Manish Narani Signed-off-by: Michal Simek --- arch/arm/dts/zynqmp.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH] arm64: zynqmp: Change compatible strings for cadence uart

2022-01-14 Thread Michal Simek
Based on Linux kernel DT binding there should be different compatible strings used that's why align zynqmp.dtsi with it. Signed-off-by: Michal Simek --- arch/arm/dts/zynqmp.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/dts/zynqmp.dtsi

[PATCH 7/8] nvme: apple: Add driver for Apple NVMe storage controller

2022-01-14 Thread Mark Kettenis
Add a driver for the NVMe storage controller integrated on Apple SoCs. This NVMe controller isn't PCI based and deviates from the NVMe standard in its implementation of the command submission queue and the integration of an NVMMU that needs to be managed. This commit tweaks the core NVMe code to

[PATCH 4/8] nvme: Introduce driver ops

2022-01-14 Thread Mark Kettenis
The NVMe storage controller integrated on Apple SoCs deviates from the NVMe standard in two aspects. It uses a "linear" submission queue and it integrates an NVMMU that needs to be programmed for each NVMe command. Introduce driver ops such that we can set up the linear submission queue and

[PATCH 3/8] arm: apple: Add RTKit support

2022-01-14 Thread Mark Kettenis
Most Apple IOPs run a firmware that is based on what Apple calls RTKit. RTKit implements a common mailbox protocol. This code provides an implementation of the AP side of this protocol, providing a function to initialize RTKit-based firmwares as well as a function to do a clean shutdown of this

[PATCH 2/8] mailbox: apple: Add driver for Apple IOP mailbox

2022-01-14 Thread Mark Kettenis
This mailbox driver provides a communication channel with the Apple IOP controllers found on Apple SoCs. These IOP controllers are used to implement various functions such as the System Manegement Controller (SMC) and NVMe storage. It allows sending and receiving a 96-bit message over a single

[PATCH 1/8] nvme: Split out PCI support

2022-01-14 Thread Mark Kettenis
Apple SoCs have an integrated NVMe controller that isn't connected over a PCIe bus. In preparation for adding support for this NVMe controller, split out the PCI support into its own file. This file is selected through a new CONFIG_NVME_PCI Kconfig option, so do a wholesale replacement of

[PATCH 6/8] power: domain: apple: Add reset support

2022-01-14 Thread Mark Kettenis
The power management controller found on Apple SoCs als provides a way to reset all devices within a power domain. This is needed to cleanly shutdown the NVMe controller before we hand over control to the OS. Signed-off-by: Mark Kettenis --- arch/arm/Kconfig | 1 +

[PATCH 8/8] configs: apple: Add NVMe boot target

2022-01-14 Thread Mark Kettenis
Add a boot target for NVMe such that we can boot from NVMe. Signed-off-by: Mark Kettenis --- include/configs/apple.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/include/configs/apple.h b/include/configs/apple.h index 3e5fb495f1..47faad8150 100644 --- a/include/configs/apple.h +++

[PATCH 5/8] nvme: Add shutdown function

2022-01-14 Thread Mark Kettenis
Add a function to disable the NVMe controller. This will be used to let the driver for the NVMe storage integrated on Apple SoCs shutdown the NVMe controller such we can shutdown the NVMe IOP controller in a clean way afterwards before handing control to the OS. Signed-off-by: Mark Kettenis ---

[PATCH 0/8] Apple M1 NVMe storage support

2022-01-14 Thread Mark Kettenis
This adds support for the (rather quirky) NVMe storage controller integrated on Apple SoCs. This makes it possible to boot from the NVMe storage that is present on all the M1 machines that Apple has released so far. The series has been designed to have as little impact on the existing NVMe

Re: [PATCH] nvme: Do not allocate 8kB buffer on stack

2022-01-14 Thread Pali Rohár
Hello! Could you please review this patch? On Thursday 09 December 2021 11:06:39 Pali Rohár wrote: > Calling 'nvme scan' followed by 'nvme detail' crashes U-Boot on Turris > Omnia with the following error: > > undefined instruction > pc : [<0a00>] lr : [<7ff80bfc>] > reloc pc

Re: [PATCH 1/1] disk: gpt: print all partitions

2022-01-14 Thread Alexandre Ghiti
Hi Heinrich, On Tue, Jan 11, 2022 at 3:48 PM Heinrich Schuchardt wrote: > > For GPT partition tables the 'part list' command stops at the first invalid > partition number. But Ubuntu has images with partitions number > > 1, 12, 13, 14, 15 > > In this case only partition 1 was listed by 'part

binman generator nodes break over read-back

2022-01-14 Thread Jan Kiszka
Hi Simon, something is fishy with generator nodes: We are playing with replacing fdts and configs in [1] with generator nodes, and that works so far - until we want to read-back the image: $ binman ls -i flash.bin binman: Node '/fit@0x28': Generator node requires 'of-list' entry

Re: [PATCH 0/3] mkimage: allow to specify signing algorithm

2022-01-14 Thread Jan Kiszka
On 14.01.22 00:28, Dhananjay Phadke wrote: On 1/13/2022 4:38 AM, Jan Kiszka wrote: On 25.11.21 20:03, Jan Kiszka wrote: Another step to decouple the FIT image specification from the actual signing: With these changes, the signature nodes can leave out an algo property, mkimage will initialize

[resent][PATCH 2/3] mkimage: Drop unused OPT_STRING constant

2022-01-14 Thread Jan Kiszka
From: Jan Kiszka The actual opt string is inlined - and different. Seems this was a left-over from older versions of 603e26f76346. Signed-off-by: Jan Kiszka --- tools/mkimage.c | 1 - 1 file changed, 1 deletion(-) diff --git a/tools/mkimage.c b/tools/mkimage.c index fbe883ce36..a4844d0f18

[resent][PATCH 3/3] mkimage: Allow to specify the signature algorithm on the command line

2022-01-14 Thread Jan Kiszka
From: Jan Kiszka This permits to prepare FIT image description that do not hard-code the final choice of the signature algorithm, possibly requiring the user to patch the sources. When -o is specified, this information is used in favor of the 'algo' property in the signature node. Furthermore,

[resent][PATCH 0/3] mkimage: allow to specify signing algorithm

2022-01-14 Thread Jan Kiszka
[resent as requested by Simon] Another step to decouple the FIT image specification from the actual signing: With these changes, the signature nodes can leave out an algo property, mkimage will initialize that as well while signing. This way, in-tree FIT source files can be prepared for gaining

[resent][PATCH 1/3] image-fit: Make string of algo parameter constant

2022-01-14 Thread Jan Kiszka
From: Jan Kiszka Modifications would be invalid. Signed-off-by: Jan Kiszka --- boot/image-fit-sig.c | 2 +- boot/image-fit.c | 8 include/image.h | 2 +- tools/image-host.c | 4 ++-- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/boot/image-fit-sig.c

Re: [PATCH v2] introduce CONFIG_DEVICE_TREE_INCLUDES

2022-01-14 Thread Rasmus Villemoes
Ping On 21/11/2021 14.52, Rasmus Villemoes wrote: > The build system already automatically looks for and includes an > in-tree *-u-boot.dtsi when building the control .dtb. However, there > are some things that are awkward to maintain in such an in-tree file, > most notably the metadata