[PATCH blktests] blktests: use consistent helper returns

[Chaitanya Kulkarni]

This is a cleanup patch which uses consistent return values
for helper functions.

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulka...@wdc.com>
---
 common/rc | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/common/rc b/common/rc
index e59ae0b..936b21d 100644
--- a/common/rc
+++ b/common/rc
@@ -51,10 +51,9 @@ _have_root() {
 _have_module() {
        if modprobe -n -q "$1"; then
                return 0
-       else
-               SKIP_REASON="$1 module is not available"
-               return 1
        fi
+       SKIP_REASON="$1 module is not available"
+       return 1
 }
 
 _have_module_param() {
@@ -62,15 +61,15 @@ _have_module_param() {
                SKIP_REASON="$1 module does not have parameter $2"
                return 1
        fi
+       return 0
 }
 
 _have_program() {
        if command -v "$1" >/dev/null 2>&1; then
                return 0
-       else
-               SKIP_REASON="$1 is not available"
-               return 1
        fi
+       SKIP_REASON="$1 is not available"
+       return 1
 }
 
 _have_src_program() {
-- 
2.9.5