| Message ID | 20211214080919.22616-1-brgl@bgdev.pl (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 36ccddf80e56b8c51604bafd449522a5271bfd35 |
| Headers | show |
| Series | [-next] selftests: gpio: gpio-sim: avoid forking test twice | expand |
On Tue, Dec 14, 2021 at 09:09:19AM +0100, Bartosz Golaszewski wrote: > Use '-o' within [] in order to avoid spawning two processes for test. Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Thanks! > Suggested-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl> > --- > tools/testing/selftests/gpio/gpio-sim.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/testing/selftests/gpio/gpio-sim.sh b/tools/testing/selftests/gpio/gpio-sim.sh > index c913d5aec768..341e3de00896 100755 > --- a/tools/testing/selftests/gpio/gpio-sim.sh > +++ b/tools/testing/selftests/gpio/gpio-sim.sh > @@ -23,7 +23,7 @@ remove_chip() { > > for FILE in $CONFIGFS_DIR/$CHIP/*; do > BANK=`basename $FILE` > - if [ "$BANK" = "live" ] || [ "$BANK" = "dev_name" ]; then > + if [ "$BANK" = "live" -o "$BANK" = "dev_name" ]; then > continue > fi > > -- > 2.30.1 >
diff --git a/tools/testing/selftests/gpio/gpio-sim.sh b/tools/testing/selftests/gpio/gpio-sim.sh index c913d5aec768..341e3de00896 100755 --- a/tools/testing/selftests/gpio/gpio-sim.sh +++ b/tools/testing/selftests/gpio/gpio-sim.sh @@ -23,7 +23,7 @@ remove_chip() { for FILE in $CONFIGFS_DIR/$CHIP/*; do BANK=`basename $FILE` - if [ "$BANK" = "live" ] || [ "$BANK" = "dev_name" ]; then + if [ "$BANK" = "live" -o "$BANK" = "dev_name" ]; then continue fi
Use '-o' within [] in order to avoid spawning two processes for test. Suggested-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl> --- tools/testing/selftests/gpio/gpio-sim.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)