[openwrt/openwrt] ramips: mt7621: use preferred logic in lib/upgrade/iodata.sh

LEDE Commits lede-commits at lists.infradead.org
Sun Feb 7 15:58:59 EST 2021


adrian pushed a commit to openwrt/openwrt.git, branch master:
https://git.openwrt.org/1c0e13db43b41a1d527428b82bf88a032a916e84

commit 1c0e13db43b41a1d527428b82bf88a032a916e84
Author: Adrian Schmutzler <freifunk at adrianschmutzler.de>
AuthorDate: Sun Feb 7 21:42:19 2021 +0100

    ramips: mt7621: use preferred logic in lib/upgrade/iodata.sh
    
    shellcheck recommends || and && over "-a" and "-o" because the
    latter are not well defined.
    
    Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
---
 target/linux/ramips/mt7621/base-files/lib/upgrade/iodata.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/target/linux/ramips/mt7621/base-files/lib/upgrade/iodata.sh b/target/linux/ramips/mt7621/base-files/lib/upgrade/iodata.sh
index 071a132e02..8303ae9922 100644
--- a/target/linux/ramips/mt7621/base-files/lib/upgrade/iodata.sh
+++ b/target/linux/ramips/mt7621/base-files/lib/upgrade/iodata.sh
@@ -29,7 +29,7 @@ iodata_mstc_upgrade_prepare() {
 		iodata_mstc_prepare_fail
 	fi
 
-	if [ -z "$persist_mtd" -o -z "$factory_mtd" ]; then
+	if [ -z "$persist_mtd" ] || [ -z "$factory_mtd" ]; then
 		echo 'cannot find mtd partition(s), "factory" or "persist"'
 		iodata_mstc_prepare_fail
 	fi
@@ -37,11 +37,11 @@ iodata_mstc_upgrade_prepare() {
 	local bootnum=$(hexdump -s 4 -n 1 -e '"%x"' ${persist_mtd})
 	local debugflag=$(hexdump -s $((dflag_offset)) -n 1 -e '"%x"' ${factory_mtd})
 
-	if [ "$bootnum" != "1" -a "$bootnum" != "2" ]; then
+	if [ "$bootnum" != "1" ] && [ "$bootnum" != "2" ]; then
 		echo "failed to get bootnum, please check the value at 0x4 in ${persist_mtd}"
 		iodata_mstc_prepare_fail
 	fi
-	if [ "$debugflag" != "0" -a "$debugflag" != "1" ]; then
+	if [ "$debugflag" != "0" ] && [ "$debugflag" != "1" ]; then
 		echo "failed to get debugflag, please check the value at ${dflag_offset} in ${factory_mtd}"
 		iodata_mstc_prepare_fail
 	fi



More information about the lede-commits mailing list