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

2021-08-25 Thread Ilias Apalodimas
[...] > > > > > > > > > > > > > > > > > > > Look I'm sorry if this all seems a bit much. My initial request was > > > > > rebuffed, other emails have been ignored and a large number of > > > > > objections have been raised. It's just too hard. As far as I can > > > > > remember, I've not come

Re: [PATCH 3/3] Convert CONFIG_SYS_LOAD_ADDR to Kconfig

2021-08-25 Thread Rick Chen
> From: U-Boot On Behalf Of Tom Rini > Sent: Monday, August 23, 2021 10:26 PM > To: u-boot@lists.denx.de > Subject: [PATCH 3/3] Convert CONFIG_SYS_LOAD_ADDR to Kconfig > > Now that we have consistent usage, migrate this symbol to Kconfig. > > Signed-off-by: Tom Rini > --- > Kconfig

Re: Usage of device-tree for blobs

2021-08-25 Thread Simon Glass
Hi Heinrich, On Wed, 25 Aug 2021 at 02:05, Heinrich Schuchardt wrote: > > Hello Simon, > > some boards like qemu-riscv64_defconfig do not use any device-tree at > build time. A device-tree is only supplied at runtime by the prior boot > stage (CONFIG_OF_PRIOR_STAGE=y). > > In

Re: [PATCH] arm: dts: imx8mp: Generate single bootable binary

2021-08-25 Thread Heiko Schocher
Hello Teresa, On 25.08.21 17:25, Teresa Remmet wrote: > binman conversion made flashing flash.bin > and u-boot.itb necessary. Update binman config > to create a single flash.bin image again. > > This updates imx8mp_evk and phyCORE-i.MX8MP as they share the > same binman config. > > Updated also

Re: [PATCH v3 04/10] verdin-imx8mm: switch to use binman to pack images

2021-08-25 Thread Heiko Schocher
Hello Marcel, On 25.08.21 17:14, Marcel Ziswiler wrote: > From: Marcel Ziswiler > > Use binman to pack images. > > Signed-off-by: Marcel Ziswiler > Reviewed-by: Heiko Thiery > Reviewed-by: Fabio Estevam > > --- > > Changes in v3: > - Case fold hex string. > - Revert binman part of

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

2021-08-25 Thread Andre Przywara
On Sat, 21 Aug 2021 19:54:02 -0500 Samuel Holland wrote: Hi Samuel, thanks for the update. > For ARMv8-A, NEON is standard, so the compiler can use it even when no > special target flags are provided. That is not entirely true, NEON is architecturally an optional feature, it's just so

Re: [PATCH 1/4] tools: docker: Bump up QEMU version to 6.1.0

2021-08-25 Thread Tom Rini
On Wed, Aug 25, 2021 at 01:00:46PM +0800, Bin Meng wrote: > On Wed, Aug 25, 2021 at 10:01 AM Tom Rini wrote: > > > > On Wed, Aug 25, 2021 at 08:05:17AM +0800, Bin Meng wrote: > > > > > At present U-Boot CI testing is still using QEMU 4.2.0 which is > > > pretty old. Let's bump up to QEMU 6.1.0. >

Re: incompatible device trees between u-boot and linux

2021-08-25 Thread Vladimir Oltean
On Wed, Aug 25, 2021 at 04:09:50PM -0400, Tom Rini wrote: > I'm saying that because it's what's been said for what feels like 10+ > years. I don't want to think how many countless hours have been spent > on that point at conferences over the years. It's not even a Linux > thing. I would swear

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

2021-08-25 Thread Tom Rini
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 and keep with the latest feature. > > Changes since v1: > - fix the inconsistency between From

Re: [PATCH v2 2/3] fs/erofs: add lz4 1.8.3 decompressor

2021-08-25 Thread Tom Rini
On Mon, Aug 23, 2021 at 08:36:45PM +0800, Huang Jianan wrote: > From: Huang Jianan > > In order to use the newest LZ4_decompress_safe_partial() which can > now decode exactly the nb of bytes requested. > > Signed-off-by: Huang Jianan > --- > fs/erofs/Makefile | 3 +- > fs/erofs/lz4.c|

Re: [PATCH v2 1/3] fs/erofs: add erofs filesystem support

2021-08-25 Thread Tom Rini
On Mon, Aug 23, 2021 at 08:36:44PM +0800, Huang Jianan wrote: > From: Huang Jianan > > This patch mainly deals with uncompressed files. > > Signed-off-by: Huang Jianan > --- > fs/Kconfig | 1 + > fs/Makefile | 1 + > fs/erofs/Kconfig| 12 ++ > fs/erofs/Makefile |

Re: How should we deal with actual hush odd behavior?

2021-08-25 Thread Tom Rini
On Fri, Aug 20, 2021 at 12:22:22PM -0600, Simon Glass wrote: > Hi Francis, > > On Fri, 20 Aug 2021 at 10:12, Francis Laniel > wrote: > > > > Hi. > > > > > > I hope you are fine and the same for your family and friends. > > > > In July, a proposal to add a new shell for U-Boot was posted on the

Re: [PATCH 00/28] Initial implementation of bootmethod/bootflow

2021-08-25 Thread Tom Rini
On Wed, Aug 25, 2021 at 11:54:58PM +0200, Mark Kettenis wrote: > > Date: Wed, 25 Aug 2021 10:56:35 -0400 > > From: Tom Rini > > > > On Wed, Aug 25, 2021 at 11:42:51PM +0900, AKASHI Takahiro wrote: > > > Simon, > > > > > > On Wed, Aug 25, 2021 at 07:11:44AM -0600, Simon Glass wrote: > > > > Hi,

Re: [PATCH 00/28] Initial implementation of bootmethod/bootflow

2021-08-25 Thread Tom Rini
On Wed, Aug 25, 2021 at 11:34:45PM +0200, Mark Kettenis wrote: > > From: Peter Robinson > > Date: Wed, 25 Aug 2021 14:29:45 +0100 > > > > > > Exactly. Keep in mind that RISC-V is looking into EBBR as well, > > > > so this is far from an 'Arm thing'. Moreover, the efi stub side > > > > of the

Re: [PATCH 00/28] Initial implementation of bootmethod/bootflow

2021-08-25 Thread Mark Kettenis
> Date: Wed, 25 Aug 2021 10:56:35 -0400 > From: Tom Rini > > On Wed, Aug 25, 2021 at 11:42:51PM +0900, AKASHI Takahiro wrote: > > Simon, > > > > On Wed, Aug 25, 2021 at 07:11:44AM -0600, Simon Glass wrote: > > > Hi, > > > > > > On Wed, 25 Aug 2021 at 04:45, Emmanuel Vadot > > > wrote: > > >

Re: [PATCH] mmc: fsl_esdhc_imx: Fix clock disable issue

2021-08-25 Thread Jaehoon Chung
On 8/17/21 6:09 PM, Ye Li wrote: > The SD clock disable is wrapped by MMC_SUPPORTS_TUNING. So it > only works when UHS is enabled. However, in SD initialization the > power cycle does not depends on UHS. But the power cycle needs > disable the SD clock before power down. > So this causes a problem

Re: [PATCH v5 5/7] mmc: zynq_sdhci: Move setting tapdelay code to driver

2021-08-25 Thread Jaehoon Chung
On 8/3/21 2:20 PM, Ashok Reddy Soma wrote: > Move tapdelay function calls to zynq_sdhci.c and make them static > inline. zynqmp_tap_delay.h has function prototypes for the functions > defined in tap_delays.c, which will not be needed anymore. > > Remove tap_delays.c and zynqmp_tap_delay.h files.

Re: [PATCH v5 4/7] mmc: zynq_sdhci: Add xilinx_pm_request() method to set tapdelays

2021-08-25 Thread Jaehoon Chung
On 8/3/21 2:20 PM, Ashok Reddy Soma wrote: > Currently xilinx sdhci driver is using zynqmp_mmio_write() to set > tapdelay values and DLL resets. Continue to use this for SPL and mini > U-Boot where U-Boot will be executed at EL3 level. > > Use firmware call xilinx_pm_request() using appropriate

Re: [PATCH 00/28] Initial implementation of bootmethod/bootflow

2021-08-25 Thread Mark Kettenis
> From: Peter Robinson > Date: Wed, 25 Aug 2021 14:29:45 +0100 > > > > Exactly. Keep in mind that RISC-V is looking into EBBR as well, > > > so this is far from an 'Arm thing'. Moreover, the efi stub side > > > of the kernel for risc-v, will only load an initrd using the > > > EFI_LOAD_FILE2

Re: [PATCH 1/1] riscv: enable booting HiFive Unmatched from SATA

2021-08-25 Thread Mark Kettenis
> From: Heinrich Schuchardt > Date: Wed, 25 Aug 2021 18:21:57 +0200 > > On the HiFive Unmatched a PCI to SATA adapter may be used to install a SATA > drive. Enable booting from it. > > Signed-off-by: Heinrich Schuchardt Reviewed-by: Mark Kettenis > --- > configs/sifive_unmatched_defconfig

Re: incompatible device trees between u-boot and linux

2021-08-25 Thread Mark Kettenis
> Date: Wed, 25 Aug 2021 18:12:20 +0300 > From: Vladimir Oltean > > On Wed, Aug 25, 2021 at 10:26:10AM -0400, Tom Rini wrote: > > On Wed, Aug 25, 2021 at 05:18:16PM +0300, Vladimir Oltean wrote: > > > On Wed, Aug 25, 2021 at 10:00:45AM -0400, Tom Rini wrote: > > > > On Wed, Aug 25, 2021 at

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

2021-08-25 Thread 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: Priyanka Jain

Re: incompatible device trees between u-boot and linux

2021-08-25 Thread Tom Rini
On Wed, Aug 25, 2021 at 06:43:23PM +0300, Vladimir Oltean wrote: > On Wed, Aug 25, 2021 at 11:24:08AM -0400, Tom Rini wrote: > > On Wed, Aug 25, 2021 at 06:12:20PM +0300, Vladimir Oltean wrote: > > > On Wed, Aug 25, 2021 at 10:26:10AM -0400, Tom Rini wrote: > > > > On Wed, Aug 25, 2021 at

Re: [PATCH] ARM: mach-at91: fix multiple cpu_reset definition when enabling SYSRESET

2021-08-25 Thread Clément Léger
Le Tue, 24 Aug 2021 09:51:24 +, a écrit : > On 8/24/21 12:37 PM, Clément Léger wrote: > > Le Mon, 23 Aug 2021 07:42:58 +, > > a écrit : > > > >> On 8/4/21 5:55 PM, Clément Léger wrote: > >>> When SYSRESET is enabled, cpu_reset function is also defined in > >>> sysreset-uclass.c

[PATCH u-boot-marvell 18/29] tools: kwbimage: Simplify iteration over version 1 optional headers

2021-08-25 Thread Marek Behún
Create macro for_each_opt_hdr_v1 and functions opt_hdr_v1_size(), opt_hdr_v1_valid_size(), opt_hdr_v1_ext(), opt_hdr_v1_first() and opt_hdr_v1_next() to simplify iteration over version 1 optional headers. This prevents ugly code repetition and makes it nicer to read. Signed-off-by:

Re: [PATCH u-boot-marvell 25/29] tools: kwboot: Support higher baudrates when booting via UART

2021-08-25 Thread Marek Behún
On Wed, 25 Aug 2021 15:46:30 +0200 Marek Behún wrote: > @@ -765,6 +1187,37 @@ kwboot_img_is_secure(void *img) > return 0; > } > > +static void * > +kwboot_img_grow_data_left(void *img, size_t *size, size_t grow) > +{ > + uint32_t hdrsz, datasz, srcaddr; > + struct main_hdr_v1

Re: [PATCH 1/1] riscv: enable booting HiFive Unmatched from SATA

2021-08-25 Thread Bin Meng
On Thu, Aug 26, 2021 at 12:21 AM Heinrich Schuchardt wrote: > > On the HiFive Unmatched a PCI to SATA adapter may be used to install a SATA > drive. Enable booting from it. > > Signed-off-by: Heinrich Schuchardt > --- > configs/sifive_unmatched_defconfig | 5 + >

[PATCH 1/1] rockchip: px30: add support for HW RNG for Odroid Go Advance

2021-08-25 Thread Chris Morgan
From: Chris Morgan The Odroid Go Advance has a hardware random number generator present. The device does not have an upstream Linux driver, but does have a U-Boot driver. Add the appropriate node so that the hardware RNG can be used in U-Boot. Signed-off-by: Chris Morgan ---

[PATCH 1/2] cmd: kaslrseed: add command to generate value from hwrng

2021-08-25 Thread Chris Morgan
From: Chris Morgan Allow the kaslr-seed value in the chosen node to be set from a hardware rng source. Tested on a Rockchip PX30 (Odroid Go Advance), you must have loaded the devicetree first and prepared it for editing. On my device the workflow goes as follows: setenv dtb_loadaddr

[PATCH 1/1] riscv: enable booting HiFive Unmatched from SATA

2021-08-25 Thread Heinrich Schuchardt
On the HiFive Unmatched a PCI to SATA adapter may be used to install a SATA drive. Enable booting from it. Signed-off-by: Heinrich Schuchardt --- configs/sifive_unmatched_defconfig | 5 + include/configs/sifive-unmatched.h | 3 +++ 2 files changed, 8 insertions(+) diff --git

Re: [PATCH v2] ARM: dts: Fix node status to "okay" on TI boards

2021-08-25 Thread Nishanth Menon
On 14:07-20210824, Roger Quadros wrote: > As per Device Tree Specification [1], the status parameter of nodes can > be "okay", "disabled", etc. "ok" is not a valid parameter. > > U-boot Driver Model does not recognize status="ok" either and treats > the node as disabled. > > [1] >

Re: [PATCH] arm: dts: imx8mp: Generate single bootable binary

2021-08-25 Thread Fabio Estevam
Hi Teresa, On Wed, Aug 25, 2021 at 12:25 PM Teresa Remmet wrote: > > binman conversion made flashing flash.bin > and u-boot.itb necessary. Update binman config > to create a single flash.bin image again. > > This updates imx8mp_evk and phyCORE-i.MX8MP as they share the > same binman config. > >

[PATCH v2] board: ea: mx7ulp_com: allocate specific region of memory to OP-TEE

2021-08-25 Thread Oleksandr Suvorov
From: Ricardo Salveti On the iMX7ULP uCOM board, OP-TEE uses the memory region defined by the maximum DRAM address minus CONFIG_OPTEE_TZDRAM_SIZE, so subtract CONFIG_OPTEE_TZDRAM_SIZE from the available DRAM size to avoid conflicts. Note the OPTEE boot process itself subtracts the DRAM region

Re: incompatible device trees between u-boot and linux

2021-08-25 Thread Vladimir Oltean
On Wed, Aug 25, 2021 at 11:24:08AM -0400, Tom Rini wrote: > On Wed, Aug 25, 2021 at 06:12:20PM +0300, Vladimir Oltean wrote: > > On Wed, Aug 25, 2021 at 10:26:10AM -0400, Tom Rini wrote: > > > On Wed, Aug 25, 2021 at 05:18:16PM +0300, Vladimir Oltean wrote: > > > > On Wed, Aug 25, 2021 at

Re: [PATCH] board: ea: mx7ulp_com: allocate specific region of memory to OP-TEE

2021-08-25 Thread Tom Rini
On Wed, Aug 25, 2021 at 06:35:05PM +0300, Oleksandr Suvorov wrote: > Hi Tom, > > > On Wed, Aug 25, 2021 at 4:46 PM Tom Rini wrote: > > > > On Wed, Aug 25, 2021 at 04:22:55PM +0300, Oleksandr Suvorov wrote: > > > > > From: Ricardo Salveti > > > > > > On the iMX7ULP uCOM board, OP-TEE uses the

Re: [PATCH] board: ea: mx7ulp_com: allocate specific region of memory to OP-TEE

2021-08-25 Thread Oleksandr Suvorov
Hi Tom, On Wed, Aug 25, 2021 at 4:46 PM Tom Rini wrote: > > On Wed, Aug 25, 2021 at 04:22:55PM +0300, Oleksandr Suvorov wrote: > > > From: Ricardo Salveti > > > > On the iMX7ULP uCOM board, OP-TEE uses the memory region defined by the > > maximum DRAM address minus CONFIG_OPTEE_TZDRAM_SIZE, so

[PATCH] arm: dts: imx8mp: Generate single bootable binary

2021-08-25 Thread Teresa Remmet
binman conversion made flashing flash.bin and u-boot.itb necessary. Update binman config to create a single flash.bin image again. This updates imx8mp_evk and phyCORE-i.MX8MP as they share the same binman config. Updated also imx8mp_evk documentation. Tested on phyCORE-i.MX8MP. Signed-off-by:

Re: incompatible device trees between u-boot and linux

2021-08-25 Thread Tom Rini
On Wed, Aug 25, 2021 at 06:12:20PM +0300, Vladimir Oltean wrote: > On Wed, Aug 25, 2021 at 10:26:10AM -0400, Tom Rini wrote: > > On Wed, Aug 25, 2021 at 05:18:16PM +0300, Vladimir Oltean wrote: > > > On Wed, Aug 25, 2021 at 10:00:45AM -0400, Tom Rini wrote: > > > > On Wed, Aug 25, 2021 at

[PATCH v3 08/10] include/configs: apalis-imx8/verdin-imx8mm: rename kernel image variable

2021-08-25 Thread Marcel Ziswiler
From: Oleksandr Suvorov Variable "kernel_image" is used in boot.scr script only, that sets its own default value to the constant string @@KERNEL_IMAGETYPE@@ in case "kernel_image" is not set. The default name of the kernel image shipped with BSP 5.x is "Image.gz". Setting kernel_image="Image" as

[PATCH v3 10/10] verdin-imx8mm: fix watchdog pinctrl issue

2021-08-25 Thread Marcel Ziswiler
From: Marcel Ziswiler Finally, found the root cause of the issue already once mentioned back here [2] which caused the following error message during boot: imx_wdt watchdog@3028: pinctrl_select_state_full: uclass_get_device_by_phandle_id: err=-19 Turns out while the watchdog node itself

[PATCH v3 09/10] verdin-imx8mm: use preboot for fdtfile evaluation

2021-08-25 Thread Marcel Ziswiler
From: Igor Opaniuk Enable and set preboot var with fdtfile evaluation. It will be checked and run immediately before starting the CONFIG_BOOTDELAY countdown and/or running the auto-boot command resp. entering interactive mode. This provides possibility to use different boot cmds in interactive

[PATCH v3 07/10] verdin-imx8mm: drop support for v1.0 hardware

2021-08-25 Thread Marcel Ziswiler
From: Max Krummenacher We drop support for Verdin iMX8M Mini V1.0B. Related-to: ELB-3551 Signed-off-by: Max Krummenacher Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam --- Changes in v3: - Add Fabio's reviewed-by tag. Changes in v2: - Drop first patch ("imx: mkimage_fit_atf:

[PATCH v3 04/10] verdin-imx8mm: switch to use binman to pack images

2021-08-25 Thread Marcel Ziswiler
From: Marcel Ziswiler Use binman to pack images. Signed-off-by: Marcel Ziswiler Reviewed-by: Heiko Thiery Reviewed-by: Fabio Estevam --- Changes in v3: - Case fold hex string. - Revert binman part of imx8mm-verdin-u-boot.dtsi to a plain copy from imx8mm-evk and postpone further

[PATCH v3 06/10] verdin-imx8mm: clean-up include order

2021-08-25 Thread Marcel Ziswiler
From: Marcel Ziswiler Alphabetically order includes. While at it also update copyright year resp. period. Signed-off-by: Marcel Ziswiler --- Changes in v3: - New patch cleaning up include order. board/toradex/verdin-imx8mm/verdin-imx8mm.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH v3 00/10] board: toradex: verdin-imx8mm: target refresh

2021-08-25 Thread Marcel Ziswiler
From: Marcel Ziswiler An assortment of fixes and improvements like an Ethernet PHY configuration fix, DEK blob encapsulation preparation, migration to using binman to pack images, SLEEP_MOCI# enablement, dropping of V1.0 hardware support [1], renaming kernel image variable, using preboot for

[PATCH v3 02/10] verdin-imx8mm: fix ethernet

2021-08-25 Thread Marcel Ziswiler
From: Marcel Ziswiler Turns out Microship (formerly Micrel) meanwhile integrated proper support for the DLL setup on their KSZ9131. Unfortunately, this conflicts with our previous board code doing that. Fix this by getting rid of our board code and just relying on the generic implementation

[PATCH v3 05/10] verdin-imx8mm: enable sleep_moci output

2021-08-25 Thread Marcel Ziswiler
From: Max Krummenacher This powers some peripherals on the carrier board e.g. the USB hub. Related-to: ELB-3206 Signed-off-by: Max Krummenacher Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam --- (no changes since v1) arch/arm/dts/imx8mm-verdin.dts | 18 ++

[PATCH v3 03/10] ARM: dts: imx8mm-verdin: prepare for dek blob encapsulation

2021-08-25 Thread Marcel Ziswiler
From: Marcel Ziswiler Prepare for DEK blob encapsulation support through "dek_blob" command. On ARMv8, u-boot runs in non-secure, thus cannot encapsulate a DEK blob for encrypted boot. The DEK blob is encapsulated by OP-TEE through a trusted application call. U-boot sends and receives the DEK

[PATCH v3 01/10] imx8m: clean-up kconfig indentation

2021-08-25 Thread Marcel Ziswiler
From: Marcel Ziswiler Replace spurious spaces with proper tabs. Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam --- (no changes since v1) arch/arm/mach-imx/imx8m/Kconfig | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git

Re: incompatible device trees between u-boot and linux

