clk: meson: fix SET_PARM macro
authorJerome Brunet <jbrunet@baylibre.com>
Thu, 9 Mar 2017 10:41:46 +0000 (11:41 +0100)
committerMichael Turquette <mturquette@baylibre.com>
Mon, 27 Mar 2017 19:29:51 +0000 (12:29 -0700)
parameter val is not enclosed in parenthesis which is buggy when given an
expression instead of a simple value

Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
Reviewed-by: Kevin Hilman <khilman@baylibre.com>
Signed-off-by: Michael Turquette <mturquette@baylibre.com>
Link: lkml.kernel.org/r/20170309104154.28295-2-jbrunet@baylibre.com

drivers/clk/meson/clkc.h

index 9bb70e7..c6be77d 100644 (file)
@@ -25,7 +25,7 @@
 #define PARM_GET(width, shift, reg)                                    \
        (((reg) & SETPMASK(width, shift)) >> (shift))
 #define PARM_SET(width, shift, reg, val)                               \
-       (((reg) & CLRPMASK(width, shift)) | (val << (shift)))
+       (((reg) & CLRPMASK(width, shift)) | ((val) << (shift)))
 
 #define MESON_PARM_APPLICABLE(p)               (!!((p)->width))