Hi Marek, On Thu, 9 Mar 2023 at 19:34, Marek Vasut <marek.va...@mailbox.org> wrote: > > On 3/9/23 23:09, Simon Glass wrote: > > Hi Marek, > > > > On Wed, 8 Mar 2023 at 15:16, Marek Vasut <marek.va...@mailbox.org> wrote: > >> > >> On 3/8/23 23:23, Simon Glass wrote: > >>> Hi Marek, > >>> > >>> On Wed, 1 Mar 2023 at 19:09, Marek Vasut > >>> <marek.vasut+rene...@mailbox.org> wrote: > >>>> > >>>> Add 'fdt memory' test which works as follows: > >>>> - Create custom FDT with /memory node, with select #*cells, map it to > >>>> sysmem > >>>> - Perform memory fixup > >>>> - Read back the /memory node and validate its content > >>>> > >>>> 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 <marek.vasut+rene...@mailbox.org> > >>>> Reviewed-by: Simon Glass <s...@chromium.org> > >>>> --- > >>>> Cc: Heinrich Schuchardt <heinrich.schucha...@canonical.com> > >>>> Cc: Simon Glass <s...@chromium.org> > >>>> Cc: Tom Rini <tr...@konsulko.com> > >>>> --- > >>>> V2: Add RB from Simon > >>>> --- > >>>> test/cmd/fdt.c | 83 ++++++++++++++++++++++++++++++++++++++++++++++++++ > >>>> 1 file changed, 83 insertions(+) > >>>> > >>> > >>> This patch and some others cause test failures. I applied those that I > >>> could to u-boot-dm/next > >>> > >>> Can you please rebase and take a look? > >> > >> Does it work any better if you also apply: > >> > >> [PATCH] console: Use only 0x00 as line separator for console recording > >> > >> ? > >> > >> If so, then just pick the aforementioned patch and the rest of this series. > > > > Yes, but one of the patches causes a fault in sandbox_spl: > > > > https://source.denx.de/u-boot/custodians/u-boot-dm/-/jobs/590598 > > None in this series, but I sent out > > [PATCH v2 1/2] test: Wrap assert macros in ({ ... }) and fix missing > semicolons > > That should help.
Actually that seems to be what causes the problem with sandbox_spl, but I didn't dig into it. But I see you sent a v2 so will look at that. Regards, Simom Applied to u-boot-dm/next, thanks!
