Re: [PATCH 2/2] ARM: dts: stm32: Add DHSOM based DRC02 board

2020-07-28 Thread Patrice CHOTARD
Hi Marek On 7/25/20 4:50 PM, Marek Vasut wrote: > Add DT for DH DRC02 unit, which is a universal controller device. > The system has two ethernet ports, two CANs, RS485 and RS232, USB, > capacitive buttons and an OLED display. > > Signed-off-by: Marek Vasut > Cc: Patrice Chotard > Cc: Patrick

Re: Kendryte build warnings

2020-07-28 Thread Sean Anderson
On 7/28/20 6:56 AM, Heinrich Schuchardt wrote: > Hello Sean, > > when I compile for Kendryte K210 I see the following build warnings: > > In file included from drivers/clk/kendryte/clk.c:15: > include/kendryte/pll.h:46: warning: "nop" redefined >46 | #define nop() > | > In file

Re: Kendryte build warnings

2020-07-28 Thread Heinrich Schuchardt
On 28.07.20 13:07, Sean Anderson wrote: > On 7/28/20 6:56 AM, Heinrich Schuchardt wrote: >> Hello Sean, >> >> when I compile for Kendryte K210 I see the following build warnings: >> >> In file included from drivers/clk/kendryte/clk.c:15: >> include/kendryte/pll.h:46: warning: "nop" redefined >>

Re: [Uboot-stm32] [PATCH] board: update test on misc_read result in board_late_init

