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 at linux.intel.com>
Intel Finland Oy
