ramips: get rid of platform_pre_upgrade()
authorRafał Miłecki <rafal@milecki.pl>
Fri, 12 Jul 2019 12:05:49 +0000 (14:05 +0200)
committerRafał Miłecki <rafal@milecki.pl>
Tue, 16 Jul 2019 17:03:42 +0000 (19:03 +0200)
The only step between platform_pre_upgrade() and platform_do_upgrade()
is switching to ramdisk. It should be fine to "mtd erase firmware" from
the later callback and get rid of the first one.

This change wasn't tested on affected target but identical code logic
was verified to work as expected on brcm47xx with initramfs firmware.

Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
target/linux/ramips/base-files/lib/upgrade/platform.sh

index c71ffc0321930b6f7d876bdb38a3f02b07756586..4eaaffe029d133fbb54b4671d728cb3965352ab9 100755 (executable)
@@ -9,18 +9,6 @@ platform_check_image() {
        return 0
 }
 
-platform_pre_upgrade() {
-       local board=$(board_name)
-
-       case "$board" in
-       mikrotik,rb750gr3|\
-       mikrotik,rbm11g|\
-       mikrotik,rbm33g)
-               [ -z "$(rootfs_type)" ] && mtd erase firmware
-               ;;
-       esac
-}
-
 platform_nand_pre_upgrade() {
        local board=$(board_name)
 
@@ -35,6 +23,14 @@ platform_nand_pre_upgrade() {
 platform_do_upgrade() {
        local board=$(board_name)
 
+       case "$board" in
+       mikrotik,rb750gr3|\
+       mikrotik,rbm11g|\
+       mikrotik,rbm33g)
+               [ -z "$(rootfs_type)" ] && mtd erase firmware
+               ;;
+       esac
+
        case "$board" in
        hiwifi,hc5962|\
        netgear,r6220|\