]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
pinctrl: sh-pfc: Make pinmux_cfg_reg.var_field_width[] variable-length
authorGeert Uytterhoeven <geert+renesas@glider.be>
Wed, 12 Dec 2018 12:38:59 +0000 (13:38 +0100)
committerGeert Uytterhoeven <geert+renesas@glider.be>
Tue, 18 Dec 2018 10:25:56 +0000 (11:25 +0100)
pinmux_cfg_reg.var_field_width[] is actually a variable-length array,
terminated by a zero, and counting at most r_width entries.
Usually the number of entries is much smaller than r_width, so the
ability to catch bugs at compile time through an "excess elements in
array initializer" warning is fairly limited.

Hence make the array variable-length, decreasing kernel size slightly.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Simon Horman <horms+renesas@verge.net.au>
drivers/pinctrl/sh-pfc/sh_pfc.h

index 4892c09b27bb43ea0587734f8d66ba155ce97dfd..46d477ff510921f36036b92bf8778296fbf3b081 100644 (file)
@@ -142,8 +142,7 @@ struct pinmux_cfg_reg {
  */
 #define PINMUX_CFG_REG_VAR(name, r, r_width, var_fw0, var_fwn...) \
        .reg = r, .reg_width = r_width, \
-       .var_field_width = (const u8 [r_width]) \
-               { var_fw0, var_fwn, 0 }, \
+       .var_field_width = (const u8 []) { var_fw0, var_fwn, 0 }, \
        .enum_ids = (const u16 [])
 
 struct pinmux_drive_reg_field {