[PATCH] arm: kirkwood: Add support for ZyXEL NSA325 board

2023-08-25 Thread Tony Dinh
ZyXEL NSA325 specifications: Marvell Kirkwood 88F6282 SoC 1.6 GHz CPU 1x GBE LAN port (Marvell MV88E1318) 512 MB RAM 128 MB Eon NAND, SLC I2C 1x USB 3.0 (on PCIe bus) 2x USB 2.0 2x SATA (hot swap slots) Serial console Signed-off-by: Tony Dinh --- arch/arm/dts/Makefile| 1

Re: [PATCH 1/2] cmd: efidebug: add uri device path

2023-08-25 Thread Heinrich Schuchardt
On 8/25/23 07:57, Masahisa Kojima wrote: On Thu, 24 Aug 2023 at 09:11, AKASHI Takahiro wrote: Hi Kojima-san, On Wed, Aug 23, 2023 at 05:37:19PM +0900, Masahisa Kojima wrote: This adds the URI device path option for 'boot add' subcommand. User can add the URI load option for downloading ISO

Re: [PATCH 3/3] cmd: let gpt_partition_entry be hexadecimal

2023-08-25 Thread Simon Glass
On Fri, 25 Aug 2023 at 19:53, Heinrich Schuchardt wrote: > > In commands like 'ls mmc 0:f' the partition number is hexadecimal. > > In command 'gpt setenv' variable gpt_partition_entry needs to be set > to a hexadecimal value to allow its use as a parameter in a > subsequent command. > >

Re: [PATCH 2/3] cmd: fix gpt enumerate

2023-08-25 Thread Simon Glass
Hi Heinrich, On Fri, 25 Aug 2023 at 19:53, Heinrich Schuchardt wrote: > > Do not assume that partitions are numbered continuously starting at 1. > > Only a single partition table type can exist on a block device. If we found > a GPT partition table, we must not re-enumerate with the MBR

Re: [PATCH 1/3] cmd: fix gpt setenv

2023-08-25 Thread Simon Glass
On Fri, 25 Aug 2023 at 19:53, Heinrich Schuchardt wrote: > > Do not assume that partitions are continuously numbered starting at 1. > > Having a partition table with a single partition 63 is valid. > > Signed-off-by: Heinrich Schuchardt > --- > cmd/gpt.c | 6 ++ > 1 file changed, 2

Re: [PATCH V4 8/8] doc: board: ti: Add BeaglePlay documentation

2023-08-25 Thread Simon Glass
Hi Robert, On Fri, 25 Aug 2023 at 19:06, Robert Nelson wrote: > > > > Just https://etcher.balena.io/ and flash an SD card with the official > > > image here: > > > https://www.beagleboard.org/distros/beagleplay-debian-11-flasher > > > > > >

Re: [PATCH v3 2/8] doc: Add gpt command documentation

2023-08-25 Thread Heinrich Schuchardt
On 8/25/23 21:38, Joshua Watt wrote: Adds initial documentation for the gpt command Thanks a lot for filling the gap. Signed-off-by: Joshua Watt --- doc/usage/cmd/gpt.rst | 141 ++ doc/usage/index.rst | 1 + 2 files changed, 142 insertions(+)

[PATCH 3/3] cmd: let gpt_partition_entry be hexadecimal

2023-08-25 Thread Heinrich Schuchardt
In commands like 'ls mmc 0:f' the partition number is hexadecimal. In command 'gpt setenv' variable gpt_partition_entry needs to be set to a hexadecimal value to allow its use as a parameter in a subsequent command. Signed-off-by: Heinrich Schuchardt --- cmd/gpt.c | 2 +- 1 file changed, 1

[PATCH 2/3] cmd: fix gpt enumerate

2023-08-25 Thread Heinrich Schuchardt
Do not assume that partitions are numbered continuously starting at 1. Only a single partition table type can exist on a block device. If we found a GPT partition table, we must not re-enumerate with the MBR partition driver which would find the protective partition. Signed-off-by: Heinrich

[PATCH 1/3] cmd: fix gpt setenv

2023-08-25 Thread Heinrich Schuchardt
Do not assume that partitions are continuously numbered starting at 1. Having a partition table with a single partition 63 is valid. Signed-off-by: Heinrich Schuchardt --- cmd/gpt.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/cmd/gpt.c b/cmd/gpt.c index

[PATCH 0/3] cmd: fix errors in gpt command

2023-08-25 Thread Heinrich Schuchardt
Consider that partitions may not be numbered continously starting at 1. Don't enumerate a block device with multiple partition drivers. Let gpt_partition_entry be a hexadecimal value. Heinrich Schuchardt (3): cmd: fix gpt setenv cmd: fix gpt enumerate cmd: let gpt_partition_entry be

Re: [PATCH V4 8/8] doc: board: ti: Add BeaglePlay documentation

2023-08-25 Thread Robert Nelson
> > Just https://etcher.balena.io/ and flash an SD card with the official > > image here: https://www.beagleboard.org/distros/beagleplay-debian-11-flasher > > > > https://files.beagle.cc/file/beagleboard-public-2021/images/am625x-emmc-flasher-debian-11.6-xfce-arm64-2023-02-04-10gb.img.xz > > OK I

Re: [PATCH v3 8/8] cmd: gpt: Add command to swap partition order

2023-08-25 Thread Heinrich Schuchardt
On 8/25/23 21:38, Joshua Watt wrote: Adds a command called "gpt swap-postition" which will swap the order two partitions are listed in the GPT partition table (but leaves them pointing to the same locations on disk). Why is this functionality needed in a boot firmware? Signed-off-by: Joshua

