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

Lan Chunhe b25806 at freescale.com
Tue Aug 16 09:24:10 CEST 2011


On Tue, 16 Aug 2011 02:12:17 +0800, Scott Wood <scottwood at freescale.com>  
wrote:

> 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.

    OK.
    Thanks.

   -Jack Lan



More information about the U-Boot mailing list