Hi Tom, On Thu, Jul 25, 2019 at 6:13 PM Tom Rini <tr...@konsulko.com> wrote: > > On Thu, Jul 25, 2019 at 04:33:21PM +0300, Sam Protsenko wrote: > > Hi Simon, > > > > On Sat, Jul 6, 2019 at 7:53 PM Simon Glass <s...@chromium.org> wrote: > > > > > > Hi, > > > > > > On Tue, 2 Jul 2019 at 12:18, Sam Protsenko <semen.protse...@linaro.org> > > > wrote: > > > > > > > > Signed-off-by: Sam Protsenko <semen.protse...@linaro.org> > > > > --- > > > > test/py/tests/{ => test_android}/test_avb.py | 0 > > > > 1 file changed, 0 insertions(+), 0 deletions(-) > > > > rename test/py/tests/{ => test_android}/test_avb.py (100%) > > > > > > > > diff --git a/test/py/tests/test_avb.py > > > > b/test/py/tests/test_android/test_avb.py > > > > similarity index 100% > > > > rename from test/py/tests/test_avb.py > > > > rename to test/py/tests/test_android/test_avb.py > > > > -- > > > > > > I'd prefer to keep tsts in one place. What is the goal of this patch? > > > > > > > Similar patch was merged recently (adding test_android/ dir): > > > > https://patchwork.ozlabs.org/patch/1128013/ > > > > So we need either: > > 1. move that test out of test_android/ dir, and drop this one > > 2. or apply this patch > > > > What would you prefer? (I've explained my perspective on that matter > > earlier in this thread). We should do either (1) or (2) to keep the > > consistency. > > Note that we've also had vboot as a sub-directory since it was > introduced and test_fs is also a sub-directory. My two cents is that > sub-directories help with organization as the number of tests increase. >
If there are no further concerns, can you please apply it for -rc2, so it doesn't get lost in ML? Thanks! > -- > Tom _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot