[libcamera-devel,1/3] utils: ipu3: Abort when sensor or media device isn't found

Message ID 20181211140028.25241-1-laurent.pinchart@ideasonboard.com
State Accepted
Commit 0d96f3e2322c5eecdd99bf134a2d80429836b778
Headers show
Series
  • [libcamera-devel,1/3] utils: ipu3: Abort when sensor or media device isn't found
Related show

Commit Message

Laurent Pinchart Dec. 11, 2018, 2 p.m. UTC
Calling exit from a function only exits from the function, it doesn't
abort the whole script. Propagate the errors to stop operation when the
sensor or media device can't be found.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
 utils/ipu3/ipu3-capture.sh | 16 ++++++----------
 utils/ipu3/ipu3-process.sh |  6 +++---
 2 files changed, 9 insertions(+), 13 deletions(-)

Patch

diff --git a/utils/ipu3/ipu3-capture.sh b/utils/ipu3/ipu3-capture.sh
index af189da3f8d8..ba6147b49799 100755
--- a/utils/ipu3/ipu3-capture.sh
+++ b/utils/ipu3/ipu3-capture.sh
@@ -20,7 +20,8 @@  find_sensor() {
 
 	bus=$(grep "$sensor_name" /sys/class/video4linux/v4l-subdev*/name | cut -d ' ' -f 2)
 	if [[ -z $bus ]]; then
-		return
+		echo "Sensor '$sensor_name' not found." >&2
+		exit 1
 	fi
 
 	echo "$sensor_name $bus"
@@ -36,8 +37,8 @@  find_media_device() {
 	done
 
 	if [[ -z $mdev ]] ; then
-	       echo "IPU3 media device not found." >&2
-	       exit 1
+		echo "IPU3 media device not found." >&2
+		exit 1
 	fi
 
 	echo $mdev
@@ -170,13 +171,8 @@  fi
 
 sensor_name=$1
 
-sensor=$(find_sensor $sensor_name)
-if [[ -z $sensor ]] ; then
-	echo "Sensor '$sensor_name' not found." >&2
-	exit 1
-fi
-
-mdev=$(find_media_device)
+sensor=$(find_sensor $sensor_name) || exit
+mdev=$(find_media_device) || exit
 mediactl="media-ctl -d $mdev"
 
 parse_pipeline $sensor
diff --git a/utils/ipu3/ipu3-process.sh b/utils/ipu3/ipu3-process.sh
index 6412aea4a3e7..78a7937fca70 100755
--- a/utils/ipu3/ipu3-process.sh
+++ b/utils/ipu3/ipu3-process.sh
@@ -23,8 +23,8 @@  find_media_device() {
 	done
 
 	if [[ -z $mdev ]] ; then
-	       echo "IPU3 media device not found." >&2
-	       exit 1
+		echo "IPU3 media device not found." >&2
+		exit 1
 	fi
 
 	echo $mdev
@@ -173,7 +173,7 @@  fi
 out_size=${out_size:-$in_size}
 vf_size=${vf_size:-$in_size}
 
-mdev=$(find_media_device)
+mdev=$(find_media_device) || exit
 mediactl="media-ctl -d $mdev"
 echo "Using device $mdev"