Messages in this thread | | | Subject | Re: [PATCH] phy: stm32: adopt dev_err_probe for regulators | From | Amelie DELAUNAY <> | Date | Tue, 26 Oct 2021 17:50:47 +0200 |
| |
Hi Fabrice,
Thanks for your patch.
On 10/25/21 4:31 PM, Fabrice Gasnier wrote: > Change stm32-usbphyc driver to use dev_err_probe(), to benefit of > devices_deferred debugfs in case of probe deferral. > > Signed-off-by: Fabrice Gasnier <fabrice.gasnier@foss.st.com>
Reviewed-by: Amelie Delaunay <amelie.delaunay@foss.st.com>
> --- > drivers/phy/st/phy-stm32-usbphyc.c | 10 ++++------ > 1 file changed, 4 insertions(+), 6 deletions(-) > > diff --git a/drivers/phy/st/phy-stm32-usbphyc.c b/drivers/phy/st/phy-stm32-usbphyc.c > index 7df6a63..115be0e 100644 > --- a/drivers/phy/st/phy-stm32-usbphyc.c > +++ b/drivers/phy/st/phy-stm32-usbphyc.c > @@ -672,17 +672,15 @@ static int stm32_usbphyc_probe(struct platform_device *pdev) > > usbphyc->vdda1v1 = devm_regulator_get(dev, "vdda1v1"); > if (IS_ERR(usbphyc->vdda1v1)) { > - ret = PTR_ERR(usbphyc->vdda1v1); > - if (ret != -EPROBE_DEFER) > - dev_err(dev, "failed to get vdda1v1 supply: %d\n", ret); > + ret = dev_err_probe(dev, PTR_ERR(usbphyc->vdda1v1), > + "failed to get vdda1v1 supply\n"); > goto clk_disable; > } > > usbphyc->vdda1v8 = devm_regulator_get(dev, "vdda1v8"); > if (IS_ERR(usbphyc->vdda1v8)) { > - ret = PTR_ERR(usbphyc->vdda1v8); > - if (ret != -EPROBE_DEFER) > - dev_err(dev, "failed to get vdda1v8 supply: %d\n", ret); > + ret = dev_err_probe(dev, PTR_ERR(usbphyc->vdda1v8), > + "failed to get vdda1v8 supply\n"); > goto clk_disable; > } > >
| |