Re: [PATCH] bootstd: Drop some TODOs

2023-08-25 Thread Tony Dinh
Hi Simon, On Fri, Aug 25, 2023 at 4:53 PM Simon Glass wrote: > > Hi Tony, > > On Fri, 25 Aug 2023 at 14:17, Tony Dinh wrote: > > > > Hi Simon, > > > > On Thu, Aug 24, 2023 at 6:48 PM Simon Glass wrote: > > > > > > The existing TODOs are done, so remove them. Add another that came up > > >

Re: [PATCH v3 6/8] cmd: gpt: Preserve type GUID if enabled

2023-08-25 Thread Heinrich Schuchardt
On 8/25/23 21:38, Joshua Watt wrote: If CONFIG_PARTITION_TYPE_GUID is enabled, the type GUID will be preserved when writing out the partition string. It was already respected when writing out partitions; this ensures that if you capture the current partition layout and write it back (such as

Re: [PATCH v3 2/8] doc: Add gpt command documentation

2023-08-25 Thread Simon Glass
Hi Joshua, On Fri, 25 Aug 2023 at 13:38, Joshua Watt wrote: > > Adds initial documentation for the gpt command > > Signed-off-by: Joshua Watt > --- > doc/usage/cmd/gpt.rst | 141 ++ > doc/usage/index.rst | 1 + > 2 files changed, 142 insertions(+) >

Re: [PATCH] bootstd: Drop some TODOs

2023-08-25 Thread Simon Glass
Hi Tony, On Fri, 25 Aug 2023 at 14:17, Tony Dinh wrote: > > Hi Simon, > > On Thu, Aug 24, 2023 at 6:48 PM Simon Glass wrote: > > > > The existing TODOs are done, so remove them. Add another that came up > > today. > > > > Signed-off-by: Simon Glass > > --- > > > > doc/develop/bootstd.rst | 4

Re: [PATCH v3 4/8] cmd: gpt: Add gpt_partition_bootable variable

2023-08-25 Thread Simon Glass
On Fri, 25 Aug 2023 at 13:38, Joshua Watt wrote: > > Adds an additional variable called gpt_partition_bootable that indicates > if the given partition is bootable or not. > > Signed-off-by: Joshua Watt > --- > cmd/gpt.c | 9 +++-- > doc/usage/cmd/gpt.rst | 3 +++ >

Re: [PATCH v3 6/8] cmd: gpt: Preserve type GUID if enabled

2023-08-25 Thread Simon Glass
On Fri, 25 Aug 2023 at 13:39, Joshua Watt wrote: > > If CONFIG_PARTITION_TYPE_GUID is enabled, the type GUID will be > preserved when writing out the partition string. It was already > respected when writing out partitions; this ensures that if you capture > the current partition layout and write

Re: [PATCH v3 8/8] cmd: gpt: Add command to swap partition order

2023-08-25 Thread Simon Glass
On Fri, 25 Aug 2023 at 13:39, Joshua Watt wrote: > > Adds a command called "gpt swap-postition" which will swap the order two > partitions are listed in the GPT partition table (but leaves them > pointing to the same locations on disk). > > Signed-off-by: Joshua Watt > --- > cmd/gpt.c

Re: [PATCH V4 8/8] doc: board: ti: Add BeaglePlay documentation

2023-08-25 Thread Simon Glass
Hi Nishanth, On Thu, 24 Aug 2023 at 19:59, Nishanth Menon wrote: > > On 19:31-20230824, Simon Glass wrote: > > > > > > > > Also I tried the mmc instructions and managed to brick the board. Is > > > Hmm.. This is just documenting what Robert has in his scripts[1] > > > > > > > there a recovery

Re: [PATCH] efi: Correct handling of frame buffer

2023-08-25 Thread Heinrich Schuchardt
On 8/25/23 21:28, Simon Glass wrote: The efi_gop driver uses private fields from the video uclass to obtain a pointer to the frame buffer. Use the platform data instead. Check the VIDEO_COPY setting to determine which frame buffer to use. Once the next stage is running (and making use of

Re: [PATCH 00/19] expo: Support a persistent configuration editor

2023-08-25 Thread Tom Rini
On Mon, 14 Aug 2023 16:40:20 -0600, Simon Glass wrote: > So far cedit does not support reading and writing the configuration. > This series add several features related to this: > > First, it adds support for using a file on a filesystem. This is in > FDT format and provides enough information

Re: [PATCH V6 19/20] board: ti: am62x: Add am62x_beagleplay_* defconfigs and env file

2023-08-25 Thread Tom Rini
On Fri, Aug 25, 2023 at 01:03:04PM -0500, Nishanth Menon wrote: > Add defconfig fragments for am625 based beagleplay and corresponding > customized environment file for beagleplay. > > Reviewed-by: Mattijs Korpershoek > Tested-by: Mattijs Korpershoek > Signed-off-by: Nishanth Menon

Re: [PATCH V6 10/20] include: env: ti: Add a generic default_findfdt.env

2023-08-25 Thread Tom Rini
On Fri, Aug 25, 2023 at 01:02:55PM -0500, Nishanth Menon wrote: > ti_mmc bootmethod uses a findfdt routine that is expected to be > implemented by all platforms. > > Define a default findfdt based on configured DEFAULT_DEVICE_TREE option > for u-boot. This saves duplication across multiple

Re: [PATCH V6 02/20] include: env: ti: mmc: envboot: Only attempt boot.scr if BOOTSTD is not enabled

2023-08-25 Thread Tom Rini
On Fri, Aug 25, 2023 at 01:02:47PM -0500, Nishanth Menon wrote: > 'script' bootmethod that should be used with CONFIG_BOOTSTD. > > Signed-off-by: Nishanth Menon Reviewed-by: Tom Rini -- Tom signature.asc Description: PGP signature

Re: [PATCH V6 07/20] configs: am62x_evm_a53_defconfig: Switch to bootstd

2023-08-25 Thread Tom Rini
On Fri, Aug 25, 2023 at 01:02:52PM -0500, Nishanth Menon wrote: > Switch to using bootstd. Note with this change, we will stop using > distro_bootcmd and instead depend entirely on bootflow method of > starting the system up. > > Suggested-by: Tom Rini > Suggested-by: Simon Glass >

Re: [PATCH V6 01/20] include: env: ti: mmc: envboot/mmcboot: Check result of mmc dev before proceeding

2023-08-25 Thread Tom Rini
On Fri, Aug 25, 2023 at 01:02:46PM -0500, Nishanth Menon wrote: > If mmc dev reports that the device is not present, there is no point in > proceeding further to attempt to load the files. > > Signed-off-by: Nishanth Menon Reviewed-by: Tom Rini -- Tom signature.asc Description: PGP

Re: [PATCH] bootstd: Drop some TODOs

2023-08-25 Thread Tony Dinh
Hi Simon, On Thu, Aug 24, 2023 at 6:48 PM Simon Glass wrote: > > The existing TODOs are done, so remove them. Add another that came up > today. > > Signed-off-by: Simon Glass > --- > > doc/develop/bootstd.rst | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git

[PATCH v3 8/8] cmd: gpt: Add command to swap partition order

2023-08-25 Thread Joshua Watt
Adds a command called "gpt swap-postition" which will swap the order two partitions are listed in the GPT partition table (but leaves them pointing to the same locations on disk). Signed-off-by: Joshua Watt --- cmd/gpt.c | 52 ---

[PATCH v3 7/8] cmd: gpt: Preserve bootable flag

2023-08-25 Thread Joshua Watt
Sets the bootable flag when constructing the partition string from the current partition configuration. This ensures that when the partitions are written back (for example, when renaming a partition), the flag is preserved. Signed-off-by: Joshua Watt --- cmd/gpt.c | 3 +++

[PATCH v3 6/8] cmd: gpt: Preserve type GUID if enabled

2023-08-25 Thread Joshua Watt
If CONFIG_PARTITION_TYPE_GUID is enabled, the type GUID will be preserved when writing out the partition string. It was already respected when writing out partitions; this ensures that if you capture the current partition layout and write it back (such as when renaming), the type GUIDs are

[PATCH v3 5/8] cmd: gpt: Add command to set bootable flags

2023-08-25 Thread Joshua Watt
Adds a command that can be used to modify the GPT partition table to indicate which partitions should have the bootable flag set Signed-off-by: Joshua Watt --- cmd/gpt.c | 80 +++ doc/usage/cmd/gpt.rst | 9 + test/py/tests/test_gpt.py

[PATCH v3 4/8] cmd: gpt: Add gpt_partition_bootable variable

2023-08-25 Thread Joshua Watt
Adds an additional variable called gpt_partition_bootable that indicates if the given partition is bootable or not. Signed-off-by: Joshua Watt --- cmd/gpt.c | 9 +++-- doc/usage/cmd/gpt.rst | 3 +++ test/py/tests/test_gpt.py | 33 + 3

[PATCH v3 3/8] tests: gpt: Remove test order dependency

2023-08-25 Thread Joshua Watt
Re-create a clean disk image for each test to prevent modifications from one test affecting another Signed-off-by: Joshua Watt --- test/py/tests/test_gpt.py | 20 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/test/py/tests/test_gpt.py

[PATCH v3 2/8] doc: Add gpt command documentation

2023-08-25 Thread Joshua Watt
Adds initial documentation for the gpt command Signed-off-by: Joshua Watt --- doc/usage/cmd/gpt.rst | 141 ++ doc/usage/index.rst | 1 + 2 files changed, 142 insertions(+) create mode 100644 doc/usage/cmd/gpt.rst diff --git a/doc/usage/cmd/gpt.rst

[PATCH v3 1/8] cmd: gpt: Remove confusing help text

2023-08-25 Thread Joshua Watt
This help text appears to be a fragment of the text shown when CONFIG_CMD_GPT_RENAME is enabled, but is confusing so remove it. Signed-off-by: Joshua Watt --- cmd/gpt.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/cmd/gpt.c b/cmd/gpt.c index 007a68eaa7..e6f7b0319a 100644 --- a/cmd/gpt.c

[PATCH v3 0/8] cmd: gpt: GPT manipulation improvements

2023-08-25 Thread Joshua Watt
Adds several improvements and additions to the gpt command processing, specifically (although not exclusively) for the purpose of supporting "ping-pong" booting when doing A/B boot partitions with u-boot itself. In this mechanism, u-boot must boot up, and then check if the correct boot partition

[PATCH] efi: Correct handling of frame buffer

2023-08-25 Thread Simon Glass
The efi_gop driver uses private fields from the video uclass to obtain a pointer to the frame buffer. Use the platform data instead. Check the VIDEO_COPY setting to determine which frame buffer to use. Once the next stage is running (and making use of U-Boot's EFI boot services) U-Boot does not

Re: [PATCH v2] Convert CFG_SYS_UBOOT_START to Kconfig

2023-08-25 Thread Jesse T
On Fri, Aug 25, 2023 at 2:06 PM Simon Glass wrote: > > On Fri, 25 Aug 2023 at 08:49, Tom Rini wrote: > > > > On Thu, Aug 24, 2023 at 09:59:48PM -0400, Jesse Taube wrote: > > > > > Commit 65cc0e2a65d2 ("global: Move remaining CONFIG_SYS_* to CFG_SYS_*") > > > renamed CONFIG_SYS_UBOOT_START to

Re: [PATCH 2/2] CI: Move to latest Ubuntu "Jammy" tag

2023-08-25 Thread Tom Rini
On Fri, Aug 25, 2023 at 12:06:09PM -0600, Simon Glass wrote: > Hi Tom, > > On Fri, 25 Aug 2023 at 11:21, Tom Rini wrote: > > > > Move to the latest "Jammy" tag from Ubuntu. > > > > Signed-off-by: Tom Rini > > --- > > .azure-pipelines.yml| 2 +- > > .gitlab-ci.yml | 2 +- > >

[PATCH V6 18/20] arm: dts: Add k3-am625-beagleplay

2023-08-25 Thread Nishanth Menon
From: Robert Nelson BeagleBoard.org BeaglePlay is an easy to use, affordable open source hardware single board computer based on the Texas Instruments AM625 SoC that allows you to create connected devices that work even at long distances using IEEE 802.15.4g LR-WPAN and IEEE 802.3cg 10Base-T1L.

Re: [PATCH v2 3/3] board: toradex: add verdin am62 support

2023-08-25 Thread Sverdlin, Alexander
Hello Marcel, On Fri, 2023-08-25 at 15:20 +0200, Alexander Sverdlin wrote: > > +Build U-Boot for R5 > > +--- > > + > > +.. code-block:: bash > > + > > +    $ export ARCH=arm > > +    $ export CROSS_COMPILE=arm-none-linux-gnueabihf- > > +    $ make verdin-am62_r5_defconfig

Re: [PATCH v2] tools: ensure zeroed padding in external FIT images

2023-08-25 Thread Simon Glass
On Fri, 25 Aug 2023 at 02:10, Roman Azarenko wrote: > > Padding the header of an external FIT image is achieved by truncating > the existing temporary FIT file to match the required alignment before > appending image data. Reusing an existing file this way means that the > padding will likely

Re: [PATCH v2] Convert CFG_SYS_UBOOT_START to Kconfig

2023-08-25 Thread Simon Glass
On Fri, 25 Aug 2023 at 08:49, Tom Rini wrote: > > On Thu, Aug 24, 2023 at 09:59:48PM -0400, Jesse Taube wrote: > > > Commit 65cc0e2a65d2 ("global: Move remaining CONFIG_SYS_* to CFG_SYS_*") > > renamed CONFIG_SYS_UBOOT_START to CFG_SYS_UBOOT_START. Unfortunately, > > this meant that the value was

Re: [PATCH v2] test: print: Fix hexdump test on 64bit systems

2023-08-25 Thread Simon Glass
On Fri, 25 Aug 2023 at 02:20, Marek Vasut wrote: > > Use the following regex to make this test compatible with > both 32bit and 64bit systems. The trick is to use %0*lx > format string for the address prefix in the test. > > " > s@\(ut_assert_nextline("\)0\+\([^:]\+\)\(:.*"\)\();\)@\1%0*lx\3, >

Re: [PATCH 2/2] CI: Move to latest Ubuntu "Jammy" tag

2023-08-25 Thread Simon Glass
Hi Tom, On Fri, 25 Aug 2023 at 11:21, Tom Rini wrote: > > Move to the latest "Jammy" tag from Ubuntu. > > Signed-off-by: Tom Rini > --- > .azure-pipelines.yml| 2 +- > .gitlab-ci.yml | 2 +- > tools/docker/Dockerfile | 2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) >

Re: [PATCH v2] dm: core: support reading a single indexed u64 value

2023-08-25 Thread Simon Glass
On Fri, 25 Aug 2023 at 03:37, Michal Simek wrote: > > Add helper function to allow reading a single indexed u64 value from a > device-tree property containing multiple u64 values, that is an array of > u64's. > > Co-developed-by: Ashok Reddy Soma > Signed-off-by: Ashok Reddy Soma >

Re: [PATCH 1/2] Reland "x86: Move FACP table into separate functions""

2023-08-25 Thread Simon Glass
Hi Andy, On Fri, 25 Aug 2023 at 07:27, Andy Shevchenko wrote: > > On Fri, Aug 25, 2023 at 04:25:29PM +0300, Andy Shevchenko wrote: > > On Fri, Aug 25, 2023 at 04:17:58PM +0300, Andy Shevchenko wrote: > > > On Fri, Aug 25, 2023 at 02:06:34PM +0300, Andy Shevchenko wrote: > > > > On Thu, Aug 24,

Re: [PATCH v1 1/2] TO BE FOLDED: x86: tangier: Add FADT to the chain

2023-08-25 Thread Simon Glass
Hi Andy, On Fri, 25 Aug 2023 at 07:52, Andy Shevchenko wrote: > > The Simon's patch missed the FADT to be added to the chain, hence all > the issues on Intel Tangier. > > Signed-off-by: Andy Shevchenko > --- > arch/x86/cpu/tangier/acpi.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git

Re: [PATCH 1/2] CI: Update to gcc-13.2.0

2023-08-25 Thread Simon Glass
On Fri, 25 Aug 2023 at 11:21, Tom Rini wrote: > > The latest kernel.org toolchains for gcc are now 13.2.0, so upgrade to > that. > > Signed-off-by: Tom Rini > --- > .azure-pipelines.yml| 2 +- > .gitlab-ci.yml | 2 +- > tools/buildman/toolchain.py | 2 +- >

Re: [PATCH] fwu: Initialize global fwu library state during CI test

2023-08-25 Thread Simon Glass
On Fri, 25 Aug 2023 at 10:00, Marek Vasut wrote: > > On 8/25/23 12:52, Sughosh Ganu wrote: > > On Wed, 23 Aug 2023 at 05:47, Marek Vasut > > wrote: > >> > >> The current CI test worked by sheer luck, the g_dev global pointer > >> in the fwu library was never initialized and the test equally well

Re: [PATCH] sunxi: psci: remove redundant initialization from psci_arch_init

2023-08-25 Thread Sam Edwards
On 8/25/23 00:20, Chen-Yu Tsai wrote: Hi Chen-Yu, IIRC the GIC manual says that the secure bit is set or cleared to select which bank of registers is accessed. Which secure bit are we talking about here? Do we mean the *configured* secure bit (SCR.NS, what the code is attempting to clear)

[PATCH V6 20/20] doc: board: ti: Add BeaglePlay documentation

2023-08-25 Thread Nishanth Menon
Add base documentation for BeaglePlay Reviewed-by: Mattijs Korpershoek Tested-by: Mattijs Korpershoek Signed-off-by: Nishanth Menon --- Cc: Heinrich Schuchardt Since V5: - Updated to point at how to get serial console - Introduce on using sd card and some tips and tricks I personally learnt

[PATCH V6 15/20] arm: mach-k3: am625: Add support for UDA FS

2023-08-25 Thread Nishanth Menon
While boot partition support with EMMC boot is useful, it is constrained by the size of boot hardware partition itself. In the case of K3 devices, tispl images can contain OP-TEE images that can substantially vary in size and the u-boot image itself can vary over time as we enable various

[PATCH V6 01/20] include: env: ti: mmc: envboot/mmcboot: Check result of mmc dev before proceeding

2023-08-25 Thread Nishanth Menon
If mmc dev reports that the device is not present, there is no point in proceeding further to attempt to load the files. Signed-off-by: Nishanth Menon --- new patch - noticed a series of 'MMC: no card present' in boot log and was wondering why, now I know - shaves off 2 seconds or so into boot

[PATCH V6 16/20] configs: am62x_evm_a53_defconfig: Disable semi-functional PSCI reset support

2023-08-25 Thread Nishanth Menon
From: Jan Kiszka At this point, system shutdown is not supported by the DM firmware that TF-A calls. As we can't de-select only this feature[1], declare complete PSCI reset support as non-functional so that we don't signal incomplete support to the OS via EFI runtime services. This makes

[PATCH V6 13/20] arm: mach-k3: am625_init: Convert rtc_erratumi2327_init to static

2023-08-25 Thread Nishanth Menon
The erratum is called locally, make it static, drop the #ifdeffery since it will only be called in R5 build and mark it potentially unused to stop compiler screaming at us. While at this, drop the redundant return for a void function. Reviewed-by: Tom Rini Reviewed-by: Mattijs Korpershoek

[PATCH V6 00/20] board: ti: Add support for BeaglePlay

2023-08-25 Thread Nishanth Menon
Alright, 6th time is a charm ;) Hopefully this takes care of all the un-addressed concerns we have had. Thanks all for pitching in, reviewing and improving and I will stop finding things to cleanup (promise). Baseline: 7c6b18fb5494 (origin/next) Merge branch

[PATCH V6 05/20] include: configs: am62x_evm: Wrap distroboot with CONFIG_DISTRO_DEFAULTS

2023-08-25 Thread Nishanth Menon
Wrap the distro_boot options with CONFIG_DISTRO_DEFAULTS. This is an intermediate step for us to switch over to CONFIG_BOOTSTD_DEFAULTS and drop this section in follow on patches. Reviewed-by: Tom Rini Tested-by: Mattijs Korpershoek Signed-off-by: Nishanth Menon --- No change other than

[PATCH V6 04/20] include: configs: am62x_evm: Drop unused SDRAM address

2023-08-25 Thread Nishanth Menon
Drop unused macro. This was meant for a second region of DDR which we do not need for AM62x evm configurations. Tested-by: Mattijs Korpershoek Signed-off-by: Nishanth Menon --- No change other than picking up reviews, tested tags along the way. V5:

[PATCH V6 11/20] board: ti: am62x: am62x.env: Use default findfdt

2023-08-25 Thread Nishanth Menon
Use the default findfdt using CONFIG_DEFAULT_DEVICE_TREE Reviewed-by: Tom Rini Tested-by: Mattijs Korpershoek Signed-off-by: Nishanth Menon --- No change that picking up Review, tested by and using new include file. V5: https://lore.kernel.org/r/20230824031101.3460411-10...@ti.com

[PATCH V6 19/20] board: ti: am62x: Add am62x_beagleplay_* defconfigs and env file

2023-08-25 Thread Nishanth Menon
Add defconfig fragments for am625 based beagleplay and corresponding customized environment file for beagleplay. Reviewed-by: Mattijs Korpershoek Tested-by: Mattijs Korpershoek Signed-off-by: Nishanth Menon --- No major changes other than explicitly calling out script in the crafted bootmeths

[PATCH V6 07/20] configs: am62x_evm_a53_defconfig: Switch to bootstd

2023-08-25 Thread Nishanth Menon
Switch to using bootstd. Note with this change, we will stop using distro_bootcmd and instead depend entirely on bootflow method of starting the system up. Suggested-by: Tom Rini Suggested-by: Simon Glass Reviewed-by: Tom Rini Tested-by: Mattijs Korpershoek Signed-off-by: Nishanth Menon ---

[PATCH V6 17/20] arm: dts: k3-am625-sk-binman: Add labels for unsigned binary

2023-08-25 Thread Nishanth Menon
Add labels for unsigned binary to permit over-ride. Reviewed-by: Mattijs Korpershoek Tested-by: Mattijs Korpershoek Signed-off-by: Nishanth Menon --- No change other than picking up reviews, tested tags along the way. V5: https://lore.kernel.org/r/20230824031101.3460411-15...@ti.com V4:

[PATCH V6 12/20] arm: mach-k3: am625_init: Use IS_ENABLED()

2023-08-25 Thread Nishanth Menon
Drop the #ifdeffery and use IS_ENABLED() inline check and let the compiler do it's thing. Reviewed-by: Tom Rini Reviewed-by: Mattijs Korpershoek Tested-by: Mattijs Korpershoek Signed-off-by: Nishanth Menon --- No change other than picking up reviews, tested tags along the way. V5:

[PATCH V6 14/20] configs: am62x_evm*: Enable EMMC_BOOT configuration

2023-08-25 Thread Nishanth Menon
Enable EMMC boot support for AM62x evm base configuration. Reviewed-by: Mattijs Korpershoek Tested-by: Mattijs Korpershoek Signed-off-by: Nishanth Menon --- No change other than picking up reviews, tested tags along the way. V5: https://lore.kernel.org/r/20230824031101.3460411-13...@ti.com

[PATCH V6 10/20] include: env: ti: Add a generic default_findfdt.env

2023-08-25 Thread Nishanth Menon
ti_mmc bootmethod uses a findfdt routine that is expected to be implemented by all platforms. Define a default findfdt based on configured DEFAULT_DEVICE_TREE option for u-boot. This saves duplication across multiple boards and handles architecture folder location changes centrally. TI ARMV7

[PATCH V6 08/20] include: configs: am62x_evm: Drop distro_bootcmd usage

2023-08-25 Thread Nishanth Menon
Now that BOOTSTD is used by default, drop un-used header file inclusion. Reviewed-by: Tom Rini Tested-by: Mattijs Korpershoek Signed-off-by: Nishanth Menon --- No change other than picking up reviews, tested tags along the way. V5: https://lore.kernel.org/r/20230824031101.3460411-7...@ti.com

[PATCH V6 06/20] board: ti: am62x: am62x.env: Add explicit boot_targets

2023-08-25 Thread Nishanth Menon
Add explicit boot_targets to indicate the specific boot sequence to follow. NOTE: The non-standard ti_mmc emulates what is done for distro_boot. With bootstd, this will eventually need to be replaced by equivalent class. Tested-by: Mattijs Korpershoek Signed-off-by: Nishanth Menon --- No

[PATCH V6 02/20] include: env: ti: mmc: envboot: Only attempt boot.scr if BOOTSTD is not enabled

2023-08-25 Thread Nishanth Menon
'script' bootmethod that should be used with CONFIG_BOOTSTD. Signed-off-by: Nishanth Menon --- Since Simon educated me to realize that the published documentation can be stale and bootmeths=script does indeed work for some time despite what the docs say.. include/env/ti/mmc.env | 4 +++- 1

[PATCH V6 09/20] include: env: ti: ti_armv7_common.env: Rename to ti_common.env

2023-08-25 Thread Nishanth Menon
ti_armv7_common does not make any more sense as it is used by armv7 and armv8 TI based platforms. Reported-by: Tom Rini Reviewed-by: Mattijs Korpershoek Reviewed-by: Tom Rini Tested-by: Mattijs Korpershoek Signed-off-by: Nishanth Menon --- No change other than picking up reviews, tested tags

[PATCH V6 03/20] include: configs: ti_armv7_common: Add documentation for protected section

2023-08-25 Thread Nishanth Menon
Make the section protected by CONFIG_DISTRO_DEFAULTS macro clear. Reviewed-by: Mattijs Korpershoek Tested-by: Mattijs Korpershoek Signed-off-by: Nishanth Menon --- No change other than picking up reviews, tested tags along the way. V5:

Re: [PATCH v1 1/1] usb: host: tegra: implement dts based xcvr setup

2023-08-25 Thread Svyatoslav Ryhel
ср, 23 серп. 2023 р. о 13:49 Thierry Reding пише: > > On Sun, Aug 20, 2023 at 09:10:17PM +0200, Marek Vasut wrote: > > On 8/20/23 20:32, Svyatoslav Ryhel wrote: > > > 20 серпня 2023 р. 21:14:15 GMT+03:00, Marek Vasut > > > написав(-ла): > > > > On 8/20/23 09:13, Svyatoslav Ryhel wrote: > > > >

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

2023-08-25 Thread Tom Rini
On Fri, Aug 25, 2023 at 10:57:40AM +0200, Marek Vasut wrote: > The following changes since commit 291055efee4e1ae4ad0b62393256514a4e76cb1e: > > Merge tag 'doc-2023-10-rc4' of > https://source.denx.de/u-boot/custodians/u-boot-efi (2023-08-23 11:03:46 > -0400) > > are available in the Git

[PATCH 1/2] CI: Update to gcc-13.2.0

2023-08-25 Thread Tom Rini
The latest kernel.org toolchains for gcc are now 13.2.0, so upgrade to that. Signed-off-by: Tom Rini --- .azure-pipelines.yml| 2 +- .gitlab-ci.yml | 2 +- tools/buildman/toolchain.py | 2 +- tools/docker/Dockerfile | 58 ++--- 4 files

[PATCH 2/2] CI: Move to latest Ubuntu "Jammy" tag

2023-08-25 Thread Tom Rini
Move to the latest "Jammy" tag from Ubuntu. Signed-off-by: Tom Rini --- .azure-pipelines.yml| 2 +- .gitlab-ci.yml | 2 +- tools/docker/Dockerfile | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.azure-pipelines.yml b/.azure-pipelines.yml index

Re: [PATCH v2 3/3] board: toradex: add verdin am62 support

2023-08-25 Thread Marcel Ziswiler
Sali Alexander On August 25, 2023 6:24:01 PM GMT+02:00, "Sverdlin, Alexander" wrote: >Hello Marcel, > >On Fri, 2023-08-25 at 15:20 +0200, Alexander Sverdlin wrote: >> > +Build U-Boot for R5 >> > +--- >> > + >> > +.. code-block:: bash >> > + >> > +    $ export ARCH=arm >> > +   

[PATCH] configs: stm32f769-disco: Enable VIDEO_LOGO flag

2023-08-25 Thread Patrice Chotard
The patch removes the legacy mode of displaying the ST logo and adopts the approach introduced by the commit 284b08fb51b6 ("board: stm32mp1: add splash screen with stmicroelectronics logo"). Signed-off-by: Patrice Chotard --- configs/stm32f769-disco_defconfig | 2 +-

Re: [PATCH 2/3] board: stm32f746-disco: refactor the display of the ST logo

2023-08-25 Thread Patrice CHOTARD
On 8/20/23 18:24, Dario Binacchi wrote: > The patch removes the legacy mode of displaying the ST logo and adopts > the approach introduced by the commit 284b08fb51b6 ("board: stm32mp1: add > splash screen with stmicroelectronics logo"). It was necessary to use a > specific logo for the

Re: [PATCH 1/3] configs: stm32f746-disco: limit resolution to 480x272

2023-08-25 Thread Patrice CHOTARD
On 8/20/23 18:24, Dario Binacchi wrote: > The patch fixes the y-resolution, which was causing the creation of a > framebuffer larger than actually needed, resulting in memory waste. > > Fixes: cc1b0e7b8e55b ("board: Add display to STM32F746 SoC discovery board") > Signed-off-by: Dario Binacchi

Re: [PATCH] fwu: Initialize global fwu library state during CI test

2023-08-25 Thread Marek Vasut
On 8/25/23 12:52, Sughosh Ganu wrote: On Wed, 23 Aug 2023 at 05:47, Marek Vasut wrote: The current CI test worked by sheer luck, the g_dev global pointer in the fwu library was never initialized and the test equally well failed on sandbox64. Trigger the main loop in sandbox tests too to

Re: [PATCH v3 1/2] usb: dwc3: Fix renaming SPL_USB_HOST_SUPPORT to SPL_USB_HOST

2023-08-25 Thread Marek Vasut
On 8/25/23 12:42, Oleksandr Suvorov wrote: In the usb/dwc3-layerscape driver the first option should be renamed to the latter as well. Do it. Fix original logic in dwc3_layerscape_bind() - do not enable Fixes: 333e4a621df ("Rename SPL_USB_HOST_SUPPORT to SPL_USB_HOST") Signed-off-by: Oleksandr

Re: [PATCH v3 1/2] dfu: mtd: fix the trace when limit is reached

2023-08-25 Thread Patrice CHOTARD
On 6/5/23 09:52, Patrick Delaunay wrote: > The offset variable = 'off' used in the error trace when limit is reach > on erase operation is incorect as 'erase_op.addr' is used in the loop. > This patch corrects the copy paste issue between the erase loop and > the write loop. > > This patch

Re: [PATCH 06/19] FMU: Avoid showing an unselectable menu option

2023-08-25 Thread Tom Rini
On Fri, Aug 25, 2023 at 05:30:17PM +0530, Sughosh Ganu wrote: > On Fri, 25 Aug 2023 at 01:29, Simon Glass wrote: > > > > Use a menuconfig to avoid showing a menu which cannot be selected in many > > cases. > > > > This option should really go with the other 'Update support'. > > > > Perhaps we

Re: [PATCH v2] Convert CFG_SYS_UBOOT_START to Kconfig

2023-08-25 Thread Tom Rini
On Thu, Aug 24, 2023 at 09:59:48PM -0400, Jesse Taube wrote: > Commit 65cc0e2a65d2 ("global: Move remaining CONFIG_SYS_* to CFG_SYS_*") > renamed CONFIG_SYS_UBOOT_START to CFG_SYS_UBOOT_START. Unfortunately, > this meant that the value was no longer available to the Makefile. This > caused imxrt

[PATCH] ARM: imx8ulp: support env in fat and ext4

2023-08-25 Thread Oleksandr Suvorov
From: Ricardo Salveti Change boot device logic to also allow environment stored in fat and in ext4 when booting from SD or eMMC. As the boot device check for SD and for eMMC was depending on ENV_IS_IN_MMC being defined, change the ifdef blocks at env_get_location to use IS_ENABLED instead for

Re: [PATCH] clk: Dont return error when assigned-clocks is empty or missing

2023-08-25 Thread Tom Rini
On Fri, Aug 25, 2023 at 09:15:09AM +0200, Michal Simek wrote: > Hi Tom, > > On 7/11/23 11:51, Ashok Reddy Soma wrote: > > There is a chance that assigned-clock-rates is given and assigned-clocks > > could be empty. Dont return error in that case, because the probe of the > > corresponding driver

Re: [PATCH V4 8/8] doc: board: ti: Add BeaglePlay documentation

2023-08-25 Thread Neha Malcom Francis
Hi Simon, Tom On 24-Aug-23 8:16 PM, Simon Glass wrote: Hi Tom, On Thu, 24 Aug 2023 at 08:44, Tom Rini wrote: On Thu, Aug 24, 2023 at 08:41:23AM -0600, Simon Glass wrote: Hi, On Thu, 24 Aug 2023 at 08:20, Tom Rini wrote: On Thu, Aug 24, 2023 at 06:46:57PM +0530, Neha Malcom Francis

Re: [PATCH] fwu: Initialize global fwu library state during CI test

2023-08-25 Thread Jassi Brar
On Tue, 22 Aug 2023 at 19:17, Marek Vasut wrote: > > The current CI test worked by sheer luck, the g_dev global pointer > in the fwu library was never initialized and the test equally well > failed on sandbox64. Trigger the main loop in sandbox tests too to > initialize that global state, and

Re: [PATCH v1 2/2] x86: Prevent from missing the FADT chaining

2023-08-25 Thread Andy Shevchenko
On Fri, Aug 25, 2023 at 04:51:41PM +0300, Andy Shevchenko wrote: ... > +static inline int acpi_add_fadt(struct acpi_ctx *ctx, struct acpi_fadt *fadt) > +{ > + acpi_add_table(ctx, fadt); > + acpi_inc(ctx, sizeof(struct acpi_fadt)); > + return 0; > +} Ideally it should be like

[PATCH v1 2/2] x86: Prevent from missing the FADT chaining

2023-08-25 Thread Andy Shevchenko
Recent approach with FADT writer shows that there is a room for subtle errors. Prevent this from happening again by introducing acpi_add_fadt() helper. Signed-off-by: Andy Shevchenko --- arch/x86/cpu/apollolake/acpi.c | 6 +- arch/x86/cpu/baytrail/acpi.c | 6 +-

[PATCH v1 1/2] TO BE FOLDED: x86: tangier: Add FADT to the chain

2023-08-25 Thread Andy Shevchenko
The Simon's patch missed the FADT to be added to the chain, hence all the issues on Intel Tangier. Signed-off-by: Andy Shevchenko --- arch/x86/cpu/tangier/acpi.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/x86/cpu/tangier/acpi.c b/arch/x86/cpu/tangier/acpi.c index

[PATCH] Revert "arm: imx: mx7: Move CONFIG_OPTEE_TZDRAM_SIZE from lib/optee"

2023-08-25 Thread Oleksandr Suvorov
From: Ricardo Salveti This reverts commit c5b68ef8af3c2f515c1f5b8d63a69359a85d753b. CONFIG_OPTEE_TZDRAM_SIZE is used by imx6-based SoCs as well. Move the option back. Signed-off-by: Ricardo Salveti Signed-off-by: Oleksandr Suvorov --- arch/arm/mach-imx/mx7/Kconfig | 8

Re: [PATCH 1/2] Reland "x86: Move FACP table into separate functions""

2023-08-25 Thread Andy Shevchenko
On Fri, Aug 25, 2023 at 04:25:29PM +0300, Andy Shevchenko wrote: > On Fri, Aug 25, 2023 at 04:17:58PM +0300, Andy Shevchenko wrote: > > On Fri, Aug 25, 2023 at 02:06:34PM +0300, Andy Shevchenko wrote: > > > On Thu, Aug 24, 2023 at 12:23:32PM -0600, Simon Glass wrote: > > > > Each board has its own

Re: [PATCH 1/2] Reland "x86: Move FACP table into separate functions""

2023-08-25 Thread Andy Shevchenko
On Fri, Aug 25, 2023 at 04:17:58PM +0300, Andy Shevchenko wrote: > On Fri, Aug 25, 2023 at 02:06:34PM +0300, Andy Shevchenko wrote: > > On Thu, Aug 24, 2023 at 12:23:32PM -0600, Simon Glass wrote: > > > Each board has its own way of creating this table. Rather than calling the > > >

Re: [PATCH v2 3/3] board: toradex: add verdin am62 support

2023-08-25 Thread Sverdlin, Alexander
Hello Marcel, On Mon, 2023-06-12 at 22:54 +0200, Marcel Ziswiler wrote: > +Build U-Boot for R5 > +--- > + > +.. code-block:: bash > + > +    $ export ARCH=arm > +    $ export CROSS_COMPILE=arm-none-linux-gnueabihf- > +    $ make verdin-am62_r5_defconfig O=/tmp/r5 > +    $ make

Re: [PATCH 1/2] Reland "x86: Move FACP table into separate functions""

2023-08-25 Thread Andy Shevchenko
On Fri, Aug 25, 2023 at 02:06:34PM +0300, Andy Shevchenko wrote: > On Thu, Aug 24, 2023 at 12:23:32PM -0600, Simon Glass wrote: > > Each board has its own way of creating this table. Rather than calling the > > acpi_create_fadt() function for each one from a common acpi_write_fadt() > > function,

Re: [PATCH RESEND v3] fpga: add inline stub for fpga_load

2023-08-25 Thread Eugen Hristev
On 8/25/23 10:21, Michal Simek wrote: Hi, On 8/8/23 12:22, Eugen Hristev wrote: In case CC_OPTIMIZE_FOR_DEBUG is set, unused code will not be optimized out, hence the reference to fpga_load will be compiled. if DM_FPGA and SPL_FPGA are not set, the build will fail with :

Re: [PATCH 1/2] Reland "x86: Move FACP table into separate functions""

2023-08-25 Thread Andy Shevchenko
On Fri, Aug 25, 2023 at 02:10:05PM +0300, Andy Shevchenko wrote: > On Fri, Aug 25, 2023 at 02:06:34PM +0300, Andy Shevchenko wrote: > > On Thu, Aug 24, 2023 at 12:23:32PM -0600, Simon Glass wrote: > > > Each board has its own way of creating this table. Rather than calling the > > >

  1   2   >