[OpenWrt-Devel] [PATCH 1/2] oxnas: drop some unused sysupgrade code
Daniel Golle
daniel at makrotopia.org
Mon Sep 21 05:37:37 EDT 2015
Hi!
First of all sorry for the delay. Both patches look correct to me,
thanks for the good work! I new tested them on Shuttle KD-20 and
sysupgrade works great with both patches applied.
On Tue, Sep 15, 2015 at 06:28:16PM +0200, Rafał Miłecki wrote:
> First of all platform_do_upgrade_phase2 doesn't seem to be defined
> anywhere. Other than that platform_do_upgrade is not likely to be ever
> called. There is a call to nand_do_platform_check which triggers
> nand_upgrade_stage1 call with "exit 0" in it.
>
> Signed-off-by: Rafał Miłecki <zajec5 at gmail.com>
Acked-by: Daniel Golle <daniel at makrotopia.org>
Cheers
Daniel
> ---
> target/linux/oxnas/base-files/lib/upgrade/platform.sh | 9 ---------
> 1 file changed, 9 deletions(-)
>
> diff --git a/target/linux/oxnas/base-files/lib/upgrade/platform.sh b/target/linux/oxnas/base-files/lib/upgrade/platform.sh
> index 8591c7d..a43391c 100644
> --- a/target/linux/oxnas/base-files/lib/upgrade/platform.sh
> +++ b/target/linux/oxnas/base-files/lib/upgrade/platform.sh
> @@ -15,15 +15,6 @@ platform_check_image() {
> return $?
> }
>
> -platform_do_upgrade() {
> - if [ $$ -ne 1 ]; then
> - echo "not PID 1, upgrade aborted."
> - return 1;
> - fi
> - platform_do_upgrade_phase2 "$1" "$CONF_TAR" "$SAVE_CONFIG"
> -}
> -
> -
> disable_watchdog() {
> killall watchdog
> ( ps | grep -v 'grep' | grep '/dev/watchdog' ) && {
> --
> 1.8.4.5
>
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
More information about the openwrt-devel
mailing list