Re: [PATCH 1/2] global: Do not default to faking missing binaries for buildman

2022-10-10 Thread Rasmus Villemoes
On 10/10/2022 17.18, Tom Rini wrote: > While it is possible and documented on how to re-run buildman to replace > faked required binary files after the fact, this behavior ends up being > more confusing than helpful in practice. Switch to requiring > BINMAN_ALLOW_MISSING=1 to be passed on the 'make

Re: [PATCH] ARM: dts: stm32: update SCMI dedicated file

2022-10-10 Thread Patrice CHOTARD
On 10/10/22 10:56, Patrick Delaunay wrote: > The PWR regulators don't need be removed as they are already deactivated. > This patches is a alignment with the accepted patch in Linux device tree > in commit a34b42f8690c ("ARM: dts: stm32: fix pwr regulators references > to use scmi"). > > Fixes:

Re: [PATCH] ARM: stm32: Enable btrfs support on DHSOM

2022-10-10 Thread Patrice CHOTARD
On 9/26/22 18:44, Marek Vasut wrote: > The btrfs filesystem provides advanced functionality like copy-on-write > and snapshots, as well as metadata and data duplication and checksumming. > Enable btrfs in U-Boot to permit even the primary partition to be btrfs > and let system boot from it. > >

Re: Bad Data CRC

2022-10-10 Thread momo aubin
Hi Simon > On 11 Oct 2022, at 12:33 AM, Simon Glass wrote: > > Hi Momo, > >> On Mon, 10 Oct 2022 at 15:31, momo aubin wrote: >> >> Hello Simon, >> >> My board boots well . When I added the second partition into my kernel I got >> this message >> Verifying Checksum ... Bad Data CRC >> what

Re: [PATCH 1/1] efi_loader: CloseProtocol in efi_fmp_find

2022-10-10 Thread Heinrich Schuchardt
On 10/11/22 02:49, AKASHI Takahiro wrote: The commit message is not accurate. On Fri, Oct 07, 2022 at 04:06:23PM +0200, Heinrich Schuchardt wrote: The CloseProtocol() boot service requires a handle as first argument. Passing the protocol interface is incorrect. Correct, but CloseProtocol

Re: [PATCH 1/1] dm: fix blk_get_devnum_by_uclass_idname()

2022-10-10 Thread Heinrich Schuchardt
On 10/11/22 01:49, Simon Glass wrote: Hi Heinrich, On Thu, 6 Oct 2022 at 14:05, Heinrich Schuchardt wrote: On 10/3/22 18:44, Simon Glass wrote: Hi Heinrich, On Mon, 3 Oct 2022 at 10:33, Heinrich Schuchardt wrote: On 10/3/22 16:57, Simon Glass wrote: Hi Heinrich, On Mon, 3 Oct 2022

Re: [PATCH] binman: Add support for symlinking images

2022-10-10 Thread Neha Malcom Francis
Hi Simon, On 11/10/22 05:19, Simon Glass wrote: Hi Neha, On Mon, 10 Oct 2022 at 05:07, Neha Malcom Francis wrote: Adding support to symlink an image packaged using binman. Signed-off-by: Neha Malcom Francis --- tools/binman/image.py | 6 ++ 1 file changed, 6 insertions(+) This lo

Re: [maemo-leste] Broken u-boot on n900 hw (possibly CONFIG_POSITION_INDEPENDENT)

2022-10-10 Thread Sicelo
On Sun, Oct 09, 2022 at 09:33:17PM +0200, Pali Rohár wrote: > Hello! > > Current u-boot from master branch does not work on n900 hw. It does not > show anything on display, there just nokia logo. > > In qemu it is working fine and all tests are passing. Also in qemu is > display working without a

Re: [PATCH 1/1] efi_loader: CloseProtocol in efi_fmp_find

2022-10-10 Thread AKASHI Takahiro
The commit message is not accurate. On Fri, Oct 07, 2022 at 04:06:23PM +0200, Heinrich Schuchardt wrote: > The CloseProtocol() boot service requires a handle as first argument. > Passing the protocol interface is incorrect. Correct, but > CloseProtocol() only has an effect if called with a non-z

Re: [PATCH 1/1] dm: fix blk_get_devnum_by_uclass_idname()

2022-10-10 Thread Simon Glass
Hi Heinrich, On Thu, 6 Oct 2022 at 14:05, Heinrich Schuchardt wrote: > > On 10/3/22 18:44, Simon Glass wrote: > > Hi Heinrich, > > > > On Mon, 3 Oct 2022 at 10:33, Heinrich Schuchardt > > wrote: > >> > >> > >> > >> On 10/3/22 16:57, Simon Glass wrote: > >>> Hi Heinrich, > >>> > >>> On Mon, 3 Oct

Re: [PATCH 1/2] global: Do not default to faking missing binaries for buildman

2022-10-10 Thread Simon Glass
Hi Tom, On Mon, 10 Oct 2022 at 09:18, Tom Rini wrote: > > While it is possible and documented on how to re-run buildman to replace > faked required binary files after the fact, this behavior ends up being > more confusing than helpful in practice. Switch to requiring > BINMAN_ALLOW_MISSING=1 to b

Re: Adding 'N:' field support to buildman's boards database

2022-10-10 Thread Simon Glass
Hi, On Thu, 6 Oct 2022 at 13:07, Simon Glass wrote: > > Hi Tom, > > On Thu, 6 Oct 2022 at 09:33, Tom Rini wrote: > > > > On Mon, Sep 12, 2022 at 04:20:07PM -0400, Tom Rini wrote: > > > > > Hey, > > > > > > So looking at series such as > > > https://patchwork.ozlabs.org/project/uboot/patch/202208

Re: [PATCH] binman: Add support for symlinking images

