All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] clk: wm831x: fix checkpatch issue for static const char * array
@ 2012-07-04  9:54 Barry Song
  2012-07-04 10:19 ` Mark Brown
  0 siblings, 1 reply; 4+ messages in thread
From: Barry Song @ 2012-07-04  9:54 UTC (permalink / raw)
  To: linux-arm-kernel

From: Barry Song <Baohua.Song@csr.com>

static const char * array should probably be static const char * const

Signed-off-by: Barry Song <Baohua.Song@csr.com>
Cc: Mark Brown <broonie@opensource.wolfsonmicro.com>
---
 drivers/clk/clk-wm831x.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/clk/clk-wm831x.c b/drivers/clk/clk-wm831x.c
index e7b7765..f601e61 100644
--- a/drivers/clk/clk-wm831x.c
+++ b/drivers/clk/clk-wm831x.c
@@ -177,7 +177,7 @@ static int wm831x_fll_set_rate(struct clk_hw *hw, unsigned long rate,
 			       WM831X_FLL_AUTO_FREQ_MASK, i);
 }
 
-static const char *wm831x_fll_parents[] = {
+static const char * const wm831x_fll_parents[] = {
 	"xtal",
 	"clkin",
 };
@@ -298,7 +298,7 @@ static void wm831x_clkout_unprepare(struct clk_hw *hw)
 	wm831x_reg_lock(wm831x);
 }
 
-static const char *wm831x_clkout_parents[] = {
+static const char * const wm831x_clkout_parents[] = {
 	"xtal",
 	"fll",
 };
-- 
1.7.1



Member of the CSR plc group of companies. CSR plc registered in England and Wales, registered number 4187346, registered office Churchill House, Cambridge Business Park, Cowley Road, Cambridge, CB4 0WZ, United Kingdom
More information can be found at www.csr.com. Follow CSR on Twitter at http://twitter.com/CSR_PLC and read our blog at www.csr.com/blog

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH] clk: wm831x: fix checkpatch issue for static const char * array
  2012-07-04  9:54 [PATCH] clk: wm831x: fix checkpatch issue for static const char * array Barry Song
@ 2012-07-04 10:19 ` Mark Brown
  2012-07-06  2:37   ` Barry Song
  0 siblings, 1 reply; 4+ messages in thread
From: Mark Brown @ 2012-07-04 10:19 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Jul 04, 2012 at 05:54:40PM +0800, Barry Song wrote:
> From: Barry Song <Baohua.Song@csr.com>
> 
> static const char * array should probably be static const char * const

We need some additional clock API patches to get merged before this is
OK, last time I checked the API wasn't const correct enough for it.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120704/dc221aa9/attachment.sig>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] clk: wm831x: fix checkpatch issue for static const char * array
  2012-07-04 10:19 ` Mark Brown
@ 2012-07-06  2:37   ` Barry Song
  2012-07-06 11:08     ` Mark Brown
  0 siblings, 1 reply; 4+ messages in thread
From: Barry Song @ 2012-07-06  2:37 UTC (permalink / raw)
  To: linux-arm-kernel

2012/7/4 Mark Brown <broonie@opensource.wolfsonmicro.com>:
> On Wed, Jul 04, 2012 at 05:54:40PM +0800, Barry Song wrote:
>> From: Barry Song <Baohua.Song@csr.com>
>>
>> static const char * array should probably be static const char * const
>
> We need some additional clock API patches to get merged before this is
> OK, last time I checked the API wasn't const correct enough for it.
>
ok. also with this conflict: "warning: initialization discards
qualifiers from pointer target type"

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] clk: wm831x: fix checkpatch issue for static const char * array
  2012-07-06  2:37   ` Barry Song
@ 2012-07-06 11:08     ` Mark Brown
  0 siblings, 0 replies; 4+ messages in thread
From: Mark Brown @ 2012-07-06 11:08 UTC (permalink / raw)
  To: linux-arm-kernel

On Fri, Jul 06, 2012 at 10:37:19AM +0800, Barry Song wrote:
> 2012/7/4 Mark Brown <broonie@opensource.wolfsonmicro.com>:

> > We need some additional clock API patches to get merged before this is
> > OK, last time I checked the API wasn't const correct enough for it.

> ok. also with this conflict: "warning: initialization discards
> qualifiers from pointer target type"

Yes, that's the issue I'm referring to - we need the clock API to mark
the inputs it takes as const before we can have the drivers declare
thinngs const.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120706/76d0bdc0/attachment.sig>

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-07-06 11:08 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-07-04  9:54 [PATCH] clk: wm831x: fix checkpatch issue for static const char * array Barry Song
2012-07-04 10:19 ` Mark Brown
2012-07-06  2:37   ` Barry Song
2012-07-06 11:08     ` Mark Brown

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.