diff options
author | Jeff Kletsky <git-commits@allycomm.com> | 2019-04-10 08:28:01 -0700 |
---|---|---|
committer | Christian Lamparter <chunkeey@gmail.com> | 2019-05-18 13:43:51 +0200 |
commit | b3770eaca39f223faecfcc0d169a1bb2f6f9464a (patch) | |
tree | 4697e1277e06775e1fdf0e36aadbe1f636b22806 /package/system/mtd/src/linksys_bootcount_fix.c | |
parent | 4bdc873a5fda60557478a4294983cf71e7eccea6 (diff) | |
download | upstream-b3770eaca39f223faecfcc0d169a1bb2f6f9464a.tar.gz upstream-b3770eaca39f223faecfcc0d169a1bb2f6f9464a.tar.bz2 upstream-b3770eaca39f223faecfcc0d169a1bb2f6f9464a.zip |
mtd: base-files: Unify dual-firmware devices (Linksys)
Consistently handle boot-count reset and upgrade across
ipq40xx, ipq806x, kirkwood, mvebu
Dual-firmware devices often utilize a specific MTD partition
to record the number of times the boot loader has initiated boot.
Most of these devices are NAND, typically with a 2k erase size.
When this code was ported to the ipq40xx platform, the device in hand
used NOR for this partition, with a 16-byte "record" size. As the
implementation of `mtd resetbc` is by-platform, the hard-coded nature
of this change prevented proper operation of a NAND-based device.
* Unified the "NOR" variant with the rest of the Linksys variants
* Added logging to indicate success and failure
* Provided a meaningful return value for scripting
* "Protected" the use of `mtd resetbc` in start-up scripts so that
failure does not end the boot sequence
* Moved Linksys-specific actions into common `/etc/init.d/bootcount`
For upgrade, these devices need to determine which partition to flash,
as well as set certain U-Boot envirnment variables to change the next
boot to the newly flashed version.
* Moved upgrade-related environment changes out of bootcount
* Combined multiple flashes of environment into single one
* Current-partition detection now handles absence of `boot_part`
Runtime-tested: Linksys EA8300
Signed-off-by: Jeff Kletsky <git-commits@allycomm.com>
Signed-off-by: Christian Lamparter <chunkeey@gmail.com>
[checkpatch.pl fixes, traded split strings for 80+ chars per line]
Diffstat (limited to 'package/system/mtd/src/linksys_bootcount_fix.c')
-rw-r--r-- | package/system/mtd/src/linksys_bootcount_fix.c | 115 |
1 files changed, 0 insertions, 115 deletions
diff --git a/package/system/mtd/src/linksys_bootcount_fix.c b/package/system/mtd/src/linksys_bootcount_fix.c deleted file mode 100644 index 3fc38012fb..0000000000 --- a/package/system/mtd/src/linksys_bootcount_fix.c +++ /dev/null @@ -1,115 +0,0 @@ -/* - * Linksys boot counter reset code for mtd - * - * Copyright (C) 2013 Jonas Gorski <jogo@openwrt.org> - * - * This program is free software; you can redistribute it and/or - * modify it under the terms of the GNU General Public License v2 - * as published by the Free Software Foundation. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software - * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. - * - */ - -#include <stdio.h> -#include <stdlib.h> -#include <stddef.h> -#include <unistd.h> -#include <fcntl.h> -#include <sys/mman.h> -#include <sys/stat.h> -#include <endian.h> -#include <string.h> -#include <errno.h> -#include <stdint.h> - -#include <sys/ioctl.h> -#include <mtd/mtd-user.h> - -#include "mtd.h" - -#define BOOTCOUNT_MAGIC 0x20110811 - -struct bootcounter { - uint32_t magic; - uint32_t count; - uint32_t checksum; -}; - -static char page[2048]; - -int mtd_resetbc(const char *mtd) -{ - struct mtd_info_user mtd_info; - struct bootcounter *curr = (struct bootcounter *)page; - unsigned int i; - int last_count = 0; - int num_bc; - int fd; - int ret; - - fd = mtd_check_open(mtd); - - if (ioctl(fd, MEMGETINFO, &mtd_info) < 0) { - fprintf(stderr, "failed to get mtd info!\n"); - return -1; - } - - num_bc = mtd_info.size / 16; - - for (i = 0; i < num_bc; i++) { - pread(fd, curr, sizeof(*curr), i * 16); - - if (curr->magic != (BOOTCOUNT_MAGIC) && curr->magic != 0xffffffff) { - fprintf(stderr, "unexpected magic %08x, bailing out\n", curr->magic); - goto out; - } - - if (curr->magic == 0xffffffff) - break; - - last_count = curr->count; - } - - /* no need to do writes when last boot count is already 0 */ - if (last_count == 0) - goto out; - - - if (i == num_bc) { - struct erase_info_user erase_info; - erase_info.start = 0; - erase_info.length = mtd_info.size; - - /* erase block */ - ret = ioctl(fd, MEMERASE, &erase_info); - if (ret < 0) { - fprintf(stderr, "failed to erase block: %i\n", ret); - return -1; - } - - i = 0; - } - - memset(curr, 0xff, 16); - - curr->magic = BOOTCOUNT_MAGIC; - curr->count = 0; - curr->checksum = BOOTCOUNT_MAGIC; - - ret = pwrite(fd, curr, 16, i * 16); - if (ret < 0) - fprintf(stderr, "failed to write: %i\n", ret); - sync(); -out: - close(fd); - - return 0; -} |