cetree instead"
>
> Add a missing condition in the sandbox driver.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2:
> don't create U_BOOT_DRVINFO for DT_PLAT_C
> (as requested in review comment by Simon)
> ---
> drivers/timer/sandbox_timer.
Hi Eugen,
On Fri, 24 Feb 2023 at 06:59, Eugen Hristev wrote:
>
> On 2/21/23 21:35, Simon Glass wrote:
> > On Sat, 18 Feb 2023 at 01:34, Heinrich Schuchardt
> > wrote:
> >>
> >> Building sandbox_spl with CONFIG_DEBUG leads to errors due to missing
> >>
On Sat, 18 Feb 2023 at 02:38, Heinrich Schuchardt
wrote:
>
> Drivers should have a maintainer.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> MAINTAINERS | 1 +
> 1 file changed, 1 insertion(+)
Reviewed-by: Simon Glass
Applied to u-boot-dm, thanks!
On Mon, Feb 13, 2023 at 12:36 AM Simon Glass wrote:
>
> Hi,
>
> On Thu, 19 Jan 2023 at 00:00, Stefan Roese wrote:
> >
> > On 1/17/23 15:08, Robert Marko wrote:
> > > Currently, pinctrl drivers are getting probed during post-bind, however
> > > that is b
This is only for posterity, since once the conversion is done, the script
is of no use.
Signed-off-by: Simon Glass
---
scripts/style.py | 180 +++
1 file changed, 180 insertions(+)
create mode 100755 scripts/style.py
Applied to u-boot-dm, thanks!
On Sun, Oct 23, 2022 at 09:56:29AM +0200, Heinrich Schuchardt wrote:
> On 10/23/22 09:50, Michal Suchánek wrote:
> > On Sat, Oct 22, 2022 at 11:52:29PM +0200, Heinrich Schuchardt wrote:
> > >
> > >
> > > Am 22. Oktober 2022 23:22:01 MESZ schrieb Michal Suchanek
> > > :
> > > > SANDBOX_BITS_PER_LON
+Marek Vasut +Bin Meng +Mark Kettenis +Tom Rini
On Wed, 1 Mar 2023 at 08:12, bluetail wrote:
>
> Hello. user kettenis aka "Mark Kettenis" guided me write my bug report
> to this email. "bluetail: please report these usb bugs upstream; they're
> almost certainly not hardware-specific"
>
> But befo
arning
> u32 *data;
> ^
> = NULL
> "
>
> Signed-off-by: Marek Vasut
> ---
> Cc: Heinrich Schuchardt
> Cc: Michal Simek
> Cc: Oleksandr Suvorov
> Cc: Simon Glass
> Cc: Stefan Roese
> Cc: Tom Rini
> ---
> boot/image-board.c | 15 +++
> 1 file changed, 11 insertions(+), 4 deletions(-)
>
Reviewed-by: Simon Glass
Hi Ilias,
On Wed, 1 Mar 2023 at 01:52, Ilias Apalodimas
wrote:
>
> Hi Simon
>
> On Sun, 26 Feb 2023 at 03:33, Simon Glass wrote:
> >
> > Provide information about the GUIDs supplied by QEMU, so far as it is
> > known.
>
> What the patch does is obvious.
> that code would work perfectly well with index -1 too.
>
> Signed-off-by: Marek Vasut
> ---
> Cc: Heinrich Schuchardt
> Cc: Simon Glass
> Cc: Tom Rini
> ---
> cmd/fdt.c | 14 ++++--
> 1 file changed, 12 insertions(+), 2 deletions(-)
>
Reviewed-by: Simon Glass
reused by other tests. No functional change.
>
> Signed-off-by: Marek Vasut
> ---
> Cc: Heinrich Schuchardt
> Cc: Simon Glass
> Cc: Tom Rini
> ---
> test/cmd/fdt.c | 124 +++--
> 1 file changed, 111 insertions(+), 13 deletions(-)
>
Reviewed-by: Simon Glass
Hi Jan,
On Tue, 28 Feb 2023 at 11:20, Jan Kiszka wrote:
>
> From: Jan Kiszka
>
> Anything that is not boot-env related is better kept there by now.
>
> At this chance, also drop a stale comment from iot2050.h
>
> Signed-off-by: Jan Kiszka
> ---
> board/siemens/iot2050/iot2050.env | 9
oc/usage/cmd/read.rst
>
Reviewed-by: Simon Glass
Please add a commit msg
> diff --git a/doc/usage/cmd/read.rst b/doc/usage/cmd/read.rst
> new file mode 100644
> index 00..705d5a3e0c
> --- /dev/null
> +++ b/doc/usage/cmd/read.rst
> @@ -0,0 +1,40 @@
> +.. SPDX-Li
On Wed, 1 Mar 2023 at 14:13, Rasmus Villemoes
wrote:
>
> Signed-off-by: Rasmus Villemoes
> ---
> configs/sandbox64_defconfig | 1 +
> configs/sandbox_defconfig | 1 +
> 2 files changed, 2 insertions(+)
Reviewed-by: Simon Glass
On Wed, 1 Mar 2023 at 14:13, Rasmus Villemoes
wrote:
>
> Signed-off-by: Rasmus Villemoes
> ---
> test/cmd/Makefile | 1 +
> test/cmd/rw.c | 104 ++
> 2 files changed, 105 insertions(+)
> create mode 100644 test/cmd/rw.
elf whether it
> was invoked as "read" or "write".
>
> Signed-off-by: Rasmus Villemoes
> ---
> cmd/Kconfig | 5 +
> cmd/Makefile | 1 +
> cmd/read.c | 29 ++---
> 3 files changed, 28 insertions(+), 7 deletions(-)
>
Reviewed-by: Simon Glass
Hi Jan,
On Tue, 28 Feb 2023 at 11:23, Jan Kiszka wrote:
>
> From: Jan Kiszka
>
> The M.2 slots of the related IOT2050 variant need to be configured
> according to the plugged cards. This tries to detect the card using the
> M.2 configuration pins of the B-key slot. If that fails, a U-Boot
> envi
s(+), 4 deletions(-)
Reviewed-by: Simon Glass
gt; to indicate the gpio resource.
>
> Signed-off-by: chao zeng
> ---
> board/siemens/iot2050/board.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Reviewed-by: Simon Glass
You might consider having a board note with this sort of information,
e.g. using sysinfo,
050/board.c| 3 ++
> configs/iot2050_pg1_defconfig| 1 +
> configs/iot2050_pg2_defconfig| 1 +
> 4 files changed, 12 insertions(+), 37 deletions(-)
Reviewed-by: Simon Glass
+
> 1 file changed, 16 insertions(+)
>
Reviewed-by: Simon Glass
standard passage[1] which should not be too far away.
For now, disable it in TPL and SPL.
[1] https://patchwork.ozlabs.org/project/uboot/cover/
20220117150428.1580273-1-...@chromium.org/
Signed-off-by: Simon Glass
Tested-by: Vagrant Cascadian
---
Changes in v2:
- Fix 'accessible&
These board have moved to standard boot but the old 'distro_bootcmd'
command is still active. Disable DISTRO_DEFAULTS to fix this.
Signed-off-by: Simon Glass
Tested-by: Vagrant Cascadian
---
(no changes since v1)
arch/arm/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletio
which have
a very small environment. Disable BOOTSTD_DEFAULTS for smartweb since
it is too close to its limit.
Signed-off-by: Simon Glass
---
(no changes since v1)
boot/Kconfig | 1 +
configs/bk4r1_defconfig | 1 +
co
This relates to booting so move it in to that Kconfig file, before
changing it.
Signed-off-by: Simon Glass
---
Changes in v2:
- Add new patch to move DISTRO_DEFAULTS into boot/
Kconfig | 27 ---
boot/Kconfig | 28 +++-
2 files changed, 27
Ramdisk relocation requires LMB, so enable it automatically to avoid
build errors.
Signed-off-by: Simon Glass
---
(no changes since v1)
boot/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/boot/Kconfig b/boot/Kconfig
index 7815341ada7..da18d4738a7 100644
--- a/boot/Kconfig
+++ b
Currently this does not allow the -lb flags unless CMD_BOOTFLOW_FULL is
enabled, which means that we need two separate boot commands. Relax this
to make things easier. It is only a small amount of extra code.
Signed-off-by: Simon Glass
---
(no changes since v1)
boot/Kconfig | 3 +--
cmd
Drop the distro-boot scripts and use standard boot instead.
Signed-off-by: Simon Glass
---
Changes in v2:
- Add new patch to move rockchip to standard boot
arch/arm/Kconfig | 1 -
include/configs/px30_common.h | 4 +--
include/configs/rk3036_common.h | 5 +--
include
At present, 'buildman -A sandbox' adds the path containing the
toolchain at present. We can assume that this is in the path and
we don't want to set CROSS_COMPILE=/bin/ so change this to align
with what MakeEnvironment() does.
Signed-off-by: Simon Glass
---
tools/buildman/t
Add these flags for the 'replace' subcommand too, to aid debugging.
Signed-off-by: Simon Glass
44 2023 -0700
---
Changes in v2:
- Add new patch to allow preserving the output dir when replacing
tools/binman/binman.rst | 6 ++
tools/binman/cmdline.py | 16 ++--
to
At present these are handled as if they are allowed to be missing, but
this is only true if the -M flag is provided. Fix this and add a test.
Signed-off-by: Simon Glass
---
Changes in v2:
- Add new patch to handle missing bintools correctly in fit
tools/binman/etype/fit.py | 2 ++
tools
And a new entry type which supports generation of x509 certificates.
This uses a new 'openssl' btool with just one operation so far.
Signed-off-by: Simon Glass
---
Changes in v2:
- Add new patch to generate x509 certificates
tools/binman/btool/openssl.py
Implement this feature since it is useful for updating FITs within an
image.
Signed-off-by: Simon Glass
---
Changes in v2:
- Update to support more replacement cases
tools/binman/binman.rst | 16 ++
tools/binman/control.py | 2 +
tools/binman
Hi Quentin,
On Mon, 6 Feb 2023 at 03:26, Quentin Schulz
wrote:
>
> Hi Simon,
>
> On 2/4/23 23:24, Simon Glass wrote:
> > Implement this feature since it is useful for updating FITs within an
> > image.
> >
> > Signed-off-by: Simon Glass
> > ---
&g
s
> ---
> doc/usage/cmd/read.rst | 44 +
> doc/usage/cmd/write.rst | 6 ++
> doc/usage/index.rst | 2 ++
> 3 files changed, 52 insertions(+)
> create mode 100644 doc/usage/cmd/read.rst
> create mode 100644 doc/usage/cmd/write.rst
>
Reviewed-by: Simon Glass
On Wed, 1 Mar 2023 at 20:07, Marek Vasut wrote:
>
> On 3/1/23 16:02, Simon Glass wrote:
> > Hi Marek,
> >
> > On Mon, 27 Feb 2023 at 12:55, Marek Vasut
> > wrote:
> >>
> >> Add 'fdt set' test which works as follows:
> >> - Crea
e still set the variable from 'fdt get addr/size' to
> address of the property or size of the property, where the later
> is 0.
>
> Signed-off-by: Marek Vasut
> Reviewed-by: Simon Glass
> ---
> Cc: Heinrich Schuchardt
> Cc: Simon Glass
> Cc: Tom Rini
&g
t;
> Signed-off-by: Marek Vasut
> ---
> Cc: Heinrich Schuchardt
> Cc: Simon Glass
> Cc: Tom Rini
> ---
> include/test/ut.h | 152 ++---
> test/cmd/pwm.c | 4 +-
> test/dm/acpigen.c | 2 +-
> test/dm/misc.c | 4 +-
&
Hi Johan,
On Thu, 2 Mar 2023 at 17:15, Johan Jonker wrote:
>
> Add devfdt_get_addr_size_index_ptr function with the same
> functionality as devfdt_get_addr_size_index, but instead
> a return pointer is given.
>
> Suggested-by: Michael Nazzareno Trimarchi
> Signed-off-by: Johan Jonker
> Reviewed
o functional change.
>
> Signed-off-by: Rasmus Villemoes
> ---
> cmd/nvedit.c | 11 ---
> 1 file changed, 4 insertions(+), 7 deletions(-)
Reviewed-by: Simon Glass
insertions(+)
> create mode 100644 doc/usage/cmd/panic.rst
Reviewed-by: Simon Glass
On Wed, 1 Mar 2023 at 20:05, Marek Vasut
wrote:
>
> Drop no longer needed { } around ut_assert*() functions in FDT test.
> No functional change.
>
> Signed-off-by: Marek Vasut
> ---
> Cc: Heinrich Schuchardt
> Cc: Simon Glass
> Cc: Tom Rini
> ---
> test/c
this
> breaks anything.
>
> Signed-off-by: Marek Vasut
> ---
> Cc: Heinrich Schuchardt
> Cc: Simon Glass
> Cc: Tom Rini
> ---
> cmd/fdt.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/cmd/fdt.c b/cmd/fdt.c
> index f38fe909c3e..04b664e652c
us files in the drivers directory that cast to
> a pointer.
>
> Signed-off-by: Johan Jonker
> Reviewed-by: Michael Trimarchi
> ---
> drivers/pci/pcie_dw_mvebu.c | 6 +++---
> drivers/spi/cadence_qspi.c | 3 +--
> 2 files changed, 4 insertions(+), 5 deletions(-)
>
Reviewed-by: Simon Glass
Hi Jan,
On Wed, 1 Mar 2023 at 23:38, Jan Kiszka wrote:
>
> On 02.03.23 00:38, Simon Glass wrote:
> > Hi Jan,
> >
> > On Tue, 28 Feb 2023 at 11:20, Jan Kiszka wrote:
> >>
> >> From: Jan Kiszka
> >>
> >> Anything that is not boot-env r
Hi Jan,
On Tue, 28 Feb 2023 at 11:21, Jan Kiszka wrote:
>
> From: Jan Kiszka
>
> There are many ways to get a signed firmware for the IOT2050 devices,
> namely for the parts under user-control. This script documents one way
> of doing it, given a signing key. Augment the board documentation with
Hi Tom,
On Fri, 3 Mar 2023 at 07:50, Tom Rini wrote:
>
> On Wed, Feb 22, 2023 at 09:34:13AM -0700, Simon Glass wrote:
>
> > Drop the invalid SPL_ in a CONFIG_IS_ENABLED() usage. Use the correct
> > X86_64 option in msr.h since SPL may be 32-bit when U-Bout proper is
> &
Hi Johan,
On Thu, 2 Mar 2023 at 17:15, Johan Jonker wrote:
>
> The fdt_addr_t and phys_addr_t size have been decoupled. A 32bit CPU
> can expect 64-bit data from the device tree parser, so use
Why is that? It seems quite inefficient.
> dev_read_addr_index_ptr instead of the dev_read_addr_index
nction.
>
> Signed-off-by: Johan Jonker
> ---
>
> Changed V6:
> use map_sysmem()
>
> Changed V5:
> new patch
> ---
> drivers/core/read.c | 15 ++-
> 1 file changed, 14 insertions(+), 1 deletion(-)
Reviewed-by: Simon Glass
Hi Tom,
On Fri, 3 Mar 2023 at 16:43, Tom Rini wrote:
>
> On Wed, Feb 22, 2023 at 09:33:41AM -0700, Simon Glass wrote:
>
> > This series was split out of the old 'split config' splc series. It
> > contains clean-up patches which do not depend on split config.
>
Hi Marek,
On Mon, 6 Mar 2023 at 12:07, Marek Vasut wrote:
>
> On 3/6/23 18:53, Simon Glass wrote:
> > Hi Marek,
> >
> > On Wed, 1 Mar 2023 at 20:04, Marek Vasut
> > wrote:
> >>
> >> The 'fdt get addr' is always assumed to be hex value, d
Hi Stephen,
On Tue, 7 Mar 2023 at 16:33, wrote:
>
> From: Stephen Carlson
>
> Adds a test for the new pci_mps command to ensure that it can set the
> Maximum Payload Size (MPS) of all devices to 256 bytes in the sandbox
> environment. Enables the pci_mps command in the sandbox environment so
> t
Hi Johan,
On Wed, 8 Mar 2023 at 00:39, Johan Jonker wrote:
>
>
>
> On 3/3/23 01:14, Johan Jonker wrote:
> > The fdt_addr_t and phys_addr_t size have been decoupled.
> > A 32bit CPU can expect 64-bit data from the device tree parser,
> > so convert regmap_init_mem_plat() input to phys_addr_t in fi
all_skips
- Sort the set directly instead of converting it to a list first
Changes in v2:
- Fix 'uncorrect' typo in subject
- Fix missing 'f' on format string
Simon Glass (4):
patman: Drop an incorrect comment about git am
patman: Refactor MakeCcFile() into two fun
Patman does not do this anymore, as of this commit:
7428dc14b0f ("patman: Remove the -a option")
Drop the comment.
Signed-off-by: Simon Glass
Reviewed-by: Douglas Anderson
---
Changes in v3:
- Mention the commit where the -a option was dropped
Changes in v2:
- Fix 'unc
This function is quite long. Moving the handling of a commit into a
separate function. This will make it easier to do the work in parallel.
Update function comments while we are here.
Signed-off-by: Simon Glass
Reviewed-by: Douglas Anderson
---
Changes in v3:
- Reword comment for all_skips
This script can take ages on some series. Try to limit the time by
using threads. If a few stubborn patches remain, show progress so the
user has some idea what is going on.
Signed-off-by: Simon Glass
Reviewed-by: Douglas Anderson
---
(no changes since v1)
tools/patmanu/func_test.py | 2
For large series this can take a while. Run checkpatch in parallel to
try to reduce the time. The checkpatch information is still reported in
sequential order, so a very slow patch at the start can still slow
things down. But overall this gives good results.
Signed-off-by: Simon Glass
Reviewed
This script can take ages on some series. Try to limit the time by
using threads. If a few stubborn patches remain, show progress so the
user has some idea what is going on.
Signed-off-by: Simon Glass
Reviewed-by: Douglas Anderson
---
(no changes since v1)
tools/patmanu/func_test.py | 2
For large series this can take a while. Run checkpatch in parallel to
try to reduce the time. The checkpatch information is still reported in
sequential order, so a very slow patch at the start can still slow
things down. But overall this gives good results.
Signed-off-by: Simon Glass
Reviewed
This function is quite long. Moving the handling of a commit into a
separate function. This will make it easier to do the work in parallel.
Update function comments while we are here.
Signed-off-by: Simon Glass
Reviewed-by: Douglas Anderson
---
Changes in v3:
- Reword comment for all_skips
At present, 'buildman -A sandbox' adds the path containing the
toolchain at present. We can assume that this is in the path and
we don't want to set CROSS_COMPILE=/bin/ so change this to align
with what MakeEnvironment() does.
Signed-off-by: Simon Glass
---
tools/buildman/t
Patman does not do this anymore, as of this commit:
7428dc14b0f ("patman: Remove the -a option")
Drop the comment.
Signed-off-by: Simon Glass
Reviewed-by: Douglas Anderson
---
Changes in v3:
- Mention the commit where the -a option was dropped
Changes in v2:
- Fix 'unc
The test case can be triggered using:
"
./u-boot -Dc 'ut fdt'
"
To dump the full output from commands used during test, add '-v' flag.
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
V2: Ad
:
"
./u-boot -Dc 'ut fdt'
"
To dump the full output from commands used during test, add '-v' flag.
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
V2: Add RB from Simon
---
test/cmd/fdt.c | 33 +++
Add list of missing tests for the 'fdt' command, currently
the missing sandbox tests are only 'fdt boardsetup' and
'fdt checksign' .
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
V2: Add RB from
lues using 'fdt get value'
The test case can be triggered using:
"
./u-boot -Dc 'ut fdt'
"
To dump the full output from commands used during test, add '-v' flag.
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
x27;-v' flag.
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
V2: Add RB from Simon
---
test/cmd/fdt.c | 79 ++
1 file changed, 79 insertions(+)
Applied to u-boot-dm/next, thanks!
The 'fdt get' command has a 'get value' subcommand, rename the fdt_test_get()
to fdt_test_get_value() to avoid confusion about what it is testing. There is
currently no get 'get name', 'get addr', 'get size' subcommand test.
Signed-off-by: Marek
-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
V2: Add RB from Simon
---
test/cmd/fdt.c | 58 --
1 file changed, 42 insertions(+), 16 deletions(-)
Applied to u-boot-dm/next, thanks!
Add helper macro to test for empty lines, which is an inobvious
wrapper around ut_assert_nextline("%s", "") .
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
V2: Add RB from Simon
---
include/test/ut.h | 4
The 'fdt' command help contains the following note:
"
Dereference aliases by omitting the leading '/', e.g. fdt print ethernet0.
"
Add test for it.
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
V
ddr var / compatible
; md $var'
"
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
V2: Add RB from Simon
---
cmd/fdt.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
Applied to u-boot-dm/next, thanks!
The 'fdt' command has a 'resize' subcommand, rename the fdt_test_resize()
to fdt_test_addr_resize() to avoid confusion about what it is testing.
There is currently no resize test.
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon
t index is set to -1 instead of 0.
This is OK, since the only place which checks for index to be less or
equal zero is the string array handling code in fdt_value_env_set() and
that code would work perfectly well with index -1 too.
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc
The command assumed 32bit pointers so far, with 64bit pointer the
command would overwrite a piece of stack. Fix it by extending the
array size to cater for 64bit pointer, and use snprintf() to avoid
writing past the end of the array ever again.
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
the full output from commands used during test, add '-v' flag.
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
V2: Add RB from Simon
---
test/cmd/fdt.c | 87 ++
1 file
fdt'
"
To dump the full output from commands used during test, add '-v' flag.
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
V2: Add RB from Simon
---
test/cmd/fdt.c | 34 ++
1
eviewed-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
V2: Add RB from Simon
---
test/cmd/fdt.c | 124 +++--
1 file changed, 111 insertions(+), 13 deletions(-)
Applied to u-boot-dm/next, thanks!
On case 'fdt bootcpu' is invoked without parameters, argv[2] is not
valid and this command would SEGFAULT in sandbox environment. Add
missing argc test to avoid the crash and rather print usage help
message.
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich Schu
"
To dump the full output from commands used during test, add '-v' flag.
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
V2: Add RB from Simon
---
test/cmd/fdt.c | 84 +
The help text references 'addr' as an optional key start address,
but the explanation references the same as 'start', make sure they
both read as 'addr'. Also update the abbreviated 'addr' in the
explanation to 'address'.
Signed-off-by: Marek
dump the full output from commands used during test, add '-v' flag.
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
V2: Add RB from Simon
---
test/cmd/fdt.c | 24
1 file changed, 24 insertions(+)
Applied to u-boot-dm/next, thanks!
ai Pham
Signed-off-by: Nam Nguyen
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
V2: Add RB from Simon
---
cmd/fdt.c | 36 ++--
1 file changed, 14 insertions(+), 22 deletions(-)
Applied to u-boot-dm/next, thanks!
Add a simple check that the PyPi packages can be built.
Signed-off-by: Simon Glass
---
Changes in v2:
- Update the cover letter
.azure-pipelines.yml | 10 ++
.gitlab-ci.yml | 6 ++
2 files changed, 16 insertions(+)
Applied to u-boot-dm/next, thanks!
Create the necessary files to build this new package.
It is not actually clear whether this is useful, since buildman has no
purpose outside U-Boot.
Move the main program into a function so that it can easily be called by
the PyPi-created script.
Signed-off-by: Simon Glass
---
(no changes
e still set the variable from 'fdt get addr/size' to
> address of the property or size of the property, where the later
> is 0.
>
> Signed-off-by: Marek Vasut
> Reviewed-by: Simon Glass
> ---
> Cc: Heinrich Schuchardt
> Cc: Simon Glass
> Cc: Tom Rini
&g
This allows tests to run in parallel and speeds up some tests markedly,
particularly with binman. Add it to the requirements.
Signed-off-by: Simon Glass
---
(no changes since v1)
test/py/requirements.txt | 1 +
1 file changed, 1 insertion(+)
Applied to u-boot-dm/next, thanks!
On case 'fdt memory' is invoked without parameters, argv[2]/argv[3]
is not valid and this command would SEGFAULT in sandbox environment.
Add missing argc test to avoid the crash and rather print usage help
message.
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich
ornously returns 'err' instead of
CMD_RET_FAILURE, fix it. Furthermore, align the number of spaces past
fdt_delprop() in error message with the rest of the code.
Signed-off-by: Marek Vasut
Reviewed-by: Simon Glass
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
V2: Add R
These can be installed with 'pip' now. Add the details for those who are
interested.
Signed-off-by: Simon Glass
---
(no changes since v1)
tools/binman/binman.rst | 13 +
tools/patman/patman.rst | 12
2 files changed, 25 insertions(+)
Applied to u-boot-dm/next, thanks!
Use this function so that the help can be found even when binman is
running from a package.
Signed-off-by: Simon Glass
---
Changes in v2:
- Add new patch to use importlib to find the help
tools/binman/control.py | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
Applied to u-boot-dm
It doesn't make much sense to expose tests when dtoc is running
outside of the U-Boot git checkout. Hide the option in this case.
Fix a long line while we are here.
Signed-off-by: Simon Glass
---
Changes in v2:
- Add a patch to hide the test options unless test code is available
tools/b
Update this so that the directory being used is declared at the top of
the file. Use pathlib as it seems to be more modern.
Signed-off-by: Simon Glass
---
(no changes since v1)
tools/dtoc/test_dtoc.py | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
Applied to u-boot-dm/next
Put this code into a function so it is easy for it be run when packaged.
Signed-off-by: Simon Glass
---
Changes in v2:
- Add patch to split out the main code into a function
tools/binman/main.py | 12
1 file changed, 8 insertions(+), 4 deletions(-)
Applied to u-boot-dm/next
Create the necessary files to build this new package.
This is needed for binman.
Move the main program into a function so that it can easily be called by
the PyPi-created script.
Signed-off-by: Simon Glass
---
(no changes since v1)
Makefile | 1 +
tools/dtoc/README.rst
Put this code into a function so it is easy for it be run when packaged.
Signed-off-by: Simon Glass
---
Changes in v2:
- Add patch to split out the main code into a function
tools/dtoc/main.py | 100 +++--
1 file changed, 52 insertions(+), 48 deletions
It doesn't make much sense to expose tests when dtoc is running
outside of the U-Boot git checkout. Hide the option in this case.
Signed-off-by: Simon Glass
---
Changes in v2:
- Add a patch to hide the test options unless test code is available
tools/dtoc/main.py | 19 +
It doesn't make much sense to expose tests when buildman is running
outside of the U-Boot git checkout. Hide the option in this case
Signed-off-by: Simon Glass
---
Changes in v2:
- Add a patch to hide the test options unless test code is available
tools/buildman/cmdline.py
called by
the PyPi-created script.
Signed-off-by: Simon Glass
---
(no changes since v1)
Makefile | 1 +
tools/buildman/pyproject.toml | 29 +
2 files changed, 30 insertions(+)
create mode 100644 tools/buildman/pyproject.toml
Applied to u-boot-dm
301 - 400 of 57637 matches
Mail list logo
