Re: [PATCH v5 15/29] acpi: Add a simple sandbox test

2020-04-15 Thread Bin Meng
On Thu, Apr 9, 2020 at 6:58 AM Simon Glass wrote: > > Add a sandbox test for the basic ACPI functionality we have so far. > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > Reviewed-by: Wolfgang Wallner > --- > > Changes in v5: > - Capitalise ACPI_OPS_PTR since it includes a comma > >

Re: [PATCH v6 09/12] arm: mvebu: enable working default boot support

2020-04-15 Thread Joel Johnson
On 2020-04-14 03:33, Stefan Roese wrote: Hi Joel, On 23.03.20 21:21, Joel Johnson wrote: With the move to driver model usage, ensure that the required driver support for SPI and MMC booting is available in SPL. Tested on SolidRun ClearFog devices. Signed-off-by: Joel Johnson Reviewed-by:

Re: [PATCH v5 14/29] acpi: Add a binding for ACPI settings in the device tree

2020-04-15 Thread Bin Meng
Hi Simon, On Thu, Apr 9, 2020 at 6:58 AM Simon Glass wrote: > > Devices need to report various identifiers in the ACPI tables. Rather than > hard-coding these in drivers it is typically better to put them in the > device tree. > > Add a binding file to describe this. > > Signed-off-by: Simon

Re: [PATCH v6 12/29] dm: core: Add basic ACPI support

2020-04-15 Thread Bin Meng
On Fri, Apr 10, 2020 at 12:28 AM Simon Glass wrote: > > ACPI (Advanced Configuration and Power Interface) is a standard for > specifying information about a platform. It is a little like device > tree but the bindings are part of the specification and it supports an > interpreted bytecode

Re: [PATCH v5 13/29] dts: Add a binding for hid-over-i2c

2020-04-15 Thread Bin Meng
On Thu, Apr 16, 2020 at 1:40 PM Bin Meng wrote: > > On Thu, Apr 9, 2020 at 6:58 AM Simon Glass wrote: > > > > Add this binding from Linux v5.4. > > > > Signed-off-by: Simon Glass > > --- > > > > Changes in v5: None > > Changes in v4: None > > Changes in v3: > > - Split out hid-over-i2c into its

Re: [PATCH v5 11/29] x86: apl: Add Global NVS table header

2020-04-15 Thread Bin Meng
On Thu, Apr 9, 2020 at 6:58 AM Simon Glass wrote: > > Add the C version of this header. It includes a few Chrome OS bits which > are disabled for a normal build. > > Signed-off-by: Simon Glass > Reviewed-by: Wolfgang Wallner > --- > > Changes in v5: > - Tweak unused naming and comments > >

Re: [PATCH v5 13/29] dts: Add a binding for hid-over-i2c

2020-04-15 Thread Bin Meng
On Thu, Apr 9, 2020 at 6:58 AM Simon Glass wrote: > > Add this binding from Linux v5.4. > > Signed-off-by: Simon Glass > --- > > Changes in v5: None > Changes in v4: None > Changes in v3: > - Split out hid-over-i2c into its own patch > > Changes in v2: None > > .../input/hid-over-i2c.txt

Re: [PATCH v6 10/29] pci: Adjust dm_pci_read_bar32() to return errors correctly

2020-04-15 Thread Bin Meng
On Fri, Apr 10, 2020 at 12:28 AM Simon Glass wrote: > > At present if reading a BAR returns 0x then the value is masked > and a different value is returned. This makes it harder to detect the > problem when debugging. > > Update the function to avoid masking in this case. > >

Re: [PATCH v5 09/29] x86: apl: Move p2sb ofdata reading to the correct method

2020-04-15 Thread Bin Meng
On Thu, Apr 9, 2020 at 6:58 AM Simon Glass wrote: > > With P2SB the initial BAR (base-address register) is set up by TPL and > this is used unchanged right through U-Boot. > > At present the reading of this address is split between the ofdata() and > probe() methods. There are a few problems that

Re: [PATCH v5 06/29] tpm: Don't cleanup unless an error happens

2020-04-15 Thread Bin Meng
On Thu, Apr 9, 2020 at 6:58 AM Simon Glass wrote: > > At present the cleanup() method is called on every transfer. It should > only be called on failing transfers. Fix this and tidy up the error > handling a little. > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > --- > > Changes in v5:

Re: [PATCH v5 05/29] tpm: cr50: Use the correct GPIO binding

2020-04-15 Thread Bin Meng
On Thu, Apr 9, 2020 at 6:58 AM Simon Glass wrote: > > This device should use ready-gpios rather than ready-gpio. Fix it. > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > --- > > Changes in v5: None > Changes in v4: None > Changes in v3: None > Changes in v2: None > >

Re: [PATCH v5 08/29] x86: Correct wording of coreboot source code

2020-04-15 Thread Bin Meng
On Thu, Apr 9, 2020 at 6:58 AM Simon Glass wrote: > > Some files are taken or modified from coreboot, but the files are > no-longer part of the coreboot project. Fix the wording in a few places. > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > Reviewed-by: Wolfgang Wallner > --- > >

Re: [PATCH v5 07/29] dm: pci: Allow disabling auto-config for a device

2020-04-15 Thread Bin Meng
On Thu, Apr 9, 2020 at 6:58 AM Simon Glass wrote: > > Add a means to avoid configuring a device when needed. Add an explanation > of why this is useful to the binding file. > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > --- > > Changes in v5: None > Changes in v4: None > Changes in

Re: [PATCH v5 04/29] tpm: cr50: Add a comment for cr50_priv

2020-04-15 Thread Bin Meng
On Thu, Apr 9, 2020 at 6:58 AM Simon Glass wrote: > > Add a comment for the private structure > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > --- > > Changes in v5: None > Changes in v4: None > Changes in v3: None > Changes in v2: > - Drop the other comment change since it is already

Re: [PATCH v5 01/29] cpu: Support querying the address width

2020-04-15 Thread Bin Meng
On Thu, Apr 9, 2020 at 6:57 AM Simon Glass wrote: > > Different CPUs may support different address widths, meaning the amount of > memory they can address. Add a property for this to the cpu_info struct. > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > --- > > Changes in v5: None >

Re: [PATCH v5 03/29] tpm: cr50: Release locality on exit

2020-04-15 Thread Bin Meng
On Thu, Apr 9, 2020 at 6:58 AM Simon Glass wrote: > > At present the cr50 driver claims the locality and does not release it for > Linux. This causes problems. Fix this by tracking what is claimed, and > adding a 'remove' method. > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > --- > >

Re: [PATCH v5 02/29] spi: Add SPI mode enums

2020-04-15 Thread Bin Meng
On Thu, Apr 9, 2020 at 6:57 AM Simon Glass wrote: > > With ACPI we need to describe the settings of the SPI bus. Add enums to > handle this. > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > Reviewed-by: Wolfgang Wallner > --- > > Changes in v5: None > Changes in v4: None > Changes in

[PATCH] libfdt: migrate fdt_ro.c to a wrapper of scripts/dtc/libfdt/fdt_ro.c

2020-04-15 Thread Masahiro Yamada
There is no essential difference between scripts/dtc/libfdt/fdt_ro.c and lib/libfdt/fdt_ro.c Migrate to a simple wrapper like the other files. Signed-off-by: Masahiro Yamada --- lib/libfdt/Makefile | 5 +- lib/libfdt/fdt_ro.c | 925 +--- 2 files

[PATCH 0/2] fixdep: fix a bug for TPL, and re-sync with Linux

2020-04-15 Thread Masahiro Yamada
Masahiro Yamada (2): fixdep: handle CONFIG_IS_ENABLE() and friends for TPL fixdep: Re-sync with Linux 5.7-rc1 scripts/basic/fixdep.c | 104 + 1 file changed, 54 insertions(+), 50 deletions(-) -- 2.25.1

[PATCH 2/2] fixdep: Re-sync with Linux 5.7-rc1

2020-04-15 Thread Masahiro Yamada
fixdep is a standalone host program, so we can just re-sync it with the latest Linux in one commit. I kept the U-Boot own code block surrounded by /* hack for U-Boot */ ... /* U-boot hack end */. Signed-off-by: Masahiro Yamada --- scripts/basic/fixdep.c | 92

[PATCH 1/2] fixdep: handle CONFIG_IS_ENABLE() and friends for TPL

2020-04-15 Thread Masahiro Yamada
Since commit f1c6e1922eb5 ("spl: dm: use CONFIG_IS_ENABLED to test for the DM option"), CONFIG_IS_ENABLED() handles CONFIG_TPL_* options, but fixdep still cannot because it hard-codes the "SPL_" prefix as follows: char tmp_buf[256] = "SPL_"; /* hack for U-Boot */ Take care of the "TPL_"

[PATCH 0/3] support separate asm-offsets.h for SPL and TPL

2020-04-15 Thread Masahiro Yamada
Bin Meng asked me how to generate correct asm-offsets headers for SPL/TPL. 1/3: fix a bug that I just happened to find when touching this line 2/3: cherry-pick some Kbuild changes. This is a good resync regardless of 3/3 3/3: change as Bin Meng requested. This complicates the build process a

[PATCH 2/3] kbuild: cherry-pick kbuild changes from Linux

2020-04-15 Thread Masahiro Yamada
Import the following commits from Linux. They are mostly nice cleanups. b42841b7bb62 kbuild: Get rid of KBUILD_STR ebf003f0cfb3 kbuild: Consolidate header generation from ASM offset information 2982c953570b kbuild: remove redundant $(wildcard ...) for cmd_files calculation 8a78756eb545 kbuild:

[PATCH 1/3] kbuild: add FORCE to dependency of $(obj)/dts/dt-platdata.o

2020-04-15 Thread Masahiro Yamada
if_changed must have FORCE as a prerequisite. Add $(obj)/dts/dt-platdata.o to 'targets' so that the corresponding .cmd file is included. Signed-off-by: Masahiro Yamada --- scripts/Makefile.spl | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/scripts/Makefile.spl

[PATCH 3/3] kbuild: SPL/TPL: generate separate asm-offsets.h for SPL and TPL

2020-04-15 Thread Masahiro Yamada
Currently generic-asm-offsets.h and asm-offsets.h are generated based on U-Boot proper config options. The same asm-offsets headers are used for building U-Boot SPL/TPL, which causes potential offset mismatch if U-Boot proper has different config options from U-Boot SPL/TPL. This commit adds:

Re: [PATCH 1/2] arm: mvebu: clearfog: add SCSI to distro bootcmd

2020-04-15 Thread Joel Johnson
On 2020-03-23 04:20, Stefan Roese wrote: Added Josua to Cc. On 29.01.20 04:59, Joel Johnson wrote: Include attempting to boot from SCSI (SATA) devices within generated board distro bootcmd environment. The reasoning for boot ordering is that MMC and USB are external and removable, while when a

Re: [PATCH] rockchip: rk3328: add rock-pi-e-rk3328_defconfig

2020-04-15 Thread b.l.huang
>Does this board dts also upstream to kernel? If yes, please split the >dts as a separate patch > >and add the commit info in the message. > Temporarily not for upstream to kernel. > >On 2020/4/13 PM 4:23, banglang.huang wrote: >> The ROCK-PI-E is a credit card size SBC based on Rockchip RK3328

