[PATCH v3 16/38] spl: Remove #ifdefs with BOOTSTAGE

2023-09-24 Thread Simon Glass
This feature has some helpers in its header file so that its functions resolve to nothing when the feature is disabled. Add a few more and use these to simplify the code. With this there are no more #ifdefs in board_init_r() Signed-off-by: Simon Glass --- (no changes since v1) common/spl

[PATCH v3 14/38] dm: core: Correct help in TPL_DM and VPL_DM

2023-09-24 Thread Simon Glass
There are copying errors in the help. Fix these. Signed-off-by: Simon Glass --- Changes in v3: - Add new patch to correct help in TPL_DM and VPL_DM drivers/core/Kconfig | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/core/Kconfig b/drivers/core/Kconfig

[PATCH v3 15/38] spl: Avoid an #ifdef when printing gd->malloc_ptr

2023-09-24 Thread Simon Glass
Use an accessor in the header file to avoid this. Signed-off-by: Simon Glass --- Changes in v3: - Rebase on the new patch Changes in v2: - Use the same condition for both pieces to avoid possible problems common/spl/spl.c | 9 + include/asm-generic/global_data.h | 6

[PATCH v3 13/38] doc: Clean up SYS_MALLOC_SIMPLE

2023-09-24 Thread Simon Glass
Move the useful help to Kconfig. Drop mention of CONFIG_SYS_MALLOC_SIMPLE since it doesn't exist. Correct a 'CONFIGSYS_MALLOC_F_LEN' typo Signed-off-by: Simon Glass --- Changes in v3: - Add new patch to clean up SYS_MALLOC_SIMPLE documentation Kconfig | 15

[PATCH v3 12/38] Tidy up uses of CONFIG_SYS_MALLOC_F_LEN

2023-09-24 Thread Simon Glass
Use CONFIG_SYS_MALLOC_F instead to of CONFIG_SYS_MALLOC_F_LEN to determine whether pre-relocation malloc() is enabled. Signed-off-by: Simon Glass --- Changes in v3: - Add new patch to tidy up uses of CONFIG_SYS_MALLOC_F_LEN Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH v3 11/38] spl: Use SYS_MALLOC_F instead of SYS_MALLOC_F_LEN

2023-09-24 Thread Simon Glass
Use the new SPL/TPL/VPL_SYS_MALLOC_F symbols to determine whether the malloc pool exists. Signed-off-by: Simon Glass --- Changes in v3: - Add new patch to use SYS_MALLOC_F instead of SYS_MALLOC_F_LEN Kconfig | 9 - arch/arm/lib/bdinfo.c

[PATCH v3 10/38] tpl: Enable CONFIG_TPL_SYS_MALLOC_F where needed

2023-09-24 Thread Simon Glass
Enable CONFIG_TPL_SYS_MALLOC_F for boards which have a non-zero value for CONFIG_TPL_SYS_MALLOC_F_LEN Note that the default is yes in most cases, so no changes are needed to board defconfig options. Signed-off-by: Simon Glass --- Changes in v3: - Add new patch to nable CONFIG_TPL_SYS_MALLOC_F

[PATCH v3 09/38] spl: Enable CONFIG_SPL_SYS_MALLOC_F where needed

2023-09-24 Thread Simon Glass
Enable CONFIG_SPL_SYS_MALLOC_F for boards which have a non-zero value for CONFIG_SPL_SYS_MALLOC_F_LEN Note that the default is yes in most cases, so no changes are needed to board defconfig options. Signed-off-by: Simon Glass --- Changes in v3: - Add new patch to enable CONFIG_SPL_SYS_MALLOC_F

[PATCH v3 08/38] spl: Create proper symbols for enabling the malloc() pool

2023-09-24 Thread Simon Glass
. This differing approach is confusing. Add a new CONFIG_SPL_SYS_MALLOC_F symbol for SPL (and similarly for TPL and VPL). Tidy up the Kconfig help for clarity. For now these symbols are not used. That is cleaned up in the following patches. Signed-off-by: Simon Glass --- Changes in v3: - Add new

[PATCH v3 06/38] spl: Avoid #ifdef with CONFIG_SPL_PAYLOAD_ARGS_ADDR

2023-09-24 Thread Simon Glass
Move the condition to the header file to improve readability. Signed-off-by: Simon Glass --- (no changes since v2) Changes in v2: - Change the approach to use the header file common/spl/spl.c | 10 -- include/system-constants.h | 7 +++ 2 files changed, 11 insertions

[PATCH v3 07/38] spl: Drop the switch() statement for OS selection

2023-09-24 Thread Simon Glass
to avoid needing #ifdef in the C code. Signed-off-by: Simon Glass --- (no changes since v1) common/spl/spl.c | 32 ++-- include/spl.h| 9 + 2 files changed, 19 insertions(+), 22 deletions(-) diff --git a/common/spl/spl.c b/common/spl/spl.c index

[PATCH v3 05/38] spl: Drop #ifdefs for BOARD_INIT and watchdog