2021-08-25 Thread Vladimir Oltean
On Wed, Aug 25, 2021 at 10:26:10AM -0400, Tom Rini wrote: > On Wed, Aug 25, 2021 at 05:18:16PM +0300, Vladimir Oltean wrote: > > On Wed, Aug 25, 2021 at 10:00:45AM -0400, Tom Rini wrote: > > > On Wed, Aug 25, 2021 at 03:58:10PM +0200, Michael Walle wrote: > > > > > > > Hi, > > > > > > > > I

Re: [PATCH 00/28] Initial implementation of bootmethod/bootflow

2021-08-25 Thread Tom Rini
On Wed, Aug 25, 2021 at 11:42:51PM +0900, AKASHI Takahiro wrote: > Simon, > > On Wed, Aug 25, 2021 at 07:11:44AM -0600, Simon Glass wrote: > > Hi, > > > > On Wed, 25 Aug 2021 at 04:45, Emmanuel Vadot wrote: > > > > > > On Tue, 24 Aug 2021 12:22:42 +0200 (CEST) > > > Mark Kettenis wrote: > > >

Re: [PATCH 00/28] Initial implementation of bootmethod/bootflow

2021-08-25 Thread AKASHI Takahiro
Simon, On Wed, Aug 25, 2021 at 07:11:44AM -0600, Simon Glass wrote: > Hi, > > On Wed, 25 Aug 2021 at 04:45, Emmanuel Vadot wrote: > > > > On Tue, 24 Aug 2021 12:22:42 +0200 (CEST) > > Mark Kettenis wrote: > > > > > > Date: Mon, 23 Aug 2021 16:01:46 -0400 > > > > From: Tom Rini > > > > > > > >

Re: incompatible device trees between u-boot and linux

2021-08-25 Thread Michael Walle
Am 2021-08-25 16:18, schrieb Vladimir Oltean: Curious that Michael mentions Ethernet and DSA on LS1028A. [..] I mean, they look pretty similar to me? The biggest difference is that the ENETC ECAM is under the /soc node in Linux, but under / in U-Boot, as well as some BAR memory areas that

[PATCH] net: remove unused CONFIG_DRIVER_AT91EMAC_*

2021-08-25 Thread Eugen Hristev
AT91EMAC driver is unused, thus removing. Signed-off-by: Eugen Hristev --- README | 11 - drivers/net/Makefile | 1 - drivers/net/at91_emac.c | 519 --- include/netdev.h | 1 - scripts/config_whitelist.txt | 2

[PATCH] ARM: at91: remove references to RM9200DK

2021-08-25 Thread Eugen Hristev
The AT91 RM9200DK board was removed long time ago. Remove existing references that were not cleaned up. Fixes: 1c85752258 ("ARM: remove broken "at91rm9200dk" board") Signed-off-by: Eugen Hristev --- arch/arm/cpu/arm920t/start.S | 14 -- arch/arm/include/asm/mach-types.h | 1 -

Re: incompatible device trees between u-boot and linux

2021-08-25 Thread Tom Rini
On Wed, Aug 25, 2021 at 05:18:16PM +0300, Vladimir Oltean wrote: > On Wed, Aug 25, 2021 at 10:00:45AM -0400, Tom Rini wrote: > > On Wed, Aug 25, 2021 at 03:58:10PM +0200, Michael Walle wrote: > > > > > Hi, > > > > > > I noticed that there is a fallback to the u-boot device tree for linux > > >

Re: incompatible device trees between u-boot and linux

2021-08-25 Thread Vladimir Oltean
On Wed, Aug 25, 2021 at 10:00:45AM -0400, Tom Rini wrote: > On Wed, Aug 25, 2021 at 03:58:10PM +0200, Michael Walle wrote: > > > Hi, > > > > I noticed that there is a fallback to the u-boot device tree for linux > > (esp. EFI boot) if no other device tree was found, see [1]. It seems this > > is

Re: [PATCH] Kconfig: Don't use RSA_FREESCALE_EXP on MX7ULP

2021-08-25 Thread Tom Rini
On Wed, Aug 25, 2021 at 04:56:51PM +0300, Oleksandr Suvorov wrote: > From: Ricardo Salveti > > The CAAM on IMX7ULP doesn't support public key hardware acceleration > (PKHA), as in other NXP parts. Disable RSA_FREESCALE_EXP for IMX7ULP > too. > > Fixed: f4e9ff7135 ("Kconfig: Don't use

Re: Pull request: SoCFPGA changes for efi-2021-10-rc3

2021-08-25 Thread Tom Rini
On Wed, Aug 25, 2021 at 11:15:08AM +, Chee, Tien Fong wrote: > Hi Tom > > Please pull the SoCFPGA changes for efi-2021-10-rc3. > > Regards > Tien Fong > > The following changes since commit 4865db07169126ca0205f1a6265adf01bd69b3df: > > Merge tag 'efi-2021-10-rc3' of >

Re: incompatible device trees between u-boot and linux

2021-08-25 Thread Tom Rini
On Wed, Aug 25, 2021 at 03:58:10PM +0200, Michael Walle wrote: > Hi, > > I noticed that there is a fallback to the u-boot device tree for linux > (esp. EFI boot) if no other device tree was found, see [1]. It seems this > is working fine for imx devices, for example, where you can just boot a >

incompatible device trees between u-boot and linux

2021-08-25 Thread Michael Walle
Hi, I noticed that there is a fallback to the u-boot device tree for linux (esp. EFI boot) if no other device tree was found, see [1]. It seems this is working fine for imx devices, for example, where you can just boot a stock installer iso via EFI. It will just work and it is quite a nice

[PATCH] Kconfig: Don't use RSA_FREESCALE_EXP on MX7ULP

2021-08-25 Thread Oleksandr Suvorov
From: Ricardo Salveti The CAAM on IMX7ULP doesn't support public key hardware acceleration (PKHA), as in other NXP parts. Disable RSA_FREESCALE_EXP for IMX7ULP too. Fixed: f4e9ff7135 ("Kconfig: Don't use RSA_FREESCALE_EXP on IMX") Signed-off-by: Ricardo Salveti Signed-off-by: Oleksandr Suvorov

[PATCH u-boot-marvell 16/29] tools: kwboot: Properly finish xmodem transfer

2021-08-25 Thread Marek Behún
From: Pali Rohár After kwboot sends EOT, BootROM sends back ACK. Add code for handling this and retry sending EOT on error. Signed-off-by: Pali Rohár [ refactored ] Signed-off-by: Marek Behún --- tools/kwboot.c | 65 +- 1 file changed, 48

[PATCH u-boot-marvell 07/29] tools: kwboot: Fix return type of kwboot_xm_makeblock() function

2021-08-25 Thread Marek Behún
From: Pali Rohár Function kwboot_xm_makeblock() always returns length of xmodem block. It is always non-negative and calculated from variable with size_t type. Set return type of this function to size_t and remove dead code which checks for negative value. Signed-off-by: Pali Rohár

[PATCH u-boot-marvell 11/29] tools: kwboot: Split sending image into header and data stages

2021-08-25 Thread Marek Behún
From: Pali Rohár This change is required to implement other features in kwboot. Split sending header and data parts of the image into two stages. Signed-off-by: Pali Rohár [ refactored ] Signed-off-by: Marek Behún --- tools/kwbimage.h | 8 +++-- tools/kwboot.c | 84

Re: [PATCH] board: ea: mx7ulp_com: allocate specific region of memory to OP-TEE

2021-08-25 Thread Fabio Estevam
Hi Oleksandr, On Wed, Aug 25, 2021 at 10:37 AM Oleksandr Suvorov wrote: > > From: Ricardo Salveti > > On the iMX7ULP uCOM board, OP-TEE uses the memory region defined by the > maximum DRAM address minus CONFIG_OPTEE_TZDRAM_SIZE, so subtract > CONFIG_OPTEE_TZDRAM_SIZE from the available DRAM

Re: [PATCH] board: ea: mx7ulp_com: allow setting CONFIG_BOOTCOMMAND in defconfig

2021-08-25 Thread Tom Rini
On Wed, Aug 25, 2021 at 04:37:49PM +0300, Oleksandr Suvorov wrote: > From: Ricardo Salveti > > Allow replacing the board defined CONFIG_BOOTCOMMAND via defconfig. > > Signed-off-by: Ricardo Salveti > Signed-off-by: Oleksandr Suvorov > --- > > include/configs/mx7ulp_com.h | 2 ++ > 1 file

[PATCH u-boot-marvell 25/29] tools: kwboot: Support higher baudrates when booting via UART

2021-08-25 Thread Marek Behún
From: Pali Rohár Add support for uploading the boot image (the data part only) at higher baudrate than the standard one. The kwboot utility already has -B option, but choosing other baudrate than the standard one (115200 Bd) can only work for debug mode, not for booting the device. The BootROM

Re: [PATCH] board: ea: mx7ulp_com: allocate specific region of memory to OP-TEE

2021-08-25 Thread Tom Rini
On Wed, Aug 25, 2021 at 04:22:55PM +0300, Oleksandr Suvorov wrote: > From: Ricardo Salveti > > On the iMX7ULP uCOM board, OP-TEE uses the memory region defined by the > maximum DRAM address minus CONFIG_OPTEE_TZDRAM_SIZE, so subtract > CONFIG_OPTEE_TZDRAM_SIZE from the available DRAM size to

[PATCH u-boot-marvell 28/29] doc/kwboot.1: Update man page

2021-08-25 Thread Marek Behún
Update man page for the kwboot utility. Signed-off-by: Marek Behún --- doc/kwboot.1 | 58 ++-- 1 file changed, 38 insertions(+), 20 deletions(-) diff --git a/doc/kwboot.1 b/doc/kwboot.1 index 1e9ca268f7..cf113b8b27 100644 --- a/doc/kwboot.1 +++

[PATCH u-boot-marvell 26/29] tools: kwboot: Allow any baudrate on Linux

