public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot-Users] [PATCH 4/17] Improve fdt move length handling.
@ 2007-07-05  1:15 Jerry Van Baren
  2007-07-05  2:39 ` Grant Likely
  0 siblings, 1 reply; 2+ messages in thread
From: Jerry Van Baren @ 2007-07-05  1:15 UTC (permalink / raw)
  To: u-boot

Make the length parameter optional: if not specified, do the move using
the current size unchanged.

Signed-off-by: Gerald Van Baren <vanbaren@cideas.com>
---
 common/cmd_fdt.c |   21 +++++++++++++++------
 1 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/common/cmd_fdt.c b/common/cmd_fdt.c
index 08fe351..a119985 100644
--- a/common/cmd_fdt.c
+++ b/common/cmd_fdt.c
@@ -115,7 +115,7 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char *argv[])
 		int  len;
 		int  err;
 
-		if (argc != 5) {
+		if (argc < 4) {
 			printf ("Usage:\n%s\n", cmdtp->usage);
 			return 1;
 		}
@@ -129,11 +129,20 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char *argv[])
 		}
 
 		newaddr = (struct fdt_header *)simple_strtoul(argv[3], NULL, 16);
-		len     =  simple_strtoul(argv[4], NULL, 16);
-		if (len < fdt_totalsize(fdt)) {
-			printf ("New length %d < existing length %d, aborting.\n",
-				len, fdt_totalsize(fdt));
-			return 1;
+
+		/*
+		 * If the user specifies a length, use that.  Otherwise use the
+		 * current length.
+		 */
+		if (argc <= 4) {
+			len = fdt_totalsize(fdt);
+		} else {
+			len = simple_strtoul(argv[4], NULL, 16);
+			if (len < fdt_totalsize(fdt)) {
+				printf ("New length 0x%X < existing length 0x%X, aborting.\n",
+					len, fdt_totalsize(fdt));
+				return 1;
+			}
 		}
 
 		/*
-- 
1.4.4.4

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [U-Boot-Users] [PATCH 4/17] Improve fdt move length handling.
  2007-07-05  1:15 [U-Boot-Users] [PATCH 4/17] Improve fdt move length handling Jerry Van Baren
@ 2007-07-05  2:39 ` Grant Likely
  0 siblings, 0 replies; 2+ messages in thread
From: Grant Likely @ 2007-07-05  2:39 UTC (permalink / raw)
  To: u-boot

On 7/4/07, Jerry Van Baren <gvb.uboot@gmail.com> wrote:
> Make the length parameter optional: if not specified, do the move using
> the current size unchanged.
>
> Signed-off-by: Gerald Van Baren <vanbaren@cideas.com>
Acked-by: Grant Likely <grant.likely@secretlab.ca>

> ---
>  common/cmd_fdt.c |   21 +++++++++++++++------
>  1 files changed, 15 insertions(+), 6 deletions(-)
>
> diff --git a/common/cmd_fdt.c b/common/cmd_fdt.c
> index 08fe351..a119985 100644
> --- a/common/cmd_fdt.c
> +++ b/common/cmd_fdt.c
> @@ -115,7 +115,7 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char *argv[])
>                 int  len;
>                 int  err;
>
> -               if (argc != 5) {
> +               if (argc < 4) {
>                         printf ("Usage:\n%s\n", cmdtp->usage);
>                         return 1;
>                 }
> @@ -129,11 +129,20 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char *argv[])
>                 }
>
>                 newaddr = (struct fdt_header *)simple_strtoul(argv[3], NULL, 16);
> -               len     =  simple_strtoul(argv[4], NULL, 16);
> -               if (len < fdt_totalsize(fdt)) {
> -                       printf ("New length %d < existing length %d, aborting.\n",
> -                               len, fdt_totalsize(fdt));
> -                       return 1;
> +
> +               /*
> +                * If the user specifies a length, use that.  Otherwise use the
> +                * current length.
> +                */
> +               if (argc <= 4) {
> +                       len = fdt_totalsize(fdt);
> +               } else {
> +                       len = simple_strtoul(argv[4], NULL, 16);
> +                       if (len < fdt_totalsize(fdt)) {
> +                               printf ("New length 0x%X < existing length 0x%X, aborting.\n",
> +                                       len, fdt_totalsize(fdt));
> +                               return 1;
> +                       }
>                 }
>
>                 /*
> --
> 1.4.4.4
>
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> U-Boot-Users mailing list
> U-Boot-Users at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/u-boot-users
>


-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
grant.likely at secretlab.ca
(403) 399-0195

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2007-07-05  2:39 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-07-05  1:15 [U-Boot-Users] [PATCH 4/17] Improve fdt move length handling Jerry Van Baren
2007-07-05  2:39 ` Grant Likely

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox