Hi Jon,
On Tue, Feb 26, 2013 at 2:30 PM, Jon Hunter <[email protected]> wrote:
[...]
>
> + if (!of_property_read_u32(np, "gpmc,bus-turnaround", &val))
> + gpmc_t->bus_turnaround = val;
> +
AFAIK, you don't need to test if the property is defined.
You can just read it straightforward, and if the property is not defined
then nothing will be stored in your variable:
of_property_read_u32(np, "gpmc,bus-turnaround", &gpmc_t->bus_turnaround);
You can simplify *a lot* your patch with this, and I believe you could
also clean
existent code.
--
Ezequiel
_______________________________________________
devicetree-discuss mailing list
[email protected]
https://lists.ozlabs.org/listinfo/devicetree-discuss