2020-07-28 Thread Patrice CHOTARD
On 7/3/20 5:45 PM, Patrick Delaunay wrote: > Update management of misc_read, which now return length of data > after the commit 8729b1ae2cbd ("misc: Update read() and write() > methods to return bytes xfered") > > Fixes: 8b8b3d6b55b9 ("stm32mp1: board: add environment variable for board id > and

Re: [Uboot-stm32] [PATCH] Revert "ARM: dts: stm32: Reinstate card detect behavior on ST boards"

2020-07-28 Thread Patrice CHOTARD
On 7/7/20 6:47 PM, Patrick Delaunay wrote: > Since commit f13ff88b61c3 ("gpio: stm32: add ops set_dir_flags") > the gpio flags are supported by stm32 pinctrl driver. > > The cd-gpios is correctly handle in U-Boot and the patch on the > device tree can be removed. > > This reverts commit

Re: [Uboot-stm32] [PATCH] configs: stm32mp15: activate CMD_ADTIMG

2020-07-28 Thread Patrice CHOTARD
On 7/6/20 3:00 PM, Patrick Delaunay wrote: > Activate the support of the command adtimg to handle android images. > > Signed-off-by: Patrick Delaunay > --- > > configs/stm32mp15_basic_defconfig | 1 + > configs/stm32mp15_trusted_defconfig | 1 + > 2 files changed, 2 insertions(+) > > diff

Re: Kendryte build warnings

2020-07-28 Thread Sean Anderson
On 7/28/20 9:20 AM, Sean Anderson wrote: > On 7/28/20 8:31 AM, Heinrich Schuchardt wrote: >> Another issue I found is that the memory size is not correctly set on >> the Kendryte as reported by bdinfo. >> >> DRAM bank = 0x >> -> start= 0x8000 >> -> size =

[PATCH v4 12/14] env: ext4: introduce new function env_ext4_save_buffer

2020-07-28 Thread Patrick Delaunay
Split the function env_ext4_save to prepare the erase support. Signed-off-by: Patrick Delaunay --- (no changes since v1) env/ext4.c | 28 1 file changed, 20 insertions(+), 8 deletions(-) diff --git a/env/ext4.c b/env/ext4.c index ac9f126bec..0a10a5e050 100644 ---

[PATCH v4 07/14] cmd: env: add env load command

2020-07-28 Thread Patrick Delaunay
Add the new command env load to load the environment from the current location gd->env_load_prio. Signed-off-by: Patrick Delaunay --- (no changes since v3) Changes in v3: - new: add 'env load' command cmd/Kconfig | 6 ++ cmd/nvedit.c | 14 ++ env/env.c | 28

[PATCH v4 02/14] env: ext4: set gd->env_valid

2020-07-28 Thread Patrick Delaunay
Add a missing initialization of gd->env_valid in env_ext4_load as it is already done in some other env device. Set gd->env_valid = ENV_VALID in env_ext4_save() and env_ext4_load(). This patch allows to have a correct information in 'env info' command. Reviewed-by: Tom Rini Signed-off-by:

[PATCH v4 00/14] env: ext4: corrections and add test for env in ext4

2020-07-28 Thread Patrick Delaunay
Hi, V4 of the serie [1]: rebase on master branch and solve the SPL size issue for board with SPL_ENV_IS_NOWHERE=y (pointed by Tom Rini in [2]) The buildman result for these boards are: - tools/buildman/buildman -b sssddsqqs

[PATCH v4 06/14] env: the ops driver load becomes mandatory in struct env_driver

2020-07-28 Thread Patrick Delaunay
The ops driver load becomes mandatory in struct env_drive, change the comment for this ops and remove unnecessary test. Signed-off-by: Patrick Delaunay --- (no changes since v3) Changes in v3: - new: load operation becomes mandatory env/env.c | 3 --- include/env_internal.h | 3

[PATCH v4 05/14] env: nowhere: add .load ops

2020-07-28 Thread Patrick Delaunay
Add the ops .load for nowhere ENV backend to load the default environment. This ops is needed for the command 'env load' Signed-off-by: Patrick Delaunay --- Changes in v4: - don't use env_import in SPL to avoid to increase its size as it is only required for 'env load' command Changes in

[PATCH v4 03/14] env: sf: avoid space in backend name

2020-07-28 Thread Patrick Delaunay
Remove space in ENV backend name for SPI Flash (SF) to avoid issue with env select command. Signed-off-by: Patrick Delaunay --- (no changes since v3) Changes in v3: - new env/sf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/env/sf.c b/env/sf.c index

[PATCH v4 09/14] configs: sandbox: activate env in ext4 support

2020-07-28 Thread Patrick Delaunay
Activate ENV in EXT4 support in sandbox. The sandbox behavior don't change; the default environment with the nowhere backend (CONFIG_ENV_IS_NOWHERE)is still used: the weak function env_get_location() return ENVL_NOWHERE for priority 0. Signed-off-by: Patrick Delaunay --- (no changes since v3)

[PATCH v1] configs: pico-imx6: convert ethernet function to DM_ETH

2020-07-28 Thread wig . cheng
From: Wig Cheng Before enable _DM_ETH: Net: FEC [PRIME] After enable DM_ETH: Net: eth0: ethernet@2188000 Here is the test commands: => dhcp BOOTP broadcast 1 DHCP client bound to address 10.88.88.152 (146 ms) *** ERROR: `serverip' not set Cannot autoload with

[PATCH v1] configs: pico-imx6: convert ethernet function to DM_ETH

2020-07-28 Thread wig . cheng
From: Wig Cheng Before enable _DM_ETH: Net: FEC [PRIME] After enable DM_ETH: Net: eth0: ethernet@2188000 Here is the test commands: => dhcp BOOTP broadcast 1 DHCP client bound to address 10.88.88.152 (146 ms) *** ERROR: `serverip' not set Cannot autoload with

RE: [PATCH 1/2] ARM: dts: stm32: Move ethernet PHY into SoM DT

2020-07-28 Thread Patrick DELAUNAY
Hi Marek, > From: Marek Vasut > Sent: samedi 25 juillet 2020 16:51 > To: u-boot@lists.denx.de > Cc: Marek Vasut ; Patrice CHOTARD > ; Patrick DELAUNAY > Subject: [PATCH 1/2] ARM: dts: stm32: Move ethernet PHY into SoM DT > Importance: High > > The PHY and the VIO regulator is populated on the

Re: Reproducibility regression with mx6cuboxi

2020-07-28 Thread Fabio Estevam
Hi Vagrant, On Sun, Jul 26, 2020 at 3:17 PM Vagrant Cascadian wrote: > That appears to build reproducibly for me on top of v2020.07. Haven't > tested if the resulting image boots. When you have a chance to test it, please reply with your Tested-by on my v2 patch. Thanks

Re: Kendryte build warnings

2020-07-28 Thread Sean Anderson
On 7/28/20 8:31 AM, Heinrich Schuchardt wrote: > On 28.07.20 13:07, Sean Anderson wrote: >> On 7/28/20 6:56 AM, Heinrich Schuchardt wrote: >>> Hello Sean, >>> >>> when I compile for Kendryte K210 I see the following build warnings: >>> >>> In file included from drivers/clk/kendryte/clk.c:15: >>>

Re: [Uboot-stm32] [PATCH 1/3] ARM: dts: rename stm32mp15xx-avenger96 ethernet0_rgmii_pins

2020-07-28 Thread Patrice CHOTARD
On 7/6/20 1:26 PM, Patrick Delaunay wrote: > Alignment with pins name used in Linux kernel v5.8. > > It is a preleminary step for device tree alignment. > > Signed-off-by: Patrick Delaunay > --- > > arch/arm/dts/stm32mp15-pinctrl.dtsi | 4 ++-- >

Re: [PATCH] arm: stm32mp: stm32prog: check result of find_mmc_device

2020-07-28 Thread Patrice CHOTARD
On 7/6/20 1:20 PM, Patrick Delaunay wrote: > Check result of find_mmc_device() before calling mmc_init(). > > This patch avoid a data abort when the command try to acces > to sd/mmc device deactivated in device tree. > > Fixes: aff4c5dd8265 ("stm32mp: stm32prog: add MMC device") > Signed-off-by:

[PATCH v4 04/14] env: correctly handle env_load_prio

2020-07-28 Thread Patrick Delaunay
Only update gd->env_load_prio in generic function env_load() and no more in the weak function env_get_location() which is called in many place (for example in env_driver_lookup, even for ENVOP_SAVE operation). This patch is a preliminary step to use env_driver_lookup()/ env_get_location() in new

[PATCH v4 01/14] env: add absolute path at CONFIG_ENV_EXT4_FILE

2020-07-28 Thread Patrick Delaunay
Add the absolute path to the default value of CONFIG_ENV_EXT4_FILE = "/uboot.env". This patch avoid the error : Saving Environment to EXT4... File System is consistent Please supply Absolute path Reviewed-by: Tom Rini Signed-off-by: Patrick Delaunay --- For information, it is the value

Re: [PATCH] phy: usbphyc: use regulator_set_enable_if_allowed for disabling vdd supply

2020-07-28 Thread Patrice CHOTARD
On 7/3/20 7:13 PM, Patrick Delaunay wrote: > Use regulator_set_enable_if_allowed() api instead of regulator_set_enable() > while disabling vdd supply. This way the driver doesn't see an error > when disabling an always-on regulator. > > This patch is needed since the commit f93fab312615 ("Revert

Re: [PATCH] stm32mp1: mmu_set_region_dcache_behaviour

2020-07-28 Thread Patrice CHOTARD
On 7/24/20 11:21 AM, Patrick Delaunay wrote: > Since commit d877f8fd0f09 ("arm: provide a function for boards init > code to modify MMU virtual-physical map") the parameter of > mmu_set_region_dcache_behaviour need to be MMU_SECTION_SIZE > aligned. > > Signed-off-by: Patrick Delaunay > --- > >

Re: [Uboot-stm32] [PATCH v2] Revert "ARM: dts: stm32: Reinstate card detect behavior on DHSOM"

2020-07-28 Thread Patrice CHOTARD
On 7/7/20 6:46 PM, Patrick Delaunay wrote: > Since commit f13ff88b61c3 ("gpio: stm32: add ops set_dir_flags") > the gpio flags are supported by stm32 pinctrl driver. > > The cd-gpios is correctly handle in U-Boot and the patch on the > device tree can be removed. > > This reverts commit

Re: [PATCH] arm: stm32mp: use correct weak function name spl_board_prepare_for_linux

2020-07-28 Thread Patrice CHOTARD
On 7/7/20 2:21 PM, Patrick Delaunay wrote: > Replace the function spl_board_prepare_for_boot_linux by the correct > name of the weak function spl_board_prepare_for_linux defined in spl.h. > > This patch avoids warning with W=1 option: > > u-boot/arch/arm/mach-stm32mp/spl.c:150:6: > warning: no

RE: [PATCH v4 0/5] add DM based reset driver for SiFive SoC's

2020-07-28 Thread Sagar Kadam
Hello Rick, > -Original Message- > From: Sagar Kadam > Sent: Monday, July 27, 2020 8:56 PM > To: u-boot@lists.denx.de > Cc: r...@andestech.com; Paul Walmsley ( Sifive) > ; pal...@dabbelt.com; anup.pa...@wdc.com; > atish.pa...@wdc.com; lu...@denx.de; Pragnesh Patel > ;

Re: [PATCH 1/6] riscv: Rework riscv timer driver to only support S-mode

2020-07-28 Thread Sean Anderson
On 7/28/20 5:11 AM, Rick Chen wrote: > Hi Sean > >> The riscv-timer driver currently serves as a shim for several riscv timer >> drivers. This is not too desirable because it bypasses the usual timer >> selection via the driver model. There is no easy way to specify an >> alternate timing driver,

[PATCH] cmd: bootz: fix device-tree overlap test

2020-07-28 Thread Baruch Siach
The address of the kernel image is stored in images->ep. zi_start is the offset of execution entry in the image, which is usually 0 for ARM zImage. Fixes boot error when ftd is stored near RAM address 0x0: ERROR: FDT image overlaps OS image (OS=0x0..0x5fd608) Fixes: fbde7589ce30 ("common:

[PATCH 2/4] azure: Drop 32-bit MSYS2 build

2020-07-28 Thread Bin Meng
As of 2020-05-17, 32-bit MSYS2 is no longer actively supported by the upstream [1]. Let's drop the 32-bit Windows host tool build. [1] https://www.msys2.org/news/#2020-05-17-32-bit-msys2-no-longer-actively-supported Signed-off-by: Bin Meng --- .azure-pipelines.yml | 16 1

[PATCH 3/4] azure: Add the missing build dependency for MSYS2 build

2020-07-28 Thread Bin Meng
Package 'flex' is needed when building the U-Boot host tool, but is currently missing in the build dependency in the CI pipeline. This is to prepare switching to an installer based CI build. Signed-off-by: Bin Meng --- .azure-pipelines.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 1/4] azure: Use a login shell everywhere for MSYS2 build

2020-07-28 Thread Bin Meng
This simplifies things a bit to just use a login shell everywhere. This keeps in sync with MSYS2 upstream commit: 9d11b7f0aa93 ("azure-pipelines: simplify things a bit"). Signed-off-by: Bin Meng --- .azure-pipelines.yml | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git

RE: [PATCH] [dh stm32mp1] remove env location override

2020-07-28 Thread Patrick DELAUNAY
Hi Jakob, > From: U-Boot On Behalf Of Jakob Riepler > Sent: lundi 27 juillet 2020 14:18 > > Overriding the environment location is not necessary as the defconfig for the > relevant boards only enable SPI flash and nowhere sources which are in the > same > order per default but having this

Re: [PATCH V2 4/7] env: Fix invalid env handling in env_init()

2020-07-28 Thread Tom Rini
On Tue, Jul 28, 2020 at 09:28:17AM +0200, Marek Vasut wrote: > On 7/24/20 4:56 PM, Tom Rini wrote: > > On Tue, Jul 07, 2020 at 08:51:36PM +0200, Marek Vasut wrote: > > > >> This fixes the case where there are multiple environment drivers, one of > >> them is the default environment one, and it is

Re: [Uboot-stm32] [PATCH 3/3] ARM: dts: stm32mp1: DT alignment with Linux kernel v5.8-rc1

2020-07-28 Thread Patrice CHOTARD
On 7/6/20 1:26 PM, Patrick Delaunay wrote: > DT alignment with Linux kernel v5.8-rc1 for the STM32MP15x soc > device tree files and the STMicroelectronics boards device tree files. > > Signed-off-by: Patrick Delaunay > --- > > arch/arm/dts/stm32mp15-pinctrl.dtsi | 562

Re: [PATCH] arm: stm32mp: move dbgmcu_init call when DT is ready

2020-07-28 Thread Patrice CHOTARD
On 7/24/20 11:13 AM, Patrick Delaunay wrote: > As the dbgmcu_init use the function bsec_dbgswenable which is based > on the DM and DT, its call can't be done before the spl is initialized > (driver model, DT and malloc) in board_init_f::spl_early_init(). > > This function call is moved later in

[PATCH] tools/imximage: fix DCD Blocks message output order

2020-07-28 Thread Matthias Schiffer
The correct order is load address, offset, length. The order was accidentally switched a while ago; make it match the HAB Blocks output and what CST expects again. Fixes: e97bdfa5da70 ("tools/imximage: share DCD information via Kconfig") Signed-off-by: Matthias Schiffer --- tools/imximage.c | 2

Re: [PATCH v1] configs: pico-imx6: convert ethernet function to DM_ETH

2020-07-28 Thread Fabio Estevam
HI Wig, On Tue, Jul 28, 2020 at 7:37 AM wrote: > > From: Wig Cheng > > Before enable _DM_ETH: > Net: FEC [PRIME] > > After enable DM_ETH: > Net: eth0: ethernet@2188000 > > Here is the test commands: > => dhcp > BOOTP broadcast 1 > DHCP client bound to address

Re: [PATCH v4 1/5] cmd: bind: allow to bind driver with driver data

2020-07-28 Thread Patrice CHOTARD
Hi Tom On 7/7/20 10:08 PM, Tom Rini wrote: > On Thu, Apr 30, 2020 at 12:06:15PM +0200, Patrice Chotard wrote: > >> Initial implementation invokes device_bind_with_driver_data() >> with driver_data parameter equal to 0. >> For driver with driver data, the bind command can't bind >> correctly this

[PATCH v1 2/2] x86: call32: Append appropriate suffixes

2020-07-28 Thread Andy Shevchenko
Assembler is not happy: arch/x86/cpu/call32.S: Assembler messages: arch/x86/cpu/call32.S:36: Warning: no instruction mnemonic suffix given and no register operands; using default for `retf' Fix this by adding appropriate suffixes to the assembler commands. Fixes: 6f92ed8f1abf ("x86: Add a way

[PATCH v1 1/2] x86: sipi_vector: Append appropriate suffixes

2020-07-28 Thread Andy Shevchenko
Assembler is not happy: arch/x86/cpu/sipi_vector.S: Assembler messages: arch/x86/cpu/sipi_vector.S:134: Warning: no instruction mnemonic suffix given and no register operands; using default for `cmp' arch/x86/cpu/sipi_vector.S:139: Warning: no instruction mnemonic suffix given and no register

Re: [PATCH V2 4/7] env: Fix invalid env handling in env_init()

2020-07-28 Thread Marek Vasut
On 7/28/20 2:39 PM, Tom Rini wrote: [...] >>> So, I am not sure about this change. Given the whole thread that ends >>> in https://lists.denx.de/pipermail/u-boot/2020-June/417433.html this >>> particular part of the function is being too clever. I think it's >>> intentionally not doing what

[PATCH v4 10/14] configs: sandbox: activate command env select and env load

2020-07-28 Thread Patrick Delaunay
Add support of environment location with the new env command: 'env select' and 'env load' The ENV backend is selected by priority order - 0 = "nowhere" (default at boot) - 1 = "EXT4" To test EXT4 env support, this backend is selected by name: > env select EXT4 Signed-off-by: Patrick Delaunay

[PATCH v4 08/14] cmd: env: add env select command

2020-07-28 Thread Patrick Delaunay
Add the new command 'env select' to force the persistent storage of environment, saved in gd->env_load_prio. Signed-off-by: Patrick Delaunay --- (no changes since v3) Changes in v3: - new: add 'env select' command cmd/Kconfig | 5 + cmd/nvedit.c | 15 +++ env/env.c |

[PATCH v4 11/14] test: environment in ext4

2020-07-28 Thread Patrick Delaunay
Add basic test to persistent environment in ext4: save and load in host ext4 file 'uboot.env'. On first execution an empty EXT4 file system is created in persistent data dir: env.ext4.img. Signed-off-by: Patrick Delaunay --- (no changes since v3) Changes in v3: - replace specific sandbox

[PATCH v4 14/14] test: sandbox: add test for erase command

2020-07-28 Thread Patrick Delaunay
Add test for the erase command tested on ENV in EXT4. Signed-off-by: Patrick Delaunay --- (no changes since v3) Changes in v3: - replace sandbox command by generic command 'env load' in test_env configs/sandbox64_defconfig| 1 + configs/sandbox_defconfig | 1 +

[PATCH v4 13/14] env: ext4: add support of command env erase

2020-07-28 Thread Patrick Delaunay
Add support of opts erase for env in ext4, this opts is used by command 'env erase'. This command only fill the env file (CONFIG_ENV_EXT4_FILE) with 0, the CRC and the saved environment becomes invalid. Signed-off-by: Patrick Delaunay --- (no changes since v2) Changes in v2: - use

RE: [PATCH 2/2] ARM: dts: stm32: Add DHSOM based DRC02 board

2020-07-28 Thread Patrick DELAUNAY
Hi Marek, > From: Marek Vasut > Sent: samedi 25 juillet 2020 16:51 > > Add DT for DH DRC02 unit, which is a universal controller device. > The system has two ethernet ports, two CANs, RS485 and RS232, USB, > capacitive buttons and an OLED display. > > Signed-off-by: Marek Vasut > Cc: Patrice

Re: [Uboot-stm32] [PATCH] ARM: dts: stm32: add reset support to uart nodes on stm32mp15x

2020-07-28 Thread Patrice CHOTARD
On 7/6/20 2:48 PM, Patrick Delaunay wrote: > STM32 serial IP can be reset via reset controller. > Add the support of reset to uart nodes on stm32mp15-u-boot.dtsi, > the ad-dons file for U-Boot. > > This patch fix issues when previous UART configuration, for example > done in TF-A or ROM code, is

Finish off common.h reduction

2020-07-28 Thread Simon Glass
Hi Tom, I thought now (after rc1) might be a good time to do this series. Does that sound right to you? Regards, Simon

Re: [Uboot-stm32] [PATCH] arm: stm32mp: move dbgmcu_init call when DT is ready

2020-07-28 Thread Patrice CHOTARD
On 7/28/20 3:53 PM, Patrice CHOTARD wrote: > On 7/24/20 11:13 AM, Patrick Delaunay wrote: >> As the dbgmcu_init use the function bsec_dbgswenable which is based >> on the DM and DT, its call can't be done before the spl is initialized >> (driver model, DT and malloc) in

Re: [PATCH 1/6] riscv: Rework riscv timer driver to only support S-mode

2020-07-28 Thread Rick Chen
Hi Sean > The riscv-timer driver currently serves as a shim for several riscv timer > drivers. This is not too desirable because it bypasses the usual timer > selection via the driver model. There is no easy way to specify an > alternate timing driver, or have the tick rate depend on the cpu's >

Kendryte build warnings

2020-07-28 Thread Heinrich Schuchardt
Hello Sean, when I compile for Kendryte K210 I see the following build warnings: In file included from drivers/clk/kendryte/clk.c:15: include/kendryte/pll.h:46: warning: "nop" redefined 46 | #define nop() | In file included from ./arch/riscv/include/asm/io.h:13, from

Re: [PATCH 2/2] configs: migrate CONFIG_SPL_LOAD_FIT_ADDRESS to defconfigs

2020-07-28 Thread Tom Rini
On Mon, Jul 06, 2020 at 03:35:01PM +0800, peng@nxp.com wrote: > From: Peng Fan > > Done with: > ./tools/moveconfig.py -S SPL_LOAD_FIT_ADDRESS > ./tools/moveconfig.py -S SPL_LOAD_FIT_ADDRESS -H > > Signed-off-by: Peng Fan Applied to u-boot/master, thanks! -- Tom signature.asc

[PATCH v5 0/3] Add support for booting into userspace fastboot

2020-07-28 Thread Roman Kovalivskyi
Android 10 adds support for dynamic partitions and in order to support them userspace fastboot must be used[1]. New tool called fastbootd is included into recovery image. Userspace fastboot works from recovery and is launched if: 1) - Dynamic partitioning is enabled 2) - Boot control block has

[PATCH v5 1/3] fastboot: Extend fastboot_set_reboot_flag with reboot reason

2020-07-28 Thread Roman Kovalivskyi
Extend fastboot_set_reboot_flag arguments with reboot reason so that it could handle different reboot cases in future. Signed-off-by: Roman Kovalivskyi --- arch/arm/mach-meson/board-common.c | 6 +- arch/arm/mach-rockchip/board.c | 6 +- board/amazon/kc1/kc1.c | 6

Re: AM335X Bootp of u-boot-spl.bin followed by tftp of u-boot.img. Is it possible?

2020-07-28 Thread Richard Lourette
Here is more information... I dumped the DTB to decompile to see what was in the device tree. I am working on a new board based upon the AM3352 that has only ethernet and a serial port. The goal is to use the device model exclusively for this. I am working with u-boot 2020.07rc4 presently. I am

[PATCH 1/1] cmd: bdinfo: cleanup phys_addr_t output

2020-07-28 Thread Heinrich Schuchardt
We currently print the memory size with at least 8 hexadecimal digits. This creates a ragged output on 64 bit boards, e.g. on a Kendryte K210: DRAM bank = 0x0002 -> start= 0x8060 -> size = 0x0020 memstart= 0x memsize =

Re: [Uboot-stm32] [PATCH 3/3] ARM: dts: stm32mp1: DT alignment with Linux kernel v5.8-rc1

2020-07-28 Thread Patrice CHOTARD
On 7/28/20 3:36 PM, Patrice CHOTARD wrote: > On 7/6/20 1:26 PM, Patrick Delaunay wrote: >> DT alignment with Linux kernel v5.8-rc1 for the STM32MP15x soc >> device tree files and the STMicroelectronics boards device tree files. >> >> Signed-off-by: Patrick Delaunay >> --- >> >>

Re: [Uboot-stm32] [PATCH 1/3] ARM: dts: rename stm32mp15xx-avenger96 ethernet0_rgmii_pins

2020-07-28 Thread Patrice CHOTARD
On 7/6/20 1:26 PM, Patrick Delaunay wrote: > Alignment with pins name used in Linux kernel v5.8. > > It is a preleminary step for device tree alignment. > > Signed-off-by: Patrick Delaunay > --- > > arch/arm/dts/stm32mp15-pinctrl.dtsi | 4 ++-- >

Re: [Uboot-stm32] [PATCH] ARM: dts: stm32mp: remove hnp-srp-disable for usbotg on dk1

2020-07-28 Thread Patrice CHOTARD
On 7/2/20 10:19 AM, Patrice CHOTARD wrote: > Hi Patrick > > On 7/1/20 1:10 PM, Patrick Delaunay wrote: >> Remove hnp-srp-disable for usbotg node because this feature is only needed >> for usbotg in HOST mode (in drivers/usb/host/dwc2.c) >> and this feature is not activated in stm32mp15 U-Boot >>

Re: [Uboot-stm32] [PATCH] ARM: dts: stm32: add reset support to uart nodes on stm32mp15x

2020-07-28 Thread Patrice CHOTARD
On 7/28/20 4:12 PM, Patrice CHOTARD wrote: > On 7/6/20 2:48 PM, Patrick Delaunay wrote: >> STM32 serial IP can be reset via reset controller. >> Add the support of reset to uart nodes on stm32mp15-u-boot.dtsi, >> the ad-dons file for U-Boot. >> >> This patch fix issues when previous UART

Re: [PATCH v4 07/27] rockchip: Enable building a SPI ROM image on bob

2020-07-28 Thread Simon Glass
Add a simple binman config and enable CONFIG_HAS_ROM so that U-Boot produces a ROM for bob. Signed-off-by: Simon Glass --- Changes in v4: - Use CONFIG_ROCKCHIP_SPI_IMAGE to select the image arch/arm/dts/rk3399-gru-u-boot.dtsi | 4 arch/arm/dts/rk3399-gru.dtsi | 2 +-

Re: [PATCH v2 02/18] Kconfig: Introduce CONFIG_XEN

2020-07-28 Thread Simon Glass
Hi Anastasiia, On Mon, 20 Jul 2020 at 05:02, Anastasiia Lukianenko wrote: > > From: Peng Fan > > Introduce CONFIG_XEN to make U-Boot could be used as bootloader > for a virtual machine. > > Without bootloader, we could successfully boot up android on XEN, but > we need need bootloader to