2023-09-24 Thread Simon Glass
Avoid using the preprocessor for these checks. Signed-off-by: Simon Glass --- (no changes since v1) common/spl/spl.c | 10 -- include/spl.h| 8 ++-- 2 files changed, 10 insertions(+), 8 deletions(-) diff --git a/common/spl/spl.c b/common/spl/spl.c index 4233390d7de2

[PATCH v3 04/38] spl: mx6: powerpc: Drop the condition on timer_init()

2023-09-24 Thread Simon Glass
It doesn't make sense to have some boards do this differently. Drop the condition in the hope that the maintainers can figure out any run-time problems. This has been tested on qemu-ppce500 Signed-off-by: Simon Glass --- Changes in v3: - Mention testing on qemu-ppce500 Changes in v2

[PATCH v3 03/38] spl: Avoid #ifdef with CONFIG_SPL_SYS_MALLOC

2023-09-24 Thread Simon Glass
Use IF_ENABLED_INT() to avoid needing to use the preprocessor. Signed-off-by: Simon Glass --- (no changes since v2) Changes in v2: - Improve readability by moving the size part to the header file common/spl/spl.c | 8 include/system-constants.h | 7 ++- 2 files changed

[PATCH] bootstd: Scan all bootdevs in a boot_targets entry

2023-09-23 Thread Simon Glass
und two. Signed-off-by: Simon Glass Reported-by: Date Huang Reported-by: Vincent Stehlé --- boot/bootdev-uclass.c | 3 ++- boot/bootflow.c | 21 +++-- test/boot/bootdev.c | 10 ++ 3 files changed, 31 insertions(+), 3 deletions(-) diff --git a/boot/bootdev-uclass.c b/b

Re: [PATCH 1/1] starfive: visionfive2: add mmc0 and nvme boot targets

2023-09-23 Thread Simon Glass
Hi Leo, On Wed, 20 Sept 2023 at 06:46, Leo Liang wrote: > > On Mon, Sep 18, 2023 at 10:32:29AM +0200, Milan P. Stanić wrote: > > boot from SDIO3.0 (mmc sdcard) first if it is plugged. > > If mmc is not plugged try to boot from emmc if it is plugged. > > If emmc is not plugged then try to boot

Re: [BUG] issues with new bootflow, uefi and virtio

2023-09-23 Thread Simon Glass
Hi Vincent, On Tue, 11 Apr 2023 at 06:00, Vincent Stehlé wrote: > > On Fri, Apr 07, 2023 at 05:31:06PM +1200, Simon Glass wrote: > .. > > > When combined with the patch from Mathew[1], it does indeed repair the > > > case of > > > efi boot with two virti

Please pull u-boot-dm/next

2023-09-23 Thread Simon Glass
Jonas Karlman (1): dm: core: Report bootph-pre-ram/sram node as pre-reloc after relocation Simon Glass (4): buildman: Keep all common output files buildman: Show progress when regenerating the board.cfg file buildman: Start the clock when the build starts kontron_sl28

Re: [PATCH v2] dm: core: Report bootph-pre-ram/sram node as pre-reloc after relocation

2023-09-23 Thread Simon Glass
the binding and driver model documentation. This changes behavior of what nodes are bound in the U-Boot proper pre-relocation phase. Change to bootph-all or add bootph-some-ram prop to restore prior behavior. Signed-off-by: Jonas Karlman Reviewed-by: Simon Glass --- Changes in v2: - Drop use

Re: [PATCH v2] cmd: dm: allow for selecting uclass and device

2023-09-23 Thread Simon Glass
Hi, On Wed, 30 Aug 2023 at 22:34, AKASHI Takahiro wrote: > > On Wed, Aug 30, 2023 at 09:48:48PM -0600, Simon Glass wrote: > > Hi AKASHI, > > > > On Tue, 22 Aug 2023 at 19:50, AKASHI Takahiro > > wrote: > > > > > > The output from "dm tree&

Re: [PATCH v2 1/4] buildman: Keep all common output files

2023-09-23 Thread Simon Glass
On Thu, Sep 07, 2023 at 10:00:17AM -0600, Simon Glass wrote: > Make a list of common output extensions and use it to ensure that the -k > option preserves all of these. > > Signed-off-by: Simon Glass > Suggested-by: Tom Rini Reviewed-by: Tom Rini -- Tom Applied to u-boot-dm/next, thanks!

Re: [PATCH v2 2/4] buildman: Show progress when regenerating the board.cfg file

2023-09-23 Thread Simon Glass
This can take a while, so show a message when starting. Signed-off-by: Simon Glass Reported-by Tom Rini --- (no changes since v1) tools/buildman/boards.py | 15 --- tools/buildman/control.py | 3 ++- 2 files changed, 14 insertions(+), 4 deletions(-) Applied to u-boot-dm/next

Re: [PATCH v2 4/4] kontron_sl28: Use u-boot-update.bin instead of u-boot.update

