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

Scott Wood scottwood at freescale.com
Mon Aug 15 20:12:17 CEST 2011


On 08/14/2011 10:40 PM, Lan Chunhe wrote:
> On Sat, 13 Aug 2011 00:24:33 +0800, Scott Wood <scottwood at freescale.com>
> 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()?
> 
>    I think that function of do_fixup_by_path should have
>    completed that function.

do_fixup_by_path_string() would just be an inline wrapper for
do_fixup_by_path() that calls strlen on the argument.

And then you could just do:

do_fixup_by_path_string(fdt, "nand_flash", "status", "okay");

I don't see a need for a function dedicated to the status property.

-Scott



More information about the U-Boot mailing list