[U-Boot] [PATCH] mflash u-boot support (a little change)
Wolfgang Denk
wd at denx.de
Wed Feb 11 22:32:08 CET 2009
Dear unsik Kim,
In message <57afda040902092222l2bb9027bvbebf67795a7add3a at mail.gmail.com> you wrote:
>
> I fixed miscalculated total sectors when using reserve area feature.
>
> Also I changed CONFIG_MG_DISK_RES's unit to KB. (Byte is too small and
> also needs sector size alignment.)
Please split these two changes into separate commits.
> diff --git a/common/cmd_mgdisk.c b/common/cmd_mgdisk.c
> new file mode 100644
> index 0000000..f2f5061
> --- /dev/null
> +++ b/common/cmd_mgdisk.c
> @@ -0,0 +1,76 @@
> +/*
> + * (C) Copyright 2009 mGine co.
> + * unsik Kim <donari75 at gmail.com>
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License as
> + * published by the Free Software Foundation; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * 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 <common.h>
> +#include <command.h>
> +
> +#if defined (CONFIG_CMD_MG_DISK)
> +
> +#include <mg_disk.h>
> +
> +int do_mg_disk_cmd (cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
> +{
> + u32 from, to, size;
> +
> + switch (argc) {
> + case 2:
> + if (!strcmp(argv[1], "init"))
> + mg_disk_init();
> + else
> + return 1;
> + break;
> + case 4:
> + from = simple_strtoul(argv[2], NULL, 0);
> + to = simple_strtoul(argv[3], NULL, 0);
> + size = simple_strtoul(argv[4], NULL, 0);
> +
> + if (!strcmp(argv[1], "read"))
> + mg_disk_read(from, (u8 *)to, size);
> + else if (!strcmp(argv[1], "write"))
> + mg_disk_write(to, (u8 *)from, size);
> + else if (!strcmp(argv[1], "readsec"))
> + mg_disk_read_sects((void *)to, from, size);
> + else if (!strcmp(argv[1], "writesec"))
> + mg_disk_write_sects((void *)from, to, size);
> + else
> + return 1;
> + break;
> + default:
> + printf("Usage:\n%s\n", cmdtp->usage);
> + return 1;
> + }
> + return 0;
> +}
> +
> +U_BOOT_CMD(
> + mgd, 5, 0, do_mg_disk_cmd,
> + "mgd - mgine m[g]flash command\n",
> + ": mgine mflash IO mode (disk) command\n"
> + " - initialize : mgd init\n"
> + " - random read : mgd read [from] [to] [size]\n"
> + " - random write : mgd write [from] [to] [size]\n"
> + " - sector read : mgd readsec [sector] [to] [counts]\n"
> + " - sector write : mgd writesec [from] [sector] [counts]\n"
> +);
This doesn't look like the changes you subscribe in the commit message
above. Instead, this looks as if iyou were adding completely new
code.
> diff --git a/common/env_mgdisk.c b/common/env_mgdisk.c
> new file mode 100644
> index 0000000..2b4949f
> --- /dev/null
> +++ b/common/env_mgdisk.c
> @@ -0,0 +1,90 @@
Ditto.
Your commit message seems terribly wrong. Please fix.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
HEALTH WARNING: Care Should Be Taken When Lifting This Product, Since
Its Mass, and Thus Its Weight, Is Dependent on Its Velocity Relative
to the User.
More information about the U-Boot
mailing list