From 1f8ad1e46ae561126f8fb196a3c677117f786f43 Mon Sep 17 00:00:00 2001
From: Leah Rowe <leah@libreboot.org>
Date: Sun, 14 May 2023 09:24:31 +0100
Subject: blobutil/extract: simplified main()

---
 resources/scripts/blobs/extract | 42 ++++++++++++++++++-----------------------
 1 file changed, 18 insertions(+), 24 deletions(-)

(limited to 'resources/scripts/blobs')

diff --git a/resources/scripts/blobs/extract b/resources/scripts/blobs/extract
index eb8fa7f9..0f1e1f3a 100755
--- a/resources/scripts/blobs/extract
+++ b/resources/scripts/blobs/extract
@@ -19,28 +19,22 @@ main()
 	board="${1}"
 	vendor_rom="${2}"
 
-	if [ ! -f "${vendor_rom}" ] ; then
-		Print_help
-		exit 1
-	fi
-
-	if [ ! -d "resources/coreboot/${board}" ]; then
-		Print_help
-		printf "build/roms %s: target not defined.\n" \
-				"${projectname}" ${board}
-		exit 1
-	fi
-
-	if [ ! -f "resources/coreboot/${board}/board.cfg" ]; then
-		Print_help
-		printf "build/roms %s: missing board.cfg.\n" ${board}
-		exit 1
-	fi
-
+	Check_board
 	Build_deps
 	Extract_blobs
 }
 
+Check_board()
+{
+	if [ ! -f "${vendor_rom}" ] ; then
+		Fail "file does not exist: ${vendor_rom}"
+	elif [ ! -d "resources/coreboot/${board}" ]; then
+		Fail "build/roms ${board}: target not defined"
+	elif [ ! -f "resources/coreboot/${board}/board.cfg" ]; then
+		Fail "build/roms ${board}: missing board.cfg"
+	fi
+}
+
 Build_deps(){
 	if [ ! -d me_cleaner ]; then
 		printf "downloading me_cleaner\n"
@@ -106,16 +100,16 @@ Extract_blobs(){
 	fi
 }
 
-Print_help(){
-	printf "Usage: ./blobutil extract {boardname} {path/to/vendor_rom}\n"
-	printf "Example: ./blobutil extract x230 12mb_flash.bin\n"
-	printf "\nYou need to specify exactly 2 arguments\n"
-}
-
 Fail(){
 	Print_help
 	printf "\n%s: ERROR: %s\n" ${sname} $@
 	exit 1 
 }
 
+Print_help(){
+	printf "Usage: ./blobutil extract {boardname} {path/to/vendor_rom}\n"
+	printf "Example: ./blobutil extract x230 12mb_flash.bin\n"
+	printf "\nYou need to specify exactly 2 arguments\n"
+}
+
 main $@
-- 
cgit v1.2.1