Re: [PATCH] rockchip: rk3328: add rock-pi-e-rk3328_defconfig

2020-04-15 Thread b.l.huang
Does this board dts also upstream to kernel? If yes, please split the dts as a separate patch and add the commit info in the message. Temporarily not for upstream to kernel. On 2020/4/13 PM 4:23, banglang.huang wrote: The ROCK-PI-E is a credit card size SBC based on Rockchip RK3328

Re: [PATCH v2 00/19] i.MXRT1050 add LCDIF support

2020-04-15 Thread Simon Glass
Hi Giulio, On Wed, 15 Apr 2020 at 14:05, Giulio Benetti wrote: > > I've pinged the wrong patchset, sorry for the noise, the right one is v3. > > -- > Giulio Benetti > Benetti Engineering sas > > On 4/15/20 9:51 PM, Giulio Benetti wrote: > > Kindly ping > > > Who are you pinging? Who is the

[PATCH] travis: Correct error checking when building boards

2020-04-15 Thread Simon Glass
At present if buildman reports an error, the travis build still succeeds. This is because the travis script does not stop when it sees errors; nor does it automatically return the exit code. Also the current error checking never triggers since 'ret' is not set. Fix this by setting 'ret'

Re: [PATCH 4/5] lib: do not provide hexdump in SPL

2020-04-15 Thread Tom Rini
On Wed, Apr 15, 2020 at 06:46:22PM +0200, Heinrich Schuchardt wrote: > SPL should not be enlarged by building with CONFIG_HEXDUMP=y. > > Signed-off-by: Heinrich Schuchardt > --- > lib/hexdump.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Does this show up as a problem somewhere?

Re: [PATCH 5/5] dlmalloc: remove unit test support in SPL

2020-04-15 Thread Tom Rini
On Wed, Apr 15, 2020 at 06:46:23PM +0200, Heinrich Schuchardt wrote: > We cannot run unit tests in SPL. So remove the unit test support. > > Signed-off-by: Heinrich Schuchardt > --- > common/dlmalloc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Does this show up as a problem

Re: [PATCH 3/5] common: image_sign_info helper functions in SPL

2020-04-15 Thread Tom Rini
On Wed, Apr 15, 2020 at 06:46:21PM +0200, Heinrich Schuchardt wrote: > Do not build image_sign_info helper functions in SPL if not needed. > > Fixes: b983cc2da0ba ("lib: rsa: decouple rsa from FIT image verification") > Signed-off-by: Heinrich Schuchardt Reviewed-by: Tom Rini -- Tom

Re: [PATCH 2/5] lib: date functions in SPL

2020-04-15 Thread Tom Rini
On Wed, Apr 15, 2020 at 06:46:20PM +0200, Heinrich Schuchardt wrote: > Date functions are only needed in SPL if SPL contains RTC drivers. > > Fixes: 05429b6cf5b3 ("rtc: move date.c from drivers/rtc/ to lib/") > Signed-off-by: Heinrich Schuchardt Does this show up as a problem somewhere? I

Re: [PATCH] drivers: mmc: rpmb: do not build for SPL

2020-04-15 Thread Tom Rini
On Wed, Apr 15, 2020 at 06:28:09PM +0200, Heinrich Schuchardt wrote: > RPMB support is used by the 'mmc rpmb' command and by the OP-TEE support. > We do not need it in SPL. > > Signed-off-by: Heinrich Schuchardt Reviewed-by: Tom Rini -- Tom signature.asc Description: PGP signature

Re: [PATCH 1/5] lib: do not build OID registry in SPL

2020-04-15 Thread Tom Rini
On Wed, Apr 15, 2020 at 06:46:19PM +0200, Heinrich Schuchardt wrote: > The OID registry is only used by crypto functions that are not built in > SPL. So we should not build it in SPL. > > Fixes: a9b45e6e8382 ("lib: add oid registry utility") > Signed-off-by: Heinrich Schuchardt > --- >

RE: [PATCH] armv8: ls1046ardb: add new 16GB udimm memory support

2020-04-15 Thread Andy Tang
+Mazyar, Hi Mazyar, Could you please address Priyanka's concern? BR, Andy > -Original Message- > From: Priyanka Jain (OSS) > Sent: 2020年4月15日 18:07 > To: Andy Tang > Cc: Mingkai Hu ; Rajesh Bhagat > ; u-boot@lists.denx.de; Andy Tang > > Subject: RE: [PATCH] armv8: ls1046ardb: add new

RE: [PATCH 3/7] ddr: altera: arria10: Change to use reset DM function

2020-04-15 Thread Tan, Ley Foon
> -Original Message- > From: Marek Vasut > Sent: Wednesday, April 15, 2020 8:43 PM > To: Tan, Ley Foon ; u-boot@lists.denx.de > Cc: Ley Foon Tan ; See, Chin Liang > ; Simon Goldschmidt > ; Chee, Tien Fong > > Subject: Re: [PATCH 3/7] ddr: altera: arria10: Change to use reset DM >

RE: [PATCH 2/7] ddr: altera: arria10: Move SDRAM driver to DM

2020-04-15 Thread Tan, Ley Foon
> -Original Message- > From: Marek Vasut > Sent: Wednesday, April 15, 2020 8:42 PM > To: Tan, Ley Foon ; u-boot@lists.denx.de > Cc: Ley Foon Tan ; See, Chin Liang > ; Simon Goldschmidt > ; Chee, Tien Fong > > Subject: Re: [PATCH 2/7] ddr: altera: arria10: Move SDRAM driver to DM > >

