All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jerry Van Baren <gvb.uboot-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Simon Glass <sjg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
Cc: Tom Rini <trini-l0cyMroinI0@public.gmane.org>,
	U-Boot Mailing List
	<u-boot-0aAXYlwwYIKGBzrmiIFOJg@public.gmane.org>,
	Devicetree Discuss
	<devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org>,
	u-boot-review-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org
Subject: Re: [U-Boot] [PATCH v4 06/19] libfdt: Add fdt_next_subnode() to permit easy subnode iteration
Date: Tue, 07 May 2013 17:48:11 -0400	[thread overview]
Message-ID: <5189769B.5060708@gmail.com> (raw)
In-Reply-To: <1367943123-16013-7-git-send-email-sjg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>

Hi Simon,

On 05/07/2013 12:11 PM, Simon Glass wrote:
> Iterating through subnodes with libfdt is a little painful to write as we
> need something like this:

[snip]

> Signed-off-by: Simon Glass <sjg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
> (Cherry-picked from dtc commit 4e76ec79)

Acked-by: Gerald Van Baren <vanbaren-He//nVnquyzQT0dZR+AlfA@public.gmane.org>

Unless I hear otherwise, I'll drop this patch from my u-boot-fdt repo 
and let you (Tom?) apply it with the "sandbox" patch series since that 
is where it came from and where it logically belongs.

> ---
> Changes in v4:
> - Bring in upstream version of fdt_first/next_subnode()
>
> Changes in v3: None
> Changes in v2: None
>
>   include/libfdt.h | 22 ++++++++++++++++++++++
>   lib/libfdt/fdt.c | 28 ++++++++++++++++++++++++++++
>   2 files changed, 50 insertions(+)

Thanks,
gvb

WARNING: multiple messages have this Message-ID (diff)
From: Jerry Van Baren <gvb.uboot@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4 06/19] libfdt: Add fdt_next_subnode() to permit easy subnode iteration
Date: Tue, 07 May 2013 17:48:11 -0400	[thread overview]
Message-ID: <5189769B.5060708@gmail.com> (raw)
In-Reply-To: <1367943123-16013-7-git-send-email-sjg@chromium.org>

Hi Simon,

On 05/07/2013 12:11 PM, Simon Glass wrote:
> Iterating through subnodes with libfdt is a little painful to write as we
> need something like this:

[snip]

> Signed-off-by: Simon Glass <sjg@chromium.org>
> (Cherry-picked from dtc commit 4e76ec79)

Acked-by: Gerald Van Baren <vanbaren@cideas.com>

Unless I hear otherwise, I'll drop this patch from my u-boot-fdt repo 
and let you (Tom?) apply it with the "sandbox" patch series since that 
is where it came from and where it logically belongs.

> ---
> Changes in v4:
> - Bring in upstream version of fdt_first/next_subnode()
>
> Changes in v3: None
> Changes in v2: None
>
>   include/libfdt.h | 22 ++++++++++++++++++++++
>   lib/libfdt/fdt.c | 28 ++++++++++++++++++++++++++++
>   2 files changed, 50 insertions(+)

Thanks,
gvb

  parent reply	other threads:[~2013-05-07 21:48 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-07 16:11 [PATCH v4 0/19] Allow images to work on sandbox Simon Glass
2013-05-07 16:11 ` [U-Boot] " Simon Glass
2013-05-07 16:11 ` [U-Boot] [PATCH v4 01/19] env: Fix minor comment typos in cmd_nvedit Simon Glass
2013-05-07 16:11 ` [U-Boot] [PATCH v4 02/19] Add minor updates to README.fdt-control Simon Glass
2013-05-07 16:11 ` [U-Boot] [PATCH v4 03/19] hash: Add a way to calculate a hash for any algortihm Simon Glass
2013-05-07 16:11 ` [U-Boot] [PATCH v4 04/19] bootstage: Don't build for HOSTCC Simon Glass
2013-05-07 16:11 ` [U-Boot] [PATCH v4 05/19] mkimage: Move ARRAY_SIZE to header file Simon Glass
     [not found] ` <1367943123-16013-1-git-send-email-sjg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
2013-05-07 16:11   ` [PATCH v4 06/19] libfdt: Add fdt_next_subnode() to permit easy subnode iteration Simon Glass
2013-05-07 16:11     ` [U-Boot] " Simon Glass
     [not found]     ` <1367943123-16013-7-git-send-email-sjg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
2013-05-07 21:48       ` Jerry Van Baren [this message]
2013-05-07 21:48         ` Jerry Van Baren
2013-05-09 19:36   ` [U-Boot] [PATCH v4 0/19] Allow images to work on sandbox Tom Rini
2013-05-09 19:36     ` Tom Rini
2013-05-10  0:17     ` Simon Glass
2013-05-10  0:17       ` Simon Glass
2013-05-07 16:11 ` [U-Boot] [PATCH v4 07/19] image: Move timestamp #ifdefs to header file Simon Glass
2013-05-07 16:11 ` [U-Boot] [PATCH v4 08/19] image: Export fit_check_ramdisk() Simon Glass
2013-05-07 16:11 ` [U-Boot] [PATCH v4 09/19] image: Split FIT code into new image-fit.c Simon Glass
2013-05-07 16:11 ` [U-Boot] [PATCH v4 10/19] image: Move HOSTCC image code to tools/ Simon Glass
2013-05-07 16:11 ` [U-Boot] [PATCH v4 11/19] image: Split hash node processing into its own function Simon Glass
2013-05-07 16:11 ` [U-Boot] [PATCH v4 12/19] image: Convert fit_image_hash_set_value() to static, and rename Simon Glass
2013-05-07 16:11 ` [U-Boot] [PATCH v4 13/19] image: Rename fit_image_check_hashes() to fit_image_verify() Simon Glass
2013-05-07 16:11 ` [U-Boot] [PATCH v4 14/19] image: Move hash checking into its own function Simon Glass
2013-05-07 16:11 ` [U-Boot] [PATCH v4 15/19] image: Move error! string to common place Simon Glass
2013-05-07 16:12 ` [U-Boot] [PATCH v4 16/19] image: Export fit_conf_get_prop_node() Simon Glass
2013-05-07 16:12 ` [U-Boot] [PATCH v4 17/19] image: Rename fit_add_hashes() to fit_add_verification_data() Simon Glass
2013-05-07 16:12 ` [U-Boot] [PATCH v4 18/19] image: Rename hash printing to fit_image_print_verification_data() Simon Glass
2013-05-07 16:12 ` [U-Boot] [PATCH v4 19/19] sandbox: image: Add support for booting images in sandbox Simon Glass
2013-05-15 12:48 ` [PATCH v4 0/19] Allow images to work on sandbox Tom Rini
2013-05-15 12:48   ` [U-Boot] " Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5189769B.5060708@gmail.com \
    --to=gvb.uboot-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
    --cc=devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \
    --cc=sjg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org \
    --cc=trini-l0cyMroinI0@public.gmane.org \
    --cc=u-boot-0aAXYlwwYIKGBzrmiIFOJg@public.gmane.org \
    --cc=u-boot-review-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.