From b6e8da2e146824d14813eee749e39a56b615e639 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
Date: Tue, 7 Apr 2015 14:14:08 +0000
Subject: [PATCH] bcm53xx: image: share function checking TRX in provided
 firmware
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Rafał Miłecki <zajec5@gmail.com>

SVN-Revision: 45292
---
 .../base-files/lib/upgrade/platform.sh        | 24 ++++++++++++++-----
 1 file changed, 18 insertions(+), 6 deletions(-)

diff --git a/target/linux/bcm53xx/base-files/lib/upgrade/platform.sh b/target/linux/bcm53xx/base-files/lib/upgrade/platform.sh
index 8055b4fa33..e174cc7beb 100644
--- a/target/linux/bcm53xx/base-files/lib/upgrade/platform.sh
+++ b/target/linux/bcm53xx/base-files/lib/upgrade/platform.sh
@@ -54,6 +54,20 @@ platform_identify() {
 	echo "unknown"
 }
 
+# $(1): image that should contain trx
+# $(2): trx offset in image
+platform_check_image_trx() {
+	local magic=$(get_magic_long_at "$1" $2)
+
+	[ "$magic" != "48445230" ] && {
+		return 1
+	}
+
+	# TODO: Check crc32
+
+	return 0
+}
+
 platform_check_image() {
 	[ "$#" -gt 1 ] && return 1
 
@@ -79,11 +93,10 @@ platform_check_image() {
 				error=1
 			}
 
-			magic=$(get_magic_long_at "$1" "$header_len")
-			[ "$magic" != "48445230" ] && {
+			if ! platform_check_image_trx "$1" "$header_len"; then
 				echo "No valid TRX firmware in the CHK image"
 				error=1
-			}
+			fi
 		;;
 		"cybertan")
 			local pattern=$(dd if="$1" bs=1 count=4 2>/dev/null | hexdump -v -e '1/1 "%c"')
@@ -95,11 +108,10 @@ platform_check_image() {
 				error=1
 			}
 
-			magic=$(get_magic_long_at "$1" 32)
-			[ "$magic" != "48445230" ] && {
+			if ! platform_check_image_trx "$1" 32; then
 				echo "No valid TRX firmware in the CyberTAN image"
 				error=1
-			}
+			fi
 		;;
 		"trx")
 		;;