2023-09-23 Thread Simon Glass
A '.update' extension does not get preserved by buildman, so change it. Signed-off-by: Simon Glass Acked-by: Michael Walle --- (no changes since v1) arch/arm/dts/fsl-ls1028a-kontron-sl28-u-boot.dtsi | 2 +- doc/board/kontron/sl28.rst| 4 ++-- 2 files changed, 3

Re: [PATCH v2 3/4] buildman: Start the clock when the build starts

2023-09-23 Thread Simon Glass
The Kconfig and maintainer processing can take a while, sometimes 5 seconds or more. This skews the timing printed by buildmand when the build completes. Start the clock when the threads start to avoid this problem. Signed-off-by: Simon Glass Suggested-by: Tom Rini --- (no changes since v1

[PATCH v3 25/27] moveconfig: Rename the tool to qconfig

2023-09-23 Thread Simon Glass
This does not move configs anymore, but queries them, based on a database it can build. Rename the tool to better reflect its purpose. Signed-off-by: Simon Glass --- (no changes since v1) tools/{moveconfig.py => qconfig.py} | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) ren

[PATCH v3 27/27] qconfig: Update the documentation

2023-09-23 Thread Simon Glass
Update qconfig's documentation to better reflect its new purpose in life. Signed-off-by: Simon Glass --- Changes in v3: - Update based on Heinrich's comments Changes in v2: - Reinstate the -C option doc/develop/index.rst | 2 +- doc/develop/{moveconfig.rst

[PATCH v3 24/27] moveconfig: Move summaries to the end

2023-09-23 Thread Simon Glass
Write the summary for -s and -b at the end, using a unified format. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 36 +++- 1 file changed, 23 insertions(+), 13 deletions(-) diff --git a/tools/moveconfig.py b/tools/moveconfig.py

[PATCH v3 26/27] qconfig: Rename the database file

2023-09-23 Thread Simon Glass
Use qconfig.db as the new name, to reflect the tool's purpose. Signed-off-by: Simon Glass --- (no changes since v1) .gitignore | 4 ++-- tools/qconfig.py | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/.gitignore b/.gitignore index 002f95de4feb..7f64446db3c8

[PATCH v3 22/27] moveconfig: Show a summary at the end

2023-09-23 Thread Simon Glass
Rather than printing all the failed boards, which are now easily visible on the terminal, just show a summary. Sort it by defconfig and drop the '_defconfig' suffix. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 28 +++- 1 file changed, 15

[PATCH v3 20/27] moveconfig: Use u_boot_pylib for terminal colour

2023-09-23 Thread Simon Glass
Use the existing terminal code to handle ANSI colours. Enable colour by default if the output is going to a terminal. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 92 +++-- 1 file changed, 38 insertions(+), 54 deletions

[PATCH v3 23/27] moveconfig: Drop the initial output

2023-09-23 Thread Simon Glass
Since moveconfig now just does what it is told (build database or sync defconfigs) we don't need to print what it is doing. Drop this info, which is of very little use. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 6 -- 1 file changed, 6 deletions(-) diff

[PATCH v3 21/27] moveconfig: Show failures in progress

2023-09-23 Thread Simon Glass
Show the number of accumulated failures when processing. Use a shorter format with colour. An unwanted space appears before the defconfig name on every item except the last. Fix that while we are here. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 26

[PATCH v3 17/27] moveconfig: Only show output when there is a reason

2023-09-23 Thread Simon Glass
in this mode. Signed-off-by: Simon Glass --- Changes in v3: - Fix 'Supress' typo tools/moveconfig.py | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tools/moveconfig.py b/tools/moveconfig.py index 727a5d6df864..568386f7b9d0 100755 --- a/tools/moveconfig.py +++ b/tools

[PATCH v3 19/27] moveconfig: Avoid showing progress at the end

2023-09-23 Thread Simon Glass
When the process is finished, moveconfig leaves a line saying that all boards were processed (for better or worse). Drop this, since it is unncessary. Future work will provide a summary at the end instead. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 3 ++- 1

[PATCH v3 18/27] moveconfig: Reduce the amount of output

2023-09-23 Thread Simon Glass
Output a single line in the case where the defconfig only has one line of output. Show the name without the _defconfig suffix, since that is the same for all boards. Use a list for the log so it is easier to process at the end. Signed-off-by: Simon Glass --- (no changes since v1) tools

[PATCH v3 13/27] moveconfig: Correct use of members not declared in __init__()

2023-09-23 Thread Simon Glass
Fix these pylint warnings. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 4 1 file changed, 4 insertions(+) diff --git a/tools/moveconfig.py b/tools/moveconfig.py index 2234f28a7724..1dff89159312 100755 --- a/tools/moveconfig.py +++ b/tools/moveconfig.py

[PATCH v3 14/27] moveconfig: Correct list-comprehension warnings

2023-09-23 Thread Simon Glass
Correct some pylint warnings about needing to use list comprehension. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tools/moveconfig.py b/tools/moveconfig.py index 1dff89159312..d46433c6d8f5

[PATCH v3 16/27] moveconfig: Fix misc pylint warnings

2023-09-23 Thread Simon Glass
Fix various remaining pylint warnings. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 23 --- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/tools/moveconfig.py b/tools/moveconfig.py index 9b716596956b..727a5d6df864 100755

[PATCH v3 15/27] moveconfig: Use an encoding with open()

2023-09-23 Thread Simon Glass
Fix pylint warnings about needing an explicit character encoding. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tools/moveconfig.py b/tools/moveconfig.py index d46433c6d8f5..9b716596956b

[PATCH v3 12/27] moveconfig: Correct non-snake variables names

2023-09-23 Thread Simon Glass
Correct some variable names that do not conform to snake case, with the three-character minimum. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 70 ++--- 1 file changed, 35 insertions(+), 35 deletions(-) diff --git a/tools

[PATCH v3 11/27] moveconfig: Correct unused variables

2023-09-23 Thread Simon Glass
Fix pylint warnings about unused variables. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 26 -- 1 file changed, 12 insertions(+), 14 deletions(-) diff --git a/tools/moveconfig.py b/tools/moveconfig.py index 42298dd5d9c8..911e272d11a3

[PATCH v3 10/27] moveconfig: Use f strings where possible

2023-09-23 Thread Simon Glass
Avoid pylint warnings by using 'f' strings where possible. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 56 ++--- 1 file changed, 28 insertions(+), 28 deletions(-) diff --git a/tools/moveconfig.py b/tools/moveconfig.py

[PATCH v3 09/27] moveconfig: Correct parameter-type warnings

2023-09-23 Thread Simon Glass
Fix pylint warnings related to parameter types. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 43 ++- 1 file changed, 22 insertions(+), 21 deletions(-) diff --git a/tools/moveconfig.py b/tools/moveconfig.py index

[PATCH v3 08/27] moveconfig: Correct some regular-expression strings

2023-09-23 Thread Simon Glass
Use the 'r' prefix for these strings to avoid pylint warnings. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tools/moveconfig.py b/tools/moveconfig.py index 36202fe5134c..0b0ce64e81fe 100755

[PATCH v3 05/27] moveconfig: Drop CONFIG-moving code

2023-09-23 Thread Simon Glass
As a step towards cleaning out old code, drop most of the code that moves CONFIG options to Kconfig. This includes parse_one_config(). Drop the ACTION_... values as well, since they are no-longer used. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 163

[PATCH v3 07/27] moveconfig: Drop suspicious boards

2023-09-23 Thread Simon Glass
This code isn't needed anymore. Drop it. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 25 - 1 file changed, 25 deletions(-) diff --git a/tools/moveconfig.py b/tools/moveconfig.py index 87a670f6e855..36202fe5134c 100755 --- a/tools

[PATCH v3 06/27] moveconfig: Drop check_defconfig() and update_dotconfig()

2023-09-23 Thread Simon Glass
These functions are not needed anymore. Drop them. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 62 ++--- 1 file changed, 2 insertions(+), 60 deletions(-) diff --git a/tools/moveconfig.py b/tools/moveconfig.py index

[PATCH v3 04/27] moveconfig: Drop unused cleanup options

2023-09-23 Thread Simon Glass
Cleaning up the README and config.h files are not needed now, since we don't have any CONFIG options to convert. Drop this code. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 254 +--- 1 file changed, 2 insertions(+), 252

[PATCH v3 03/27] moveconfig: Correct ordering of asteval import

2023-09-23 Thread Simon Glass
This should be after the standard imports. Move it to avoid a lot of pylint warnings. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/moveconfig.py b/tools/moveconfig.py index 2f7dee88c7ee

[PATCH v3 02/27] moveconfig: Avoid deprecation warning for setDaemon

2023-09-23 Thread Simon Glass
Use the recommended new way of setting a thread to be a daemon. This avoids a warning: DeprecationWarning: setDaemon() is deprecated, set the daemon attribute instead Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH v3 01/27] moveconfig: Drop -H option

