On 1/30/19 10:53, Jorge Ramirez wrote: >> + priv->vbus = devm_regulator_get_optional(priv->dev, "vbus"); > get_optional means that if vbus-supply is not found, rather than > returning a dummy regulator object this will fail with -ENODEV. on this subject, is it intentional that devm_reset_control_get_optional behaves differently (ie, it returns a NULL if it cant be found) IMO all *_get_optional should be semantically as well as functionally equivalent to avoid confusions like the one I had.
