RE: [PATCH 2/2] spi: nxp_fspi: Implement errata workaround for LS1028A

2021-03-21 Thread Kuldeep Singh
+ Ye Li > -Original Message- > From: Kuldeep Singh > Sent: Monday, March 22, 2021 12:00 PM > To: Jagan Teki ; u-boot@lists.denx.de > Cc: Kuldeep Singh > Subject: [PATCH 2/2] spi: nxp_fspi: Implement errata workaround for LS1028A > > Errata ERR050568 description says that "Flash access b

[PATCH 2/2] spi: nxp_fspi: Implement errata workaround for LS1028A

2021-03-21 Thread Kuldeep Singh
Errata ERR050568 description says that "Flash access by FlexSPI AHB command may not work with platform frequency equal to 300 MHz" on LS1028A. By default, smaller length reads(equal to RX FIFO size) are done by IP bus and larger length reads using AHB bus. For adding errata workaround, use IP bus

[PATCH 1/2] spi: nxp-fspi: Add support for IP read only

2021-03-21 Thread Kuldeep Singh
Add support for disabling AHB bus and read entire flash contents via IP bus only. Please note, this enables IP bus read using a quirk which can be enabled directly in device-type data or in existence of an errata where AHB bus may need to be disabled. Signed-off-by: Kuldeep Singh --- drivers/spi

Re: [PATCH 5/5] Convert CONFIG_MISC_INIT_F to Kconfig

2021-03-21 Thread Stefan Roese
On 22.03.21 06:13, Simon Glass wrote: This converts the following to Kconfig: CONFIG_MISC_INIT_F Signed-off-by: Simon Glass Reviewed-by: Stefan Roese Thanks, Stefan --- common/Kconfig | 6 ++ configs/MPC8349ITXGP_defconfig | 1 + configs/MPC8349ITX

[PATCH] sandbox: Support signal handling only when requested

2021-03-21 Thread Simon Glass
At present if sandbox crashes it prints a message and tries to exit. But with the recently introduced signal handler, it often seems to get stuck in a loop until the stack overflows: Segmentation violation Segmentation violation Segmentation violation Segmentation violation Segmentation violat

[PATCH 5/5] Convert CONFIG_MISC_INIT_F to Kconfig

2021-03-21 Thread Simon Glass
This converts the following to Kconfig: CONFIG_MISC_INIT_F Signed-off-by: Simon Glass --- common/Kconfig | 6 ++ configs/MPC8349ITXGP_defconfig | 1 + configs/MPC8349ITX_LOWBOOT_defconfig | 1 + configs/MPC8349ITX_defconfig | 1 + configs/kmcoge4_defco

[PATCH 4/5] moveconfig: Handle binary files cleanly

2021-03-21 Thread Simon Glass
Some files are not actually source code and thus can produce unicode errors. Report this and continue. Signed-off-by: Simon Glass --- tools/moveconfig.py | 16 ++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff --git a/tools/moveconfig.py b/tools/moveconfig.py index 9514d9a

[PATCH 3/5] log: Fix up debug_cond() when LOG is enabled

2021-03-21 Thread Simon Glass
At present debug() statements can cause debuf output to appear when LOG is enabled but DEBUG is not. This is not intended and it seems that the condition is wrong. Fix it. Signed-off-by: Simon Glass --- include/log.h | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git

[PATCH 1/5] dtoc: Show driver warnings once at the end

2021-03-21 Thread Simon Glass
At present warnings are shown as soon as they are discovered in the source scannner. But the function that detects them may be called multiple times. Collect all the warnings and show them at the end. Signed-off-by: Simon Glass --- tools/dtoc/dtb_platdata.py | 6 -- tools/dtoc/src_scan.p

[PATCH 2/5] dtoc: Improve handling of reg properties

2021-03-21 Thread Simon Glass
This existing code assumes that a reg property is larger than one cell, but this is not always the case. Fix this assumption. Also if a node's parent is missing the #address-cells and #size-cells properties we use 2 as a default for each. But this should not happen in practice. More likely the pro

[PATCH 0/5] misc: Various misc patches

2021-03-21 Thread Simon Glass
This series includes some minor tools updates and conversion of CONFIG_MISC_INIT_F to Kconfig. Simon Glass (5): dtoc: Show driver warnings once at the end dtoc: Improve handling of reg properties log: Fix up debug_cond() when LOG is enabled moveconfig: Handle binary files cleanly Conver

Re: [PATCH 02/16] drivers: reset: Handle gracefully NULL pointers

2021-03-21 Thread Kishon Vijay Abraham I
Hi Simon, On 12/03/21 10:15 am, Simon Glass wrote: > Hi Kishon, > > On Tue, 9 Mar 2021 at 05:27, Kishon Vijay Abraham I wrote: >> >> From: Jean-Jacques Hiblot >> >> Prepare the way for a managed reset API by handling NULL pointers without >> crashing nor failing. >> >> Signed-off-by: Jean-Jacqu

Re: [PATCH 01/16] dm: core: Add helper to compare node names

2021-03-21 Thread Kishon Vijay Abraham I
Hi Simon, On 12/03/21 10:15 am, Simon Glass wrote: > Hi Kishon, > > On Tue, 9 Mar 2021 at 05:27, Kishon Vijay Abraham I wrote: >> >> Add helper to compare node names. >> >> Signed-off-by: Kishon Vijay Abraham I >> --- >> drivers/core/ofnode.c | 13 + >> include/dm/ofnode.h | 9 +

[PATCH] patman: Continue on if warnings are found outside a commit

2021-03-21 Thread Simon Glass
While we cannot know which commit the warning relates to, this should not be fatal. Print the warning and carry on. Signed-off-by: Simon Glass --- tools/patman/patchstream.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/patman/patchstream.py b/tools/patman/patch

Re: [PATCH 1/2] riscv: simply longjmp

2021-03-21 Thread Sean Anderson
On 3/21/21 6:19 AM, Heinrich Schuchardt wrote: The value returned by setjmp must be nonzero. If zero is passed as parameter it must be replaced by 1. This patch reduces the code size a bit. Signed-off-by: Heinrich Schuchardt --- arch/riscv/lib/setjmp.S | 8 ++-- 1 file changed, 2 insert

Re: [PATCH 1/2] mx23_olinuxino: enable device tree

2021-03-21 Thread Sean Anderson
On 3/21/21 9:34 PM, Trevor Woerner wrote: Add the dts file for the Olimex Olinuxino from the linux kernel, and enable its use in this machine's defconfig. Signed-off-by: Trevor Woerner --- arch/arm/dts/Makefile| 3 + arch/arm/dts/imx23-olinuxino.dts | 131 ++

RE: [PATCH] imx8mn_ddr4_evk: Allow booting the kernel by default

2021-03-21 Thread Peng Fan
> Subject: [PATCH] imx8mn_ddr4_evk: Allow booting the kernel by default > > Currently it is not possible to boot Linux from the SD card by default. > > Make the necessary adjustments to allow it. > > Reported-by: Peter Bergin > Signed-off-by: Fabio Estevam > --- > configs/imx8mn_ddr4_evk_defc

Re: [ANN] U-Boot v2021.04-rc4 released

2021-03-21 Thread Trevor Woerner
On Sun 2021-03-21 @ 09:28:21 PM, Tom Rini wrote: > On Sun, Mar 21, 2021 at 09:09:46PM -0400, Trevor Woerner wrote: > > On Wed 2021-03-17 @ 10:37:18 AM, Fabio Estevam wrote: > > > On Wed, Mar 17, 2021 at 10:22 AM Trevor Woerner > > > wrote: > > > > However, I am interested in preserving the olimex

[PATCH 2/2] mx23_olinuxino: convert MMC to driver model

2021-03-21 Thread Trevor Woerner
Convert the Olimex Olinuxino board's support for MMC to driver model following Fabio Estevam's excellent example from: commit: 23013aa9619881290dbeb6217f1fab863869050e: mx23evk: Convert to driver model Signed-off-by: Trevor Woerner --- arch/arm/dts/imx23-olinuxino.dts

[PATCH 1/2] mx23_olinuxino: enable device tree

2021-03-21 Thread Trevor Woerner
Add the dts file for the Olimex Olinuxino from the linux kernel, and enable its use in this machine's defconfig. Signed-off-by: Trevor Woerner --- arch/arm/dts/Makefile| 3 + arch/arm/dts/imx23-olinuxino.dts | 131 +++ configs/mx23_olinuxino_defconfig |

Re: [ANN] U-Boot v2021.04-rc4 released

2021-03-21 Thread Tom Rini
On Sun, Mar 21, 2021 at 09:09:46PM -0400, Trevor Woerner wrote: > On Wed 2021-03-17 @ 10:37:18 AM, Fabio Estevam wrote: > > On Wed, Mar 17, 2021 at 10:22 AM Trevor Woerner wrote: > > > However, I am interested in preserving the olimex board (i.e. > > > configs/mx23_olinuxino_defconfig). So I could

Re: [PATCH 1/2] of: addr: Abort address translation for parent nodes missing 'ranges'

2021-03-21 Thread Bin Meng
Hi Dario, On Sun, Mar 21, 2021 at 11:19 PM Dario Binacchi wrote: > > Hi Tom, > > > Il 18/03/2021 20:51 Tom Rini ha scritto: > > > > > > On Thu, Mar 18, 2021 at 08:27:49AM +0100, Dario Binacchi wrote: > > > Hi Bin, > > > > > > > Il 17/03/2021 02:26 Bin Meng ha scritto: > > > > > > > > > > > > Hi

Re: [PATCH v4 2/2] x86: coral: Show memory config and SKU ID on startup

2021-03-21 Thread Bin Meng
Hi Simon, On Sun, Mar 21, 2021 at 11:51 AM Simon Glass wrote: > > Hi Bin, > > On Mon, 15 Mar 2021 at 18:30, Simon Glass wrote: > > > > Hi Bin, > > > > On Tue, 9 Mar 2021 at 15:34, Bin Meng wrote: > > > > > > Hi Simon, > > > > > > On Sun, Feb 7, 2021 at 12:13 AM Simon Glass wrote: > > > > > > >

Re: [ANN] U-Boot v2021.04-rc4 released

2021-03-21 Thread Trevor Woerner
On Wed 2021-03-17 @ 10:37:18 AM, Fabio Estevam wrote: > On Wed, Mar 17, 2021 at 10:22 AM Trevor Woerner wrote: > > However, I am interested in preserving the olimex board (i.e. > > configs/mx23_olinuxino_defconfig). So I could take a look at that one first > > unless, Marek, you would like to take

Re: [PATCH 14/26] power: pca9450: add a new parameter for power_pca9450_init

2021-03-21 Thread Jaehoon Chung
On 3/19/21 4:57 PM, Peng Fan (OSS) wrote: > From: Peng Fan > > Currently PCA9450 might have address 0x25 or 0x35, so let user > choose the address. > > Signed-off-by: Peng Fan Reviewed-by: Jaehoon Chung Best Regards, Jaehoon Chung > --- > board/freescale/imx8mp_evk/spl.c | 2 +- > board/p

Re: [PATCH] azure: Use --board flag with sandbox_spl

2021-03-21 Thread Tom Rini
On Mon, Mar 22, 2021 at 08:22:53AM +1300, Simon Glass wrote: > At present there is only one board which uses sandbox SPL. But with > sandbox_noinst being added, this is no longer true. Add a --board flag > so that we just build one board on azure, as is done in gitlab. > > Signed-off-by: Simon Gl