2023-09-23 Thread Simon Glass
Drop this option, which is no longer needed now that we have converted all CONFIG options to Kconfig. Signed-off-by: Simon Glass --- (no changes since v1) tools/moveconfig.py | 18 +++--- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/tools/moveconfig.py b/tools

[PATCH v3 00/27] moveconfig: Drop old code and tidy up

2023-09-23 Thread Simon Glass
the -C option Simon Glass (27): moveconfig: Drop -H option moveconfig: Avoid deprecation warning for setDaemon moveconfig: Correct ordering of asteval import moveconfig: Drop unused cleanup options moveconfig: Drop CONFIG-moving code moveconfig: Drop check_defconfig

Re: [PATCH] arm: rpi: set boot_targets with correct macro expansion

2023-09-23 Thread Simon Glass
Hi Date, On Sat, 23 Sept 2023 at 12:34, Date Huang wrote: > > rpi uboot will only try to load mmcnr@7e30.bootdev > which is wlan interface if boot_targets only has `mmc` > Change it to `mmc0 mmc1 mmc2 usb0 pxe dhcp` > > Signed-off-by: Date Huang > --- > board/raspberrypi/rpi/rpi.env | 2 +-

Re: Reading from ISO9660 in U-Boot

2023-09-23 Thread Simon Glass
Hi Heinrich, On Sat, 23 Sept 2023 at 12:41, Heinrich Schuchardt wrote: > > > > Am 23. September 2023 16:36:32 MESZ schrieb Simon Glass : > >Hi Heinrich, > > > >On Fri, 22 Sept 2023 at 19:38, Heinrich Schuchardt > >wrote: > >> > >> On 9/23

