| Date | Mon, 13 Oct 2014 12:35:12 +0200 | From | Mark Brown <> | Subject | Re: [RESUBMIT PATCH v4 7/8] regulator: sky81452: Add compatible string for device binding |
| |
On Mon, Oct 13, 2014 at 02:47:41PM +0900, Gyungoh Yoo wrote:
> - np = of_get_child_by_name(dev->parent->of_node, "regulator"); > - if (unlikely(!np)) { > + if (!np) { > dev_err(dev, "regulator node not found"); > - return NULL; > + return ERR_PTR(-ENODATA); > }
If you're doing anything here you should convert the code to use the simplified interface for parsing regulator data based on putting the subnode name and regualtor name in the regulator descriptor. [unhandled content-type:application/pgp-signature] |