Re: rk3399-gru-kevin: issues on bringup

2020-07-28 Thread Simon Glass
Hi Marty, On Tue, 21 Jul 2020 at 21:07, Marty E. Plummer wrote: > > On Tue, Jul 21, 2020 at 10:21:52AM -0600, Simon Glass wrote: > > Hi Marty, > > > > Did you check spl_boot_device()? > > > After sending the initial email I noticed your binman work, which does > some of the stuff I think I need.

Re: [PATCH v4 1/2] arm: move CONFIG_PREBOOT="usb start" to KConfig

2020-07-28 Thread Simon Glass
On Sun, 19 Jul 2020 at 07:56, Jonas Smedegaard wrote: > > This commit moves CONFIG_PREBOOT="usb start" to common/KConfig > for all boards also declaring USB_KEYBOARD. > > Besides simplifying defconfig files, this also enables support for > board-specific CONFIG_PREBOOT for sunxi boards: > commit

Re: [PATCH v4 20/27] Makefile: Warn against using CONFIG_SPL_FIT_GENERATOR

2020-07-28 Thread Simon Glass
This option is used to run arch-specific shell scripts which produce .its files which are used to produce FIT images. We already have binman which is designed to produce firmware images. It is more powerful and has tests. So this option should be deprecated and not used. Existing uses should be

Re: [PATCH v4 09/27] tegra: Don't enable binman on ARMv8 SoCs

2020-07-28 Thread Simon Glass
At present only the ARMv7 tegra SoCs actually use binman to create an image. Change the config to reflect this, since otherwise running binman will produce an error. Signed-off-by: Simon Glass --- (no changes since v1) arch/arm/mach-tegra/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1

Re: [PATCH 1/1] dm: remove superfluous comment for union ofnode_union

2020-07-28 Thread Simon Glass
On Fri, 24 Jul 2020 at 10:39, Heinrich Schuchardt wrote: > > "future live tree" does not make sense anymore as we have CONFIG_OF_LIVE. > > Signed-off-by: Heinrich Schuchardt > --- > include/dm/ofnode.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Reviewed-by: Simon Glass

Re: [PATCH v4 02/27] rockchip: Allow Bob to use SPI boot

2020-07-28 Thread Simon Glass
Bob is a Chromebook and can boot from SPI flash. Add it to the condition check for this. Signed-off-by: Simon Glass --- (no changes since v1) arch/arm/mach-rockchip/spl.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) Applied to u-boot-dm, thanks!

Re: [PATCH v4 22/27] rockchip: Convert evb-rk3229 over to use binman

2020-07-28 Thread Simon Glass
At present this board uses a custom script to produce the .its file. Update it to use binman instead. Binman can create all the images that are needed. Signed-off-by: Simon Glass --- (no changes since v3) Changes in v3: - Drop leftover debugging configs/evb-rk3229_defconfig | 2 +- 1 file

[PATCH v5 2/3] fastboot: Add support for 'reboot fastboot' command

2020-07-28 Thread Roman Kovalivskyi
Android 10 adds support for dynamic partitions and in order to support this userspace fastboot must be used[1]. New tool fastbootd is included into recovery. Userspace fastboot works from recovery and is launched if: 1) - Dynamic partitioning is enabled 2) - Boot control block has 'boot-fastboot'

[PATCH v5 3/3] fastboot: Add default fastboot_set_reboot_flag implementation

2020-07-28 Thread Roman Kovalivskyi
Default implementation of fastboot_set_reboot_flag function that depends on "bcb" commands could be used in general case if there are no need to make any platform-specific implementation, otherwise it could be disabled via Kconfig option FASTBOOT_USE_BCB_SET_REBOOT_FLAG. Please note that

Re: [PATCH] dtoc: add coverage test for unicode error

2020-07-28 Thread Walter Lozano
Hi Simon, On 27/7/20 20:35, Simon Glass wrote: Hi Walter, On Mon, 20 Jul 2020 at 12:29, Walter Lozano wrote: Add an additional test to dtoc in order improve the coverage, specifically to take into account the case of unicode error when scanning drivers. Signed-off-by: Walter Lozano ---

[PATCH v2] dtoc: add coverage test for unicode error

2020-07-28 Thread Walter Lozano
Add an additional test to dtoc in order improve the coverage, specifically to take into account the case of unicode error when scanning drivers. Signed-off-by: Walter Lozano --- Changes in v2: - Add missing files - Extend scan_drivers to use both relative and absolute paths

Re: [PATCH] Convert CONFIG_ENV_OVERWRITE to Kconfig

2020-07-28 Thread Tom Rini
On Fri, Jul 03, 2020 at 06:48:56AM -0500, Adam Ford wrote: > This converts the following to Kconfig: >CONFIG_ENV_OVERWRITE > > Signed-off-by: Adam Ford > > diff --git a/arch/arm/mach-kirkwood/include/mach/config.h > b/arch/arm/mach-kirkwood/include/mach/config.h > index

Re: [PATCH] Convert CONFIG_BOOTP_DNS2 to Kconfig

2020-07-28 Thread Tom Rini
On Fri, Jul 03, 2020 at 08:45:28AM -0500, Adam Ford wrote: > The Kconfig already has BOOTP_DNS2, but not all boards have been > converted. This finishes converting the following to Kconfig: >CONFIG_BOOTP_DNS2 > > Signed-off-by: Adam Ford This migration was done in a previous patch, so

Re: [PATCH] Convert CONFIG_BOOTP_SEND_HOSTNAME to Kconfig

2020-07-28 Thread Tom Rini
On Fri, Jul 03, 2020 at 09:00:14AM -0500, Adam Ford wrote: > This converts the following to Kconfig: >CONFIG_BOOTP_SEND_HOSTNAME > > Signed-off-by: Adam Ford > > diff --git a/README b/README > index 60d8e77256..6bac583666 100644 Applied to u-boot/master, thanks! -- Tom signature.asc

Re: [PATCH 2/2] Convert CONFIG_SPL_NAND_BASE et al to Kconfig

2020-07-28 Thread Tom Rini
On Fri, Jul 03, 2020 at 08:09:45AM -0500, Adam Ford wrote: > This converts the following to Kconfig: >CONFIG_SPL_NAND_BASE >CONFIG_SPL_NAND_IDENT > > Signed-off-by: Adam Ford > > diff --git a/README b/README > index b6d368041c..60d8e77256 100644 Applied to u-boot/master, thanks! --

Re: [PATCH 1/2] Convert CONFIG_SPL_NAND_DRIVERS et al to Kconfig

2020-07-28 Thread Tom Rini
On Fri, Jul 03, 2020 at 08:09:44AM -0500, Adam Ford wrote: > This converts the following to Kconfig: >CONFIG_SPL_NAND_DRIVERS >CONFIG_SPL_NAND_ECC >CONFIG_SPL_NAND_SIMPLE > > Signed-off-by: Adam Ford Note: While you're the listed maintainer of da850evm_nand, the patch as sent also

Re: [PATCH] Convert CONFIG_DRIVER_TI_EMAC_USE_RMII to Kconfig

2020-07-28 Thread Tom Rini
On Fri, Jul 03, 2020 at 08:27:12AM -0500, Adam Ford wrote: > This converts the following to Kconfig: >CONFIG_DRIVER_TI_EMAC_USE_RMII > > Signed-off-by: Adam Ford > > diff --git a/configs/am3517_evm_defconfig b/configs/am3517_evm_defconfig > index 58fb91c1c5..a78a596d4d 100644 Applied to

Re: [PATCH] Convert CONFIG_SYS_MMCSD_FS_BOOT_PARTITION to Kconfig

2020-07-28 Thread Tom Rini
On Fri, Jul 03, 2020 at 10:17:30AM -0500, Adam Ford wrote: > This converts the following to Kconfig: >CONFIG_SYS_MMCSD_FS_BOOT_PARTITION > > Signed-off-by: Adam Ford > > diff --git a/README b/README > index 6bac583666..b45c7b27d7 100644 Applied to u-boot/master, thanks! -- Tom

Re: [PATCH 1/2] Kconfig: add SPL_LOAD_FIT_ADDRESS

2020-07-28 Thread Tom Rini
On Mon, Jul 06, 2020 at 03:35:00PM +0800, peng@nxp.com wrote: > From: Peng Fan > > Add SPL_LOAD_FIT_ADDRESS to make user could add it in defconfig > > Signed-off-by: Peng Fan Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH] configs: Remove dead CONFIG options

2020-07-28 Thread Tom Rini
On Fri, Jul 03, 2020 at 09:06:36AM -0500, Adam Ford wrote: > BOOTP_DEFAULT is defined in several boards, but this config > option is never checked or used. > > This patch removes this config option from config files and > the whitelist.txt > > Signed-off-by: Adam Ford > > diff --git

RE: [PATCH] Clean up the Spansion/Cypress/Infineon device ID table

2020-07-28 Thread Bacem Daassi
The device s25fl512s is not offered with neither 64kB sector size, nor 512kB sector size, that's why s25fl512s_64k and s25fl512s_512k need to be removed from the device ID table. The device ID entries "s25fl512s_256k" and "s25fl512s" are exactly the same, that's why the entry "s25fl512s_256k"

