# Re: \[PATCH v4 03/24\] clk: pwm: use pwm\_get\_xxx() helpers where appropriate

- From: Michael Turquette \<hidden\>
- Date: 2015-12-30 21:04:07 
- List: linux-input
- Page: https://ratatoskr.run/linux-input/2015/12/4650787
- Thread: https://ratatoskr.run/linux-pwm/2015/11/4621011/t.md

```text
Hi Boris,

Quoting Boris Brezillon (2015-11-16 00:56:26)
> diff --git a/drivers/clk/clk-pwm.c b/drivers/clk/clk-pwm.c
> index 328fcfc..b6306a2 100644
> --- a/drivers/clk/clk-pwm.c
> +++ b/drivers/clk/clk-pwm.c
> @@ -71,22 +71,23 @@ static int clk_pwm_probe(struct platform_device *pdev)
>         if (IS_ERR(pwm))
>                 return PTR_ERR(pwm);
>  
> -       if (!pwm->period) {
> +       if (!pwm_get_period((pwm))) {

The change itself looks fine, but the semantic patch added extra parens.
Can you remove them? After doing so feel free to add:

Acked-by: Michael Turquette <redacted>
```