RE: [PATCH 5/7] ddr: altera: arria10: Add RAM size check

2020-04-15 Thread Tan, Ley Foon
> -Original Message- > From: Marek Vasut > Sent: Wednesday, April 15, 2020 8:44 PM > To: Tan, Ley Foon ; u-boot@lists.denx.de > Cc: Ley Foon Tan ; See, Chin Liang > ; Simon Goldschmidt > ; Chee, Tien Fong > > Subject: Re: [PATCH 5/7] ddr: altera: arria10: Add RAM size check > > On

Re: [PATCH v3 00/19] i.MXRT1050 add LCDIF support

2020-04-15 Thread Fabio Estevam
Hi Giulio, On Wed, Apr 8, 2020 at 12:10 PM Giulio Benetti wrote: > > This patchset add support for LCDIF on i.MXRT1050 evk. This requires > PLL5 to be setup, mxsfb needs to use display_timing to retrieve if Lcd > has inverted PIXCLOCK from dts. > > With this patchset applied we temporary loose

RE: [PATCH 1/7] ddr: altera: arria10: Fix incorrect address for mpu1

2020-04-15 Thread Tan, Ley Foon
> -Original Message- > From: Marek Vasut > Sent: Wednesday, April 15, 2020 8:38 PM > To: Tan, Ley Foon ; u-boot@lists.denx.de > Cc: Ley Foon Tan ; See, Chin Liang > ; Simon Goldschmidt > ; Chee, Tien Fong > > Subject: Re: [PATCH 1/7] ddr: altera: arria10: Fix incorrect address for mpu1 >

RE: [PATCH] imx8qxp_mek: Start with USDHC2 unpowered

2020-04-15 Thread Peng Fan
> Subject: [PATCH] imx8qxp_mek: Start with USDHC2 unpowered > > After triggering a "reboot" command in Linux the following hang in SPL is > observed: > > U-Boot SPL 2020.04 (Apr 15 2020 - 10:49:29 -0300) Normal Boot > WDT: Not found! > Trying to boot from MMC2_2 > spl: mmc init failed with

RE: [PATCH] drivers: mmc: rpmb: do not build for SPL

2020-04-15 Thread Peng Fan
> Subject: [PATCH] drivers: mmc: rpmb: do not build for SPL > > RPMB support is used by the 'mmc rpmb' command and by the OP-TEE > support. > We do not need it in SPL. > > Signed-off-by: Heinrich Schuchardt Reviewed-by: Peng Fan > --- > drivers/mmc/Makefile | 2 +- > 1 file changed, 1

Re: [PATCH] tiny-printf: Support %i

2020-04-15 Thread Marek Vasut
On 4/15/20 1:01 AM, Tom Rini wrote: > On Tue, Apr 14, 2020 at 05:22:19PM +0200, Marek Vasut wrote: >> On 4/14/20 4:11 PM, Tom Rini wrote: >>> On Tue, Apr 14, 2020 at 03:40:14PM +0200, Marek Vasut wrote: On 4/14/20 3:26 PM, Tom Rini wrote: > On Tue, Apr 14, 2020 at 02:24:18PM +0200, Marek

Re: [PATCH] tiny-printf: Support %i

2020-04-15 Thread Marek Vasut
On 4/15/20 2:26 AM, Simon Glass wrote: > Hi Marek, Hi, > On Tue, 14 Apr 2020 at 10:32, Marek Vasut wrote: >> >> On 4/14/20 4:41 PM, Simon Glass wrote: >>> Hi Marek, >>> >>> On Tue, 14 Apr 2020 at 07:41, Marek Vasut wrote: On 4/14/20 3:26 PM, Tom Rini wrote: > On Tue, Apr 14, 2020

Re: [PATCH 6/7] ddr: altera: arria10: Change %i to %u for printf

2020-04-15 Thread Marek Vasut
On 4/15/20 7:44 PM, Tom Rini wrote: > On Wed, Apr 15, 2020 at 05:16:52PM +0200, Marek Vasut wrote: >> On 4/15/20 5:14 PM, Tom Rini wrote: >>> On Wed, Apr 15, 2020 at 04:58:31PM +0200, Marek Vasut wrote: On 4/15/20 4:56 PM, Tom Rini wrote: > On Wed, Apr 15, 2020 at 02:45:08PM +0200, Marek

[PATCH 1/1] drivers: crypto: rsa_mod_exp: avoid DM_FLAG_PRE_RELOC

2020-04-15 Thread Heinrich Schuchardt
Function rsa_verify_key() is not called before relocation. So there is no need to load the UCLASS_MOD_EXP drivers before relocation. This avoid a failure to boot for pine64-lts_defconfig with CONFIG_RSA=y. Signed-off-by: Heinrich Schuchardt --- drivers/crypto/fsl/fsl_rsa.c| 1 -

Re: dm: crash for DM_FLAG_PRE_RELOC

2020-04-15 Thread Heinrich Schuchardt
On 4/15/20 9:22 PM, Simon Glass wrote: > Hi Heinrich, > > On Wed, 15 Apr 2020 at 12:33, Fabio Estevam wrote: >> >> Changing Ruchika's address to the NXP one. >> >> On Wed, Apr 15, 2020 at 3:28 PM Heinrich Schuchardt >> wrote: >>> >>> Hello Simon, >>> >>> I try to boot pine64-lts_defconfig +

Re: [PATCH v3 00/19] i.MXRT1050 add LCDIF support