2021-08-25 Thread Marek Behún
From: Pali Rohár The A38x platform supports more baudrates than just those defined by the Bn constants, and some of them are higher than the highest Bn baudrate (the highest is 4 MBd while A38x support 5.15 MBd). On Linux, add support for arbitrary baudrates. (Since there is no standard POSIX

[PATCH u-boot-marvell 27/29] tools: kwboot: Add Pali and Marek as authors

2021-08-25 Thread Marek Behún
From: Pali Rohár Add Pali and Marek as another authors of the kwboot utility. Signed-off-by: Pali Rohár Signed-off-by: Marek Behún --- tools/kwboot.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/kwboot.c b/tools/kwboot.c index f60cfd130e..b183e4b35b 100644 --- a/tools/kwboot.c

[PATCH u-boot-marvell 29/29] MAINTAINERS: Add entry for kwbimage / kwboot tools

2021-08-25 Thread Marek Behún
Add entry for these tools with Marek, Pali and Stefan as maintainers. Signed-off-by: Marek Behún --- MAINTAINERS | 10 ++ 1 file changed, 10 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 776ff703b9..d4b21dd660 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -772,6 +772,16 @@

[PATCH u-boot-marvell 22/29] tools: kwbimage: Refactor image_version()

2021-08-25 Thread Marek Behún
Rename this function to kwbimage_version() and don't cast argument if not needed. Signed-off-by: Marek Behún --- tools/kwbimage.c | 8 tools/kwbimage.h | 4 ++-- tools/kwboot.c | 4 ++-- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/tools/kwbimage.c

[PATCH u-boot-marvell 24/29] tools: kwboot: Round up header size to 128 B when patching

2021-08-25 Thread Marek Behún
From: Pali Rohár The beginning of image data must be sent in a separate xmodem block; the block must not contain end of header with the beginning of data. Therefore we need to ensure that the image header size is a multiple of xmodem block size (which is 128 B). Read the file into a malloc()ed

[PATCH u-boot-marvell 21/29] tools: kwboot: Patch destination address to DDR area for SPI image

2021-08-25 Thread Marek Behún
From: Pali Rohár SPI/NOR kwbimage may have destination address set to 0x, which means that the image is not downloaded to DDR but rather it is executed directly from SPI/NOR. In this case execution address is set to SPI/NOR area. When patching image to UART type, change destination and

[PATCH u-boot-marvell 23/29] tools: kwbimage: Refactor kwbimage header size determination

2021-08-25 Thread Marek Behún
Add functions kwbheader_size() and kwbheader_size_for_csum(). Refactor code determining header size to use these functions. Refactor header checksum determining function. Remove stuff that is not needed anymore. This simplifies the code a little and fixes one instance of validating header size

[PATCH u-boot-marvell 20/29] tools: kwboot: Patch source address in image header

2021-08-25 Thread Marek Behún
From: Pali Rohár Some image types have source address in non-bytes unit; for example for SATA images, it is in 512 B units. We need to multiply by unit size when patching image type to UART. Signed-off-by: Pali Rohár [ refactored ] Signed-off-by: Marek Behún --- tools/kwboot.c | 26

[PATCH u-boot-marvell 17/29] tools: kwboot: Always call kwboot_img_patch_hdr()

2021-08-25 Thread Marek Behún
From: Pali Rohár The kwboot_img_patch_hdr() function already decides if header patching is needed. Always call this function and deprecate the unneeded command line option `-p`. Signed-off-by: Pali Rohár Reviewed-by: Marek Behún --- tools/kwboot.c | 23 ++- 1 file

[PATCH u-boot-marvell 12/29] tools: kwboot: Use a function to check whether received byte is a Xmodem reply

2021-08-25 Thread Marek Behún
This is a non-functional change that should make the code more readable. Signed-off-by: Marek Behún --- tools/kwboot.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/tools/kwboot.c b/tools/kwboot.c index 7f231c0823..2e5684b91c 100644 --- a/tools/kwboot.c +++

[PATCH u-boot-marvell 10/29] tools: kwboot: Print newline on error when progress was not completed

2021-08-25 Thread Marek Behún
From: Pali Rohár When progress was not completed, current terminal position is in progress bar. So print newline before printing error message to make error message more readable. Signed-off-by: Pali Rohár Reviewed-by: Marek Behún --- tools/kwboot.c | 1 + 1 file changed, 1 insertion(+)

[PATCH u-boot-marvell 06/29] tools: kwboot: Fix kwboot_xm_sendblock() function when kwboot_tty_recv() fails

2021-08-25 Thread Marek Behún
From: Pali Rohár When kwboot_tty_recv() fails or times out, it does not set the `c` variable to NAK. The variable is then compared, while it holds either an undefined value or a value from previous iteration. Set `c` to NAK so that the other side will try to resend current block, and remove the

[PATCH u-boot-marvell 15/29] tools: kwboot: Allow greater timeout when executing header code

2021-08-25 Thread Marek Behún
When executing header code (which contains U-Boot SPL in most cases), wait 10s after every non-xmodem character received (i.e. printed by U-Boot SPL) before timing out. Sometimes DDR training, which runs in SPL, may be slow. Signed-off-by: Marek Behún --- tools/kwboot.c | 44

[PATCH u-boot-marvell 19/29] tools: kwboot: Don't patch image header if signed

2021-08-25 Thread Marek Behún
From: Pali Rohár It is not possible to modify image with secure header due to cryptographic signature. Signed-off-by: Pali Rohár [ refactored ] Signed-off-by: Marek Behún --- tools/kwboot.c | 27 --- 1 file changed, 24 insertions(+), 3 deletions(-) diff --git

[PATCH u-boot-marvell 13/29] tools: kwboot: Allow non-xmodem text output from BootROM only in a specific case

2021-08-25 Thread Marek Behún
From: Pali Rohár When sending image header / image data, BootROM does not send any non-xmodem text output. We should therefore interpret unknown bytes in the xmodem protocol as errors and resend current packet. This should improve the transfer in case there are errors on the UART line. Text

[PATCH u-boot-marvell 14/29] tools: kwboot: Print new line after SPL output

2021-08-25 Thread Marek Behún
There is no separation between output from the code from binary header (U-Boot SPL in most cases) and subsequent kwboot output. Print '\n' to make distinguishing these two easier. Signed-off-by: Marek Behún --- tools/kwboot.c | 5 + 1 file changed, 5 insertions(+) diff --git

[PATCH u-boot-marvell 09/29] tools: kwboot: Fix printing progress

2021-08-25 Thread Marek Behún
From: Pali Rohár Ensure that `pos` is still in range up to the `width` so printing 100% works also for bigger images. After printing 100% progress reset it to zero, so that next progressbar can be started. Signed-off-by: Pali Rohár Reviewed-by: Marek Behún --- tools/kwboot.c | 9 +++-- 1

[PATCH u-boot-marvell 08/29] tools: kwboot: Fix comparison of integers with different size

2021-08-25 Thread Marek Behún
The compiler complains that we are comparing int with size_t when compiled with -W. Signed-off-by: Marek Behún --- tools/kwboot.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/tools/kwboot.c b/tools/kwboot.c index 88353d19c0..3d9f73e697 100644 --- a/tools/kwboot.c +++

[PATCH u-boot-marvell 05/29] tools: kwboot: Print version information header

2021-08-25 Thread Marek Behún
From: Pali Rohár Print kwboot's (U-Boot's) version when printing usage. Signed-off-by: Pali Rohár Reviewed-by: Marek Behún --- tools/kwboot.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/kwboot.c b/tools/kwboot.c index 22cdd137ad..454339db14 100644 --- a/tools/kwboot.c +++

