Please pull u-boot-dm/next

Hi Tom,
This is for the -next branch
https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/15568
The following changes since commit bcf343146ff365a88481b9a80920ed146c6dee5b:
Merge tag 'dm-next-9mar23' of https://source.denx.de/u-boot/custodians/u-boot-dm into next (2023-03-09 11:22:50 -0500)
are available in the Git repository at:
git://git.denx.de/u-boot-dm.git tags/dm-next-12mar23
for you to fetch changes up to 7bb28330f7b4d20bcf844ec306dcfbb41296cfa8:
buildman: Fix CROSS_COMPILE output for sandbox (2023-03-12 11:37:34 -0600)
---------------------------------------------------------------- More tests and fixes for fdt command binman signing feature fix buildman -A bug introduced recently
---------------------------------------------------------------- Ivan Mikhaylov (3): binman: add documentation for binman sign option binman: add sign option for binman binman: add tests for sign option
Marek Vasut (10): test: Wrap assert macros in ({ ... }) and fix missing semicolons test: cmd: fdt: Drop new unneeded curly brackets console: Use only 0x00 as line separator for console recording cmd: fdt: Drop the 0x prefix test: cmd: fdt: Test fdt print and list test: cmd: fdt: Test fdt header test: cmd: fdt: Test fdt memory test: cmd: fdt: Test fdt rsvmem test: cmd: fdt: Test fdt chosen test: cmd: fdt: Test fdt apply
Roman Kopytin (1): tools: add fdt_add_pubkey
Simon Glass (2): Revert "buildman: Correct CROSS_COMPILE output for sandbox" buildman: Fix CROSS_COMPILE output for sandbox
cmd/fdt.c | 2 +- common/console.c | 2 +- common/main.c | 1 + include/test/ut.h | 152 ++++++++----- test/cmd/fdt.c | 562 ++++++++++++++++++++++++++++++++++++++++++++++++- test/cmd/pwm.c | 4 +- test/dm/acpigen.c | 2 +- test/dm/misc.c | 4 +- test/dm/phy.c | 8 +- test/dm/scmi.c | 4 +- test/lib/kconfig.c | 10 +- test/lib/kconfig_spl.c | 6 +- test/unicode_ut.c | 6 +- tools/.gitignore | 1 + tools/Makefile | 3 + tools/binman/binman.rst | 47 +++++ tools/binman/cmdline.py | 13 ++ tools/binman/control.py | 30 ++- tools/binman/entry.py | 3 + tools/binman/etype/fit.py | 16 ++ tools/binman/ftest.py | 93 ++++++++ tools/binman/test/280_fit_sign.dts | 63 ++++++ tools/binman/test/281_sign_non_fit.dts | 65 ++++++ tools/buildman/toolchain.py | 4 +- tools/fdt_add_pubkey.c | 138 ++++++++++++ 25 files changed, 1155 insertions(+), 84 deletions(-) create mode 100644 tools/binman/test/280_fit_sign.dts create mode 100644 tools/binman/test/281_sign_non_fit.dts create mode 100644 tools/fdt_add_pubkey.c
Regards, Simon

On Sun, Mar 12, 2023 at 09:10:19PM -0600, Simon Glass wrote:
Hi Tom,
This is for the -next branch
https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/15568
The following changes since commit bcf343146ff365a88481b9a80920ed146c6dee5b:
Merge tag 'dm-next-9mar23' of https://source.denx.de/u-boot/custodians/u-boot-dm into next (2023-03-09 11:22:50 -0500)
are available in the Git repository at:
git://git.denx.de/u-boot-dm.git tags/dm-next-12mar23
for you to fetch changes up to 7bb28330f7b4d20bcf844ec306dcfbb41296cfa8:
buildman: Fix CROSS_COMPILE output for sandbox (2023-03-12 11:37:34 -0600)
There's something, or somethings, broken here now. Take a look at: https://source.denx.de/u-boot/u-boot/-/jobs/593223#L904 where now there's a bunch of warnings but we don't fail the build (nor in https://source.denx.de/u-boot/u-boot/-/jobs/593261 which is building the whole arch, not just a test.py special build). That shouldn't happen, nor should there be these warnings to start with.

Hi Tom,
On Mon, 13 Mar 2023 at 13:36, Tom Rini trini@konsulko.com wrote:
On Sun, Mar 12, 2023 at 09:10:19PM -0600, Simon Glass wrote:
Hi Tom,
This is for the -next branch
https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/15568
The following changes since commit bcf343146ff365a88481b9a80920ed146c6dee5b:
Merge tag 'dm-next-9mar23' of https://source.denx.de/u-boot/custodians/u-boot-dm into next (2023-03-09 11:22:50 -0500)
are available in the Git repository at:
git://git.denx.de/u-boot-dm.git tags/dm-next-12mar23
for you to fetch changes up to 7bb28330f7b4d20bcf844ec306dcfbb41296cfa8:
buildman: Fix CROSS_COMPILE output for sandbox (2023-03-12 11:37:34 -0600)
There's something, or somethings, broken here now. Take a look at: https://source.denx.de/u-boot/u-boot/-/jobs/593223#L904 where now there's a bunch of warnings but we don't fail the build (nor
That seems to have been going for a while. See:
https://source.denx.de/u-boot/u-boot/-/jobs/593528
I don't think it has anything to do with this pull request.
in https://source.denx.de/u-boot/u-boot/-/jobs/593261 which is building the whole arch, not just a test.py special build). That shouldn't happen, nor should there be these warnings to start with.
This is a warning I reported to Marek and he sent a patch. I will redo the pull with that incorporated.
Regards, Simon

On Tue, Mar 14, 2023 at 04:07:57PM -0600, Simon Glass wrote:
Hi Tom,
On Mon, 13 Mar 2023 at 13:36, Tom Rini trini@konsulko.com wrote:
On Sun, Mar 12, 2023 at 09:10:19PM -0600, Simon Glass wrote:
Hi Tom,
This is for the -next branch
https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/15568
The following changes since commit bcf343146ff365a88481b9a80920ed146c6dee5b:
Merge tag 'dm-next-9mar23' of https://source.denx.de/u-boot/custodians/u-boot-dm into next (2023-03-09 11:22:50 -0500)
are available in the Git repository at:
git://git.denx.de/u-boot-dm.git tags/dm-next-12mar23
for you to fetch changes up to 7bb28330f7b4d20bcf844ec306dcfbb41296cfa8:
buildman: Fix CROSS_COMPILE output for sandbox (2023-03-12 11:37:34 -0600)
There's something, or somethings, broken here now. Take a look at: https://source.denx.de/u-boot/u-boot/-/jobs/593223#L904 where now there's a bunch of warnings but we don't fail the build (nor
That seems to have been going for a while. See:
I don't see it there, no. The problem, to be clear, is warnings in C NOT leading to CI failing.

Hi Tom,
On Wed, 15 Mar 2023 at 08:49, Tom Rini trini@konsulko.com wrote:
On Tue, Mar 14, 2023 at 04:07:57PM -0600, Simon Glass wrote:
Hi Tom,
On Mon, 13 Mar 2023 at 13:36, Tom Rini trini@konsulko.com wrote:
On Sun, Mar 12, 2023 at 09:10:19PM -0600, Simon Glass wrote:
Hi Tom,
This is for the -next branch
https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/15568
The following changes since commit bcf343146ff365a88481b9a80920ed146c6dee5b:
Merge tag 'dm-next-9mar23' of https://source.denx.de/u-boot/custodians/u-boot-dm into next (2023-03-09 11:22:50 -0500)
are available in the Git repository at:
git://git.denx.de/u-boot-dm.git tags/dm-next-12mar23
for you to fetch changes up to 7bb28330f7b4d20bcf844ec306dcfbb41296cfa8:
buildman: Fix CROSS_COMPILE output for sandbox (2023-03-12 11:37:34 -0600)
There's something, or somethings, broken here now. Take a look at: https://source.denx.de/u-boot/u-boot/-/jobs/593223#L904 where now there's a bunch of warnings but we don't fail the build (nor
That seems to have been going for a while. See:
I don't see it there, no. The problem, to be clear, is warnings in C NOT leading to CI failing.
OK that's fine, but what does it have to do with this pull request?
I can take a look at the problem, but it does not seem related.
Regards, Simon

