* [PATCH 0/4] ARM: CSR: cleanup some minor coding-style issues
@ 2011-09-23 5:51 Barry Song
2011-09-23 5:51 ` [PATCH 1/4] ARM: CSR: timer: do not initialise statics to 0 or NULL Barry Song
` (4 more replies)
0 siblings, 5 replies; 6+ messages in thread
From: Barry Song @ 2011-09-23 5:51 UTC (permalink / raw)
To: linux-arm-kernel
Barry Song (4):
ARM: CSR: timer: do not initialise statics to 0 or NULL
ARM: CSR: timer: space required before the open parenthesis '('
ARM: CSR: prima2: fix trailing whitespace
ARM: CSR: clock: Fix indentation
arch/arm/mach-prima2/clock.c | 4 ++--
arch/arm/mach-prima2/prima2.c | 2 +-
arch/arm/mach-prima2/timer.c | 4 ++--
3 files changed, 5 insertions(+), 5 deletions(-)
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 [flat|nested] 6+ messages in thread
* [PATCH 1/4] ARM: CSR: timer: do not initialise statics to 0 or NULL
2011-09-23 5:51 [PATCH 0/4] ARM: CSR: cleanup some minor coding-style issues Barry Song
@ 2011-09-23 5:51 ` Barry Song
2011-09-23 5:51 ` [PATCH 2/4] ARM: CSR: timer: space required before the open parenthesis '(' Barry Song
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Barry Song @ 2011-09-23 5:51 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Barry Song <Baohua.Song@csr.com>
---
arch/arm/mach-prima2/timer.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-prima2/timer.c b/arch/arm/mach-prima2/timer.c
index ed7ec48..225ed3b 100644
--- a/arch/arm/mach-prima2/timer.c
+++ b/arch/arm/mach-prima2/timer.c
@@ -133,7 +133,7 @@ static struct irqaction sirfsoc_timer_irq = {
/* Overwrite weak default sched_clock with more precise one */
unsigned long long notrace sched_clock(void)
{
- static int is_mapped = 0;
+ static int is_mapped;
/*
* sched_clock is called earlier than .init of sys_timer
--
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] 6+ messages in thread
* [PATCH 2/4] ARM: CSR: timer: space required before the open parenthesis '('
2011-09-23 5:51 [PATCH 0/4] ARM: CSR: cleanup some minor coding-style issues Barry Song
2011-09-23 5:51 ` [PATCH 1/4] ARM: CSR: timer: do not initialise statics to 0 or NULL Barry Song
@ 2011-09-23 5:51 ` Barry Song
2011-09-23 5:51 ` [PATCH 3/4] ARM: CSR: prima2: fix trailing whitespace Barry Song
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Barry Song @ 2011-09-23 5:51 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Barry Song <Baohua.Song@csr.com>
---
arch/arm/mach-prima2/timer.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-prima2/timer.c b/arch/arm/mach-prima2/timer.c
index 225ed3b..26ab6fe 100644
--- a/arch/arm/mach-prima2/timer.c
+++ b/arch/arm/mach-prima2/timer.c
@@ -140,7 +140,7 @@ unsigned long long notrace sched_clock(void)
* if we map timer memory in .init of sys_timer, system
* will panic due to illegal memory access
*/
- if(!is_mapped) {
+ if (!is_mapped) {
sirfsoc_of_timer_map();
is_mapped = 1;
}
--
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] 6+ messages in thread
* [PATCH 3/4] ARM: CSR: prima2: fix trailing whitespace
2011-09-23 5:51 [PATCH 0/4] ARM: CSR: cleanup some minor coding-style issues Barry Song
2011-09-23 5:51 ` [PATCH 1/4] ARM: CSR: timer: do not initialise statics to 0 or NULL Barry Song
2011-09-23 5:51 ` [PATCH 2/4] ARM: CSR: timer: space required before the open parenthesis '(' Barry Song
@ 2011-09-23 5:51 ` Barry Song
2011-09-23 5:51 ` [PATCH 4/4] ARM: CSR: clock: Fix indentation Barry Song
2011-09-30 19:56 ` [PATCH 0/4] ARM: CSR: cleanup some minor coding-style issues Arnd Bergmann
4 siblings, 0 replies; 6+ messages in thread
From: Barry Song @ 2011-09-23 5:51 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Barry Song <Baohua.Song@csr.com>
---
arch/arm/mach-prima2/prima2.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-prima2/prima2.c b/arch/arm/mach-prima2/prima2.c
index f57124b..0ddf446 100644
--- a/arch/arm/mach-prima2/prima2.c
+++ b/arch/arm/mach-prima2/prima2.c
@@ -1,5 +1,5 @@
/*
- * Defines machines for CSR SiRFprimaII
+ * Defines machines for CSR SiRFprimaII
*
* Copyright (c) 2011 Cambridge Silicon Radio Limited, a CSR plc group company.
*
--
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] 6+ messages in thread
* [PATCH 4/4] ARM: CSR: clock: Fix indentation
2011-09-23 5:51 [PATCH 0/4] ARM: CSR: cleanup some minor coding-style issues Barry Song
` (2 preceding siblings ...)
2011-09-23 5:51 ` [PATCH 3/4] ARM: CSR: prima2: fix trailing whitespace Barry Song
@ 2011-09-23 5:51 ` Barry Song
2011-09-30 19:56 ` [PATCH 0/4] ARM: CSR: cleanup some minor coding-style issues Arnd Bergmann
4 siblings, 0 replies; 6+ messages in thread
From: Barry Song @ 2011-09-23 5:51 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Barry Song <Baohua.Song@csr.com>
---
arch/arm/mach-prima2/clock.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/mach-prima2/clock.c b/arch/arm/mach-prima2/clock.c
index 615a4e7..aebad7e 100644
--- a/arch/arm/mach-prima2/clock.c
+++ b/arch/arm/mach-prima2/clock.c
@@ -350,10 +350,10 @@ static struct clk_lookup onchip_clks[] = {
.clk = &clk_mem,
}, {
.dev_id = "sys",
- .clk = &clk_sys,
+ .clk = &clk_sys,
}, {
.dev_id = "io",
- .clk = &clk_io,
+ .clk = &clk_io,
},
};
--
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] 6+ messages in thread
* [PATCH 0/4] ARM: CSR: cleanup some minor coding-style issues
2011-09-23 5:51 [PATCH 0/4] ARM: CSR: cleanup some minor coding-style issues Barry Song
` (3 preceding siblings ...)
2011-09-23 5:51 ` [PATCH 4/4] ARM: CSR: clock: Fix indentation Barry Song
@ 2011-09-30 19:56 ` Arnd Bergmann
4 siblings, 0 replies; 6+ messages in thread
From: Arnd Bergmann @ 2011-09-30 19:56 UTC (permalink / raw)
To: linux-arm-kernel
On Friday 23 September 2011, Barry Song wrote:
> Barry Song (4):
> ARM: CSR: timer: do not initialise statics to 0 or NULL
> ARM: CSR: timer: space required before the open parenthesis '('
> ARM: CSR: prima2: fix trailing whitespace
> ARM: CSR: clock: Fix indentation
>
> arch/arm/mach-prima2/clock.c | 4 ++--
> arch/arm/mach-prima2/prima2.c | 2 +-
> arch/arm/mach-prima2/timer.c | 4 ++--
> 3 files changed, 5 insertions(+), 5 deletions(-)
Hi Barry,
Sorry for the late reply. This series and the one before all look good
to me, but I fear I'm losing track of the patches.
Can you send me pull requests for each set (fixes, cleanups, other
changes) so I can be sure I get everything? If you have trouble
with git.kernel.org still being down, I can apply the patches
manually and let you double-check them, but that would be more
work for both of us I think.
Arnd
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-09-30 19:56 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-09-23 5:51 [PATCH 0/4] ARM: CSR: cleanup some minor coding-style issues Barry Song
2011-09-23 5:51 ` [PATCH 1/4] ARM: CSR: timer: do not initialise statics to 0 or NULL Barry Song
2011-09-23 5:51 ` [PATCH 2/4] ARM: CSR: timer: space required before the open parenthesis '(' Barry Song
2011-09-23 5:51 ` [PATCH 3/4] ARM: CSR: prima2: fix trailing whitespace Barry Song
2011-09-23 5:51 ` [PATCH 4/4] ARM: CSR: clock: Fix indentation Barry Song
2011-09-30 19:56 ` [PATCH 0/4] ARM: CSR: cleanup some minor coding-style issues Arnd Bergmann
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.