Thread (5 messages) 5 messages, 4 authors, 2016-12-07

[PATCH] clk: uniphier: Fix build with gcc-4.4.

From: Masahiro Yamada <hidden>
Date: 2016-12-03 01:24:51
Also in: linux-clk, lkml

Hi Vinson,

2016-12-03 9:37 GMT+09:00 Vinson Lee [off-list ref]:
gcc-4.4 has issues with anonymous unions in initializers.

  CC      drivers/clk/uniphier/clk-uniphier-sys.o
drivers/clk/uniphier/clk-uniphier-sys.c:45: error: unknown field ?factor? specified in initializer

Fixes: 1574d5722636 ("clk: uniphier: remove unneeded member name for union")
Signed-off-by: Vinson Lee <redacted>

This driver has COMPILE_TEST option, but kbuild test robot
did not mention about this.



This is a bad way of fixing, I think.
(what if a new member is inserted before the union in the future?)

Rather, please revert the bad commit.

quoted hunk ↗ jump to hunk
                .name = "sd" #ch "-sel",                                \
                .type = UNIPHIER_CLK_TYPE_MUX,                          \
                .idx = -1,                                              \
-               .mux = {                                                \
+               { .mux = {                                              \
                        .parent_names = {                               \
                                "sd-44m",                               \
                                "sd-33m",                               \
@@ -63,7 +63,7 @@
                                0x00001200,                             \
                                0x00001300,                             \
                        },                                              \
-               },                                                      \
+               } },                                                    \
        },                                                              \

No, please do not do this.






-- 
Best Regards
Masahiro Yamada
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help