Re: [PATCH] rpi: Use OF_BOARD rather than OF_EMBED

2023-09-23 Thread Simon Glass
Hi, On Tue, 28 Feb 2023 at 10:25, Tom Rini wrote: > > On Tue, Feb 28, 2023 at 03:56:05PM +, Peter Robinson wrote: > > > The use of OF_EMBED isn't recommended for standard use and the OF_BOARD > > is actually a better fit for the Raspberry Pi as the prior firmware > > provides a DT that can

Re: [PATCH v2 27/27] qconfig: Update the documentation

2023-09-23 Thread Simon Glass
Hi Heinrich, On Thu, 14 Sept 2023 at 20:39, Heinrich Schuchardt wrote: > > > > Am 15. September 2023 02:22:29 MESZ schrieb Simon Glass : > >Update qconfig's documentation to better reflect its new purpose in life. > > > >Signed-off-by: Simon Glass > >---

Please pull u-boot-dm

2023-09-23 Thread Simon Glass
): sandbox: test: Fix typo in test.dts trace: Use 64bit variable for start and len trace: Move trace_clocks description above record offset calculation trace: Fix alignment logic in flyrecord header Rong Tao (1): binman: Fix SyntaxWarning: invalid escape sequence '\(' Simon

Re: [PATCH] tools: Fix patman launcher script.

2023-09-23 Thread Simon Glass
> > Signed-off-by: Maxim Cournoyer > --- > > tools/patman/pyproject.toml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Reviewed-by: Simon Glass Applied to u-boot-dm, thanks!

Re: [PATCH] buildman: Fix full help for Python 3.8

2023-09-23 Thread Simon Glass
With Python versions older than 3.9 Buildman produces an error on start-up. Fix this with a workaround for importlib. There is already a workaround for v3.6 but I am not sure if that is still functioning. Signed-off-by: Simon Glass --- tools/buildman/main.py | 10 -- 1 file changed, 8

Re: [PATCH v2] binman: Fix SyntaxWarning: invalid escape sequence '\('

2023-09-23 Thread Simon Glass
On 9/11/23 6:36 AM, Simon Glass wrote: > Hi, > > On Tue, 5 Sept 2023 at 19:34, 荣涛 wrote: >> Thanks, Simon >> >> The following lines is my systeme ENV and make log >> >> $ cat /etc/os-release >> NAME="Fedora Linux" >> VERSION="

Re: [PATCH] patman: Respect include directive on Git config lookup

2023-09-23 Thread Simon Glass
licitly to support such use cases. > > Signed-off-by: Fei Shao > --- > > tools/patman/gitutil.py | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Simon Glass Applied to u-boot-dm, thanks!

Re: [PATCH v2 1/3] trace: Use 64bit variable for start and len

2023-09-23 Thread Simon Glass
tputq() requires variables to have 64bit width that's why make them 64bit to clean alignment requirement. Signed-off-by: Michal Simek Reviewed-by: Simon Glass --- (no changes since v1) tools/proftool.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) Applied to u-boot-dm, thanks!

Re: [PATCH v2 2/3] trace: Move trace_clocks description above record offset calculation

2023-09-23 Thread Simon Glass
Flyrecord tracing data are page aligned that's why it is necessary to calculate alignment properly. Because trace_clocks description is the part of record length it is necessary to have information about length earlier. Signed-off-by: Michal Simek Reviewed-by: Simon Glass --- (no changes since

Re: [PATCH v2 3/3] trace: Fix alignment logic in flyrecord header

2023-09-23 Thread Simon Glass
chal Simek Reviewed-by: Simon Glass --- Changes in v2: - s/start_addr/start_ofs/g' tools/proftool.c | 31 +-- 1 file changed, 29 insertions(+), 2 deletions(-) Applied to u-boot-dm, thanks!

Re: Reading from ISO9660 in U-Boot

2023-09-23 Thread Simon Glass
Hi Heinrich, On Fri, 22 Sept 2023 at 19:38, Heinrich Schuchardt wrote: > > On 9/23/23 00:13, Simon Glass wrote: > > Hi Heinrich & Bin, > > > > I'd like to be able to figure out in U-Boot what OS is on a USB stick. > > For example, with the Ubuntu inst

Re: Reading from ISO9660 in U-Boot

2023-09-23 Thread Simon Glass
Hi Heinrich, On Fri, 22 Sept 2023 at 19:02, Heinrich Schuchardt wrote: > > On 9/23/23 00:13, Simon Glass wrote: > > Hi Heinrick & Bin, > > > > I'd like to be able to figure out in U-Boot what OS is on a USB stick. > > For example, with the Ubuntu inst

Re: [PATCH 09/32] spl: Avoid an #ifdef when printing gd->malloc_ptr

2023-09-22 Thread Simon Glass
Hi Tom, On Fri, 22 Sept 2023 at 13:28, Tom Rini wrote: > > On Fri, Sep 22, 2023 at 12:27:36PM -0600, Simon Glass wrote: > > Hi Tom, > > > > On Thu, 21 Sept 2023 at 09:36, Tom Rini wrote: > > > > > > On Wed, Sep 20, 2023 at 07:03:34P

Reading from ISO9660 in U-Boot

2023-09-22 Thread Simon Glass
Hi Heinrick & Bin, I'd like to be able to figure out in U-Boot what OS is on a USB stick. For example, with the Ubuntu installer, I can boot it (through grub), but I cannot see how to read anything useful from the USB stick that would indicate that it is Ubuntu, what version it is, etc. Does

[PATCH v2 4/4] usb: Avoid unbinding devices in use by bootflows

2023-09-22 Thread Simon Glass
when booting a distro from USB. This was found using a device with 4 bootflows, the last of which was USB. Signed-off-by: Simon Glass --- Changes in v2: - Add new patch to avoid unbinding devices in use by bootflows boot/bootm.c | 2 +- common/usb.c | 7

[PATCH v2 1/4] efi: Correct handling of frame buffer

2023-09-22 Thread Simon Glass
t;Things that are private to the uclass: don't use these in the driver") which is why this was missed when the VIDEO_COPY feature was added. Signed-off-by: Simon Glass Fixes: 8f661a5b662 ("efi_loader: gop: Expose fb when 32bpp") --- Changes in v2: - Rebase to -next - Add some more comm

[PATCH v2 3/4] x86: coreboot: Add a boot script

2023-09-22 Thread Simon Glass
Provide the user with a list of available boot options. Selecting one causes it to be booted. Pressing causes U-Boot to return to the command-line prompt. Signed-off-by: Simon Glass --- Changes in v2: - Add new patch to add a coreboot boot script configs/coreboot64_defconfig | 1 + configs

[PATCH v2 2/4] bootstd: Add a return code to bootflow menu

2023-09-22 Thread Simon Glass
Return an error when the user does not select an OS, so we know whether to boot or not. Signed-off-by: Simon Glass --- Changes in v2: - Add new patch to add a return code to bootflow menu cmd/bootflow.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/cmd/bootflow.c b

[PATCH v2 0/4] Resolve issues with booting distros on x86

2023-09-22 Thread Simon Glass
more comments to the header file - Add fixes tag - Add new patch to add a return code to bootflow menu - Add new patch to add a coreboot boot script - Add new patch to avoid unbinding devices in use by bootflows Simon Glass (4): efi: Correct handling of frame buffer bootstd: Add a return code

Re: [PATCH] dt-bindings: mtd: Add a schema for binman

2023-09-22 Thread Simon Glass
Hi Rob, On Fri, 22 Sept 2023 at 13:43, Rob Herring wrote: > > On Fri, Sep 22, 2023 at 1:12 PM Simon Glass wrote: > > > > Hi Rob, > > > > On Fri, 22 Sept 2023 at 11:46, Rob Herring wrote: > > > > > > On Fri, Sep 22, 2023 at 11:

Re: [Question] TI's u-boot.img is built twice

2023-09-22 Thread Simon Glass
Hi Andrew, On Fri, 22 Sept 2023 at 12:44, Andrew Davis wrote: > > On 9/22/23 1:29 PM, Simon Glass wrote: > > Hi Andrew, > > > > On Fri, 22 Sept 2023 at 12:14, Andrew Davis wrote: > >> > >> On 9/22/23 11:40 AM, Simon Glass wrote: > >>> Hi And

[PATCH v2] dt-bindings: mtd: Add a schema for binman

2023-09-22 Thread Simon Glass
://lore.kernel.org/u-boot/20230821180220.2724080-3-...@chromium.org/ [3] https://www.spinics.net/lists/devicetree/msg626149.html Signed-off-by: Simon Glass --- Changes in v2: - Use "binman" for compatible instead of "u-boot,binman" - Significantly rework the patch - Use make dt_binding_c

Re: [Question] TI's u-boot.img is built twice

2023-09-22 Thread Simon Glass
Hi Andrew, On Fri, 22 Sept 2023 at 12:14, Andrew Davis wrote: > > On 9/22/23 11:40 AM, Simon Glass wrote: > > Hi Andrew, > > > > On Fri, 22 Sept 2023 at 10:35, Andrew Davis wrote: > >> > >> On 9/22/23 10:01 AM, Simon Glass wrote: > >>>

Re: [PATCH v3 5/9] board_f: Fix corruption of relocaddr

2023-09-22 Thread Simon Glass
Hi Devarsh, On Tue, 12 Sept 2023 at 08:35, Devarsh Thakkar wrote: > > Hi Simon, > > On 11/09/23 04:44, Simon Glass wrote: > > Hi Devarsh, > > > > On Thu, 17 Aug 2023 at 09:10, Tom Rini wrote: > >> > >> On Wed, Aug 16, 2023 at 09:16:0

