[U-Boot] [PATCH] Fix OneNAND read_oob/write_oob functions compatability
Scott Wood
scottwood at freescale.com
Tue Aug 19 00:08:44 CEST 2008
On Sat, Aug 16, 2008 at 02:32:16PM +0900, Kyungmin Park wrote:
> Also sync with kernel OneNAND codes
Either you synced with a rather old version of the kernel's
onenand_base.c, or you left out the Nokia/Adrian Hunter copyright and
attribution.
> diff --git a/common/cmd_onenand.c b/common/cmd_onenand.c
> index 5e2062b..8d87b78 100644
> --- a/common/cmd_onenand.c
> +++ b/common/cmd_onenand.c
> @@ -85,15 +85,25 @@ int do_onenand(cmd_tbl_t * cmdtp, int flag, int argc, char *argv[])
> ulong addr = simple_strtoul(argv[2], NULL, 16);
> ulong ofs = simple_strtoul(argv[3], NULL, 16);
> size_t len = simple_strtoul(argv[4], NULL, 16);
> - size_t retlen = 0;
> int oob = strncmp(argv[1], "read.oob", 8) ? 0 : 1;
> + struct mtd_oob_ops ops;
> +
> + ops.mode = MTD_OOB_PLACE;
I think this should be MTD_OOB_RAW.
> diff --git a/include/onenand_uboot.h b/include/onenand_uboot.h
> index 4260ee7..2886a6f 100644
> --- a/include/onenand_uboot.h
> +++ b/include/onenand_uboot.h
> @@ -15,24 +15,18 @@
> #define __UBOOT_ONENAND_H
>
> #include <linux/types.h>
> -
> -struct kvec {
> - void *iov_base;
> - size_t iov_len;
> -};
> -
> -typedef int spinlock_t;
> -typedef int wait_queue_head_t;
> +#include <mtd_uboot.h>
This is the second time you've posted this patch without adding
mtd_uboot.h...
-Scott
More information about the U-Boot
mailing list