On Sun, Dec 15, 2024 at 04:00:06PM -0500, Mingwei Zheng wrote: > Add check for the return value of clk_enable() to catch the potential > error. > > Fixes: 9ba96ae5074c ("usb: omap1: Tahvo USB transceiver driver") > Signed-off-by: Mingwei Zheng <zmw12306@xxxxxxxxx> > Signed-off-by: Jiasheng Jiang <jiashengjiangcool@xxxxxxxxx> > --- > drivers/usb/phy/phy-tahvo.c | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/drivers/usb/phy/phy-tahvo.c b/drivers/usb/phy/phy-tahvo.c > index ae7bf3ff89ee..0b8801318cd2 100644 > --- a/drivers/usb/phy/phy-tahvo.c > +++ b/drivers/usb/phy/phy-tahvo.c > @@ -342,8 +342,11 @@ static int tahvo_usb_probe(struct platform_device *pdev) > mutex_init(&tu->serialize); > > tu->ick = devm_clk_get(&pdev->dev, "usb_l4_ick"); > - if (!IS_ERR(tu->ick)) > - clk_enable(tu->ick); > + if (!IS_ERR(tu->ick)) { > + ret = clk_enable(tu->ick); > + if (ret) > + return ret; > + } Same questions, why the signed-off-by order and how was this found and tested? thanks, greg k-h