Re: [PATCH 6/8] arm: dts: Sync the sun50i-a64.dtsi from Linux 5.8-rc1

2020-07-28 Thread André Przywara
On 27/07/2020 14:16, Heinrich Schuchardt wrote: Hi Peter, Heinrich, > On 26.07.20 12:17, Peter Robinson wrote: >> On Thu, Jul 23, 2020 at 12:15 AM André Przywara >> wrote: >>> >>> On 22/07/2020 15:18, Peter Robinson wrote: Sync the Allwinner A64 sun50i-a64.dtsi from Linux. >>> >>> Hi

[PATCH] mmc: am654_sdhci: Use MMC_MODES_END value instead of hardcoded value

2020-07-28 Thread Faiz Abbas
The hardcoded array size leads to array overflows with changes in speed modes enum in mmc core. Use MMC_MODES_END for otap_del_sel array declaration to fix this. Signed-off-by: Faiz Abbas --- drivers/mmc/am654_sdhci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v2] configs: pico-imx6: convert ethernet function to DM_ETH

2020-07-28 Thread wig . cheng
From: Wig Cheng Before enable _DM_ETH: Net: FEC [PRIME] After enable DM_ETH: Net: eth0: ethernet@2188000 Here is the test commands: => dhcp BOOTP broadcast 1 DHCP client bound to address 10.88.88.152 (146 ms) *** ERROR: `serverip' not set Cannot autoload with TFTPGET =>

Re: [PATCH v2 06/18] arm: mach-k3: sysfw-loader: Add support for rom loading sysfw image

2020-07-28 Thread Lokesh Vutla
On 27/07/20 10:47 pm, Suman Anna wrote: > Hi Lokesh, > > On 7/27/20 4:45 AM, Lokesh Vutla wrote: >> Starting J7200 SoC, ROM supports for loading sysfw directly from boot >> image. In such cases, SPL need not load sysfw from boot media, but need >> to receive boot notification message from

[PATCH v1 2/3] configs: pico-imx6ul: convert DM_VIDEO

2020-07-28 Thread wig . cheng
From: Wig Cheng Fix compile warning messages. Signed-off-by: Wig Cheng --- configs/pico-imx6ul_defconfig | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/configs/pico-imx6ul_defconfig b/configs/pico-imx6ul_defconfig index e8cbf3bc1e..f02d370b7d 100644 ---

[PATCH v1 3/3] pico-imx6ul: convert ethernet function to DM_ETH

2020-07-28 Thread wig . cheng
From: Wig Cheng - Remove pinmux definition from pico-imx6ul.c - Enable NET_RANDOM_ETHADDR for temporary solution, because micrel_ksz8xxx driver does not support DM_ETH yet, so cannot read MAC address directly. Before enable DM_ETH: Net: FEC [PRIME] After enable DM_ETH: Net: Warning:

[PATCH v2] travis: update sbsigntools to 0.9.4

2020-07-28 Thread AKASHI Takahiro
This version of sbsigntools is a prerequisite in testing "intermediate certificates" support as part of UEFI secure boot as I added '--addcert' option to 'sbsign' command: === commit 7d6210e4b1fd Author: AKASHI Takahiro Date: Thu Jun 4 16:50:22 2020 +0900 sbsign: allow for adding

[PATCH v2] board: Add support for iMX8QXP AI_ML board

2020-07-28 Thread Manivannan Sadhasivam
This commit adds initial board support for iMX8QXP AI_ML board from Einfochips. This board is one of the 96Boards Consumer Edition and AI boards of the 96Boards family based on i.MX8QXP SoC from NXP/Freescale. This initial supports contains following peripherals which are tested and known to

[PATCH v2 0/5] cmd: Fix 'md' and add a memory-search command

2020-07-28 Thread Simon Glass
Add various fixes and improvements to this command that were missed in the original version. Unfortunately I forgot to send v2. Changes in v2: - Add new patch to check part of a console line or skip it - Rename the Kconfig to CONFIG_CMD_MEM_SEARCH - Use a separate var for the remaining search

[PATCH v2 5/5] cmd: Update the memory-search command

2020-07-28 Thread Simon Glass
Add various fixes and improvements to this command that were missed in the original version. Unfortunately I forgot to send v2. - Fix Kconfig name - Use a separate variable for the remaining search length - Correct a minor bug - Move into a separate test suite - Add -q flag to the 'quiet' test to

[PATCH v2 1/5] test: Add a way to check part of a console line or skip it

2020-07-28 Thread Simon Glass
Some lines of the output are not worth testing, or not worth testing in their entirety. For example, when checking a hex dump we know that the hex-dump routine can display ASCII so we only need to check the hex bytes, not the ASCII dump. Add a new test macros which can check only part of a console

[PATCH v2 2/5] console: Always define the console-recording functions

2020-07-28 Thread Simon Glass
On boards without console recording these function are currently missing. It is more convenient for them to be present but to return dummy values. That way if we know that a test needs recording, we can check if it is available, and skip the test if not, while avoiding #ifdefs. Update the header

[PATCH v2 4/5] test: Add a flag for tests that need console recording

2020-07-28 Thread Simon Glass
Allow tests that need console recording to be marked, so they can be skipped if it is not available. Signed-off-by: Simon Glass --- (no changes since v1) include/test/test.h | 1 + test/cmd_ut.c | 10 ++ 2 files changed, 11 insertions(+) diff --git a/include/test/test.h

  1   2   3   >