On Tue, 2015-12-08 at 22:51 +0100, Wolfram Sang wrote:
> On Tue, Dec 08, 2015 at 02:03:23PM +0100, Wolfram Sang wrote:
> >
> > > Too many && use_defaults. What about
> > >
> > > memset(t, 0, sizeof(*t));
> > >
> > > device_property_read_u32(dev, "i2c-scl-internal-delay-ns", &t-
> > > > scl_int_delay_ns);
> > >
> > > if (!use_defaults)
> > > return;
> >
> > I like this! Thanks for the input.
>
> Oops, too enthusiastic. This skips parsing all the other
> properties...
> The defaults should only be applied if reading the properties fails.
Looks like property stuff is not destructive in case of error, so, what
about
paramX = use_defaults ? defaultX : 0;
device_property_read…(…, ¶mX);
Or maybe just be a bit verbose like the original variant
int ret;
ret = device_property_read…(…);
if (ret && use_defaults) {
…
}
Among all variants I like the latter one here. What do you think?
--
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