2022-10-10 Thread Simon Glass
Hi Neha, On Mon, 10 Oct 2022 at 05:07, Neha Malcom Francis wrote: > > Adding support to symlink an image packaged using binman. > > Signed-off-by: Neha Malcom Francis > --- > tools/binman/image.py | 6 ++ > 1 file changed, 6 insertions(+) > This looks fine as a feature, but please add a te

Re: [PATCH v2] board_f: Add support for CONFIG_OF_BOARD_FIXUP for XIP images

2022-10-10 Thread Simon Glass
Hi, On Sun, 28 Aug 2022 at 09:19, Marek Behún wrote: > > From: Pali Rohár > > When U-Boot is running from flash memory (execute in place) then > gd->fdt_blob before relocation points to read-only flash memory. > > So U-Boot calls board_fix_fdt() with read-only gd->fdt_blob pointer which > cause

Re: [PATCH v2 2/2] common: board_f: Print information for all sysresets

2022-10-10 Thread Simon Glass
On Mon, 10 Oct 2022 at 12:30, Michal Suchanek wrote: > > Boards can have multiple sysresets, iterate all when printing sysreset > info. > > Fixes: 23471aed5c ("board_f: Add reset status printing") > Signed-off-by: Michal Suchanek > --- > > v2: > - do not use errno_str > - rename flag variable >

Re: [PATCH v2 1/2] sandbox: Initialize sysreset before relocation

2022-10-10 Thread Simon Glass
On Mon, 10 Oct 2022 at 12:30, Michal Suchanek wrote: > > Without this the early sysreset code cannot be tested. > > Signed-off-by: Michal Suchanek > --- > v2: use u-boot,dm-pre-proper rather than u-boot,dm-pre-reloc - sandbox > SPL cannot build otherwise > --- > arch/sandbox/dts/sandbox.dtsi

Re: [PATCH 2/2] buildman: Add --allow-missing-binaries flag to build with BINMAN_ALLOW_MISSING=1

2022-10-10 Thread Simon Glass
Hi Tom, On Mon, 10 Oct 2022 at 09:18, Tom Rini wrote: > > Add a new flag to buildman so that we will in turn pass > BINMAN_ALLOW_MISSING=1 to 'make'. Make use of this flag in CI. > > Cc: Rasmus Villemoes > Cc: Simon Glass > Signed-off-by: Tom Rini > --- > .azure-pipelines.yml| 2 +

Re: Bad Data CRC

2022-10-10 Thread Simon Glass
Hi Momo, On Mon, 10 Oct 2022 at 15:31, momo aubin wrote: > > Hello Simon, > > My board boots well . When I added the second partition into my kernel I got > this message > Verifying Checksum ... Bad Data CRC > what could be the root cause ? > > Which data is talking about ? u boot data or kernel

Re: [PATCH v4 04/21] dm: blk: Add probe in blk_first_device/blk_next_device

2022-10-10 Thread Simon Glass
Hi Michal, On Mon, 10 Oct 2022 at 15:33, Michal Suchánek wrote: > > On Mon, Oct 10, 2022 at 09:49:20PM +0200, Michal Suchánek wrote: > > On Sun, Oct 02, 2022 at 07:10:40PM -0600, Simon Glass wrote: > > > Hi Michal, > > > > > > On Sun, 2 Oct 2022 at 13:34, Michal Suchánek wrote: > > > > > > > > O

Re: [PATCH 4/5] binman: Use an exit code when blobs are missing

2022-10-10 Thread Simon Glass
Hi Tom, On Mon, 10 Oct 2022 at 14:41, Tom Rini wrote: > > On Mon, Oct 10, 2022 at 02:00:31PM -0600, Simon Glass wrote: > > > At present binman returns success when told to handle missing blobs. > > This is confusing this in fact the resulting image cannot work. > > > > Use exit code 103 to signal

Re: [PATCH v6 4/8] net: dsa: allow rcv() and xmit() to be optional

2022-10-10 Thread Simon Glass
Hi Tim, On Mon, 10 Oct 2022 at 10:40, Tim Harvey wrote: > > Allow rcv() and xmit() dsa driver ops to be optional in case a driver > does not care to mangle a packet as in U-Boot only one network port is > enabled at a time and thus no packet mangling is necessary. > > Suggested-by: Vladimir Oltea

Re: [PATCH 1/1] cmd: CMD_CLS should not depend on video

2022-10-10 Thread Simon Glass
On Fri, 7 Oct 2022 at 12:38, Heinrich Schuchardt wrote: > > The cls command works fine on the serial console. There is no reason to > let it depend on the availability of video. > > Signed-off-by: Heinrich Schuchardt > --- > cmd/Kconfig | 1 - > 1 file changed, 1 deletion(-) Reviewed-by: Simon

Re: [PATCH 1/1] doc: describe usage of virt-make-fs for testing

2022-10-10 Thread Simon Glass
On Fri, 7 Oct 2022 at 17:28, Heinrich Schuchardt wrote: > > We want tests to avoid the usage of sudo. Describe that virt-make-fs can > generate disk images without being root. > > Signed-off-by: Heinrich Schuchardt > --- > doc/develop/py_testing.rst | 19 +++ > 1 file changed, 19

Re: [PATCH v4 04/21] dm: blk: Add probe in blk_first_device/blk_next_device

2022-10-10 Thread Michal Suchánek
On Mon, Oct 10, 2022 at 09:49:20PM +0200, Michal Suchánek wrote: > On Sun, Oct 02, 2022 at 07:10:40PM -0600, Simon Glass wrote: > > Hi Michal, > > > > On Sun, 2 Oct 2022 at 13:34, Michal Suchánek wrote: > > > > > > On Thu, Sep 29, 2022 at 04:00:26AM -0600, Simon Glass wrote: > > > > Hi Michal, >

Bad Data CRC

2022-10-10 Thread momo aubin
Hello Simon, My board boots well . When I added the second partition into my kernel I got this message *Verifying Checksum ... Bad Data CRC* what could be the root cause ? Which data is talking about ? u boot data or kernel data ? Thanks in advance.

Re: [PULL] u-boot-usb/master

2022-10-10 Thread Tom Rini
On Mon, Oct 10, 2022 at 09:06:12PM +0200, Marek Vasut wrote: > The following changes since commit 73e741b8ee46eba4e02a3bcc758dbd70540ed3c1: > > Merge tag 'u-boot-nand-20221009' of > https://source.denx.de/u-boot/custodians/u-boot-nand-flash (2022-10-10 > 08:17:08 -0400) > > are available in th

[PATCH] for debug purpose only: add print to debug odd behavior.

2022-10-10 Thread Francis Laniel
Hi. I am really sorry for the delay in answer. Sadly, I am not able to send a v5 for the moment. Indeed, I am still struggling with the same bug since a lot of time... It happens in the Azure pipeline but also locally with test.py. I am sure the bug is related to memory management as the setexp

Re: [PATCH 4/5] binman: Use an exit code when blobs are missing

2022-10-10 Thread Tom Rini
On Mon, Oct 10, 2022 at 02:00:31PM -0600, Simon Glass wrote: > At present binman returns success when told to handle missing blobs. > This is confusing this in fact the resulting image cannot work. > > Use exit code 103 to signal this problem, with a -W option to convert > it to a warning. > > S

Re: [PATCH 1/5] buildman: Handle the MAINTAINERS 'N' tag

2022-10-10 Thread Tom Rini
On Mon, Oct 10, 2022 at 02:00:28PM -0600, Simon Glass wrote: > This is needed for some soon-to-be-applied patches. Scan the configs/ > directory to see if any of the files match. > > Signed-off-by: Simon Glass > Suggested-by: Tom Rini Putting this on my local branch with many defconfigs listed

[PATCH] watchdog: omap_wdt: Switch required include for watchdog defines

2022-10-10 Thread Tom Rini
All of the required values for using the omap_wdt.c driver are found in and this is what is indirectly pulled in via when it exists. Signed-off-by: Tom Rini --- drivers/watchdog/omap_wdt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/watchdog/omap_wdt.c b/drivers

Re: [PATCH 2/5] Makefile: Correct a missing FORCE on the binman rule

2022-10-10 Thread Pali Rohár
On Monday 10 October 2022 14:00:29 Simon Glass wrote: > This is required for if_changed to work correctly. Add it. That is truth. Reviewed-by: Pali Rohár > > Signed-off-by: Simon Glass > --- > > Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Makefile b/Ma

[PATCH 4/5] binman: Use an exit code when blobs are missing

2022-10-10 Thread Simon Glass
At present binman returns success when told to handle missing blobs. This is confusing this in fact the resulting image cannot work. Use exit code 103 to signal this problem, with a -W option to convert it to a warning. Signed-off-by: Simon Glass --- tools/binman/binman.rst | 4 tools/bin

[PATCH 5/5] buildman: Detect binman reporting missing blobs

2022-10-10 Thread Simon Glass
Buildman should consider a build as a success (with warnings) if missing blobs have been dealt with by binman, even though buildman itself returns and error code overall. This is how other warnings are dealt with. We cannot easily access the 103 exit code, so detect the problem in the output. Wit

[PATCH 3/5] doc: Correct the path to the Makefile documentation

2022-10-10 Thread Simon Glass
This is out-of-date now. Fix it. Signed-off-by: Simon Glass --- scripts/Kbuild.include | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/Kbuild.include b/scripts/Kbuild.include index 9c14310ad40..62e0207f91b 100644 --- a/scripts/Kbuild.include +++ b/scripts/Kbuild.incl

[PATCH 2/5] Makefile: Correct a missing FORCE on the binman rule

2022-10-10 Thread Simon Glass
This is required for if_changed to work correctly. Add it. Signed-off-by: Simon Glass --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 45f10759a13..2e8ae768c51 100644 --- a/Makefile +++ b/Makefile @@ -,7 +,7 @@ endef PHONY += in

[PATCH 1/5] buildman: Handle the MAINTAINERS 'N' tag

2022-10-10 Thread Simon Glass
This is needed for some soon-to-be-applied patches. Scan the configs/ directory to see if any of the files match. Signed-off-by: Simon Glass Suggested-by: Tom Rini --- tools/buildman/boards.py | 11 +++ 1 file changed, 11 insertions(+) diff --git a/tools/buildman/boards.py b/tools/bui

[PATCH 0/5] Makefile: Deal with missing blobs consistently

2022-10-10 Thread Simon Glass
Missing blobs should cause the build (with make) to fail, but at present success is returned. This is because binman currently produces an exit code of 0 in this case. Of course this is not correct, since the images cannot actually be used. This series fixes that and adjusts buildman to deal sens

Re: [PATCH v4 04/21] dm: blk: Add probe in blk_first_device/blk_next_device

2022-10-10 Thread Michal Suchánek
On Sun, Oct 02, 2022 at 07:10:40PM -0600, Simon Glass wrote: > Hi Michal, > > On Sun, 2 Oct 2022 at 13:34, Michal Suchánek wrote: > > > > On Thu, Sep 29, 2022 at 04:00:26AM -0600, Simon Glass wrote: > > > Hi Michal, > > > > > > On Sun, 25 Sept 2022 at 02:28, Michal Suchanek wrote: > > > > > > >

Re: Pull request for efi-2023-01-rc1-2

2022-10-10 Thread Tom Rini
On Mon, Oct 10, 2022 at 06:13:24PM +0200, Heinrich Schuchardt wrote: > The following changes since commit 73e741b8ee46eba4e02a3bcc758dbd70540ed3c1: > > Merge tag 'u-boot-nand-20221009' of > https://source.denx.de/u-boot/custodians/u-boot-nand-flash (2022-10-10 > 08:17:08 -0400) > > are availab

Re: Broken watchdog in u-boot master branch

2022-10-10 Thread Pali Rohár
On Monday 10 October 2022 15:24:13 Tom Rini wrote: > On Mon, Oct 10, 2022 at 02:14:25PM -0400, Tom Rini wrote: > > On Mon, Oct 10, 2022 at 08:01:23PM +0200, Pali Rohár wrote: > > > On Monday 10 October 2022 13:56:10 Tom Rini wrote: > > > > On Mon, Oct 10, 2022 at 07:44:05PM +0200, Pali Rohár wrote:

Re: Broken watchdog in u-boot master branch

2022-10-10 Thread Tom Rini
On Mon, Oct 10, 2022 at 02:14:25PM -0400, Tom Rini wrote: > On Mon, Oct 10, 2022 at 08:01:23PM +0200, Pali Rohár wrote: > > On Monday 10 October 2022 13:56:10 Tom Rini wrote: > > > On Mon, Oct 10, 2022 at 07:44:05PM +0200, Pali Rohár wrote: > > > > On Monday 10 October 2022 13:40:38 Tom Rini wrote:

[PULL] u-boot-usb/master

2022-10-10 Thread Marek Vasut
The following changes since commit 73e741b8ee46eba4e02a3bcc758dbd70540ed3c1: Merge tag 'u-boot-nand-20221009' of https://source.denx.de/u-boot/custodians/u-boot-nand-flash (2022-10-10 08:17:08 -0400) are available in the Git repository at: git://source.denx.de/u-boot-usb.git master for

[PATCH v2 2/2] common: board_f: Print information for all sysresets

2022-10-10 Thread Michal Suchanek
Boards can have multiple sysresets, iterate all when printing sysreset info. Fixes: 23471aed5c ("board_f: Add reset status printing") Signed-off-by: Michal Suchanek --- v2: - do not use errno_str - rename flag variable --- common/board_f.c | 29 ++--- 1 file changed, 1

[PATCH v2 1/2] sandbox: Initialize sysreset before relocation

2022-10-10 Thread Michal Suchanek
Without this the early sysreset code cannot be tested. Signed-off-by: Michal Suchanek --- v2: use u-boot,dm-pre-proper rather than u-boot,dm-pre-reloc - sandbox SPL cannot build otherwise --- arch/sandbox/dts/sandbox.dtsi | 1 + arch/sandbox/dts/test.dts | 2 ++ 2 files changed, 3 insert

Re: Broken watchdog in u-boot master branch

2022-10-10 Thread Tom Rini
On Mon, Oct 10, 2022 at 08:01:23PM +0200, Pali Rohár wrote: > On Monday 10 October 2022 13:56:10 Tom Rini wrote: > > On Mon, Oct 10, 2022 at 07:44:05PM +0200, Pali Rohár wrote: > > > On Monday 10 October 2022 13:40:38 Tom Rini wrote: > > > > On Mon, Oct 10, 2022 at 07:22:56PM +0200, Pali Rohár wrot

Re: Broken watchdog in u-boot master branch

2022-10-10 Thread Pali Rohár
On Monday 10 October 2022 13:56:10 Tom Rini wrote: > On Mon, Oct 10, 2022 at 07:44:05PM +0200, Pali Rohár wrote: > > On Monday 10 October 2022 13:40:38 Tom Rini wrote: > > > On Mon, Oct 10, 2022 at 07:22:56PM +0200, Pali Rohár wrote: > > > > On Monday 10 October 2022 12:28:18 Tom Rini wrote: > > >

Re: Broken watchdog in u-boot master branch

2022-10-10 Thread Tom Rini
On Mon, Oct 10, 2022 at 07:44:05PM +0200, Pali Rohár wrote: > On Monday 10 October 2022 13:40:38 Tom Rini wrote: > > On Mon, Oct 10, 2022 at 07:22:56PM +0200, Pali Rohár wrote: > > > On Monday 10 October 2022 12:28:18 Tom Rini wrote: > > > > On Sun, Oct 09, 2022 at 09:12:25PM +0200, Pali Rohár wrot

Re: Broken watchdog in u-boot master branch

2022-10-10 Thread Pali Rohár
On Monday 10 October 2022 13:40:38 Tom Rini wrote: > On Mon, Oct 10, 2022 at 07:22:56PM +0200, Pali Rohár wrote: > > On Monday 10 October 2022 12:28:18 Tom Rini wrote: > > > On Sun, Oct 09, 2022 at 09:12:25PM +0200, Pali Rohár wrote: > > > > Hello! Watchdog code seems to be broken in u-boot master

Re: Broken watchdog in u-boot master branch

2022-10-10 Thread Tom Rini
On Mon, Oct 10, 2022 at 07:22:56PM +0200, Pali Rohár wrote: > On Monday 10 October 2022 12:28:18 Tom Rini wrote: > > On Sun, Oct 09, 2022 at 09:12:25PM +0200, Pali Rohár wrote: > > > Hello! Watchdog code seems to be broken in u-boot master branch. > > > On Nokia N900 I'm getting following message i

Re: Broken watchdog in u-boot master branch

2022-10-10 Thread Pali Rohár
On Monday 10 October 2022 12:28:18 Tom Rini wrote: > On Sun, Oct 09, 2022 at 09:12:25PM +0200, Pali Rohár wrote: > > Hello! Watchdog code seems to be broken in u-boot master branch. > > On Nokia N900 I'm getting following message in qemu: > > > > cyclic function rx51_watchdog took too long: 1u

Re: [PATCH] Nokia RX-51: Fix compilation with non-zero CONFIG_SYS_TEXT_BASE

