Re: [PATCH v2 0/6] net: dwc_eth_qos: add support of device tree configuration for reset delay

2021-07-09 Thread Ramon Fried
On Fri, Jul 9, 2021 at 1:35 PM Patrick DELAUNAY wrote: > > Hi Ramon, > > On 4/26/21 5:46 PM, Patrick Delaunay wrote: > > It is the V2 version of [1] after Marek remarks: all commit are new. > > > > I change the DWC_ETH_QOS STM32 variant by using generic eth phy driver. > > > > This driver is updat

Re: [PATCH] net: Use phys_addr_t for SMC net device addresses

2021-07-09 Thread Ramon Fried
On Fri, Jul 9, 2021 at 12:53 PM Peter Hoyes wrote: > > From: Peter Hoyes > > Use same type as eth_device->iobase and support addresses greater > than INT_MAX. > > Signed-off-by: Peter Hoyes > --- > drivers/net/smc9.c | 2 +- > drivers/net/smc911x.c | 2 +- > include/netdev.h | 4 ++--

[PATCH] cmd: ubi: add a command to swap volumes

2021-07-09 Thread Ayoub Zaki
This commit adds the command ubi swap to swap an ubi volumes. The format of the command is: ubi swap . To enable this command, the option CMD_UBI_SWAPVOL must be selected. Signed-off-by: Ayoub Zaki --- cmd/Kconfig | 8 cmd/ubi.c | 54 +

[RESEND RFC PATCH v1 6/6] arm: dts: rockchip: add rk3318 A95X Z2 board

2021-07-09 Thread Johan Jonker
The rk3318 A95X Z2 boards are sold as TV box. No further documentation is given, but from the dts files extracted it seems that the rk3318 processor is simulair to the rk3328. This dts file contains only the basic nodes that have support in the mainline kernel. Included extra dtsi file for U-boot s

[PATCH] net: Use phys_addr_t for SMC net device addresses

2021-07-09 Thread Peter Hoyes
From: Peter Hoyes Use same type as eth_device->iobase and support addresses greater than INT_MAX. Signed-off-by: Peter Hoyes --- drivers/net/smc9.c | 2 +- drivers/net/smc911x.c | 2 +- include/netdev.h | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers

Build a standalone ext4 file system driver

2021-07-09 Thread Feng Libo
Uboot implements the simple file protocol of ext4 file system, already. However, the driver is embedded in Uboot. Is it possible to build a standalone ext4 file system driver for UEFI, in the format of efi, just like the helloworld.efi. So I can use the UEFI shell command: load to load the drive

Illegal instruction - While starting Linux kernel.

