| Message ID | 2791943.MBZ53mI7Gp@wuerfel (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Arnd, Am Freitag, 1. Januar 2016, 14:42:52 schrieb Arnd Bergmann: > The latest changes to the rockchip clk implementation cause > tons of warnings and/or errors (depending on the configuration): > > WARNING: vmlinux.o(.data+0x125394): Section mismatch in reference from the > variable __compound_literal.125 to the (unknown reference) > .init.rodata:(unknown) > > The reason is a compount literal in a macro that refers to an > object in the .init.rodata section: > > #define COMPOSITE_FRACMUX(_id, cname, pname, f, mo, df, go, gs, gf, ch) \ > { \ > ... > .child = &(struct rockchip_clk_branch)ch, \ > } > > 'ch' here is placed in .data, and I could find no way to override that > using an attribute or pragma because there is no identifier associated > with it. "[PATCH] clk: rockchip: fix section mismatches with new child-clocks" [0] should be in Mike's + Stephen's inbox since last week as well, which moves the offending new elements into separate entities, which can have __initdata attributes again. Heiko [0] http://www.spinics.net/lists/arm-kernel/msg471295.html
On Friday 01 January 2016 18:06:30 Heiko Stübner wrote: > > "[PATCH] clk: rockchip: fix section mismatches with new child-clocks" [0] > > should be in Mike's + Stephen's inbox since last week as well, which moves the > offending new elements into separate entities, which can have __initdata > attributes again. > > > Heiko > > [0] http://www.spinics.net/lists/arm-kernel/msg471295.html The patch looks good, but for some reason, the next-20151223 kernel had no problem and next-20151231 was broken, the top commits in drivers/clk/rockchips are: commit a915e30dd26ea5f3cc2e2c044aba38ee5973d3fa Merge: ce6dd266d535 b0158bb27c7b Author: Michael Turquette <mturquette@baylibre.com> Date: Wed Dec 23 13:08:56 2015 -0800 Merge branch 'clk-rockchip' into clk-next commit b0158bb27c7b6e9843f541c17b24dbd964b76db6 Author: Xing Zheng <zhengxing@rock-chips.com> Date: Tue Dec 22 22:28:01 2015 +0100 clk: rockchip: rk3036: include downstream muxes into fractional dividers Use the newly introduced possibility to combine the fractional dividers Arnd
Am Freitag, 1. Januar 2016, 22:50:43 schrieb Arnd Bergmann: > On Friday 01 January 2016 18:06:30 Heiko Stübner wrote: > > "[PATCH] clk: rockchip: fix section mismatches with new child-clocks" [0] > > > > should be in Mike's + Stephen's inbox since last week as well, which moves > > the offending new elements into separate entities, which can have > > __initdata attributes again. > > > > > > Heiko > > > > [0] http://www.spinics.net/lists/arm-kernel/msg471295.html > > The patch looks good, but for some reason, the next-20151223 kernel had no > problem and next-20151231 was broken, the top commits in > drivers/clk/rockchips are: That is correct. next-20151223 did not contain the offending patches yet. After the patches got merged into the clock-tree the kbuild-robot alerted us to the __initdata issue, so I created the linked patch as fixup. Heiko > > commit a915e30dd26ea5f3cc2e2c044aba38ee5973d3fa > Merge: ce6dd266d535 b0158bb27c7b > Author: Michael Turquette <mturquette@baylibre.com> > Date: Wed Dec 23 13:08:56 2015 -0800 > > Merge branch 'clk-rockchip' into clk-next > > commit b0158bb27c7b6e9843f541c17b24dbd964b76db6 > Author: Xing Zheng <zhengxing@rock-chips.com> > Date: Tue Dec 22 22:28:01 2015 +0100 > > clk: rockchip: rk3036: include downstream muxes into fractional dividers > > Use the newly introduced possibility to combine the fractional dividers > > > Arnd
Quoting Heiko Stübner (2016-01-01 14:05:10) > Am Freitag, 1. Januar 2016, 22:50:43 schrieb Arnd Bergmann: > > On Friday 01 January 2016 18:06:30 Heiko Stübner wrote: > > > "[PATCH] clk: rockchip: fix section mismatches with new child-clocks" [0] > > > > > > should be in Mike's + Stephen's inbox since last week as well, which moves > > > the offending new elements into separate entities, which can have > > > __initdata attributes again. > > > > > > > > > Heiko > > > > > > [0] http://www.spinics.net/lists/arm-kernel/msg471295.html > > > > The patch looks good, but for some reason, the next-20151223 kernel had no > > problem and next-20151231 was broken, the top commits in > > drivers/clk/rockchips are: > > That is correct. next-20151223 did not contain the offending patches yet. > After the patches got merged into the clock-tree the kbuild-robot alerted us > to the __initdata issue, so I created the linked patch as fixup. The fix has been pushed to the clk mirror. The next -next should no longer have the section mismatch warnings. Regards, Mike > > > Heiko > > > > > commit a915e30dd26ea5f3cc2e2c044aba38ee5973d3fa > > Merge: ce6dd266d535 b0158bb27c7b > > Author: Michael Turquette <mturquette@baylibre.com> > > Date: Wed Dec 23 13:08:56 2015 -0800 > > > > Merge branch 'clk-rockchip' into clk-next > > > > commit b0158bb27c7b6e9843f541c17b24dbd964b76db6 > > Author: Xing Zheng <zhengxing@rock-chips.com> > > Date: Tue Dec 22 22:28:01 2015 +0100 > > > > clk: rockchip: rk3036: include downstream muxes into fractional dividers > > > > Use the newly introduced possibility to combine the fractional dividers > > > > > > Arnd >
diff --git a/drivers/clk/rockchip/clk.h b/drivers/clk/rockchip/clk.h index 3f71ee5d1bc9..5cadf5e3fb16 100644 --- a/drivers/clk/rockchip/clk.h +++ b/drivers/clk/rockchip/clk.h @@ -244,7 +244,7 @@ struct clk *rockchip_clk_register_inverter(const char *name, void __iomem *reg, int shift, int flags, spinlock_t *lock); -#define PNAME(x) static const char *const x[] __initconst +#define PNAME(x) static const char *const x[] enum rockchip_clk_branch_type { branch_composite,
The latest changes to the rockchip clk implementation cause tons of warnings and/or errors (depending on the configuration): WARNING: vmlinux.o(.data+0x125394): Section mismatch in reference from the variable __compound_literal.125 to the (unknown reference) .init.rodata:(unknown) The reason is a compount literal in a macro that refers to an object in the .init.rodata section: #define COMPOSITE_FRACMUX(_id, cname, pname, f, mo, df, go, gs, gf, ch) \ { \ ... .child = &(struct rockchip_clk_branch)ch, \ } 'ch' here is placed in .data, and I could find no way to override that using an attribute or pragma because there is no identifier associated with it. This patch avoids the warning by placing the names in .rodata rather than .init.rodata. However, it is not an ideal solution because we really want all those data structures to be discarded along with the initdata. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Fixes: 8ca1ca8f6039 ("clk: rockchip: handle mux dependency of fractional dividers")