U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH 1/2 v5] fdt: Add a do_fixup_by_path_string() function
@ 2011-08-30  3:41 Chunhe Lan
  2011-08-30  5:20 ` Wolfgang Denk
  0 siblings, 1 reply; 3+ messages in thread
From: Chunhe Lan @ 2011-08-30  3:41 UTC (permalink / raw)
  To: u-boot

The do_fixup_by_path_string() will set the specified node's property to the
value contained in "status". It would just be an inline wrapper for
do_fixup_by_path() that calls strlen on the argument.

Signed-off-by: Chunhe Lan <Chunhe.Lan@freescale.com>
---
 include/fdt_support.h |    7 +++++++
 1 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/include/fdt_support.h b/include/fdt_support.h
index 863024f..1de4a1d 100644
--- a/include/fdt_support.h
+++ b/include/fdt_support.h
@@ -36,6 +36,13 @@ void do_fixup_by_path(void *fdt, const char *path, const char *prop,
 		      const void *val, int len, int create);
 void do_fixup_by_path_u32(void *fdt, const char *path, const char *prop,
 			  u32 val, int create);
+
+static inline void do_fixup_by_path_string(void *fdt, const char *path,
+					   const char *prop, const char *status)
+{
+	do_fixup_by_path(fdt, path, prop, status, strlen(status) + 1, 1);
+}
+
 void do_fixup_by_prop(void *fdt,
 		      const char *pname, const void *pval, int plen,
 		      const char *prop, const void *val, int len,
-- 
1.5.6.5

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

* [U-Boot] [PATCH 1/2 v5] fdt: Add a do_fixup_by_path_string() function
  2011-08-30  3:41 [U-Boot] [PATCH 1/2 v5] fdt: Add a do_fixup_by_path_string() function Chunhe Lan
@ 2011-08-30  5:20 ` Wolfgang Denk
  2011-08-30  7:25   ` Chunhe Lan
  0 siblings, 1 reply; 3+ messages in thread
From: Wolfgang Denk @ 2011-08-30  5:20 UTC (permalink / raw)
  To: u-boot

Dear Chunhe Lan,

In message <1314675674-9825-1-git-send-email-Chunhe.Lan@freescale.com> you wrote:
> The do_fixup_by_path_string() will set the specified node's property to the
> value contained in "status". It would just be an inline wrapper for
> do_fixup_by_path() that calls strlen on the argument.
> 
> Signed-off-by: Chunhe Lan <Chunhe.Lan@freescale.com>
> ---
>  include/fdt_support.h |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)

This is v5, but there is no log of changes.

This change log is MANDATORY!!

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
Men don't talk peace unless they're ready to back it up with war.
	-- Col. Green, "The Savage Curtain", stardate 5906.4

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

* [U-Boot] [PATCH 1/2 v5] fdt: Add a do_fixup_by_path_string() function
  2011-08-30  5:20 ` Wolfgang Denk
@ 2011-08-30  7:25   ` Chunhe Lan
  0 siblings, 0 replies; 3+ messages in thread
From: Chunhe Lan @ 2011-08-30  7:25 UTC (permalink / raw)
  To: u-boot

On Tue, 30 Aug 2011 13:20:48 +0800, Wolfgang Denk <wd@denx.de> wrote:

> Dear Chunhe Lan,
>
> In message <1314675674-9825-1-git-send-email-Chunhe.Lan@freescale.com>  
> you wrote:
>> The do_fixup_by_path_string() will set the specified node's property to  
>> the
>> value contained in "status". It would just be an inline wrapper for
>> do_fixup_by_path() that calls strlen on the argument.
>>
>> Signed-off-by: Chunhe Lan <Chunhe.Lan@freescale.com>
>> ---
>>  include/fdt_support.h |    7 +++++++
>>  1 files changed, 7 insertions(+), 0 deletions(-)
>
> This is v5, but there is no log of changes.
>
> This change log is MANDATORY!!

    OK.
    Thanks.

    -Jack Lan

> Best regards,
>
> Wolfgang Denk

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

end of thread, other threads:[~2011-08-30  7:25 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-08-30  3:41 [U-Boot] [PATCH 1/2 v5] fdt: Add a do_fixup_by_path_string() function Chunhe Lan
2011-08-30  5:20 ` Wolfgang Denk
2011-08-30  7:25   ` Chunhe Lan

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