Re: [U-Boot] [PATCH 1/2] fdt: Add a fdt_set_node_status function

2011-08-16 Thread Lan Chunhe
On Tue, 16 Aug 2011 02:12:17 +0800, Scott Wood wrote: > On 08/14/2011 10:40 PM, Lan Chunhe wrote: >> On Sat, 13 Aug 2011 00:24:33 +0800, Scott Wood >> wrote: >> >>> fdt_get_alias() is not needed -- do_fixup_by_path() calls >>> fdt_path_offset() which handles aliases. >>> >>> Instead of somethi

Re: [U-Boot] [PATCH 1/2] fdt: Add a fdt_set_node_status function

2011-08-15 Thread Scott Wood
On 08/14/2011 10:40 PM, Lan Chunhe wrote: > On Sat, 13 Aug 2011 00:24:33 +0800, Scott Wood > wrote: > >> fdt_get_alias() is not needed -- do_fixup_by_path() calls >> fdt_path_offset() which handles aliases. >> >> Instead of something specific to status, how about adding >> do_fixup_by_path_string

Re: [U-Boot] [PATCH 1/2] fdt: Add a fdt_set_node_status function

2011-08-14 Thread Lan Chunhe
On Sat, 13 Aug 2011 00:24:33 +0800, Scott Wood wrote: > On 08/11/2011 10:18 PM, Chunhe Lan wrote: >> Add a fdt_set_node_status function that will set the specified >> node's status to the value contained in "status". If the node >> doesn't have "status" property that will be created. >> >> Sign

Re: [U-Boot] [PATCH 1/2] fdt: Add a fdt_set_node_status function

2011-08-14 Thread Lan Chunhe
On Mon, 15 Aug 2011 08:08:04 +0800, Tabi Timur-B04825 wrote: > On Thu, Aug 11, 2011 at 11:18 PM, Chunhe Lan > wrote: > >> +/* >> + * Sets the specified node's status to the value contained in "status". >> + * If the first character of the specified path is "/" then we use >> + * alias as a p

Re: [U-Boot] [PATCH 1/2] fdt: Add a fdt_set_node_status function

2011-08-14 Thread Tabi Timur-B04825
On Thu, Aug 11, 2011 at 11:18 PM, Chunhe Lan wrote: > +/* > + * Sets the specified node's status to the value contained in "status". > + * If the first character of the specified path is "/" then we use > + * alias as a path. Otherwise, we look for an alias of that name. > + */ > +void fdt_set_no

Re: [U-Boot] [PATCH 1/2] fdt: Add a fdt_set_node_status function

2011-08-12 Thread Scott Wood
On 08/11/2011 10:18 PM, Chunhe Lan wrote: > Add a fdt_set_node_status function that will set the specified > node's status to the value contained in "status". If the node > doesn't have "status" property that will be created. > > Signed-off-by: Chunhe Lan > --- > common/fdt_support.c | 17 +++

[U-Boot] [PATCH 1/2] fdt: Add a fdt_set_node_status function

2011-08-11 Thread Chunhe Lan
Add a fdt_set_node_status function that will set the specified node's status to the value contained in "status". If the node doesn't have "status" property that will be created. Signed-off-by: Chunhe Lan --- common/fdt_support.c | 17 - include/fdt_support.h |3 +++ 2 file