Re: [PATCH 09/32] spl: Avoid an #ifdef when printing gd->malloc_ptr

2023-09-22 Thread Simon Glass
Hi Tom, On Thu, 21 Sept 2023 at 09:36, Tom Rini wrote: > > On Wed, Sep 20, 2023 at 07:03:34PM -0600, Simon Glass wrote: > > Hi Tom, > > > > On Wed, 30 Aug 2023 at 15:39, Tom Rini wrote: > > > > > > On Wed, Aug 30, 2023 at 12:04:40PM -0600,

Re: [PATCH 4/9] configs: sandbox: Enable GETOPT for sandbox and sandbox64 target

2023-09-22 Thread Simon Glass
On Wed, 20 Sept 2023 at 16:58, Marek Vasut wrote: > > Enable GETOPT so that 'bdinfo' command with getopt() support can be > tested in CI. > > Signed-off-by: Marek Vasut > --- > Cc: Bin Meng > Cc: Mario Six > Cc: Nikhil M Jain > Cc: Simon Glass > --- &

Re: [PATCH 7/9] test: bdinfo: Test bdinfo -h

2023-09-22 Thread Simon Glass
On Wed, 20 Sept 2023 at 16:58, Marek Vasut wrote: > > The bdinfo -h should print error message that -h is an unknown > parameter and then command help text. Test the expected output. > > Signed-off-by: Marek Vasut > --- > Cc: Bin Meng > Cc: Mario Six > Cc: Nikhi

Re: [PATCH 6/9] test: bdinfo: Test both bdinfo and bdinfo -a

2023-09-22 Thread Simon Glass
in Meng > Cc: Mario Six > Cc: Nikhil M Jain > Cc: Simon Glass > --- > test/cmd/bdinfo.c | 17 - > 1 file changed, 12 insertions(+), 5 deletions(-) > Reviewed-by: Simon Glass

Re: [RFC 6/6] test: dm: add SCMI pinctrl test

2023-09-22 Thread Simon Glass
Hi, On Sun, 10 Sept 2023 at 23:47, AKASHI Takahiro wrote: > > On Sun, Sep 10, 2023 at 01:13:30PM -0600, Simon Glass wrote: > > Hi AKASHI, > > > > On Tue, 5 Sept 2023 at 20:41, AKASHI Takahiro > > wrote: > > > > > > In this test, SCMI-based pin

Re: [PATCH 5/9] test: bdinfo: Rename bdinfo_test_move() to bdinfo_test_full()

2023-09-22 Thread Simon Glass
Cc: Nikhil M Jain > Cc: Simon Glass > --- > test/cmd/bdinfo.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > Reviewed-by: Simon Glass

Re: [PATCH 1/9] cmd: bdinfo: Optionally use getopt and implement bdinfo -a

2023-09-22 Thread Simon Glass
is implemented in preparation for other > more fine-grained options. > > Signed-off-by: Marek Vasut > --- > Cc: Bin Meng > Cc: Mario Six > Cc: Nikhil M Jain > Cc: Simon Glass > --- > cmd/bdinfo.c | 29 + > 1 file changed, 25 insertions(+), 4 deletions(-) Reviewed-by: Simon Glass

Re: [PATCH 8/9] test: bdinfo: Test bdinfo -m

2023-09-22 Thread Simon Glass
On Wed, 20 Sept 2023 at 16:58, Marek Vasut wrote: > > The bdinfo -m should print only the board memory layout. > Test the expected output. > > Signed-off-by: Marek Vasut > --- > Cc: Bin Meng > Cc: Mario Six > Cc: Nikhil M Jain > Cc: Simon Glass &

Re: [PATCH v4] bootstd: sata: Add bootstd support for ahci sata

2023-09-22 Thread Simon Glass
+++ > 5 files changed, 112 insertions(+), 4 deletions(-) > create mode 100644 drivers/ata/sata_bootdev.c > Reviewed-by: Simon Glass

Re: [PATCH 2/9] cmd: bdinfo: Implement support for printing memory layout via bdinfo -m

2023-09-22 Thread Simon Glass
On Wed, 20 Sept 2023 at 16:58, Marek Vasut wrote: > > Add support for printing memory layout only via 'bdinfo -m' . > > Signed-off-by: Marek Vasut > --- > Cc: Bin Meng > Cc: Mario Six > Cc: Nikhil M Jain > Cc: Simon Glass > --- > cmd/bdinfo.c | 5 - >

Re: [PATCH 9/9] test: bdinfo: Test bdinfo -e

2023-09-22 Thread Simon Glass
On Wed, 20 Sept 2023 at 16:58, Marek Vasut wrote: > > The bdinfo -e should print only the board ethernet settings. > Test the expected output. > > Signed-off-by: Marek Vasut > --- > Cc: Bin Meng > Cc: Mario Six > Cc: Nikhil M Jain > Cc: Simon Glass &

