lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 12 Apr 2012 11:49:23 +0530 From: Viresh Kumar <viresh.kumar@...com> To: Mike Turquette <mturquette@...aro.org> Cc: "arnd.bergmann@...aro.org" <arnd.bergmann@...aro.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>, "linaro-dev@...ts.linaro.org" <linaro-dev@...ts.linaro.org>, "patches@...aro.org" <patches@...aro.org>, Shawn Guo <shawn.guo@...aro.org>, Olof Johansson <olof@...om.net>, Russell King <linux@....linux.org.uk>, Sascha Hauer <s.hauer@...gutronix.de>, Richard Zhao <richard.zhao@...aro.org>, Saravana Kannan <skannan@...eaurora.org>, Mark Brown <broonie@...nsource.wolfsonmicro.com>, Andrew Lunn <andrew@...n.ch>, Rajendra Nayak <rnayak@...com> Subject: Re: [PATCH 07/13] clk: add "const" for clk_ops of basic clks On 4/12/2012 6:32 AM, Mike Turquette wrote: > From: Shawn Guo <shawn.guo@...aro.org> > > The clk_ops of basic clks should have "const" to match the definition > in "struct clk" and clk_register prototype. > > Signed-off-by: Shawn Guo <shawn.guo@...aro.org> > Signed-off-by: Mike Turquette <mturquette@...aro.org> > Cc: Arnd Bergman <arnd.bergmann@...aro.org> > Cc: Olof Johansson <olof@...om.net> > Cc: Russell King <linux@....linux.org.uk> > Cc: Sascha Hauer <s.hauer@...gutronix.de> > Cc: Richard Zhao <richard.zhao@...aro.org> > Cc: Saravana Kannan <skannan@...eaurora.org> > Cc: Mark Brown <broonie@...nsource.wolfsonmicro.com> > Cc: Andrew Lunn <andrew@...n.ch> > Cc: Rajendra Nayak <rnayak@...com> > Cc: Viresh Kumar <viresh.kumar@...com> > --- > drivers/clk/clk-divider.c | 2 +- > drivers/clk/clk-fixed-rate.c | 2 +- > drivers/clk/clk-gate.c | 2 +- > drivers/clk/clk-mux.c | 2 +- > include/linux/clk-private.h | 8 ++++---- > 5 files changed, 8 insertions(+), 8 deletions(-) > > diff --git a/drivers/clk/clk-divider.c b/drivers/clk/clk-divider.c > index 231cd6e..b1c4b02 100644 > --- a/drivers/clk/clk-divider.c > +++ b/drivers/clk/clk-divider.c > @@ -146,7 +146,7 @@ static int clk_divider_set_rate(struct clk_hw *hw, unsigned long rate) > return 0; > } > > -struct clk_ops clk_divider_ops = { > +const struct clk_ops clk_divider_ops = { > .recalc_rate = clk_divider_recalc_rate, > .round_rate = clk_divider_round_rate, > .set_rate = clk_divider_set_rate, > diff --git a/drivers/clk/clk-fixed-rate.c b/drivers/clk/clk-fixed-rate.c > index 651b06f..027e477 100644 > --- a/drivers/clk/clk-fixed-rate.c > +++ b/drivers/clk/clk-fixed-rate.c > @@ -33,7 +33,7 @@ static unsigned long clk_fixed_rate_recalc_rate(struct clk_hw *hw, > return to_clk_fixed_rate(hw)->fixed_rate; > } > > -struct clk_ops clk_fixed_rate_ops = { > +const struct clk_ops clk_fixed_rate_ops = { > .recalc_rate = clk_fixed_rate_recalc_rate, > }; > EXPORT_SYMBOL_GPL(clk_fixed_rate_ops); > diff --git a/drivers/clk/clk-gate.c b/drivers/clk/clk-gate.c > index b688f47..fe2ff9e 100644 > --- a/drivers/clk/clk-gate.c > +++ b/drivers/clk/clk-gate.c > @@ -98,7 +98,7 @@ static int clk_gate_is_enabled(struct clk_hw *hw) > return reg ? 1 : 0; > } > > -struct clk_ops clk_gate_ops = { > +const struct clk_ops clk_gate_ops = { > .enable = clk_gate_enable, > .disable = clk_gate_disable, > .is_enabled = clk_gate_is_enabled, > diff --git a/drivers/clk/clk-mux.c b/drivers/clk/clk-mux.c > index 45cad61..5424488 100644 > --- a/drivers/clk/clk-mux.c > +++ b/drivers/clk/clk-mux.c > @@ -82,7 +82,7 @@ static int clk_mux_set_parent(struct clk_hw *hw, u8 index) > return 0; > } > > -struct clk_ops clk_mux_ops = { > +const struct clk_ops clk_mux_ops = { > .get_parent = clk_mux_get_parent, > .set_parent = clk_mux_set_parent, > }; > diff --git a/include/linux/clk-private.h b/include/linux/clk-private.h > index 5e4312b..5f4ccd7 100644 > --- a/include/linux/clk-private.h > +++ b/include/linux/clk-private.h > @@ -55,7 +55,7 @@ struct clk { > * alternative macro for static initialization > */ > > -extern struct clk_ops clk_fixed_rate_ops; > +extern const struct clk_ops clk_fixed_rate_ops; > > #define DEFINE_CLK_FIXED_RATE(_name, _flags, _rate, \ > _fixed_rate_flags) \ > @@ -78,7 +78,7 @@ extern struct clk_ops clk_fixed_rate_ops; > .flags = _flags, \ > }; > > -extern struct clk_ops clk_gate_ops; > +extern const struct clk_ops clk_gate_ops; > > #define DEFINE_CLK_GATE(_name, _parent_name, _parent_ptr, \ > _flags, _reg, _bit_idx, \ > @@ -110,7 +110,7 @@ extern struct clk_ops clk_gate_ops; > .flags = _flags, \ > }; > > -extern struct clk_ops clk_divider_ops; > +extern const struct clk_ops clk_divider_ops; > > #define DEFINE_CLK_DIVIDER(_name, _parent_name, _parent_ptr, \ > _flags, _reg, _shift, _width, \ > @@ -143,7 +143,7 @@ extern struct clk_ops clk_divider_ops; > .flags = _flags, \ > }; > > -extern struct clk_ops clk_mux_ops; > +extern const struct clk_ops clk_mux_ops; > > #define DEFINE_CLK_MUX(_name, _parent_names, _parents, _flags, \ > _reg, _shift, _width, \ Reviewed-by: Viresh Kumar <viresh.kumar@...com> -- viresh -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists