>> -    avr_clock = *(u32*)of_get_property(avr, "clock-frequency", &len);
>> -    phys_addr = ((u32*)of_get_property(avr, "reg", &len))[0];
>> +    phys_addr = of_get_property(avr, "reg", &len);
>> +    avr_clock_prop = of_get_property(avr, "clock-frequency", &len);
>
> If you don't use the value of the len variable, then you can pass NULL 
> to
> of_get_property.

OTOH, it is almost always an error to not use the length (missing
error check).


Segher

_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to