diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2009-11-27 15:26:17 +0300 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2009-11-27 15:26:17 +0300 |
commit | 86cfb70ca5f2bde11f2d071bc59db75291d8552f (patch) | |
tree | 8f26107f9c1394a0e87609eb5c29e58cecd166b6 /miscutils | |
parent | 160b9ca7318fb63e9432c1028f52373aea6166f9 (diff) |
flash_eraseall: stop using obsolete mtd/jffs2-user.h; code shrink
function old new delta
show_progress 68 67 -1
flash_eraseall_main 1007 882 -125
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 0/2 up/down: 0/-126) Total: -126 bytes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'miscutils')
-rw-r--r-- | miscutils/flash_eraseall.c | 47 |
1 files changed, 30 insertions, 17 deletions
diff --git a/miscutils/flash_eraseall.c b/miscutils/flash_eraseall.c index ba0a6b515..ca00a130e 100644 --- a/miscutils/flash_eraseall.c +++ b/miscutils/flash_eraseall.c @@ -12,22 +12,35 @@ #include "libbb.h" #include <mtd/mtd-user.h> -#include <mtd/jffs2-user.h> +#include <linux/jffs2.h> #define OPTION_J (1 << 0) #define OPTION_Q (1 << 1) #define IS_NAND (1 << 2) #define BBTEST (1 << 3) -struct globals { - /* This is used in the cpu_to_je/je_to_cpu macros in jffs2_user.h */ - int tgt_endian; -}; -#define G (*(struct globals*)&bb_common_bufsiz1) -#define target_endian (G.tgt_endian) -#define INIT_G() do { \ - target_endian = __BYTE_ORDER; \ -} while (0) +/* mtd/jffs2-user.h used to have this atrocity: +extern int target_endian; + +#define t16(x) ({ __u16 __b = (x); (target_endian==__BYTE_ORDER)?__b:bswap_16(__b); }) +#define t32(x) ({ __u32 __b = (x); (target_endian==__BYTE_ORDER)?__b:bswap_32(__b); }) + +#define cpu_to_je16(x) ((jint16_t){t16(x)}) +#define cpu_to_je32(x) ((jint32_t){t32(x)}) +#define cpu_to_jemode(x) ((jmode_t){t32(x)}) + +#define je16_to_cpu(x) (t16((x).v16)) +#define je32_to_cpu(x) (t32((x).v32)) +#define jemode_to_cpu(x) (t32((x).m)) + +but mtd/jffs2-user.h is gone now (at least 2.6.31.6 does not have it anymore) +*/ + +/* We always use native endianness */ +#undef cpu_to_je16 +#undef cpu_to_je32 +#define cpu_to_je16(v) ((jint16_t){(v)}) +#define cpu_to_je32(v) ((jint32_t){(v)}) static uint32_t crc32(uint32_t val, const void *ss, int len, uint32_t *crc32_table) @@ -40,9 +53,11 @@ static uint32_t crc32(uint32_t val, const void *ss, int len, static void show_progress(mtd_info_t *meminfo, erase_info_t *erase) { - printf("\rErasing %d Kibyte @ %x -- %2llu %% complete.", - (unsigned)meminfo->erasesize / 1024, erase->start, - (unsigned long long) erase->start * 100 / meminfo->size); + printf("\rErasing %u Kibyte @ %x - %2u%% complete.", + (unsigned)meminfo->erasesize / 1024, + erase->start, + (unsigned) ((unsigned long long) erase->start * 100 / meminfo->size) + ); fflush_all(); } @@ -57,17 +72,15 @@ int flash_eraseall_main(int argc UNUSED_PARAM, char **argv) unsigned int flags; char *mtd_name; - INIT_G(); opt_complementary = "=1"; flags = BBTEST | getopt32(argv, "jq"); mtd_name = argv[optind]; - xstat(mtd_name, &st); + fd = xopen(mtd_name, O_RDWR); + fstat(fd, &st); if (!S_ISCHR(st.st_mode)) bb_error_msg_and_die("%s: not a char device", mtd_name); - fd = xopen(mtd_name, O_RDWR); - xioctl(fd, MEMGETINFO, &meminfo); erase.length = meminfo.erasesize; if (meminfo.type == MTD_NANDFLASH) |