Re: [PATCH] usb: ohci-spear: Remove unnecessary NULL check before clk_disable_unprepare()

From: Alan Stern
Date: Tue Jun 17 2025 - 10:27:30 EST


On Tue, Jun 17, 2025 at 12:20:50PM +0800, Chen Ni wrote:
> clk_disable_unprepare() already checks NULL by using IS_ERR_OR_NULL.
> Remove unneeded NULL check for clk here.
>
> Signed-off-by: Chen Ni <nichen@xxxxxxxxxxx>
> ---

Acked-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>

> drivers/usb/host/ohci-spear.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/usb/host/ohci-spear.c b/drivers/usb/host/ohci-spear.c
> index d7131e5a4477..6843d7cb3f9f 100644
> --- a/drivers/usb/host/ohci-spear.c
> +++ b/drivers/usb/host/ohci-spear.c
> @@ -103,8 +103,7 @@ static void spear_ohci_hcd_drv_remove(struct platform_device *pdev)
> struct spear_ohci *sohci_p = to_spear_ohci(hcd);
>
> usb_remove_hcd(hcd);
> - if (sohci_p->clk)
> - clk_disable_unprepare(sohci_p->clk);
> + clk_disable_unprepare(sohci_p->clk);
>
> usb_put_hcd(hcd);
> }
> --
> 2.25.1
>