2022-10-10 Thread Pali Rohár
On Monday 10 October 2022 12:21:56 Tom Rini wrote: > On Mon, Oct 10, 2022 at 06:18:08PM +0200, Pali Rohár wrote: > > On Monday 10 October 2022 09:54:02 Tom Rini wrote: > > > On Sun, Oct 09, 2022 at 09:37:13PM +0200, Pali Rohár wrote: > > > > > > > For some unknown reason GNU assembler version 2.31

Re: [PATCH v2 2/2] mach-meson: g12a: reset usb controller in reset_misc()

2022-10-10 Thread Marek Vasut
On 10/10/22 18:22, Neil Armstrong wrote: Hi, Hi, On 10/10/2022 18:09, Marek Vasut wrote: On 10/7/22 11:38, Mattijs Korpershoek wrote: On some g12a boards like the VIM3L and the SEI610, with some USB cables/hosts, there is a long (5s) delay before between "fastboot reboot" and the host detec

Re: [PATCH v3] schemas: Add schema for U-Boot driver model 'phase tags'

2022-10-10 Thread Tom Rini
On Mon, Oct 10, 2022 at 11:34:00AM -0500, Rob Herring wrote: > On Tue, Oct 4, 2022 at 6:22 PM Simon Glass wrote: > > > > Until recently it has not been possible to add any U-Boot-specific > > properties to the device tree schema. Now that it is somewhat separated > > from Linux and people are feel

[PATCH 3/3] qemu: Try to automatically boot from the QEMU firmware device (qfw)

2022-10-10 Thread Andre Przywara
At the moment the QEMU boot sequence tries various (storage) devices when trying to find a payload to boot. To simplify starting a specific kernel and initrd, there is also the qfw command, which can use the files specified on the QEMU command line, via the -kernel and -initrd options. Add this com

[PATCH 2/3] qfw: return failure when no kernel could be loaded

2022-10-10 Thread Andre Przywara
When we try to load a kernel via the QEMU firmware device, we currently "return -1;" if no kernel was specified on the QEMU command line. This leads to the usage output, which is confusing (since nothing on the command line was really wrong), but also somewhat hides the actual error message. Retur

[PATCH 1/3] qfw: store loaded file size in environment variable

2022-10-10 Thread Andre Przywara
At the moment the QEMU firmware command just prints the size of the loaded binaries on the console. To go with all the other load methods, and many boot scripts' expectations, also store the size of the file loaded last in the environment variable "filesize". We first put the kernel size in there,

[PATCH 0/3] qemu: allow auto-booting through firmware device

2022-10-10 Thread Andre Przywara
Hi, when running QEMU and specifying both a firmware binary (--bios) and a kernel (-kernel) on the command line, QEMU will run the firmware, but will also record information about the provided kernel in the QEMU firmware device. U-Boot's "qfw" command can load the kernel to a provided address then

Re: [PATCH v3] schemas: Add schema for U-Boot driver model 'phase tags'

2022-10-10 Thread Tom Rini
On Mon, Oct 10, 2022 at 11:34:00AM -0500, Rob Herring wrote: > On Tue, Oct 4, 2022 at 6:22 PM Simon Glass wrote: > > > > Until recently it has not been possible to add any U-Boot-specific > > properties to the device tree schema. Now that it is somewhat separated > > from Linux and people are feel

[PATCH v6 7/8] net: add MV88E61xx DSA driver

2022-10-10 Thread Tim Harvey
Add a DSA driver for the MV88E6xxx compatible Ethernet switches. Cc: Marek Behún Cc: Vladimir Oltean Signed-off-by: Tim Harvey Reviewed-by: Vladimir Oltean Reviewed-by: Fabio Estevam --- v6: - update commit msg (s/MV88E61xx/MV88E6xxx) - remove GbE from commit msg and Kconfig desc - squash

[PATCH v6 8/8] board: gw_ventana: enable MV88E6XXX DSA support

2022-10-10 Thread Tim Harvey
Add MV88E6XXX DSA support: - update dt to provide internal MDIO bus and port handles. U-Boot requires a more restrictive subset of the dt bindings required by Linux for the sake of simplifying code - update defconfig to remove old driver and enable new one - replace mv88e61xx_hw_reset weak

[PATCH v6 6/8] net: fec: add support for DM_MDIO

2022-10-10 Thread Tim Harvey
Add support for DM_MDIO by registering a UCLASS_MDIO driver and attempting to use it. This is necessary if wanting to use a DSA driver for example hanging off of the FEC MAC. Care is taken to fallback to non DM_MDIO mii bus as several boards define DM_MDIO without having the proper device-tree con

[PATCH v6 4/8] net: dsa: allow rcv() and xmit() to be optional

2022-10-10 Thread Tim Harvey
Allow rcv() and xmit() dsa driver ops to be optional in case a driver does not care to mangle a packet as in U-Boot only one network port is enabled at a time and thus no packet mangling is necessary. Suggested-by: Vladimir Oltean Signed-off-by: Tim Harvey Reviewed-by: Vladimir Oltean Reviewed-

[PATCH v6 5/8] net: ksz9477: remove unnecessary xmit and recv functions

2022-10-10 Thread Tim Harvey
Remove the unnecessary xmit and recv functions. Signed-off-by: Tim Harvey Reviewed-by: Vladimir Oltean Reviewed-by: Fabio Estevam --- v6: - no changes v5: - added Fabio's rb tag v4: - no changes v3: - added Vladimir's rb tag v2: new patch --- drivers/net/ksz9477.c | 23

[PATCH v6 3/8] net: dsa: ensure dsa driver has proper ops

2022-10-10 Thread Tim Harvey
Add a function to sanity check a dsa driver having proper ops. Suggested-by: Vladimir Oltean Signed-off-by: Tim Harvey Reviewed-by: Vladimir Oltean Reviewed-by: Fabio Estevam --- v6: - no changes v5: - added Fabio's rb tag v4: - no changes v3: - added Vladimir's rb tag v2: new patch --- n

[PATCH v6 2/8] net: dsa: move cpu port probe to dsa_post_probe

