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

> On 08/14/2011 10:40 PM, Lan Chunhe wrote:
>> On Sat, 13 Aug 2011 00:24:33 +0800, Scott Wood <scottw...@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

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to