On 17.09.2011, at 18:48, Blue Swirl wrote:
> On Wed, Sep 14, 2011 at 8:42 AM, Alexander Graf <ag...@suse.de> wrote:
>> We have a qemu internal abstraction layer on FDT. While I'm not fully
>> convinced
>> we need it at all, it's missing the nop_node functionality that we now need
>> on e500. So let's add it and think about the general future of that API
>> later.
>>
>> Signed-off-by: Alexander Graf <ag...@suse.de>
>> ---
>> device_tree.c | 11 +++++++++++
>> device_tree.h | 1 +
>> 2 files changed, 12 insertions(+), 0 deletions(-)
>>
>> diff --git a/device_tree.c b/device_tree.c
>> index 3a224d1..23e89e3 100644
>> --- a/device_tree.c
>> +++ b/device_tree.c
>> @@ -107,3 +107,14 @@ int qemu_devtree_setprop_string(void *fdt, const char
>> *node_path,
>>
>> return fdt_setprop_string(fdt, offset, property, string);
>> }
>> +
>> +int qemu_devtree_nop_node(void *fdt, const char *node_path)
>> +{
>> + int offset;
>> +
>> + offset = fdt_path_offset(fdt, node_path);
>> + if (offset < 0)
>
> -EBRACES
That code gets changed right after that patch anyways. The version that
actually ends up being visible to the user has proper braces :)
Alex
