Re: [PATCH v2 2/2] phy: qualcomm: usb: Add Super-Speed PHY driver

From: Jorge Ramirez
Date: Wed Jan 30 2019 - 07:28:05 EST


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.