On Mon, Aug 14, 2023 at 4:21 AM Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote:
>
> Hi Rob,
>
> On Tue, Aug 1, 2023 at 10:50 PM Rob Herring <robh+dt@xxxxxxxxxx> wrote:
> > On Fri, Jul 28, 2023 at 2:50 AM Geert Uytterhoeven
> > <geert+renesas@xxxxxxxxx> wrote:
> > > When of_overlay_fdt_apply() fails, the changeset may be partially
> > > applied, and the caller is still expected to call of_overlay_remove() to
> > > clean up this partial state. However, overlay_17 is the only test that
> > > takes care of cleaning up after an (expected) failure.
> > >
> > > Instead of adding cleanup code to each individual test, extend
> > > overlay_info with the optional expected return value of
> > > of_overlay_remove(), and handle cleanup in the overlay_data_apply()
> > > helper. While at it, simplify the end marker in the overlay_info table.
> > >
> > > Update the expected error output for errors during the newly cleanup.
> > >
> > > Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> > > ---
> > > v2:
> > > - No changes.
>
> > > --- a/drivers/of/unittest.c
> > > +++ b/drivers/of/unittest.c
>
> > > @@ -3689,9 +3715,19 @@ static __init void of_unittest_overlay_high_level(void)
> > > unittest(overlay_data_apply("overlay_bad_phandle", NULL),
> > > "Adding overlay 'overlay_bad_phandle' failed\n");
> > >
> > > + EXPECT_BEGIN(KERN_ERR,
> > > + "OF: changeset: remove_property failed @/testcase-data-2/substation@100/hvac-medium-2/name");
> > > + EXPECT_BEGIN(KERN_ERR,
> > > + "OF: resolver: overlay phandle fixup failed: -22");
> > > +
> > > unittest(overlay_data_apply("overlay_bad_symbol", NULL),
> > > "Adding overlay 'overlay_bad_symbol' failed\n");
> > >
> > > + EXPECT_END(KERN_ERR,
> > > + "OF: resolver: overlay phandle fixup failed: -22");
> >
> > I'm seeing "OF: Error reverting changeset (-19)" here instead.
> > Cut-n-paste error?
>
> Indeed. Thanks for pointing this out!
I've fixed this up along with the other EXPECTs that changed due to my
rework and applied the series.
Thanks,
Rob