Re: [PATCH 3/9] cmd: bdinfo: Implement support for printing ethernet settings via bdinfo -e

2023-09-22 Thread Simon Glass
On Wed, 20 Sept 2023 at 16:58, Marek Vasut wrote: > > Add support for printing ethernet settings only via 'bdinfo -e' . > > Signed-off-by: Marek Vasut > --- > Cc: Bin Meng > Cc: Mario Six > Cc: Nikhil M Jain > Cc: Simon Glass > --- > cmd/bdinfo.c | 7 ++

Re: [PATCH v2] bootstd: Make efi_mgr bootmeth work for non-sandbox setups

2023-09-22 Thread Simon Glass
On Tue, 12 Sept 2023 at 14:06, Mark Kettenis wrote: > > > From: Simon Glass > > Date: Sun, 10 Sep 2023 13:13:16 -0600 > > > > Hi Mark, > > > > On Sun, 10 Sept 2023 at 08:57, Mark Kettenis > > wrote: > > > > > > &g

Re: [PATCH] dt-bindings: mtd: Add a schema for binman

2023-09-22 Thread Simon Glass
Hi Rob, On Fri, 22 Sept 2023 at 11:46, Rob Herring wrote: > > On Fri, Sep 22, 2023 at 11:01:18AM -0600, Simon Glass wrote: > > Hi Rob, > > > > On Fri, 22 Sept 2023 at 10:00, Rob Herring wrote: > > > > > > On Thu, Sep 21, 2023 at 1:45 PM Simon Glass

Re: [PATCH] dt-bindings: mtd: Add a schema for binman

2023-09-22 Thread Simon Glass
Hi Rob, On Fri, 22 Sept 2023 at 10:00, Rob Herring wrote: > > On Thu, Sep 21, 2023 at 1:45 PM Simon Glass wrote: > > > > Binman[1] is a tool for creating firmware images. It allows you to > > combine various binaries and place them in an output file. > > > >

Re: [Question] TI's u-boot.img is built twice

2023-09-22 Thread Simon Glass
Hi Andrew, On Fri, 22 Sept 2023 at 10:35, Andrew Davis wrote: > > On 9/22/23 10:01 AM, Simon Glass wrote: > > Hi Masahiro, > > > > On Thu, 21 Sept 2023 at 09:36, Masahiro Yamada wrote: > >> > >> Hi. > >> > >> Since the TI

Re: [PATCH 1/4] mkimage: also honour -B even without external data

2023-09-22 Thread Simon Glass
Hi Rasmus, On Thu, 21 Sept 2023 at 01:57, Rasmus Villemoes wrote: > > On 21/09/2023 03.02, Simon Glass wrote: > > Hi Rasmus, > > > > On Tue, 19 Sept 2023 at 05:37, Rasmus Villemoes > > wrote: > >> > >> In some cases, using the "external dat

Re: [PATCH] dt-bindings: mtd: Add a schema for binman

2023-09-22 Thread Simon Glass
Hi Krzysztof, On Fri, 22 Sept 2023 at 01:02, Krzysztof Kozlowski wrote: > > On 21/09/2023 20:45, Simon Glass wrote: > > Binman[1] is a tool for creating firmware images. It allows you to > > combine various binaries and place them in an output file. > > > > Binma

Re: [PATCH] dt-bindings: mtd: Add a schema for binman

2023-09-22 Thread Simon Glass
Hi Alper, On Fri, 22 Sept 2023 at 07:57, Alper Nebi Yasak wrote: > > On 2023-09-21 21:45 +03:00, Simon Glass wrote: > > Binman[1] is a tool for creating firmware images. It allows you to > > combine various binaries and place them in an output file. > > > > Binma

Re: [Question] TI's u-boot.img is built twice

2023-09-22 Thread Simon Glass
Hi Masahiro, On Fri, 22 Sept 2023 at 01:19, Masahiro Yamada wrote: > > On Fri, Sep 22, 2023 at 2:27 PM Neha Malcom Francis wrote: > > > > Hi Masahiro > > > > On 21/09/23 21:06, Masahiro Yamada wrote: > > > Hi. > > > > > > Since the TI platform migrated to binman, > > > u-boot.img is built

Re: [Question] TI's u-boot.img is built twice

2023-09-22 Thread Simon Glass
Hi Masahiro, On Thu, 21 Sept 2023 at 09:36, Masahiro Yamada wrote: > > Hi. > > Since the TI platform migrated to binman, > u-boot.img is built twice. > > It is created by "mkimage -E", > then overwritten by binman. > > > So, the data are embedded in the FIT structure > instead of being appended.

[PATCH] dt-bindings: mtd: Add a schema for binman

2023-09-21 Thread Simon Glass
/devicetree/msg626149.html Signed-off-by: Simon Glass --- .../bindings/mtd/partitions/binman.yaml | 50 +++ .../bindings/mtd/partitions/binman/entry.yaml | 61 +++ .../bindings/mtd/partitions/partitions.yaml | 1 + MAINTAINERS | 5

<    16   17   18   19   20   21   22   23   24   25   >