# Re: \[PATCH 1/2\] Input: at32psif: handle clk\_enable return value

- From: Fabio Estevam \<festevam@gmail.com\>
- Date: 2015-09-11 03:10:34 
- List: linux-input
- Page: https://ratatoskr.run/linux-input/2015/09/8926798
- Thread: https://ratatoskr.run/linux-input/2015/09/8926797/t.md

```text
On Fri, Sep 11, 2015 at 12:02 AM, WEN Pingbo [ref] wrote:

> -       clk_enable(psif->pclk);
> +       retval = clk_enable(psif->pclk);
> +       if (retval < 0)
> +               dev_err(&psif->pdev->dev,
> +                       "could not enable pclk, ret %d\n", retval);

Shouldn't you add a 'return' here to prevent the code to proceed with
the failed clock?
```