Reduce unnecessary flash wear and be tidy:
- Run the extraction only if necessary
- Extract temporary file to /tmp
- cleanup after execution

Tested-by: Roger Pueyo Centelles <[email protected]>
Signed-off-by: Thibaut VARÈNE <[email protected]>
---
 .../ath79/mikrotik/base-files/lib/functions/mikrotik-caldata.sh  | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git 
a/target/linux/ath79/mikrotik/base-files/lib/functions/mikrotik-caldata.sh 
b/target/linux/ath79/mikrotik/base-files/lib/functions/mikrotik-caldata.sh
index 9c4016ee5d..71a1bf02f3 100644
--- a/target/linux/ath79/mikrotik/base-files/lib/functions/mikrotik-caldata.sh
+++ b/target/linux/ath79/mikrotik/base-files/lib/functions/mikrotik-caldata.sh
@@ -9,13 +9,18 @@ mikrotik_caldata_extract() {
        local offset=$(($2))
        local count=$(($3))
        local mtd
-       local erdfile="/lib/firmware/erd.bin"
+       local erdfile="/tmp/erd.bin"
+       local fwfile="/lib/firmware/${FIRMWARE}"
+
+       [ -e $fwfile ] && exit 0
 
        mtd=$(find_mtd_chardev $part)
        [ -n "$mtd" ] || caldata_die "no mtd device found for partition $part"
 
        rbextract -e $mtd $erdfile
 
-       dd if=$erdfile of=/lib/firmware/$FIRMWARE iflag=skip_bytes bs=$count 
skip=$offset count=1 2>/dev/null || \
+       dd if=$erdfile of=$fwfile iflag=skip_bytes bs=$count skip=$offset 
count=1 2>/dev/null || \
                caldata_die "failed to extract calibration data from $mtd"
+
+       rm -f $erdfile
 }
-- 
2.11.0


_______________________________________________
openwrt-devel mailing list
[email protected]
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to