2021-07-09 Thread Venkatakrishnan Sutharsan
Hi, I have been writing SPI Driver in U-Boot for a RISCV Board and is currently trying to use that to boot linux kernel from SD Card. Right now, I have the card detected and can view files in various partitions in the SD Card. After that I try to copy the uImage (created using mkimage command and

Re: Please pull u-boot-marvell/master

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

Re: [PATCH] arm: armv8: Fix warning about redeclaring global functions as weak

2021-07-09 Thread Tom Rini
On Tue, Jun 29, 2021 at 07:33:04PM -0400, Tom Rini wrote: > As seen with clang-12: > warning: __asm_invalidate_l3_dcache changed binding to STB_WEAK > > As we indeed use ENTRY and then declare the function weak manually. Use > the WEAK declarative from instead. > > Signed-off-by: Tom Rini Ap

Re: [PATCH] arm: bootm: Disable LMB reservation for command line and board info on arm64

2021-07-09 Thread Tom Rini
On Sat, May 29, 2021 at 01:34:32PM +0200, Marek Vasut wrote: > On arm64, board info is not applicable and kernel command line patched into > the DT, so the LMB reservation here makes no sense anymore. On legacy arm32, > this might still be necessary on systems which do not use DT or use legacy > A

Re: [PATCH] arch: cache: cp15: Add mmu_set_region_dcache_behaviour() when SYS_DCACHE_OFF is enable

2021-07-09 Thread Tom Rini
On Wed, Feb 24, 2021 at 01:48:42PM +0100, Patrice Chotard wrote: > From: Patrice Chotard > > Fix following compilation issue when SYS_DCACHE_OFF is enable: > drivers/misc/scmi_agent.c:128: undefined reference to > `mmu_set_region_dcache_behaviour' > > when SYS_DCACHE_OFF is enable, mmu_set_reg

Re: [PATCH v4] armv8: Handle EL2 Host mode

2021-07-09 Thread Tom Rini
On Wed, Feb 10, 2021 at 08:14:55PM +0100, Mark Kettenis wrote: > On implementations that support VHE, the layout of the CPTR_EL2 > register depends on whether HCR_EL2.E2H is set. If the bit is > set, CPTR_EL2 uses the same layout as CPACR_EL1 and can in fact > be accessed through that register.

Re: [PATCH u-boot-mvebu 00/31] kwboot / kwbimage improvements

2021-07-09 Thread Pali Rohár
On Saturday 10 July 2021 02:43:12 Pali Rohár wrote: > On Saturday 10 July 2021 02:31:32 Stefan Roese wrote: > > Could you perhaps add this "SPL returns to BootROM" support as an > > optional feature that can be selected on a per-board basis? > > Hi Stefan! I was thinking about it and it should not

Re: [PATCH u-boot-mvebu 00/31] kwboot / kwbimage improvements

2021-07-09 Thread Pali Rohár
On Saturday 10 July 2021 02:31:32 Stefan Roese wrote: > Could you perhaps add this "SPL returns to BootROM" support as an > optional feature that can be selected on a per-board basis? Hi Stefan! I was thinking about it and it should not be hard. Based on defconfig options you decide if you want to

Re: [PATCH u-boot-mvebu 00/31] kwboot / kwbimage improvements

2021-07-09 Thread Stefan Roese
Hi Marek, On 09.07.21 16:08, Marek Behún wrote: On Fri, 9 Jul 2021 14:35:15 +0200 Stefan Roese wrote: Thanks. And could you please also do one (or more) boot-time comparisons, old vs. new version? This way we can see, if and how the boot-time is affected (perhaps improved, which would be grea

IMX8M Mini HAB secure boot - working?

2021-07-09 Thread Tim Harvey
Greetings, Has anyone successfully used secure boot with IMX8M Mini or other IMX8M? Peng's recent series got merged with the exception of what looks like the addition of couple of 'caam' commands to blob/deblob DEK's. There are no guides yet however I'm following the guides for the downstream NXP

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

2021-07-09 Thread Julius Werner
> - "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 already though about it and proposed to have build co

Re: [PATCH u-boot-mvebu v2] arm: mvebu: armada-3720: remove unused config option

2021-07-09 Thread Pali Rohár
On Friday 09 July 2021 17:40:59 Marek Behún wrote: > The config option CONFIG_DEBUG_UART_CLOCK is not used by Armada 3720's > serial driver (it wasn't even before the recent update of that driver). > > Even if it was used, the value was incorrect (the frequency of the clock > is 25 MHz, not 25.804

RZ/G2 USB quirks U-Boot

2021-07-09 Thread Adam Ford
I recently broke my dominant wrist, so I apologize if this message is hard to read. I have been running some tests with the RZ/G2 MNH processors, and I noticed a difference in behavior between a cold boot, a soft reset, and a hardware reset. If I'm in u-boot, an issue the reset instruction, and I

Re: [PATCH] get_maintainer.pl: update from Linux kernel v5.13-rc6

2021-07-09 Thread Tom Rini
On Thu, Jul 08, 2021 at 08:22:35AM -0400, Trevor Woerner wrote: > ping? > > Tom, have you had a chance to try out this update? Thanks, I need to do a build/related PR shortly and I'll pick this up then. > > On Tue, Jun 15, 2021 at 3:30 AM Trevor Woerner wrote: > > > Update U-Boot's version of

RE: [PATCHv2 2/4] tegra: Test on CONFIG_CMD_USB being enabled for distro bootcmd

2021-07-09 Thread Tom Warren
Acked-by twar...@nvidia.com -Original Message- From: Tom Rini Sent: Friday, July 9, 2021 7:12 AM To: u-boot@lists.denx.de Cc: Tom Warren Subject: [PATCHv2 2/4] tegra: Test on CONFIG_CMD_USB being enabled for distro bootcmd External email: Use caution opening links or attachments Reu

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

2021-07-09 Thread Tom Rini
On Fri, Jul 09, 2021 at 05:00:09PM +0200, Marek Behún wrote: > On Fri, 9 Jul 2021 16:56:13 +0200 > 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

Re: [PATCH] arm64: Update memcpy_{from, to}io() helpers

2021-07-09 Thread Tom Rini
On Fri, Feb 26, 2021 at 01:44:51PM +0100, Patrice Chotard wrote: > At early U-Boot stage, before relocation, MMU is not yet configured > and disabled. DDR may not be configured with the correct memory > attributes (can be configured in MT_DEVICE instead of MT_MEMORY). > > In this case, usage of m

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

2021-07-09 Thread Manish Pandey2
Please find my replies To Julius's question: Just to clarify: are you using "bloblist" as a general term for the concept of a simple linked list of tagged data blobs, or to refer specifically to the U-Boot implementation with that name? The existing TF-A implementation (bl_aux_params) is basica

[PATCH u-boot-mvebu v2] arm: mvebu: armada-3720: remove unused config option

2021-07-09 Thread Marek Behún
The config option CONFIG_DEBUG_UART_CLOCK is not used by Armada 3720's serial driver (it wasn't even before the recent update of that driver). Even if it was used, the value was incorrect (the frequency of the clock is 25 MHz, not 25.8048 MHz). Remove it from config files and set the default valu

Re: [PATCH u-boot-mvebu] arm: mvebu: armada-3720: remove unused config option

2021-07-09 Thread Marek Behún
Please ignore this. It seems that the option is needed, because it does not have a default. A different way to solve this is needed. Will post another patch. Marek

Re: [PATCH u-boot-mvebu] arm: mvebu: armada-3720: remove unused config option

2021-07-09 Thread Pali Rohár
On Friday 09 July 2021 17:27:02 Marek Behún wrote: > The config option CONFIG_DEBUG_UART_CLOCK is not used by Armada 3720's > serial driver (it wasn't even before the recent update of that driver). > > Remove it. > > Signed-off-by: Marek Behún > Cc: Stefan Roese > Cc: Pali Rohár Reviewed-by:

Re: [PATCH] Kconfig.boot: Make 0x0 the default SYS_TEXT_BASE for POSITION_INDEPENDENT

2021-07-09 Thread Tom Rini
On Fri, Jul 09, 2021 at 05:02:19PM +0200, Mark Kettenis wrote: > > From: Tom Rini > > Date: Fri, 9 Jul 2021 10:39:21 -0400 > > > > When we build U-Boot with POSITION_INDEPENDENT we must have > > SYS_TEXT_BASE be set to zero. Make this the default in that case. > > Makes sense. I had to set th

[PATCH u-boot-mvebu] arm: mvebu: armada-3720: remove unused config option

2021-07-09 Thread Marek Behún
The config option CONFIG_DEBUG_UART_CLOCK is not used by Armada 3720's serial driver (it wasn't even before the recent update of that driver). Remove it. Signed-off-by: Marek Behún Cc: Stefan Roese Cc: Pali Rohár --- configs/mvebu_db-88f3720_defconfig | 1 - configs/mvebu_espressobin

Re: [PATCH] Kconfig.boot: Make 0x0 the default SYS_TEXT_BASE for POSITION_INDEPENDENT

2021-07-09 Thread Mark Kettenis
> From: Tom Rini > Date: Fri, 9 Jul 2021 10:39:21 -0400 > > When we build U-Boot with POSITION_INDEPENDENT we must have > SYS_TEXT_BASE be set to zero. Make this the default in that case. Makes sense. I had to set this for the (still in progress) Apple M1 support. So: Reviewed-by: Mark Kett

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

2021-07-09 Thread Marek Behún
On Fri, 9 Jul 2021 16:56:13 +0200 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 CO

Re: [PATCH u-boot-mvebu 00/31] kwboot / kwbimage improvements

2021-07-09 Thread Marek Behún
On Fri, 09 Jul 2021 17:54:25 +0300 Baruch Siach wrote: > Hi Marek, Pali, > > On Thu, Jul 08 2021, Marek Behún wrote: > > Hi Stefan and others, > > > > this is a series of improvements to kwboot, kwbimage and mvebu. > > > > The main goal of this series is to correctly use BootROM's code > > for l

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

2021-07-09 Thread Marek Behún
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 --- configs/turris_mox_defconfig | 3 +++ configs/turris_omnia_defconfig

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

2021-07-09 Thread Marek Behún
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 option to spi_flash_probe(), and with the ab

Re: [PATCH u-boot-mvebu 00/31] kwboot / kwbimage improvements

2021-07-09 Thread Baruch Siach
Hi Marek, Pali, On Thu, Jul 08 2021, Marek Behún wrote: > Hi Stefan and others, > > this is a series of improvements to kwboot, kwbimage and mvebu. > > The main goal of this series is to correctly use BootROM's code > for loading U-Boot from NOR / NAND: currently only SPL is read by > BootROM and

Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-07-09 Thread Tim Harvey
On Wed, Jul 7, 2021 at 5:58 AM Teresa Remmet wrote: > > Factor out the common node settings for dm-spl and dm-pre-reloc > and move them to imx8mp-u-boot.dtsi > > Signed-off-by: Teresa Remmet > Reviewed-by: Fabio Estevam > Reviewed-by: Heiko Schocher > --- > Changes in v3: > - Moved binman nodes

[PATCH] Kconfig.boot: Make 0x0 the default SYS_TEXT_BASE for POSITION_INDEPENDENT

2021-07-09 Thread Tom Rini
When we build U-Boot with POSITION_INDEPENDENT we must have SYS_TEXT_BASE be set to zero. Make this the default in that case. Signed-off-by: Tom Rini --- common/Kconfig.boot | 1 + 1 file changed, 1 insertion(+) diff --git a/common/Kconfig.boot b/common/Kconfig.boot index 89a3161f1faa..f078743

[PATCH] Makefile: add support to build dedicated devicetree

2021-07-09 Thread Kory Maincent
Add the support to build a dedicated devicetree located in arch/$(ARCH)/dts This devicetree does not need to be listed in the relevant devicetree makefile. Just run the following command to build the foo devicetree placed before at the right path: $ make foo.dtb Signed-off-by: Kory Maincent ---

Re: [PATCH v3 19/20] env: ti: j721e-evm: Add env variable to power on & reset QSGMII PHY in J7200 EVM

2021-07-09 Thread Tom Rini
On Fri, Jul 09, 2021 at 08:02:03PM +0530, Kishon Vijay Abraham I wrote: > Hi Tom, > > On 11/05/21 8:03 pm, Tom Rini wrote: > > On Tue, May 11, 2021 at 07:28:10PM +0530, Kishon Vijay Abraham I wrote: > >> Hi Tom, > >> > >> On 07/05/21 10:44 pm, Tom Rini wrote: > >>> On Tue, May 04, 2021 at 04:11:54

Re: [PATCH v3 19/20] env: ti: j721e-evm: Add env variable to power on & reset QSGMII PHY in J7200 EVM

2021-07-09 Thread Kishon Vijay Abraham I
Hi Tom, On 11/05/21 8:03 pm, Tom Rini wrote: > On Tue, May 11, 2021 at 07:28:10PM +0530, Kishon Vijay Abraham I wrote: >> Hi Tom, >> >> On 07/05/21 10:44 pm, Tom Rini wrote: >>> On Tue, May 04, 2021 at 04:11:54PM +0530, Kishon Vijay Abraham I wrote: >>> MAIN CPSW0 requires the PHY to be power

Re: [PATCH v3 13/20] ARM: dts: k3-j721e: Add the entries required for USB3 support on USB0

2021-07-09 Thread Tom Rini
On Fri, Jul 09, 2021 at 07:36:45PM +0530, Kishon Vijay Abraham I wrote: > Hi Tom, > > On 07/05/21 10:44 pm, Tom Rini wrote: > > On Tue, May 04, 2021 at 04:11:48PM +0530, Kishon Vijay Abraham I wrote: > > > >> Partially sync with Linux's dts to add the entries required for USB3 > >> support on USB

[PATCHv2 4/4] usb: Enforce DM_USB migration for USB_HOST devices.

2021-07-09 Thread Tom Rini
As the deadline for migration to DM_USB, when using a USB host controller has now gone two years past the deadline, enforce migration. This is done by: - Ensuring that all host controller options (other than the very legacy old MUSB ones) now select USB_HOST. USB_HOST now enforces DM_USB and

[PATCHv2 3/4] treewide: Test on CONFIG_USB_HOST (or CONFIG_CMD_USB) not CONFIG_USB

2021-07-09 Thread Tom Rini
As the logic here is only used when we have a USB host controller, test on CONFIG_USB_HOST rather than CONFIG_USB in general. This lets us move towards using CONFIG_USB only as a menu symbol to say that we have some form of USB, and then USB_HOST or USB_GADGET depending on the role that USB plays

[PATCHv2 2/4] tegra: Test on CONFIG_CMD_USB being enabled for distro bootcmd

2021-07-09 Thread Tom Rini
Reuse the common logic to allow for BOOT_TARGET_DEVICES to list USB as a possibility if we're building for a platform that will have USB but not if we don't, so that we don't hit the link-time check for trying to have USB boot on a non-USB system. Cc: Tom Warren Signed-off-by: Tom Rini --- Chang

[PATCHv2 1/4] ge_bx50v3: Remove unused USB related defines

2021-07-09 Thread Tom Rini
These USB defines do not change the build as there is no USB support enabled currently. Cc: Ian Ray Cc: Sebastian Reichel Signed-off-by: Tom Rini --- Changes in v2: - New patch --- include/configs/ge_bx50v3.h | 11 --- 1 file changed, 11 deletions(-) diff --git a/include/configs/ge_bx

Re: [PATCH u-boot-mvebu 00/31] kwboot / kwbimage improvements

2021-07-09 Thread Marek Behún
On Fri, 9 Jul 2021 14:35:15 +0200 Stefan Roese wrote: > Thanks. And could you please also do one (or more) boot-time > comparisons, old vs. new version? This way we can see, if and how the > boot-time is affected (perhaps improved, which would be great) by this > patch series. For Turris Omnia,

Re: [PATCH v3 13/20] ARM: dts: k3-j721e: Add the entries required for USB3 support on USB0

2021-07-09 Thread Kishon Vijay Abraham I
Hi Tom, On 07/05/21 10:44 pm, Tom Rini wrote: > On Tue, May 04, 2021 at 04:11:48PM +0530, Kishon Vijay Abraham I wrote: > >> Partially sync with Linux's dts to add the entries required for USB3 >> support on USB0. >> Note that the default mode is still "peripheral" not "host". USB3 is >> supporte

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

2021-07-09 Thread Patrice CHOTARD
Hi Patrick On 7/8/21 11:17 AM, Patrick Delaunay wrote: > 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

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

2021-07-09 Thread Patrice CHOTARD
Hi Patrick One typo below On 7/8/21 11:17 AM, Patrick Delaunay wrote: > 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 documen

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

2021-07-09 Thread Patrice CHOTARD
Hi Patrick On 7/8/21 11:17 AM, Patrick Delaunay wrote: > 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

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

2021-07-09 Thread Patrice CHOTARD
Hi Patrick On 7/8/21 11:17 AM, 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. > > These nodes are only required in trusted boot, when TF-

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

2021-07-09 Thread Patrice CHOTARD
Hi Patrick On 7/8/21 11:17 AM, Patrick Delaunay wrote: > 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 > extract

Re: [PATCH] stm32mp1: add pull-up for gpio button PA13 and PA14

2021-07-09 Thread Patrice CHOTARD
Hi Patrick On 7/9/21 9:53 AM, Patrick Delaunay wrote: > When a push-button is released and PA13/PA14 are defined as input (high-Z) > the LED should not be active as the circuit is open but a small current > leak through PCB or push-button close the circuit and allows a small LED > bias giving erro

Re: [PATCH u-boot-mvebu 00/31] kwboot / kwbimage improvements

2021-07-09 Thread Stefan Roese
Hi Marek, On 09.07.21 13:22, Marek Behún wrote: Hi Stefan On Fri, 9 Jul 2021 08:05:40 +0200 Stefan Roese wrote: The main goal of this series is to correctly use BootROM's code for loading U-Boot from NOR / NAND: currently only SPL is read by BootROM and the main U-Boot is read by SPL. By usi

[PATCH] clk: stm32mp1: add support of missing SPI clocks

2021-07-09 Thread Patrick Delaunay
Add the missing SPI clock even if these instances are not available on STMicroelectronics boards: SPI2_K, SPI3_K, SPI4_K, SPI6_K. With this patch, the SPI2 / SPI3 / SPI4 / SPI6 instances can be used on customer design without the clock driver error: stm32mp1_clk_get_id: clk id 131 not found Rev

Re: [PATCH 3/6] dts: synquacer: Add partition information to the spi-nor

2021-07-09 Thread Marek Behún
On Fri, 9 Jul 2021 20:27:18 +0900 Masami Hiramatsu wrote: > Hi Marek, > > 2021年7月9日(金) 19:54 Marek Behún : > > > > On Fri, 9 Jul 2021 19:50:27 +0900 > > Masami Hiramatsu wrote: > > > > > Add partition information to the spi-nor flash. > > > This is required for accessing NOR flash via mtdpar

[PATCH v3 1/6] mmc: zynq_sdhci: Resolve uninitialized return value

2021-07-09 Thread Ashok Reddy Soma
set_phase() functions are not modifying the ret value and returning the same uninitialized ret, return 0 instead. Keep the return type as int to return errors when the tapdelay's are set via xilinx_pm_request() in future. Signed-off-by: Ashok Reddy Soma --- Changes in v3: - Updated commmit des

[PATCH v3 3/6] mmc: zynq_sdhci: Use Mask writes for Tap delays

2021-07-09 Thread Ashok Reddy Soma
Restrict tap_delay value to the allowed size(8bits for itap and 6 bits for otap) before writing to the tap delay register. Clear ITAP and OTAP delay bits before updating with the new tap value for Versal platform. Signed-off-by: Ashok Reddy Soma --- Changes in v3: - Updated macro's with BIT()

[PATCH v3 2/6] mmc: zynq_sdhci: Allow configuring zero Tap values

2021-07-09 Thread Ashok Reddy Soma
Allow configuring ITAP and OTAP values with zero to avoid failures in some cases (one of them is SD boot mode). Legacy, SDR12 modes require to program the ITAP and OTAP values as zero, whereas for SDR50 and SDR104 modes ITAP value is zero. In SD boot mode firmware configures the SD ITAP and OTAP v

[PATCH v3 5/6] mmc: zynq_sdhci: Fix kernel doc warnings

2021-07-09 Thread Ashok Reddy Soma
From: Michal Simek Fix these kernel doc warnings: drivers/mmc/zynq_sdhci.c:181: warning: contents before sections drivers/mmc/zynq_sdhci.c:236: warning: contents before sections drivers/mmc/zynq_sdhci.c:291: warning: contents before sections drivers/mmc/zynq_sdhci.c:297: warning: Function paramet

[PATCH v3 6/6] mmc: zynq_sdhci: Make variables/structure static

2021-07-09 Thread Ashok Reddy Soma
From: Michal Simek All these variables/structure are local and should be static. Issues are reported by sparse: drivers/mmc/zynq_sdhci.c:49:11: warning: symbol 'zynqmp_iclk_phases' was not declared. Should it be static? drivers/mmc/zynq_sdhci.c:50:11: warning: symbol 'zynqmp_oclk_phases' was no

[PATCH v3 4/6] mmc: zynq_sdhci: Split set_tapdelay function to in and out

2021-07-09 Thread Ashok Reddy Soma
Split arasan_zynqmp_set_tapdelay() to handle input and output tapdelays separately. This is required to handle zero values for ITAP and OTAP values. If we dont split, we will have to remove the if() in the function, which makes ITAP values to be overwritten when OTAP values are called to set and vi

[PATCH v3 0/6] This patch set fixes minor issues related to tapdelays

2021-07-09 Thread Ashok Reddy Soma
This patch set fixes below issues in zynq_sdhc driver - Fix issues in tap delay functions where it returns uninitialized values - Allow configuring zero tap delay values - Split tapdelay functions to set input and output tap delay's separately. - Fix kernel doc warnings - Make local structures

Re: [PATCH 3/6] dts: synquacer: Add partition information to the spi-nor

2021-07-09 Thread Masami Hiramatsu
Hi Marek, 2021年7月9日(金) 19:54 Marek Behún : > > On Fri, 9 Jul 2021 19:50:27 +0900 > Masami Hiramatsu wrote: > > > Add partition information to the spi-nor flash. > > This is required for accessing NOR flash via mtdparts. > > > > Signed-off-by: Masami Hiramatsu > > --- > > .../dts/synquacer-sc2a

Re: [PATCH u-boot-mvebu 00/31] kwboot / kwbimage improvements

2021-07-09 Thread Marek Behún
Hi Stefan On Fri, 9 Jul 2021 08:05:40 +0200 Stefan Roese wrote: > > The main goal of this series is to correctly use BootROM's code > > for loading U-Boot from NOR / NAND: currently only SPL is read by > > BootROM and the main U-Boot is read by SPL. By using BootROM to also > > load main U-Boot

RE: [PATCH v2 1/6] mmc: zynq_sdhci: Resolve uninitialized return value

2021-07-09 Thread Ashok Reddy Soma
Hi Michal, > -Original Message- > From: Michal Simek > Sent: Friday, July 9, 2021 4:35 PM > To: Ashok Reddy Soma ; u-boot@lists.denx.de > Cc: peng@nxp.com; jh80.ch...@samsung.com; git ; > mon...@monstr.eu; somaashokre...@gmail.com > Subject: Re: [PATCH v2 1/6] mmc: zynq_sdhci: Resolve

RE: [PATCH v2 3/6] mmc: zynq_sdhci: Use Mask writes for Tap delays

2021-07-09 Thread Ashok Reddy Soma
HI Michal, > -Original Message- > From: Michal Simek > Sent: Friday, July 9, 2021 4:37 PM > To: Ashok Reddy Soma ; u-boot@lists.denx.de > Cc: peng@nxp.com; jh80.ch...@samsung.com; git ; > mon...@monstr.eu; somaashokre...@gmail.com > Subject: Re: [PATCH v2 3/6] mmc: zynq_sdhci: Use Mas

Re: [PATCH v2 3/6] mmc: zynq_sdhci: Use Mask writes for Tap delays

2021-07-09 Thread Michal Simek
On 7/9/21 12:47 PM, Ashok Reddy Soma wrote: > Restrict tap_delay value to the allowed size(8bits for itap and 6 bits > for otap) before writing to the tap delay register. > > Clear ITAP and OTAP delay bits before updating with the new tap value > for Versal platform. > > Signed-off-by: Ashok R

Re: [PATCH v2 1/6] mmc: zynq_sdhci: Resolve uninitialized return value

2021-07-09 Thread Michal Simek
On 7/9/21 12:46 PM, Ashok Reddy Soma wrote: > set_phase() functions are not modifying the ret value and returning > the same uninitialized ret, return 0 instead. > > Signed-off-by: Ashok Reddy Soma > --- > > Changes in v2: > - Changed "@degree" to "@degrees:" in function descriptions of tap

RE: [PATCH 1/6] mmc: zynq_sdhci: Resolve uninitialized return value

2021-07-09 Thread Ashok Reddy Soma
Hi Jaehoon, > -Original Message- > From: Jaehoon Chung > Sent: Friday, July 9, 2021 4:21 PM > To: Ashok Reddy Soma ; u-boot@lists.denx.de > Cc: peng@nxp.com; git ; mon...@monstr.eu; > somaashokre...@gmail.com > Subject: Re: [PATCH 1/6] mmc: zynq_sdhci: Resolve uninitialized return val

Re: [PATCH 3/6] dts: synquacer: Add partition information to the spi-nor

2021-07-09 Thread Marek Behún
On Fri, 9 Jul 2021 19:50:27 +0900 Masami Hiramatsu wrote: > Add partition information to the spi-nor flash. > This is required for accessing NOR flash via mtdparts. > > Signed-off-by: Masami Hiramatsu > --- > .../dts/synquacer-sc2a11-developerbox-u-boot.dtsi | 42 > 1 f

Re: [PATCH 2/6] configs: synquacer: Make U-Boot binary position independent

2021-07-09 Thread Marek Behún
On Fri, 9 Jul 2021 19:50:18 +0900 Masami Hiramatsu wrote: > Make the U-Boot bianry for SynQuacer position independent so *binary

[PATCH 6/6] configs: synquacer: Enable UEFI secure boot

2021-07-09 Thread Masami Hiramatsu
Enable UEFI secure boot on synquacer. Note that unless user setup their keys, the secure boot will not work. Signed-off-by: Masami Hiramatsu --- configs/synquacer_developerbox_defconfig |1 + 1 file changed, 1 insertion(+) diff --git a/configs/synquacer_developerbox_defconfig b/configs/syn

[PATCH 5/6] configs: synquacer: Drop Ext2/4 support by default

2021-07-09 Thread Masami Hiramatsu
Since the U-Boot for the SynQuacer DeveloperBox is designed for compatible with EDK2 boot, we don't need to support Ext2/4 fs support by default. Drop it. Signed-off-by: Masami Hiramatsu --- configs/synquacer_developerbox_defconfig |2 -- 1 file changed, 2 deletions(-) diff --git a/configs/

[PATCH 4/6] configs: synquacer: Remove mtdparts settings and update DFU setting

2021-07-09 Thread Masami Hiramatsu
Since MTD partitions are based on the devicetree name, remove unneeded mtdparts settings and update DFU setting. Signed-off-by: Masami Hiramatsu --- configs/synquacer_developerbox_defconfig |2 -- include/configs/synquacer.h |2 +- 2 files changed, 1 insertion(+), 3 deletion

[PATCH 3/6] dts: synquacer: Add partition information to the spi-nor

2021-07-09 Thread Masami Hiramatsu
Add partition information to the spi-nor flash. This is required for accessing NOR flash via mtdparts. Signed-off-by: Masami Hiramatsu --- .../dts/synquacer-sc2a11-developerbox-u-boot.dtsi | 42 1 file changed, 42 insertions(+) diff --git a/arch/arm/dts/synquacer-sc2a11-

[PATCH 2/6] configs: synquacer: Make U-Boot binary position independent

2021-07-09 Thread Masami Hiramatsu
Make the U-Boot bianry for SynQuacer position independent so that the previous bootloader (SCP firmware or BL2) can load the U-Boot anywhere. Signed-off-by: Masami Hiramatsu --- configs/synquacer_developerbox_defconfig |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/con

[PATCH 1/6] board: synquacer: Initialize SCBM SMMU at board_init()

2021-07-09 Thread Masami Hiramatsu
Since the SCBM SMMU is not only connected to the NETSEC but also shared with the F_SDH30 (eMMC controller), that should be initialized at board level instead of NETSEC. Move the SMMU initialization code into board support and call it from board_init(). Without this fix, if the NETSEC is disabled,

[PATCH 0/6] arm64: synquacer: DeveloperBox updates

2021-07-09 Thread Masami Hiramatsu
Hi, Here is the series of patches to update the DeveloperBox support. [1/6] board: synquacer: Initialize SCBM SMMU at board_init() - This is a driver cleanup, split the SMMU setup from the driver. [2/6] configs: synquacer: Make U-Boot binary position independent - This makes U-Boot position

Re: [PATCH 1/6] mmc: zynq_sdhci: Resolve uninitialized return value

2021-07-09 Thread Jaehoon Chung
Hi Ashok, On 7/9/21 7:15 PM, Ashok Reddy Soma wrote: > Hi Jaehoon, > >> -Original Message- >> From: Jaehoon Chung >> Sent: Friday, July 9, 2021 3:08 PM >> To: Ashok Reddy Soma ; u-boot@lists.denx.de >> Cc: peng@nxp.com; git ; mon...@monstr.eu; >> somaashokre...@gmail.com >> Subject:

[PATCH v2 4/6] mmc: zynq_sdhci: Split set_tapdelay function to in and out

2021-07-09 Thread Ashok Reddy Soma
Split arasan_zynqmp_set_tapdelay() to handle input and output tapdelays separately. This is required to handle zero values for ITAP and OTAP values. If we dont split, we will have to remove the if() in the function, which makes ITAP values to be overwritten when OTAP values are called to set and vi

[PATCH v2 6/6] mmc: zynq_sdhci: Make variables/structure static

2021-07-09 Thread Ashok Reddy Soma
From: Michal Simek All these variables/structure are local and should be static. Issues are reported by sparse: drivers/mmc/zynq_sdhci.c:49:11: warning: symbol 'zynqmp_iclk_phases' was not declared. Should it be static? drivers/mmc/zynq_sdhci.c:50:11: warning: symbol 'zynqmp_oclk_phases' was no

[PATCH v2 3/6] mmc: zynq_sdhci: Use Mask writes for Tap delays

2021-07-09 Thread Ashok Reddy Soma
Restrict tap_delay value to the allowed size(8bits for itap and 6 bits for otap) before writing to the tap delay register. Clear ITAP and OTAP delay bits before updating with the new tap value for Versal platform. Signed-off-by: Ashok Reddy Soma --- (no changes since v1) drivers/mmc/zynq_sdhc

[PATCH v2 0/6] This patch set fixes minor issues related to tapdelays

2021-07-09 Thread Ashok Reddy Soma
This patch set fixes below issues in zynq_sdhc driver - Fix issues in tap delay functions where it returns uninitialized values - Allow configuring zero tap delay values - Split tapdelay functions to set input and output tap delay's separately. - Fix kernel doc warnings - Make local structures

[PATCH v2 2/6] mmc: zynq_sdhci: Allow configuring zero Tap values

2021-07-09 Thread Ashok Reddy Soma
Allow configuring ITAP and OTAP values with zero to avoid failures in some cases (one of them is SD boot mode). Legacy, SDR12 modes require to program the ITAP and OTAP values as zero, whereas for SDR50 and SDR104 modes ITAP value is zero. In SD boot mode firmware configures the SD ITAP and OTAP v

[PATCH v2 5/6] mmc: zynq_sdhci: Fix kernel doc warnings

2021-07-09 Thread Ashok Reddy Soma
From: Michal Simek Fix these kernel doc warnings: drivers/mmc/zynq_sdhci.c:181: warning: contents before sections drivers/mmc/zynq_sdhci.c:236: warning: contents before sections drivers/mmc/zynq_sdhci.c:291: warning: contents before sections drivers/mmc/zynq_sdhci.c:354: warning: contents before

[PATCH v2 1/6] mmc: zynq_sdhci: Resolve uninitialized return value

2021-07-09 Thread Ashok Reddy Soma
set_phase() functions are not modifying the ret value and returning the same uninitialized ret, return 0 instead. Signed-off-by: Ashok Reddy Soma --- Changes in v2: - Changed "@degree" to "@degrees:" in function descriptions of tap delay functions drivers/mmc/zynq_sdhci.c | 24 ++--

[PATCH] efi_loader: replace board_get_usable_ram_top by gd->ram_top

2021-07-09 Thread Patrick Delaunay
As gd->ram_top = board_get_usable_ram_top() in board_r the EFI loader don't need to call this function again and after relocation. This patch avoid issue if board assumed that this function is called only one time and before relocation. Signed-off-by: Patrick Delaunay --- https://source.denx.de

Re: [PATCH v2 0/6] net: dwc_eth_qos: add support of device tree configuration for reset delay

2021-07-09 Thread Patrick DELAUNAY
Hi Ramon, On 4/26/21 5:46 PM, Patrick Delaunay wrote: It is the V2 version of [1] after Marek remarks: all commit are new. I change the DWC_ETH_QOS STM32 variant by using generic eth phy driver. This driver is updated to use the gpio reset and assert/deassert delay from DT with the generic bin

RE: [PATCH 5/6] mmc: zynq_sdhci: Fix kernel doc warnings

2021-07-09 Thread Ashok Reddy Soma
Hi Jaehoon, > -Original Message- > From: Jaehoon Chung > Sent: Friday, July 9, 2021 3:12 PM > To: Ashok Reddy Soma ; u-boot@lists.denx.de > Cc: peng@nxp.com; git ; mon...@monstr.eu; > somaashokre...@gmail.com; Michal Simek > Subject: Re: [PATCH 5/6] mmc: zynq_sdhci: Fix kernel doc wa

RE: [PATCH 1/6] mmc: zynq_sdhci: Resolve uninitialized return value

2021-07-09 Thread Ashok Reddy Soma
Hi Jaehoon, > -Original Message- > From: Jaehoon Chung > Sent: Friday, July 9, 2021 3:08 PM > To: Ashok Reddy Soma ; u-boot@lists.denx.de > Cc: peng@nxp.com; git ; mon...@monstr.eu; > somaashokre...@gmail.com > Subject: Re: [PATCH 1/6] mmc: zynq_sdhci: Resolve uninitialized return val

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

2021-07-09 Thread Daniel Thompson
On Fri, Jul 09, 2021 at 09:05:09AM +0200, François Ozog wrote: > Le ven. 9 juil. 2021 à 03:09, Julius Werner a écrit : > > > > Of course every project would like not to change... > > > > > > For TF-A I wonder whether it will/should in fact use devicetree if there > > is a lot of complex data? TBD

Re: [PATCH] arm: stm32mp1: force boot_device variable for invalid TAMP register value

2021-07-09 Thread Patrice CHOTARD
Hi Patrick On 7/8/21 10:53 AM, Patrick Delaunay wrote: > When the TAMP register 20 have an invalid value (0x0 for example after > TAMPER error) the "boot_device" U-Boot env variable have no value and > no error is displayed in U-Boot log. > > The STM32MP boot command bootcmd_stm32mp failed with s

Re: [PATCH 5/6] mmc: zynq_sdhci: Fix kernel doc warnings

2021-07-09 Thread Jaehoon Chung
Hi Ashok, On 7/9/21 4:17 PM, Ashok Reddy Soma wrote: > From: Michal Simek > > Fix these kernel doc warnings: > drivers/mmc/zynq_sdhci.c:181: warning: contents before sections > drivers/mmc/zynq_sdhci.c:187: warning: Function parameter or member 'degrees' > not described in 'sdhci_zynqmp_sdcardc

Re: [PATCH 1/6] mmc: zynq_sdhci: Resolve uninitialized return value

2021-07-09 Thread Jaehoon Chung
Hi Ashok, On 7/9/21 4:17 PM, Ashok Reddy Soma wrote: > set_phase() functions are not modifying the ret value and returning > the same uninitialized ret, return 0 instead. Why didn't you change from int to void? Best Regards, Jaehoon Chung > > Signed-off-by: Ashok Reddy Soma > --- > > drive

Re: [PATCH 4/4] mmc: arm_pl180_mmci: Add configuration for ST-Ericsson Ux500v2

2021-07-09 Thread Jaehoon Chung
On 7/6/21 11:54 PM, Stephan Gerhold wrote: > For the eMMC on ST-Ericsson Ux500v2 we need slightly different > configuration values. Use the existing switch statement to match > the peripheral ID of Ux500v2 (0x10480180) and override the necessary > values to make the eMMC work on devices with ST-Eri

Re: [PATCH 3/4] mmc: arm_pl180_mmci: Simplify code using mmc_of_parse()

2021-07-09 Thread Jaehoon Chung
On 7/6/21 11:54 PM, Stephan Gerhold wrote: > Simplify the code a bit by using the common mmc_of_parse() function > instead of duplicating the device tree parsing code. We can still get > a default value for cfg->f_max by assigning it before calling > mmc_of_parse(). > > Another advantage of this r

Re: [PATCH 2/4] mmc: arm_pl180_mmci: Simplify code using dev_read_addr_ptr()

2021-07-09 Thread Jaehoon Chung
On 7/6/21 11:54 PM, Stephan Gerhold wrote: > Simplify the code a bit by using dev_read_addr_ptr() instead of > dev_read_addr(). This avoids having to cast explicitly to void*. > > Signed-off-by: Stephan Gerhold Reviewed-by: Jaehoon Chung Best Regards, Jaehoon Chung > --- > > drivers/mmc/arm

Re: [PATCH 1/4] mmc: arm_pl180_mmci: Don't bind to all arm,primecell devices

2021-07-09 Thread Jaehoon Chung
On 7/6/21 11:54 PM, Stephan Gerhold wrote: > The arm,primecell compatible is used for lots of different types > of devices, e.g. I2C, SPI, coresight, ... We really should not bind > the MMC driver to all of them. > > Looking through the device trees in U-Boot there seems to be always > a second co

Re: [PATCH v4 4/5] stm32mp1: spl: Configure TrustZone controller for OP-TEE

2021-07-09 Thread Patrick DELAUNAY
Hi, On 5/31/21 7:43 PM, Alexandru Gagniuc wrote: OP-TEE is very particular about how the TZC should be configured. When booting an OP-TEE payload, an incorrect TZC configuration will result in a panic. Most information can be derived from the SPL devicetree. The only information we don't have i

  1   2   >