[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