[PATCH u-boot-marvell 04/29] tools: kwboot: Refactor and fix writing buffer

2021-08-25 Thread Marek Behún
There are 3 instances in kwboot.c where we need to write() a given buffer whole (iteratively writing until all data are written), and 2 of those instances are wrong, for they do not increment the buffer pointer. Refactor the code into a new function kwboot_write() where it is fixed.

[PATCH u-boot-marvell 03/29] tools: kwboot: Make the quit sequence buffer const

2021-08-25 Thread Marek Behún
This buffer is never written to. Make it const. Signed-off-by: Marek Behún --- tools/kwboot.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/kwboot.c b/tools/kwboot.c index e6e99849a7..f18f6d2134 100644 --- a/tools/kwboot.c +++ b/tools/kwboot.c @@ -460,7 +460,7 @@

[PATCH u-boot-marvell 01/29] tools: kwbimage: Fix printf format warning

2021-08-25 Thread Marek Behún
On 32-bit ARM the compiler complains: tools/kwbimage.c:547: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘unsigned int’ Fix this by using %zu instead of %lu format specifier.

[PATCH u-boot-marvell 00/29] kwboot higher baudrate

2021-08-25 Thread Marek Behún
Hello Stefan and others, this series adds support for booting Marvell platforms via UART (those bootable with kwboot) at higher baudrates. Tested on Turris Omnia up to 5.15 MBd, which is 44x faster than 115200 Bd. The user can direct kwboot to use higher baudrate via the -B option. (BTW this

[PATCH u-boot-marvell 02/29] tools: kwboot: Fix buffer overflow in kwboot_terminal()

2021-08-25 Thread Marek Behún
The `in` variable is set to -1 in kwboot_terminal() if stdin is not a tty. In this case we should not look whether -1 is set in fd_set, for it can lead to a buffer overflow, which can be reproduced with echo "xyz" | ./tools/kwboot -t /dev/ttyUSB0 Signed-off-by: Marek Behún --- tools/kwboot.c

[PATCH] board: ea: mx7ulp_com: allow setting CONFIG_BOOTCOMMAND in defconfig

2021-08-25 Thread Oleksandr Suvorov
From: Ricardo Salveti Allow replacing the board defined CONFIG_BOOTCOMMAND via defconfig. Signed-off-by: Ricardo Salveti Signed-off-by: Oleksandr Suvorov --- include/configs/mx7ulp_com.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/configs/mx7ulp_com.h

[PATCH] board: ea: mx7ulp_com: allocate specific region of memory to OP-TEE

2021-08-25 Thread Oleksandr Suvorov
From: Ricardo Salveti On the iMX7ULP uCOM board, OP-TEE uses the memory region defined by the maximum DRAM address minus CONFIG_OPTEE_TZDRAM_SIZE, so subtract CONFIG_OPTEE_TZDRAM_SIZE from the available DRAM size to avoid conflicts. Note the OPTEE boot process itself subtracts the DRAM region

Re: [PATCH 00/28] Initial implementation of bootmethod/bootflow

2021-08-25 Thread Peter Robinson
> > Exactly. Keep in mind that RISC-V is looking into EBBR as well, so this is > > far from an 'Arm thing'. Moreover, the efi stub side of the kernel for > > risc-v, will only load an initrd using the EFI_LOAD_FILE2 protocol we added > > support for. So right now the only way to properly boot a

[PATCH v1] driver: spi: add bcm iproc qspi support.

2021-08-25 Thread Bharat Kumar Reddy Gooty
From: Rayagonda Kokatanur IPROC qspi driver supports both BSPI and MSPI modes. Signed-off-by: Rayagonda Kokatanur Signed-off-by: Bharat Gooty --- drivers/spi/Kconfig | 6 + drivers/spi/Makefile | 1 + drivers/spi/iproc_qspi.c | 736 +++

Re: [PATCH 00/28] Initial implementation of bootmethod/bootflow

2021-08-25 Thread Simon Glass
Hi Ilias, On Tue, 24 Aug 2021 at 03:30, Ilias Apalodimas wrote: > > Hi Tom, > > > > > > > > > > > [...] > > > > > > > > > The series is available at u-boot-dm/bmea-working > > > > > > > > > > > > > > My question / concern is this. Would the next step here be to > > > > > > > implement the

Re: [PATCH 00/28] Initial implementation of bootmethod/bootflow

2021-08-25 Thread Simon Glass
Hi, On Wed, 25 Aug 2021 at 04:45, Emmanuel Vadot wrote: > > On Tue, 24 Aug 2021 12:22:42 +0200 (CEST) > Mark Kettenis wrote: > > > > Date: Mon, 23 Aug 2021 16:01:46 -0400 > > > From: Tom Rini > > > > > > On Mon, Aug 23, 2021 at 11:25:42AM -0600, Simon Glass wrote: > > > > Hi Mark, > > > > > >

  1   2   >