2020-04-15 Thread Giulio Benetti
Kindly ping -- Giulio Benetti Benetti Engineering sas On 4/8/20 5:10 PM, Giulio Benetti wrote: This patchset add support for LCDIF on i.MXRT1050 evk. This requires PLL5 to be setup, mxsfb needs to use display_timing to retrieve if Lcd has inverted PIXCLOCK from dts. With this patchset applied

Re: [PATCH v2 00/19] i.MXRT1050 add LCDIF support

2020-04-15 Thread Giulio Benetti
I've pinged the wrong patchset, sorry for the noise, the right one is v3. -- Giulio Benetti Benetti Engineering sas On 4/15/20 9:51 PM, Giulio Benetti wrote: Kindly ping

Re: [PATCH v2 1/3] environment: distro: Add SF distro command

2020-04-15 Thread Tom Rini
On Fri, Apr 10, 2020 at 11:56:29PM +0530, Jagan Teki wrote: > Add distro boot command support for SPI flash (SF). > > This distro boot will read the boot script at specific > location at the flash and start sourcing the same. > > This file need to include on required include/config > file. > >

Re: [PATCH v2 1/2] arm: dts: bcm283x: Allow UARTs to work before relocation

2020-04-15 Thread Tom Rini
On Tue, Apr 14, 2020 at 08:23:10PM -0600, Simon Glass wrote: > Hi, > > On Sun, 22 Mar 2020 at 21:16, Simon Glass wrote: > > > > At present the pinctrl nodes are not enabled in pre-relocation U-Boot so > > the UARTs do not correctly select the pinconfig to enable the UART pins. > > Fix this so

Re: [PULL] Pull request: u-boot-stm/next =u-boot-stm32-20200415

2020-04-15 Thread Tom Rini
; The following changes since commit 142a07f2a44262d76fec609e1fcde51794a456eb: > >   Merge branch 'master' of git://git.denx.de/u-boot-marvell (2020-04-14 > 08:47:07 -0400) > > are available in the Git repository at: > >   https://gitlab.denx.de/u-boot/custodians/u-boot-stm >

Re: [PATCH v2 00/19] i.MXRT1050 add LCDIF support

2020-04-15 Thread Giulio Benetti
Kindly ping -- Giulio Benetti Benetti Engineering sas On 3/22/20 11:48 PM, Giulio Benetti wrote: This patchset passed travis: https://travis-ci.org/github/giuliobenetti/u-boot-imxrt/builds/665577905?utm_medium=notification_source=email And also: ./tools/buildman/buildman.py

Re: dm: crash for DM_FLAG_PRE_RELOC

2020-04-15 Thread Simon Glass
Hi Heinrich, On Wed, 15 Apr 2020 at 12:33, Fabio Estevam wrote: > > Changing Ruchika's address to the NXP one. > > On Wed, Apr 15, 2020 at 3:28 PM Heinrich Schuchardt > wrote: > > > > Hello Simon, > > > > I try to boot pine64-lts_defconfig + CONFIG_RSA=y. > > > > SPL loads BL31. BL31 starts

Re: [PATCH v4] dm: uclass: don't assign aliased seq numbers

2020-04-15 Thread Simon Glass
Hi Michael, On Wed, 15 Apr 2020 at 09:22, Michael Walle wrote: > > Hi Simon, > > Am 2020-04-15 15:56, schrieb Simon Glass: > > +Tom > > > > Hi, > > > > On Sat, 14 Mar 2020 at 14:33, wrote: > >> > >> On 03. 03. 20 8:47, Michael Walle wrote: > >> > If there are aliases for an uclass, set the base

[PATCH 1/1] .mailmap: map Ruchika Gupta's mail address

2020-04-15 Thread Heinrich Schuchardt
Freescale mail addresses are not valid anymore. Reported-by: Fabio Estevam Signed-off-by: Heinrich Schuchardt --- .mailmap | 1 + 1 file changed, 1 insertion(+) diff --git a/.mailmap b/.mailmap index 61802f97f6..8250015453 100644 --- a/.mailmap +++ b/.mailmap @@ -36,6 +36,7 @@ Prabhakar

Re: dm: crash for DM_FLAG_PRE_RELOC

2020-04-15 Thread Fabio Estevam
Changing Ruchika's address to the NXP one. On Wed, Apr 15, 2020 at 3:28 PM Heinrich Schuchardt wrote: > > Hello Simon, > > I try to boot pine64-lts_defconfig + CONFIG_RSA=y. > > SPL loads BL31. BL31 starts U-Boot. But before any output appears SPL is > restarted. So some crash must occur. > > If

dm: crash for DM_FLAG_PRE_RELOC

2020-04-15 Thread Heinrich Schuchardt
Hello Simon, I try to boot pine64-lts_defconfig + CONFIG_RSA=y. SPL loads BL31. BL31 starts U-Boot. But before any output appears SPL is restarted. So some crash must occur. If I remove the flag DM_FLAG_PRE_RELOC in drivers/crypto/rsa_mod_exp/mod_exp_sw.c, booting works fine. Assigning the

[PATCH v4] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Harald Seiler
Use DM_ETH instead of legacy networking. Add VIO as a fixed regulator to the relevant device-trees and augment the FEC node with properties for the reset GPIO. It should be noted that the relevant properties for the reset GPIO already exist in the PHY node (reset-gpios, reset-delay-us,

[PATCH] imx8qxp_mek: Start with USDHC2 unpowered

2020-04-15 Thread Fabio Estevam
After triggering a "reboot" command in Linux the following hang in SPL is observed: U-Boot SPL 2020.04 (Apr 15 2020 - 10:49:29 -0300) Normal Boot WDT: Not found! Trying to boot from MMC2_2 spl: mmc init failed with error: -70 SPL: failed to boot from all boot devices ### ERROR ### Please RESET

Re: [PATCH v3] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Marek Vasut
On 4/15/20 7:53 PM, Harald Seiler wrote: > Hello Marek, > > On Wed, 2020-04-15 at 19:02 +0200, Marek Vasut wrote: >> On 4/15/20 5:54 PM, Harald Seiler wrote: >>> Use DM_ETH instead of legacy networking. Add VIO as a fixed regulator >>> to the relevant device-trees and augment the FEC node with

Re: [PATCH] ARM: dts: add missing meson-gxl-s805x-libretech-ac-u-boot.dtsi file

2020-04-15 Thread Jerome Brunet
On Wed 15 Apr 2020 at 17:58, Neil Armstrong wrote: > The libretech-ac u-boot.dtsi file is missing to enabled DT nodes changes > to enable Video output on U-Boot. > > Fixes: 671b1db8f8 ("arm64: dts: meson-gx: vpu should be probed before > relocation") > Reported-by: Jerome Brunet >

Re: [PATCH v3] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Harald Seiler
Hello Marek, On Wed, 2020-04-15 at 19:02 +0200, Marek Vasut wrote: > On 4/15/20 5:54 PM, Harald Seiler wrote: > > Use DM_ETH instead of legacy networking. Add VIO as a fixed regulator > > to the relevant device-trees and augment the FEC node with properties > > for the reset GPIO. > > > > It

Re: [PATCH 6/7] ddr: altera: arria10: Change %i to %u for printf

2020-04-15 Thread Tom Rini
On Wed, Apr 15, 2020 at 05:16:52PM +0200, Marek Vasut wrote: > On 4/15/20 5:14 PM, Tom Rini wrote: > > On Wed, Apr 15, 2020 at 04:58:31PM +0200, Marek Vasut wrote: > >> On 4/15/20 4:56 PM, Tom Rini wrote: > >>> On Wed, Apr 15, 2020 at 02:45:08PM +0200, Marek Vasut wrote: > On 4/15/20 11:00

Re: [PATCH v3] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Fabio Estevam
Hi Harald, On Wed, Apr 15, 2020 at 2:32 PM Harald Seiler wrote: > Yes, you are right, I will change this in v4. Interestingly, it works > with both ACTIVE_LOW and ACTIVE_HIGH but removing the regulator entirely > breaks it. Seems a bit weird to me ... Due to historical reasons, the GPIO

Re: [PATCH v3] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Harald Seiler
Hello Fabio, On Wed, 2020-04-15 at 13:15 -0300, Fabio Estevam wrote: > Hi Harald, > > On Wed, Apr 15, 2020 at 12:54 PM Harald Seiler wrote: > > > +/ { > > + fec_vio: regulator-fec { > > + compatible = "regulator-fixed"; > > + > > + regulator-name = "fec-vio";

Re: [PATCH 6/7] ddr: altera: arria10: Change %i to %u for printf

2020-04-15 Thread Marek Vasut
On 4/15/20 5:14 PM, Tom Rini wrote: > On Wed, Apr 15, 2020 at 04:58:31PM +0200, Marek Vasut wrote: >> On 4/15/20 4:56 PM, Tom Rini wrote: >>> On Wed, Apr 15, 2020 at 02:45:08PM +0200, Marek Vasut wrote: On 4/15/20 11:00 AM, Ley Foon Tan wrote: > Tiny printf doesn't support %i, change to

Re: [PATCH] pwm: Add PWM driver for SiFive SoC

2020-04-15 Thread Heiko Schocher
Hello Yash Shah, Am 15.04.2020 um 08:32 schrieb Yash Shah: Adds a PWM driver for PWM chip present in SiFive's HiFive Unleashed SoC This driver is simple port of Linux pwm sifive driver. So please add more information from exatly which linux version this patch is from, see:

Re: [PATCH v3] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Marek Vasut
On 4/15/20 5:54 PM, Harald Seiler wrote: > Use DM_ETH instead of legacy networking. Add VIO as a fixed regulator > to the relevant device-trees and augment the FEC node with properties > for the reset GPIO. > > It should be noted that the relevant properties for the reset GPIO > already exist in

[PATCH 1/5] lib: do not build OID registry in SPL

2020-04-15 Thread Heinrich Schuchardt
The OID registry is only used by crypto functions that are not built in SPL. So we should not build it in SPL. Fixes: a9b45e6e8382 ("lib: add oid registry utility") Signed-off-by: Heinrich Schuchardt --- lib/Makefile | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH 3/5] common: image_sign_info helper functions in SPL

2020-04-15 Thread Heinrich Schuchardt
Do not build image_sign_info helper functions in SPL if not needed. Fixes: b983cc2da0ba ("lib: rsa: decouple rsa from FIT image verification") Signed-off-by: Heinrich Schuchardt --- Kconfig | 2 +- common/Kconfig | 11 +++ common/Makefile | 2 +- 3 files changed, 13

[PATCH 5/5] dlmalloc: remove unit test support in SPL

2020-04-15 Thread Heinrich Schuchardt
We cannot run unit tests in SPL. So remove the unit test support. Signed-off-by: Heinrich Schuchardt --- common/dlmalloc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/common/dlmalloc.c b/common/dlmalloc.c index dade68faf7..db5ab55ed3 100644 --- a/common/dlmalloc.c +++

[PATCH 4/5] lib: do not provide hexdump in SPL

2020-04-15 Thread Heinrich Schuchardt
SPL should not be enlarged by building with CONFIG_HEXDUMP=y. Signed-off-by: Heinrich Schuchardt --- lib/hexdump.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/hexdump.c b/lib/hexdump.c index bf14b5bdbd..a3f219a874 100644 --- a/lib/hexdump.c +++ b/lib/hexdump.c @@

[PATCH 0/5] lib: reduce SPL size

2020-04-15 Thread Heinrich Schuchardt
Several configuration options inject code into SPL even if it is not needed. Heinrich Schuchardt (5): lib: do not build OID registry in SPL lib: date functions in SPL common: image_sign_info helper functions in SPL lib: do not provide hexdump in SPL dlmalloc: remove unit test support in

[PATCH 2/5] lib: date functions in SPL

2020-04-15 Thread Heinrich Schuchardt
Date functions are only needed in SPL if SPL contains RTC drivers. Fixes: 05429b6cf5b3 ("rtc: move date.c from drivers/rtc/ to lib/") Signed-off-by: Heinrich Schuchardt --- lib/Makefile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/Makefile b/lib/Makefile index

Re: [PATCH v5 14/29] acpi: Add a binding for ACPI settings in the device tree

2020-04-15 Thread Andy Shevchenko
On Wed, Apr 15, 2020 at 7:16 PM Wolfgang Wallner wrote: > I have learned through the previous review discussions aspects about this > binding which are not captured in the current patch. I tried to incorporate > these findings, the modified text is proposed below. > > Additionally, I realized

[PATCH] drivers: mmc: rpmb: do not build for SPL

2020-04-15 Thread Heinrich Schuchardt
RPMB support is used by the 'mmc rpmb' command and by the OP-TEE support. We do not need it in SPL. Signed-off-by: Heinrich Schuchardt --- drivers/mmc/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mmc/Makefile b/drivers/mmc/Makefile index

Antwort: [PATCH v5 14/29] acpi: Add a binding for ACPI settings in the device tree

2020-04-15 Thread Wolfgang Wallner
Hi Simon, -"Simon Glass" schrieb: - >Betreff: [PATCH v5 14/29] acpi: Add a binding for ACPI settings in >the device tree > >Devices need to report various identifiers in the ACPI tables. Rather >than >hard-coding these in drivers it is typically better to put them in >the >device

Re: [PATCH v3] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Fabio Estevam
Hi Harald, On Wed, Apr 15, 2020 at 12:54 PM Harald Seiler wrote: > +/ { > + fec_vio: regulator-fec { > + compatible = "regulator-fixed"; > + > + regulator-name = "fec-vio"; > + gpio = < 7 GPIO_ACTIVE_HIGH>; By looking at your board code, this

Re: [PATCH 01/10] generic-phy: add generic_phy_get_by_node()

2020-04-15 Thread Neil Armstrong
Tom, Marek, On 30/03/2020 11:27, Neil Armstrong wrote: > Add generic_phy_get_by_node() to get a PHY phandle from a node instead > of a udevice. > > Signed-off-by: Neil Armstrong > --- > drivers/phy/phy-uclass.c | 17 - > include/generic-phy.h| 27 +++

Re: Please pull u-boot-marvell/master (watchdog related)

2020-04-15 Thread Tom Rini
On Wed, Apr 15, 2020 at 11:43:41AM +0200, Stefan Roese wrote: > Hi Tom, > > please pull the following watchdog related patches: > Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH] arm: meson: imply DM_RNG

2020-04-15 Thread Neil Armstrong
On 05/04/2020 12:20, Heinrich Schuchardt wrote: > All ARCH_MESON boards have a hardware random number generator. So we should > enable building the RNG driver. > > Signed-off-by: Heinrich Schuchardt > --- > arch/arm/Kconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git

Re: [PATCH 1/1] configs: correct SMBIOS information for ODROID-C2

2020-04-15 Thread Neil Armstrong
Hi, On 07/04/2020 08:32, Heinrich Schuchardt wrote: > CONFIG_SMBIOS_MANUFACTURER="amlogic" > CONFIG_SMBIOS_PRODUCT_NAME="p200" > > is incorrect for the ODROID-C2. > > Show correct board name and manufacturer in SMBIOS. > > Signed-off-by: Heinrich Schuchardt > --- >

Dummy device tree for verified boot

2020-04-15 Thread Gamble, Bradley
Hello, I am currently attempting to utilize Verified Boot to verify FIT images on my T104x-based device. I am building U-Boot with the standard options for Verified Boot (CONFIG_FIT, CONFIG_FIT_SIGNATURE, CONFIG_RSA, CONFIG_OF_CONTROL and CONFIG_OF_SEPARATE). When building U-Boot I need to

[PATCH] ARM: dts: add missing meson-gxl-s805x-libretech-ac-u-boot.dtsi file

2020-04-15 Thread Neil Armstrong
The libretech-ac u-boot.dtsi file is missing to enabled DT nodes changes to enable Video output on U-Boot. Fixes: 671b1db8f8 ("arm64: dts: meson-gx: vpu should be probed before relocation") Reported-by: Jerome Brunet Signed-off-by: Neil Armstrong ---

[PATCH v3] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Harald Seiler
Use DM_ETH instead of legacy networking. Add VIO as a fixed regulator to the relevant device-trees and augment the FEC node with properties for the reset GPIO. It should be noted that the relevant properties for the reset GPIO already exist in the PHY node but U-Boot currently ignores those and

Re: [PATCH v4] dm: uclass: don't assign aliased seq numbers

2020-04-15 Thread Michael Walle
Hi Simon, Am 2020-04-15 15:56, schrieb Simon Glass: +Tom Hi, On Sat, 14 Mar 2020 at 14:33, wrote: On 03. 03. 20 8:47, Michael Walle wrote: > If there are aliases for an uclass, set the base for the "dynamically" > allocated numbers next to the highest alias. > > Please note, that this

Re: [PATCH v2] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Marek Vasut
On 4/15/20 5:17 PM, Harald Seiler wrote: > Hello Marek, > > On Wed, 2020-04-15 at 16:53 +0200, Marek Vasut wrote: >> On 4/15/20 4:48 PM, Harald Seiler wrote: >>> Use DM_ETH instead of legacy networking. >> >> Some more descriptive commit message would help. >> >> [...] >> >>> diff --git

Re: [PATCH v2] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Harald Seiler
Hello Marek, On Wed, 2020-04-15 at 16:53 +0200, Marek Vasut wrote: > On 4/15/20 4:48 PM, Harald Seiler wrote: > > Use DM_ETH instead of legacy networking. > > Some more descriptive commit message would help. > > [...] > > > diff --git a/arch/arm/dts/imx6qdl-dhcom-u-boot.dtsi > >

Re: Antwort: Re: Re: Re: [PATCH v3 13/29] dts: Add a binding for hid-over-i2c

2020-04-15 Thread Andy Shevchenko
On Wed, Apr 15, 2020 at 06:15:39PM +0300, Andy Shevchenko wrote: > On Wed, Apr 15, 2020 at 04:57:33PM +0200, Wolfgang Wallner wrote: > > > On Wed, Apr 15, 2020 at 5:00 PM Wolfgang Wallner > > > wrote: > > > > -"Andy Shevchenko" schrieb: - > > > > > Betreff: Re: Re: [PATCH v3 13/29] dts:

Re: Antwort: Re: Re: Re: [PATCH v3 13/29] dts: Add a binding for hid-over-i2c

2020-04-15 Thread Andy Shevchenko
On Wed, Apr 15, 2020 at 04:57:33PM +0200, Wolfgang Wallner wrote: > > On Wed, Apr 15, 2020 at 5:00 PM Wolfgang Wallner > > wrote: > > > -"Andy Shevchenko" schrieb: - > > > > Betreff: Re: Re: [PATCH v3 13/29] dts: Add a binding for hid-over-i2c > > > > > > > > On Wed, Apr 8, 2020 at 11:40

Re: [PATCH 6/7] ddr: altera: arria10: Change %i to %u for printf

2020-04-15 Thread Tom Rini
On Wed, Apr 15, 2020 at 04:58:31PM +0200, Marek Vasut wrote: > On 4/15/20 4:56 PM, Tom Rini wrote: > > On Wed, Apr 15, 2020 at 02:45:08PM +0200, Marek Vasut wrote: > >> On 4/15/20 11:00 AM, Ley Foon Tan wrote: > >>> Tiny printf doesn't support %i, change to %u. > >>> > >>> Signed-off-by: Ley Foon

Re: [PATCH 6/7] ddr: altera: arria10: Change %i to %u for printf

2020-04-15 Thread Marek Vasut
On 4/15/20 4:56 PM, Tom Rini wrote: > On Wed, Apr 15, 2020 at 02:45:08PM +0200, Marek Vasut wrote: >> On 4/15/20 11:00 AM, Ley Foon Tan wrote: >>> Tiny printf doesn't support %i, change to %u. >>> >>> Signed-off-by: Ley Foon Tan >>> --- >>> drivers/ddr/altera/sdram_arria10.c | 2 +- >>> 1 file

Antwort: Re: Re: Re: [PATCH v3 13/29] dts: Add a binding for hid-over-i2c

2020-04-15 Thread Wolfgang Wallner
Hi Andy, -"Andy Shevchenko" schrieb: - > On Wed, Apr 15, 2020 at 5:00 PM Wolfgang Wallner > wrote: > > -"Andy Shevchenko" schrieb: - > > > Betreff: Re: Re: [PATCH v3 13/29] dts: Add a binding for hid-over-i2c > > > > > > On Wed, Apr 8, 2020 at 11:40 PM Andy Shevchenko > > >

Re: [PATCH 6/7] ddr: altera: arria10: Change %i to %u for printf

2020-04-15 Thread Tom Rini
On Wed, Apr 15, 2020 at 02:45:08PM +0200, Marek Vasut wrote: > On 4/15/20 11:00 AM, Ley Foon Tan wrote: > > Tiny printf doesn't support %i, change to %u. > > > > Signed-off-by: Ley Foon Tan > > --- > > drivers/ddr/altera/sdram_arria10.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [PATCH v2] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Marek Vasut
On 4/15/20 4:48 PM, Harald Seiler wrote: > Use DM_ETH instead of legacy networking. Some more descriptive commit message would help. [...] > diff --git a/arch/arm/dts/imx6qdl-dhcom-u-boot.dtsi > b/arch/arm/dts/imx6qdl-dhcom-u-boot.dtsi > new file mode 100644 > index ..88840bb45920

[PATCH v2] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Harald Seiler
Use DM_ETH instead of legacy networking. Signed-off-by: Harald Seiler --- Notes: Changes in v2: - Move reset and VIO to device-tree - Always enable the clock, not just if CONFIG_FEC_MXC=y arch/arm/dts/imx6dl-dhcom-pdk2-u-boot.dtsi | 6 +++

Re: [PATCH v6 12/29] dm: core: Add basic ACPI support

2020-04-15 Thread Simon Glass
Hi Bin, On Wed, 15 Apr 2020 at 08:38, Bin Meng wrote: > > Hi Simon, > > On Wed, Apr 15, 2020 at 10:31 PM Bin Meng wrote: > > > > Hi Simon, > > > > On Fri, Apr 10, 2020 at 3:36 AM Simon Glass wrote: > > > > > > On Thu, 9 Apr 2020 at 11:40, Andy Shevchenko > > > wrote: > > > > > > > > On Thu,

  1   2   3   >