2022-10-10 Thread Tim Harvey
In order to ensure that a DSA driver probe gets called before dsa_ops->port_probe move the port_probe of the cpu_port to a post-probe function. Signed-off-by: Tim Harvey Reviewed-by: Ramon Fried Reviewed-by: Vladimir Oltean Reviewed-by: Fabio Estevam --- v6: - removed unnecessary semicolon v5

[PATCH v6 0/8] Add MV88E6xxx DSA driver and use on gwventana

2022-10-10 Thread Tim Harvey
This series adds a DSA driver for the MV88E6xxx based on drivers/net/phy/mv88e61xx and uses it in the gwventana_gw5904_defconfig. The hope is that the other three boards that use the MV88E61xx driver can move to this as well eventually so that we can remove the non-dm driver and the 4 Kconfig opti

[PATCH v6 1/8] net: mdio-uclass: scan for dm mdio children on post-bind

2022-10-10 Thread Tim Harvey
If a DM_MDIO driver is used we need to scan the subnodes as well. Signed-off-by: Tim Harvey Signed-off-by: Vladimir Oltean Reviewed-by: Ramon Fried Reviewed-by: Fabio Estevam --- v6: - no changes v5: - added Fabio's rb tag v4: - no changes v3: - no changes v2: - added Ramon's rb tag ---

Re: [PATCH v3] schemas: Add schema for U-Boot driver model 'phase tags'

2022-10-10 Thread Rob Herring
On Tue, Oct 4, 2022 at 6:22 PM Simon Glass wrote: > > Until recently it has not been possible to add any U-Boot-specific > properties to the device tree schema. Now that it is somewhat separated > from Linux and people are feeling the enormous pain of the bifurcated > schema, it seems like a good

Re: Broken watchdog in u-boot master branch

2022-10-10 Thread Tom Rini
On Sun, Oct 09, 2022 at 09:12:25PM +0200, Pali Rohár wrote: > Hello! Watchdog code seems to be broken in u-boot master branch. > On Nokia N900 I'm getting following message in qemu: > > cyclic function rx51_watchdog took too long: 1us vs 1000us max, disabling > > Seems that watchdog core code

Re: [PATCH v2 2/2] mach-meson: g12a: reset usb controller in reset_misc()

2022-10-10 Thread Neil Armstrong
Hi, On 10/10/2022 18:09, Marek Vasut wrote: On 10/7/22 11:38, Mattijs Korpershoek wrote: On some g12a boards like the VIM3L and the SEI610, with some USB cables/hosts, there is a long (5s) delay before between "fastboot reboot" and the host detecting a USB reset. This breaks tools relying on "

Re: [PATCH] Nokia RX-51: Fix compilation with non-zero CONFIG_SYS_TEXT_BASE

2022-10-10 Thread Tom Rini
On Mon, Oct 10, 2022 at 06:18:08PM +0200, Pali Rohár wrote: > On Monday 10 October 2022 09:54:02 Tom Rini wrote: > > On Sun, Oct 09, 2022 at 09:37:13PM +0200, Pali Rohár wrote: > > > > > For some unknown reason GNU assembler version 2.31.1 (arm-linux-gnueabi-as > > > from Debian Buster) cannot com

Re: Broken watchdog in u-boot master branch

2022-10-10 Thread Pali Rohár
On Monday 10 October 2022 09:55:12 Tom Rini wrote: > On Sun, Oct 09, 2022 at 09:12:25PM +0200, Pali Rohár wrote: > > > Hello! Watchdog code seems to be broken in u-boot master branch. > > On Nokia N900 I'm getting following message in qemu: > > > > cyclic function rx51_watchdog took too long: 100

Re: [PATCH] Nokia RX-51: Fix compilation with non-zero CONFIG_SYS_TEXT_BASE

2022-10-10 Thread Pali Rohár
On Monday 10 October 2022 09:54:02 Tom Rini wrote: > On Sun, Oct 09, 2022 at 09:37:13PM +0200, Pali Rohár wrote: > > > For some unknown reason GNU assembler version 2.31.1 (arm-linux-gnueabi-as > > from Debian Buster) cannot compile following code from located in file > > board/nokia/rx51/lowlevel

Pull request for efi-2023-01-rc1-2

2022-10-10 Thread Heinrich Schuchardt
The following changes since commit 73e741b8ee46eba4e02a3bcc758dbd70540ed3c1: Merge tag 'u-boot-nand-20221009' of https://source.denx.de/u-boot/custodians/u-boot-nand-flash (2022-10-10 08:17:08 -0400) are available in the Git repository at: https://source.denx.de/u-boot/custodians/u-boot-efi

Re: [PATCH v2 2/2] mach-meson: g12a: reset usb controller in reset_misc()

2022-10-10 Thread Marek Vasut
On 10/7/22 11:38, Mattijs Korpershoek wrote: On some g12a boards like the VIM3L and the SEI610, with some USB cables/hosts, there is a long (5s) delay before between "fastboot reboot" and the host detecting a USB reset. This breaks tools relying on "fastboot reboot fastboot" which assume that 1s

Re: [PATCH v2 1/2] usb: gadget: fastboot: detach usb on reboot commands

2022-10-10 Thread Marek Vasut
On 10/7/22 11:38, Mattijs Korpershoek wrote: When host issues "fastboot reboot fastboot", it's expected that the board drops the USB connection before resetting. On some boards, such as Khadas VIM3L and SEI610, this is not the case. We observe the following error: $ fastboot reboot fastboot Reb

Re: [PATCH v5 2/3] arch/riscv: add semihosting support for RISC-V

2022-10-10 Thread Sean Anderson
On 9/23/22 3:03 AM, Kautuk Consul wrote: > We add RISC-V semihosting based serial console for JTAG based early > debugging. > > The RISC-V semihosting specification is available at: > https://github.com/riscv/riscv-semihosting-spec/blob/main/riscv-semihosting-spec.adoc > > Signed-off-by: Anup

Re: [PATCH v2 01/38] Makefile: Fake external blobs by default with binman

2022-10-10 Thread Simon Glass
Hi Tom, On Mon, 10 Oct 2022 at 09:36, Tom Rini wrote: > > On Mon, Oct 10, 2022 at 09:33:59AM -0600, Simon Glass wrote: > > Hi Tom, > > > > On Mon, 10 Oct 2022 at 09:24, Tom Rini wrote: > > > > > > On Mon, Oct 10, 2022 at 09:19:39AM -0600, Simon Glass wrote: > > > > Hi Rasmus, > > > > > > > > On

Re: [PATCH v5 1/3] lib: Add common semihosting library

2022-10-10 Thread Sean Anderson
On 9/23/22 3:03 AM, Kautuk Consul wrote: > We factor out the arch-independent parts of the ARM semihosting > implementation as a common library so that it can be shared > with RISC-V. > > Signed-off-by: Kautuk Consul > --- > arch/arm/Kconfig | 46 - > arch/arm/lib/semihosti

Re: [PATCH] fastboot: mmc: switch to user hwpart after erase/flash

2022-10-10 Thread Sean Anderson
sc); > + > if (blks != blks_size) { > pr_err("failed erasing from device %d\n", dev_desc->devnum); > fastboot_fail("failed erasing from device", response); > diff --git a/include/mmc.h b/include/mmc.h > index 027e8bcc73a6..cdd457a60a5b 100644 > --- a/include/mmc.h > +++ b/include/mmc.h > @@ -362,6 +362,7 @@ enum mmc_voltage { > /* The number of MMC physical partitions. These consist of: > * boot partitions (2), general purpose partitions (4) in MMC v4.4. > */ > +#define MMC_PART_USERDATA0 > #define MMC_NUM_BOOT_PARTITION 2 > #define MMC_PART_RPMB 3 /* RPMB partition number */ > > > --- > base-commit: f5717231abad983b4d8f87db612ae60dec86cb1b > change-id: 20221010-switch-hwpart-7fed7746db59 > > Best regards, >

Re: [PATCH v2 01/38] Makefile: Fake external blobs by default with binman

2022-10-10 Thread Tom Rini
On Mon, Oct 10, 2022 at 09:33:59AM -0600, Simon Glass wrote: > Hi Tom, > > On Mon, 10 Oct 2022 at 09:24, Tom Rini wrote: > > > > On Mon, Oct 10, 2022 at 09:19:39AM -0600, Simon Glass wrote: > > > Hi Rasmus, > > > > > > On Mon, 10 Oct 2022 at 04:16, Rasmus Villemoes > > > wrote: > > > > > > > > O

Re: [PATCH v2 01/38] Makefile: Fake external blobs by default with binman

2022-10-10 Thread Simon Glass
Hi Tom, On Mon, 10 Oct 2022 at 09:24, Tom Rini wrote: > > On Mon, Oct 10, 2022 at 09:19:39AM -0600, Simon Glass wrote: > > Hi Rasmus, > > > > On Mon, 10 Oct 2022 at 04:16, Rasmus Villemoes > > wrote: > > > > > > On 10/01/2022 04.13, Simon Glass wrote: > > > > This behaviour is necessary with boa

Re: [PATCH v2 01/38] Makefile: Fake external blobs by default with binman

2022-10-10 Thread Tom Rini
On Mon, Oct 10, 2022 at 09:19:39AM -0600, Simon Glass wrote: > Hi Rasmus, > > On Mon, 10 Oct 2022 at 04:16, Rasmus Villemoes > wrote: > > > > On 10/01/2022 04.13, Simon Glass wrote: > > > This behaviour is necessary with boards where the binman description > > > requires processing external blobs

Re: [PATCH v2 01/38] Makefile: Fake external blobs by default with binman

2022-10-10 Thread Simon Glass
Hi Rasmus, On Mon, 10 Oct 2022 at 04:16, Rasmus Villemoes wrote: > > On 10/01/2022 04.13, Simon Glass wrote: > > This behaviour is necessary with boards where the binman description > > requires processing external blobs, since these may be missing. > > > > Enable it by default, so that CI is hap

[PATCH 2/2] buildman: Add --allow-missing-binaries flag to build with BINMAN_ALLOW_MISSING=1

2022-10-10 Thread Tom Rini
Add a new flag to buildman so that we will in turn pass BINMAN_ALLOW_MISSING=1 to 'make'. Make use of this flag in CI. Cc: Rasmus Villemoes Cc: Simon Glass Signed-off-by: Tom Rini --- .azure-pipelines.yml| 2 +- .gitlab-ci.yml | 6 +++--- tools/buildman/builder.py

[PATCH 1/2] global: Do not default to faking missing binaries for buildman

2022-10-10 Thread Tom Rini
While it is possible and documented on how to re-run buildman to replace faked required binary files after the fact, this behavior ends up being more confusing than helpful in practice. Switch to requiring BINMAN_ALLOW_MISSING=1 to be passed on the 'make' line to enable this behavior. Cc: Rasmus V

Re: [PATCH] avb: Extend support to non-eMMC interfaces

2022-10-10 Thread Mattijs Korpershoek
Hi Alistair, Thank you for your patch. On lun., sept. 26, 2022 at 22:02, Alistair Delva wrote: > From: Jiyong Park > > Previously Android AVB supported block devices only on eMMC. This change > eliminates the restriction by using the generic block driver model. > > The `avb init' command is mo

Re: [PATCH v2 01/38] Makefile: Fake external blobs by default with binman

2022-10-10 Thread Tom Rini
On Mon, Oct 10, 2022 at 12:15:58PM +0200, Rasmus Villemoes wrote: > On 10/01/2022 04.13, Simon Glass wrote: > > This behaviour is necessary with boards where the binman description > > requires processing external blobs, since these may be missing. > > > > Enable it by default, so that CI is happy

Re: [REGRESSION]: v2022.07: SHA256 hash is broken on imx8m series with CAAM enabled

2022-10-10 Thread Fabio Estevam
Hi Rasmus, On Mon, Oct 10, 2022 at 11:07 AM Rasmus Villemoes wrote: > This is still broken in v2022.10. Are there any plans to fix this? If I recall correctly the fix for this issue is to sync imx8m dts files with Linux so that the following commit can be in place: https://git.kernel.org/pub/sc

Re: [REGRESSION]: v2022.07: SHA256 hash is broken on imx8m series with CAAM enabled

2022-10-10 Thread Rasmus Villemoes
On 15/07/2022 14.40, ZHIZHIKIN Andrey wrote: > Hello Gaurav, > > In the new v2022.07, I've stumbled upon the issue with calculating the SHA256 > of > memory blocks with CAAM hashing. This causes the FIT image not to pass the > hash > validation, and also `sha256` command not operable. > > I'm a

Re: Broken watchdog in u-boot master branch

2022-10-10 Thread Tom Rini
On Sun, Oct 09, 2022 at 09:12:25PM +0200, Pali Rohár wrote: > Hello! Watchdog code seems to be broken in u-boot master branch. > On Nokia N900 I'm getting following message in qemu: > > cyclic function rx51_watchdog took too long: 1us vs 1000us max, disabling > > Seems that watchdog core cod

Re: [PATCH] Nokia RX-51: Fix compilation with non-zero CONFIG_SYS_TEXT_BASE

2022-10-10 Thread Tom Rini
On Sun, Oct 09, 2022 at 09:37:13PM +0200, Pali Rohár wrote: > For some unknown reason GNU assembler version 2.31.1 (arm-linux-gnueabi-as > from Debian Buster) cannot compile following code from located in file > board/nokia/rx51/lowlevel_init.S: > > kernoffs: > .word KERNEL_OFFSET - (. - C

Re: [PATCH 2/7] mtd/nand: try to erase bad blocks only if scrub flag is provided

2022-10-10 Thread Mikhail Kshevetskiy
On Thu, 6 Oct 2022 20:09:11 +0200 Michael Nazzareno Trimarchi wrote: > [External email] > > > > > > Hi Mikhail > > On Thu, Oct 6, 2022 at 6:17 PM Mikhail Kshevetskiy > wrote: > > > > On Thu, 6 Oct 2022 18:03:17 +0200 > > Michael Nazzareno Trimarchi wrote: > > > > > [External email] > > >

Re: Pull request for u-boot-nand-20221009

2022-10-10 Thread Tom Rini
On Sun, Oct 09, 2022 at 05:03:52PM +0200, Dario Binacchi wrote: > Hi Tom, > > The following changes since commit f5717231abad983b4d8f87db612ae60dec86cb1b: > > Merge branch '2022-10-07-riscv-toolchain-update' (2022-10-07 11:25:05 > -0400) > > are available in the Git repository at: > > http

Re: Pull request for efi-2023-01-rc1

2022-10-10 Thread Tom Rini
On Fri, Oct 07, 2022 at 12:33:33AM +0200, Heinrich Schuchardt wrote: > Dear Tom, > > The following changes since commit 2afa989fbecf25ea23902a9c480f179ba608785a: > > Merge https://source.denx.de/u-boot/custodians/u-boot-marvell > (2022-10-06 08:44:23 -0400) > > are available in the Git reposi

Re: Broken commit de47ff536363289f92f85ed1e4901724d238432d

2022-10-10 Thread Marek Behún
On Sun, 9 Oct 2022 12:32:02 -0400 Tom Rini wrote: > On Sun, Oct 09, 2022 at 03:10:45PM +0200, Pali Rohár wrote: > > On Sunday 09 October 2022 08:45:03 Tom Rini wrote: > > > On Sun, Oct 09, 2022 at 02:41:19PM +0200, Pali Rohár wrote: > > > > On Friday 26 August 2022 10:53:58 Tom Rini wrote:

[PATCH] binman: Add support for symlinking images

2022-10-10 Thread Neha Malcom Francis
Adding support to symlink an image packaged using binman. Signed-off-by: Neha Malcom Francis --- tools/binman/image.py | 6 ++ 1 file changed, 6 insertions(+) diff --git a/tools/binman/image.py b/tools/binman/image.py index afc4b4d643..191d121d88 100644 --- a/tools/binman/image.py +++ b/too

Re: [u-boot][PATCH] spl: spl_legacy: Fix NAND boot on OMAP3 BeagleBoard

2022-10-10 Thread Roger Quadros
+Dario On 29/09/2022 13:11, Roger Quadros wrote: > OMAP3 BeagleBoard NAND boot hangs when spl_load_legacy_img() tries > to read the header into 'struct hdr' which is allocated on the > stack. > > As the header has already been read once before by spl_nand.c, > we can avoid the extra header read h

Re: [PATCH v2 01/38] Makefile: Fake external blobs by default with binman

2022-10-10 Thread Rasmus Villemoes
On 10/01/2022 04.13, Simon Glass wrote: > This behaviour is necessary with boards where the binman description > requires processing external blobs, since these may be missing. > > Enable it by default, so that CI is happy. Warnings indicate that a valid > image is not produced, as with the --allo

[PATCH] ARM: dts: stm32: update SCMI dedicated file

2022-10-10 Thread Patrick Delaunay
The PWR regulators don't need be removed as they are already deactivated. This patches is a alignment with the accepted patch in Linux device tree in commit a34b42f8690c ("ARM: dts: stm32: fix pwr regulators references to use scmi"). Fixes: 69ef98b209e7 ("ARM: dts: stm32mp15: alignment with v5.19"

  1   2   >