> 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? I agree and fixed it like this. -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-rockchip/attachments/20151214/7783c2db/attachment.sig>