On Thu, Mar 16, 2023 at 07:58:33AM -0600, Simon Glass wrote:
Hi Tom,
On Wed, 15 Mar 2023 at 08:49, Tom Rini trini@konsulko.com wrote:
On Tue, Mar 14, 2023 at 04:07:57PM -0600, Simon Glass wrote:
Hi Tom,
On Mon, 13 Mar 2023 at 13:36, Tom Rini trini@konsulko.com wrote:
On Sun, Mar 12, 2023 at 09:10:19PM -0600, Simon Glass wrote:
Hi Tom,
This is for the -next branch
https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/15568
The following changes since commit bcf343146ff365a88481b9a80920ed146c6dee5b:
Merge tag 'dm-next-9mar23' of https://source.denx.de/u-boot/custodians/u-boot-dm into next (2023-03-09 11:22:50 -0500)
are available in the Git repository at:
git://git.denx.de/u-boot-dm.git tags/dm-next-12mar23
for you to fetch changes up to 7bb28330f7b4d20bcf844ec306dcfbb41296cfa8:
buildman: Fix CROSS_COMPILE output for sandbox (2023-03-12 11:37:34 -0600)
There's something, or somethings, broken here now. Take a look at: https://source.denx.de/u-boot/u-boot/-/jobs/593223#L904 where now there's a bunch of warnings but we don't fail the build (nor
That seems to have been going for a while. See:
I don't see it there, no. The problem, to be clear, is warnings in C NOT leading to CI failing.
OK that's fine, but what does it have to do with this pull request?
I don't know when this regression happened, only that I saw it, and the warnings at the same time. So I'm seeing now (after sorting out some failing lab equipment) if the "C warnings cause CI to fail" changes are in master, or next, or neither.

On Thu, Mar 16, 2023 at 10:56:17AM -0400, Tom Rini wrote:
On Thu, Mar 16, 2023 at 07:58:33AM -0600, Simon Glass wrote:
Hi Tom,
On Wed, 15 Mar 2023 at 08:49, Tom Rini trini@konsulko.com wrote:
On Tue, Mar 14, 2023 at 04:07:57PM -0600, Simon Glass wrote:
Hi Tom,
On Mon, 13 Mar 2023 at 13:36, Tom Rini trini@konsulko.com wrote:
On Sun, Mar 12, 2023 at 09:10:19PM -0600, Simon Glass wrote:
Hi Tom,
This is for the -next branch
https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/15568
The following changes since commit bcf343146ff365a88481b9a80920ed146c6dee5b:
Merge tag 'dm-next-9mar23' of https://source.denx.de/u-boot/custodians/u-boot-dm into next (2023-03-09 11:22:50 -0500)
are available in the Git repository at:
git://git.denx.de/u-boot-dm.git tags/dm-next-12mar23
for you to fetch changes up to 7bb28330f7b4d20bcf844ec306dcfbb41296cfa8:
buildman: Fix CROSS_COMPILE output for sandbox (2023-03-12 11:37:34 -0600)
There's something, or somethings, broken here now. Take a look at: https://source.denx.de/u-boot/u-boot/-/jobs/593223#L904 where now there's a bunch of warnings but we don't fail the build (nor
That seems to have been going for a while. See:
I don't see it there, no. The problem, to be clear, is warnings in C NOT leading to CI failing.
OK that's fine, but what does it have to do with this pull request?
I don't know when this regression happened, only that I saw it, and the warnings at the same time. So I'm seeing now (after sorting out some failing lab equipment) if the "C warnings cause CI to fail" changes are in master, or next, or neither.
Alright, so, there was something very odd with what got triggered before, as I cannot make CI pass when it should fail with a simple #warning. I'll review this PR fully now.

On Sun, Mar 12, 2023 at 09:10:19PM -0600, Simon Glass wrote:
Hi Tom,
This is for the -next branch
https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/15568
The following changes since commit bcf343146ff365a88481b9a80920ed146c6dee5b:
Merge tag 'dm-next-9mar23' of https://source.denx.de/u-boot/custodians/u-boot-dm into next (2023-03-09 11:22:50 -0500)
are available in the Git repository at:
git://git.denx.de/u-boot-dm.git tags/dm-next-12mar23
for you to fetch changes up to 7bb28330f7b4d20bcf844ec306dcfbb41296cfa8:
buildman: Fix CROSS_COMPILE output for sandbox (2023-03-12 11:37:34 -0600)
Applied to u-boot/next, thanks!
participants (2)
-
Simon Glass
-
Tom Rini