Thread (46 messages) 46 messages, 8 authors, 2016-03-16

[PATCH v4 03/24] clk: pwm: use pwm_get_xxx() helpers where appropriate

From: mturquette@baylibre.com (Michael Turquette)
Date: 2015-12-30 21:04:07
Also in: linux-clk, linux-fbdev, linux-input, linux-leds, linux-pwm, linux-rockchip, lkml

Hi Boris,

Quoting Boris Brezillon (2015-11-16 00:56:26)
quoted hunk ↗ jump to hunk
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 <mturquette@baylibre.com>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help