[PATCH] azure: Use --board flag with sandbox_spl

2021-03-21 Thread Simon Glass
At present there is only one board which uses sandbox SPL. But with sandbox_noinst being added, this is no longer true. Add a --board flag so that we just build one board on azure, as is done in gitlab. Signed-off-by: Simon Glass --- .azure-pipelines.yml | 2 +- 1 file changed, 1 insertion(+),

Re: [PATCH 1/6] usb: Return -ENOSYS when system call is not available

2021-03-21 Thread Marek Vasut
On 3/21/21 5:12 AM, Simon Glass wrote: Hi Marek, On Sun, 21 Mar 2021 at 17:09, Marek Vasut wrote: On 3/21/21 4:18 AM, Simon Glass wrote: Update usb_gadget_release() to use -ENOSYS, which is the correct error code for U-Boot. It would be good if you could explain _why_ this change is requir

Re: [PATCH 0/6] Use -ENOSYS consistently

2021-03-21 Thread Tom Rini
On Sun, Mar 21, 2021 at 04:18:18PM +1300, Simon Glass wrote: > A few places use -ENOTSUPP when they should use -ENOSYS. In two cases both > are used. This little series tidies this up. > > > Simon Glass (6): > usb: Return -ENOSYS when system call is not available > spi: Return -ENOSYS when s

Re: [PATCH 1/2] of: addr: Abort address translation for parent nodes missing 'ranges'

2021-03-21 Thread Dario Binacchi
Hi Tom, > Il 18/03/2021 20:51 Tom Rini ha scritto: > > > On Thu, Mar 18, 2021 at 08:27:49AM +0100, Dario Binacchi wrote: > > Hi Bin, > > > > > Il 17/03/2021 02:26 Bin Meng ha scritto: > > > > > > > > > Hi Dario, > > > > > > On Wed, Mar 17, 2021 at 4:57 AM Dario Binacchi wrote: > > > > >

Re: Please pull u-boot-dm/next take 2

2021-03-21 Thread Tom Rini
On Sun, Mar 21, 2021 at 03:36:41PM +1300, Simon Glass wrote: > Hi Tom, > > This is for the -next branch > > https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/6849 > > > The following changes since commit f879f2621b42529cf5f88912a07d2866cfc8fd45: > > Merge branch 'next' of git:/

Re: Help needed with uboot for CHIP

2021-03-21 Thread Gunjan Gupta
I figured out the why SPL was not loading U-Boot from NAND. Apparently the logic to use PIO instead of DMA that was introduced in 6ddbb1e936c78cdef1e7395039fa7020c5c75326 may be was not as generic as it was supposed

[PATCH 0/2] riscv: simply longjmp()

2021-03-21 Thread Heinrich Schuchardt
The implementation of longjmp() is simplified. A unit test for longjmp() is provided. For testing use CONFIG_UNIT_TEST=y CONFIG_CMD_SETEXPR=n and execute ut lib Heinrich Schuchardt (2): riscv: simply longjmp test: unit test for longjmp arch/riscv/lib/setjmp.S | 8 ++-- t

[PATCH 1/2] riscv: simply longjmp

2021-03-21 Thread Heinrich Schuchardt
The value returned by setjmp must be nonzero. If zero is passed as parameter it must be replaced by 1. This patch reduces the code size a bit. Signed-off-by: Heinrich Schuchardt --- arch/riscv/lib/setjmp.S | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/arch/riscv/li

[PATCH 2/2] test: unit test for longjmp

2021-03-21 Thread Heinrich Schuchardt
Provide a unit test for the longjmp() library function Signed-off-by: Heinrich Schuchardt --- test/lib/Makefile | 1 + test/lib/longjmp.c | 44 2 files changed, 45 insertions(+) create mode 100644 test/lib/longjmp.c diff --git a/test/lib/Makefile