On Mon, 2015-04-20 at 07:47 +0200, Heiko Schocher wrote:
> @@ -322,7 +213,12 @@ int do_nand_env_oob(cmd_tbl_t *cmdtp, int argc, char
> *const argv[])
> goto usage;
>
> /* We don't care about size, or maxsize. */
> - if (arg_off(argv[2], &idx, &addr, &maxsize, &maxsize)) {
> + if (arg_off(argv[2], &idx, &addr, &maxsize, &maxsize,
> + MTD_DEV_TYPE_NAND, nand_info[idx].size)) {
> + puts("Offset or partition name expected\n");
> + return 1;
> + }
Use only one tab per indentation level.
> diff --git a/include/linux/mtd/mtd.h b/include/linux/mtd/mtd.h
> index 8666413..2861af5 100644
> --- a/include/linux/mtd/mtd.h
> +++ b/include/linux/mtd/mtd.h
> @@ -482,5 +482,12 @@ int add_mtd_device(struct mtd_info *mtd);
> int del_mtd_device(struct mtd_info *mtd);
> int add_mtd_partitions(struct mtd_info *, const struct mtd_partition *, int);
> int del_mtd_partitions(struct mtd_info *);
> +
> +int str2off(const char *p, loff_t *num);
> +int str2long(const char *p, ulong *num);
These should be moved somewhere more generic, especially if they're no
longer file-local.
> +int arg_off(const char *arg, int *idx, loff_t *off, loff_t *size,
> + loff_t *maxsize, int devtype, int chipsize);
> +int arg_off_size(int argc, char *const argv[], int *idx, loff_t *off,
> + loff_t *size, loff_t *maxsize, int devtype, int chipsize);
Add an mtd prefix.
-Scott
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
