Thanks Simon On Fri, 2 Jun 2023 at 00:51, Simon Glass <s...@chromium.org> wrote: > > On Thu, 1 Jun 2023 at 00:18, Ilias Apalodimas > <ilias.apalodi...@linaro.org> wrote: > > > > One out comments is off by one, adjust it > > > > Signed-off-by: Ilias Apalodimas <ilias.apalodi...@linaro.org> > > --- > > test/dm/tpm.c | 9 +++++---- > > 1 file changed, 5 insertions(+), 4 deletions(-) > > > > Reviewed-by: Simon Glass <s...@chromium.org> > > (I'm not sure I understand the commit message though) >
I'll fix it during the merge, it was supposed to say "One of our comments" > > > diff --git a/test/dm/tpm.c b/test/dm/tpm.c > > index 3defb3c3da1f..cde933ab2848 100644 > > --- a/test/dm/tpm.c > > +++ b/test/dm/tpm.c > > @@ -98,10 +98,11 @@ static int test_tpm_autostart(struct unit_test_state > > *uts, > > > > if (reinit) > > ut_assertok(tpm_init(dev)); > > - /* > > - * tpm_auto_start will rerun tpm_init() if reinit, but handles the > > - * -EBUSY return code internally. > > - */ > > + > > + /* > > + * tpm_auto_start will rerun tpm_init() if reinit, but handles the > > + * -EBUSY return code internally. > > + */ > > ut_assertok(tpm_auto_start(dev)); > > > > return 0; > > -- > > 2.39.2 > >
