From 82e3d3379b7204bd81033c96b6279b537186ed85 Mon Sep 17 00:00:00 2001
From: Damien Lespiau <damien.lespiau@intel.com>
Date: Thu, 10 Jul 2014 14:03:37 +0100
Subject: [PATCH] drv_module_reload: Don't declare success when failing

We weren't returning straight away when failing to unload the driver, so
the test happilly executed gem_suspend and printed ""module successfully
loaded again".

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
---
 tests/drv_module_reload | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/tests/drv_module_reload b/tests/drv_module_reload
index c1fd3957..66cd6bbb 100755
--- a/tests/drv_module_reload
+++ b/tests/drv_module_reload
@@ -33,10 +33,9 @@ rmmod drm &> /dev/null
 
 if lsmod | grep i915 &> /dev/null ; then
 	echo WARNING: i915.ko still loaded!
-	exitcode=1
+	exit 1
 else
 	echo module successfully unloaded
-	exitcode=0
 fi
 
 modprobe i915
@@ -47,4 +46,4 @@ modprobe snd_hda_intel
 # try to run something
 $SOURCE_DIR/gem_exec_nop > /dev/null && echo "module successfully loaded again"
 
-exit $exitcode
+exit 0