clk: pxa: Constify static struct clk_ops (6487649e) · Commits · 戴 / test

Constify a couple of static struct clk_ops that are not modified. Their only usage is inside the macros and their address is passed to clk_register_composite() which takes pointers to const struct clk_ops. This allows the compiler to put them in read-only memory. Signed-off-by:

Rikard Falkeborn <rikard.falkeborn@gmail.com> Link: https://lore.kernel.org/r/20200922184715.1854-1-rikard.falkeborn@gmail.com Signed-off-by:

Stephen Boyd <sboyd@kernel.org>

drivers/clk/pxa/clk-pxa.h

+4 −4

Original line number	Diff line number	Diff line
		@@ -19,11 +19,11 @@
		#define MUX_RO_RATE_RO_OPS(name, clk_name) \
		static struct clk_hw name ## _mux_hw; \
		static struct clk_hw name ## _rate_hw; \
		static struct clk_ops name ## _mux_ops = { \
		static const struct clk_ops name ## _mux_ops = { \
		.get_parent = name ## _get_parent, \
		.set_parent = dummy_clk_set_parent, \
		}; \
		static struct clk_ops name ## _rate_ops = { \
		static const struct clk_ops name ## _rate_ops = { \
		.recalc_rate = name ## _get_rate, \
		}; \
		static struct clk * __init clk_register_ ## name(void) \
		@@ -53,7 +53,7 @@

		#define RATE_OPS(name, clk_name) \
		static struct clk_hw name ## _rate_hw; \
		static struct clk_ops name ## _rate_ops = { \
		static const struct clk_ops name ## _rate_ops = { \
		.recalc_rate = name ## _get_rate, \
		.set_rate = name ## _set_rate, \
		.determine_rate = name ## _determine_rate, \

Admin message