mvebu: sysupgrade: sdcard: fix platform_do_upgrade_sdcard
authorKlaus Kudielka <klaus.kudielka@gmail.com>
Sun, 28 Jul 2019 13:54:21 +0000 (15:54 +0200)
committerHauke Mehrtens <hauke@hauke-m.de>
Fri, 16 Aug 2019 20:53:06 +0000 (22:53 +0200)
Unconditionally execute the final case statement, even if the disk layout
changed. This is necessary, to keep the original Turris Omnia flash
instructions working: The disk layout WILL change, when switching from
TurrisOS to OpenWRT. Without updating the uboot environment at the same
time, the user would end up with an unbootable system.

Fixes commit 2e5a0b81ec ("mvebu: sysupgrade: sdcard: keep user added ...")

Signed-off-by: Klaus Kudielka <klaus.kudielka@gmail.com>
target/linux/mvebu/base-files/lib/upgrade/sdcard.sh

index 43fc2504fc795968bd5d2786bdd733b71575dd38..5ca5e71fe7ff9cb923df87339f60ec29362adee9 100644 (file)
@@ -70,26 +70,24 @@ platform_do_upgrade_sdcard() {
                # will be missing if it overlaps with the old partition 2
                partx -d - "/dev/$diskdev"
                partx -a - "/dev/$diskdev"
-
-               return 0
+       else
+               #write uboot image
+               get_image "$@" | dd of="$diskdev" bs=512 skip=1 seek=1 count=2048 conv=fsync
+               #iterate over each partition from the image and write it to the boot disk
+               while read part start size; do
+                       if export_partdevice partdev $part; then
+                               echo "Writing image to /dev/$partdev..."
+                               get_image "$@" | dd of="/dev/$partdev" ibs="512" obs=1M skip="$start" count="$size" conv=fsync
+                       else
+                               echo "Unable to find partition $part device, skipped."
+                       fi
+               done < /tmp/partmap.image
+
+               #copy partition uuid
+               echo "Writing new UUID to /dev/$diskdev..."
+               get_image "$@" | dd of="/dev/$diskdev" bs=1 skip=440 count=4 seek=440 conv=fsync
        fi
 
-       #write uboot image
-       get_image "$@" | dd of="$diskdev" bs=512 skip=1 seek=1 count=2048 conv=fsync
-       #iterate over each partition from the image and write it to the boot disk
-       while read part start size; do
-               if export_partdevice partdev $part; then
-                       echo "Writing image to /dev/$partdev..."
-                       get_image "$@" | dd of="/dev/$partdev" ibs="512" obs=1M skip="$start" count="$size" conv=fsync
-               else
-                       echo "Unable to find partition $part device, skipped."
-               fi
-       done < /tmp/partmap.image
-
-       #copy partition uuid
-       echo "Writing new UUID to /dev/$diskdev..."
-       get_image "$@" | dd of="/dev/$diskdev" bs=1 skip=440 count=4 seek=440 conv=fsync
-
        case "$board" in
        cznic,turris-omnia)
                fw_setenv openwrt_bootargs 'earlyprintk console=ttyS0,115200 root=/dev/mmcblk0p2 rootfstype=auto rootwait'