[U-Boot] [PATCH 05/15] fdt: fdt addr w/o any args reports back the current working address

Wolfgang Denk
2008-08-18T23:07:13Z

Hi Jerry,

In message <1218806685-3615-5-git-send-email-galak@kernel.crashing.org> Kumar Gala wrote:
> Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
> ---
> common/cmd_fdt.c | 8 ++++++++
> 1 files changed, 8 insertions(+), 0 deletions(-)
>
> diff --git a/common/cmd_fdt.c b/common/cmd_fdt.c
> index 436fec9..c129993 100644
> --- a/common/cmd_fdt.c
> +++ b/common/cmd_fdt.c
> @@ -67,6 +67,14 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char *argv[])
> /*
> * Set the address [and length] of the fdt.
> */
> + if (argc == 2) {
> + if (!fdt_valid()) {
> + return 1;
> + }
> + printf("The address of the fdt is %p\n", working_fdt);
> + return 0;
> + }
> +
> working_fdt = (struct fdt_header *)simple_strtoul(argv[2], NULL, 16);
>
> if (!fdt_valid()) {

This looks ok to me. Will you add this, or should I pull it directly?

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
A Puritan is someone who is deathly afraid that someone, somewhere,
is having fun.