* [PATCH 2/9] clocksource: parisc initialize list value
2007-03-30 18:45 [PATCH 1/9] clocksource: arm initialize list value Daniel Walker
@ 2007-03-30 18:45 ` Daniel Walker
2007-03-30 18:45 ` [PATCH 3/9] clocksource: avr32 " Daniel Walker
` (6 subsequent siblings)
7 siblings, 0 replies; 14+ messages in thread
From: Daniel Walker @ 2007-03-30 18:45 UTC (permalink / raw)
To: linux-kernel; +Cc: akpm, johnstul
[-- Attachment #1: clocksource_parisc_list_init.patch --]
[-- Type: text/plain, Size: 607 bytes --]
Update arch/parisc/ with list initialization.
Signed-Off-By: Daniel Walker <dwalker@mvista.com>
---
arch/parisc/kernel/time.c | 1 +
1 file changed, 1 insertion(+)
Index: linux-2.6.20/arch/parisc/kernel/time.c
===================================================================
--- linux-2.6.20.orig/arch/parisc/kernel/time.c
+++ linux-2.6.20/arch/parisc/kernel/time.c
@@ -184,6 +184,7 @@ static struct clocksource clocksource_cr
.mult = 0, /* to be set */
.shift = 22,
.flags = CLOCK_SOURCE_IS_CONTINUOUS,
+ .list = LIST_HEAD_INIT(clocksource_cr16.list),
};
#ifdef CONFIG_SMP
--
^ permalink raw reply [flat|nested] 14+ messages in thread* [PATCH 3/9] clocksource: avr32 initialize list value
2007-03-30 18:45 [PATCH 1/9] clocksource: arm initialize list value Daniel Walker
2007-03-30 18:45 ` [PATCH 2/9] clocksource: parisc " Daniel Walker
@ 2007-03-30 18:45 ` Daniel Walker
2007-03-30 18:45 ` [PATCH 4/9] clocksource: mips " Daniel Walker
` (5 subsequent siblings)
7 siblings, 0 replies; 14+ messages in thread
From: Daniel Walker @ 2007-03-30 18:45 UTC (permalink / raw)
To: linux-kernel; +Cc: akpm, johnstul
[-- Attachment #1: clocksource_avr32_list_init.patch --]
[-- Type: text/plain, Size: 1161 bytes --]
Update arch/avre32/ with list initialization.
Signed-Off-By: Daniel Walker <dwalker@mvista.com>
---
arch/avr32/kernel/time.c | 1 +
arch/avr32/mach-at32ap/time-tc.c | 1 +
2 files changed, 2 insertions(+)
Index: linux-2.6.20/arch/avr32/kernel/time.c
===================================================================
--- linux-2.6.20.orig/arch/avr32/kernel/time.c
+++ linux-2.6.20/arch/avr32/kernel/time.c
@@ -45,6 +45,7 @@ struct clocksource __weak clocksource_av
.mask = CLOCKSOURCE_MASK(32),
.shift = 16,
.flags = CLOCK_SOURCE_IS_CONTINUOUS,
+ .list = LIST_HEAD_INIT(clocksource_avr32.list),
};
irqreturn_t __weak timer_interrupt(int irq, void *dev_id);
Index: linux-2.6.20/arch/avr32/mach-at32ap/time-tc.c
===================================================================
--- linux-2.6.20.orig/arch/avr32/mach-at32ap/time-tc.c
+++ linux-2.6.20/arch/avr32/mach-at32ap/time-tc.c
@@ -50,6 +50,7 @@ struct clocksource clocksource_avr32 = {
.mask = CLOCKSOURCE_MASK(16),
.shift = 16,
.flags = CLOCK_SOURCE_IS_CONTINUOUS,
+ .list = LIST_HEAD_INIT(clocksource_avr32.list),
};
static void avr32_timer_ack(void)
--
^ permalink raw reply [flat|nested] 14+ messages in thread* [PATCH 4/9] clocksource: mips initialize list value
2007-03-30 18:45 [PATCH 1/9] clocksource: arm initialize list value Daniel Walker
2007-03-30 18:45 ` [PATCH 2/9] clocksource: parisc " Daniel Walker
2007-03-30 18:45 ` [PATCH 3/9] clocksource: avr32 " Daniel Walker
@ 2007-03-30 18:45 ` Daniel Walker
2007-03-30 18:45 ` [PATCH 5/9] clocksource: i386 " Daniel Walker
` (4 subsequent siblings)
7 siblings, 0 replies; 14+ messages in thread
From: Daniel Walker @ 2007-03-30 18:45 UTC (permalink / raw)
To: linux-kernel; +Cc: akpm, johnstul
[-- Attachment #1: clocksource_mips_list_init.patch --]
[-- Type: text/plain, Size: 617 bytes --]
Update arch/mips/ with list initialization.
Signed-Off-By: Daniel Walker <dwalker@mvista.com>
---
arch/mips/kernel/time.c | 1 +
1 file changed, 1 insertion(+)
Index: linux-2.6.19/arch/mips/kernel/time.c
===================================================================
--- linux-2.6.19.orig/arch/mips/kernel/time.c
+++ linux-2.6.19/arch/mips/kernel/time.c
@@ -308,6 +308,7 @@ struct clocksource clocksource_mips = {
.name = "MIPS",
.mask = 0xffffffff,
.flags = CLOCK_SOURCE_IS_CONTINUOUS,
+ .list = LIST_HEAD_INIT(clocksource_mips.list),
};
static void __init init_mips_clocksource(void)
--
^ permalink raw reply [flat|nested] 14+ messages in thread* [PATCH 5/9] clocksource: i386 initialize list value
2007-03-30 18:45 [PATCH 1/9] clocksource: arm initialize list value Daniel Walker
` (2 preceding siblings ...)
2007-03-30 18:45 ` [PATCH 4/9] clocksource: mips " Daniel Walker
@ 2007-03-30 18:45 ` Daniel Walker
2007-03-30 18:45 ` [PATCH 6/9] clocksource: x86_64 " Daniel Walker
` (3 subsequent siblings)
7 siblings, 0 replies; 14+ messages in thread
From: Daniel Walker @ 2007-03-30 18:45 UTC (permalink / raw)
To: linux-kernel; +Cc: akpm, johnstul
[-- Attachment #1: clocksource_i386_list_init.patch --]
[-- Type: text/plain, Size: 1976 bytes --]
Update arch/i386/ with list initialization.
Signed-Off-By: Daniel Walker <dwalker@mvista.com>
---
arch/i386/kernel/hpet.c | 1 +
arch/i386/kernel/i8253.c | 1 +
arch/i386/kernel/tsc.c | 1 +
arch/i386/kernel/vmitime.c | 1 +
4 files changed, 4 insertions(+)
Index: linux-2.6.20/arch/i386/kernel/hpet.c
===================================================================
--- linux-2.6.20.orig/arch/i386/kernel/hpet.c
+++ linux-2.6.20/arch/i386/kernel/hpet.c
@@ -215,6 +215,7 @@ static struct clocksource clocksource_hp
.mask = HPET_MASK,
.shift = HPET_SHIFT,
.flags = CLOCK_SOURCE_IS_CONTINUOUS,
+ .list = LIST_HEAD_INIT(clocksource_hpet.list),
};
/*
Index: linux-2.6.20/arch/i386/kernel/i8253.c
===================================================================
--- linux-2.6.20.orig/arch/i386/kernel/i8253.c
+++ linux-2.6.20/arch/i386/kernel/i8253.c
@@ -193,6 +193,7 @@ static struct clocksource clocksource_pi
.mask = CLOCKSOURCE_MASK(32),
.mult = 0,
.shift = 20,
+ .list = LIST_HEAD_INIT(clocksource_pit.list),
};
static int __init init_pit_clocksource(void)
Index: linux-2.6.20/arch/i386/kernel/tsc.c
===================================================================
--- linux-2.6.20.orig/arch/i386/kernel/tsc.c
+++ linux-2.6.20/arch/i386/kernel/tsc.c
@@ -279,6 +279,7 @@ static struct clocksource clocksource_ts
.shift = 22,
.flags = CLOCK_SOURCE_IS_CONTINUOUS |
CLOCK_SOURCE_MUST_VERIFY,
+ .list = LIST_HEAD_INIT(clocksource_tsc.list),
};
void mark_tsc_unstable(char *reason)
Index: linux-2.6.20/arch/i386/kernel/vmitime.c
===================================================================
--- linux-2.6.20.orig/arch/i386/kernel/vmitime.c
+++ linux-2.6.20/arch/i386/kernel/vmitime.c
@@ -116,6 +116,7 @@ static struct clocksource clocksource_vm
.mult = 0, /* to be set */
.shift = 22,
.flags = CLOCK_SOURCE_IS_CONTINUOUS,
+ .list = LIST_HEAD_INIT(clocksource_vmi.list),
};
--
^ permalink raw reply [flat|nested] 14+ messages in thread* [PATCH 6/9] clocksource: x86_64 initialize list value
2007-03-30 18:45 [PATCH 1/9] clocksource: arm initialize list value Daniel Walker
` (3 preceding siblings ...)
2007-03-30 18:45 ` [PATCH 5/9] clocksource: i386 " Daniel Walker
@ 2007-03-30 18:45 ` Daniel Walker
2007-03-30 18:45 ` [PATCH 7/9] clocksource: driver " Daniel Walker
` (2 subsequent siblings)
7 siblings, 0 replies; 14+ messages in thread
From: Daniel Walker @ 2007-03-30 18:45 UTC (permalink / raw)
To: linux-kernel; +Cc: akpm, johnstul
[-- Attachment #1: clocksource_x86_64_list_init.patch --]
[-- Type: text/plain, Size: 1105 bytes --]
Update arch/x86_64/ with list initialization.
Signed-Off-By: Daniel Walker <dwalker@mvista.com>
---
arch/x86_64/kernel/hpet.c | 1 +
arch/x86_64/kernel/tsc.c | 1 +
2 files changed, 2 insertions(+)
Index: linux-2.6.20/arch/x86_64/kernel/hpet.c
===================================================================
--- linux-2.6.20.orig/arch/x86_64/kernel/hpet.c
+++ linux-2.6.20/arch/x86_64/kernel/hpet.c
@@ -131,6 +131,7 @@ struct clocksource clocksource_hpet = {
.shift = HPET_SHIFT,
.flags = CLOCK_SOURCE_IS_CONTINUOUS,
.vread = vread_hpet,
+ .list = LIST_HEAD_INIT(clocksource_hpet.list),
};
int hpet_arch_init(void)
Index: linux-2.6.20/arch/x86_64/kernel/tsc.c
===================================================================
--- linux-2.6.20.orig/arch/x86_64/kernel/tsc.c
+++ linux-2.6.20/arch/x86_64/kernel/tsc.c
@@ -195,6 +195,7 @@ static struct clocksource clocksource_ts
.flags = CLOCK_SOURCE_IS_CONTINUOUS |
CLOCK_SOURCE_MUST_VERIFY,
.vread = vread_tsc,
+ .list = LIST_HEAD_INIT(clocksource_tsc.list),
};
void mark_tsc_unstable(char *reason)
--
^ permalink raw reply [flat|nested] 14+ messages in thread* [PATCH 7/9] clocksource: driver initialize list value
2007-03-30 18:45 [PATCH 1/9] clocksource: arm initialize list value Daniel Walker
` (4 preceding siblings ...)
2007-03-30 18:45 ` [PATCH 6/9] clocksource: x86_64 " Daniel Walker
@ 2007-03-30 18:45 ` Daniel Walker
2007-03-30 18:45 ` [PATCH 8/9] clocksource: s390 " Daniel Walker
2007-03-30 18:45 ` [PATCH 9/9] clocksource: refactor duplicate registration checking Daniel Walker
7 siblings, 0 replies; 14+ messages in thread
From: Daniel Walker @ 2007-03-30 18:45 UTC (permalink / raw)
To: linux-kernel; +Cc: akpm, johnstul
[-- Attachment #1: clocksource_drivers_list_init.patch --]
[-- Type: text/plain, Size: 1631 bytes --]
Update drivers/clocksource/ with list initialization.
Signed-Off-By: Daniel Walker <dwalker@mvista.com>
---
drivers/clocksource/acpi_pm.c | 1 +
drivers/clocksource/cyclone.c | 1 +
drivers/clocksource/scx200_hrt.c | 1 +
3 files changed, 3 insertions(+)
Index: linux-2.6.19/drivers/clocksource/acpi_pm.c
===================================================================
--- linux-2.6.19.orig/drivers/clocksource/acpi_pm.c
+++ linux-2.6.19/drivers/clocksource/acpi_pm.c
@@ -74,6 +74,7 @@ static struct clocksource clocksource_ac
.mult = 0, /*to be caluclated*/
.shift = 22,
.flags = CLOCK_SOURCE_IS_CONTINUOUS,
+ .list = LIST_HEAD_INIT(clocksource_acpi_pm.list),
};
Index: linux-2.6.19/drivers/clocksource/cyclone.c
===================================================================
--- linux-2.6.19.orig/drivers/clocksource/cyclone.c
+++ linux-2.6.19/drivers/clocksource/cyclone.c
@@ -32,6 +32,7 @@ static struct clocksource clocksource_cy
.mult = 10,
.shift = 0,
.flags = CLOCK_SOURCE_IS_CONTINUOUS,
+ .list = LIST_HEAD_INIT(clocksource_cyclone.list),
};
static int __init init_cyclone_clocksource(void)
Index: linux-2.6.19/drivers/clocksource/scx200_hrt.c
===================================================================
--- linux-2.6.19.orig/drivers/clocksource/scx200_hrt.c
+++ linux-2.6.19/drivers/clocksource/scx200_hrt.c
@@ -58,6 +58,7 @@ static struct clocksource cs_hrt = {
.read = read_hrt,
.mask = CLOCKSOURCE_MASK(32),
.flags = CLOCK_SOURCE_IS_CONTINUOUS,
+ .list = LIST_HEAD_INIT(cs_hrt.list),
/* mult, shift are set based on mhz27 flag */
};
--
^ permalink raw reply [flat|nested] 14+ messages in thread* [PATCH 8/9] clocksource: s390 initialize list value
2007-03-30 18:45 [PATCH 1/9] clocksource: arm initialize list value Daniel Walker
` (5 preceding siblings ...)
2007-03-30 18:45 ` [PATCH 7/9] clocksource: driver " Daniel Walker
@ 2007-03-30 18:45 ` Daniel Walker
2007-03-30 19:09 ` Heiko Carstens
2007-03-30 18:45 ` [PATCH 9/9] clocksource: refactor duplicate registration checking Daniel Walker
7 siblings, 1 reply; 14+ messages in thread
From: Daniel Walker @ 2007-03-30 18:45 UTC (permalink / raw)
To: linux-kernel; +Cc: akpm, johnstul
[-- Attachment #1: clocksource_s390_init_list.patch --]
[-- Type: text/plain, Size: 561 bytes --]
Update arch/s390 with list initialization.
Signed-Off-By: Daniel Walker <dwalker@mvista.com>
---
arch/s390/kernel/time.c | 1 +
1 file changed, 1 insertion(+)
Index: linux-2.6.20/arch/s390/kernel/time.c
===================================================================
--- linux-2.6.20.orig/arch/s390/kernel/time.c
+++ linux-2.6.20/arch/s390/kernel/time.c
@@ -314,6 +314,7 @@ static struct clocksource clocksource_to
.mult = 1000,
.shift = 12,
.flags = CLOCK_SOURCE_IS_CONTINUOUS,
+ .list = LIST_HEAD_INIT(clocksource_tod.list),
};
--
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH 8/9] clocksource: s390 initialize list value
2007-03-30 18:45 ` [PATCH 8/9] clocksource: s390 " Daniel Walker
@ 2007-03-30 19:09 ` Heiko Carstens
2007-03-30 19:19 ` Daniel Walker
0 siblings, 1 reply; 14+ messages in thread
From: Heiko Carstens @ 2007-03-30 19:09 UTC (permalink / raw)
To: Daniel Walker; +Cc: linux-kernel, akpm, johnstul
On Fri, Mar 30, 2007 at 11:45:49AM -0700, Daniel Walker wrote:
> Update arch/s390 with list initialization.
>
> Signed-Off-By: Daniel Walker <dwalker@mvista.com>
>
> ---
> arch/s390/kernel/time.c | 1 +
> 1 file changed, 1 insertion(+)
>
> Index: linux-2.6.20/arch/s390/kernel/time.c
> ===================================================================
> --- linux-2.6.20.orig/arch/s390/kernel/time.c
> +++ linux-2.6.20/arch/s390/kernel/time.c
> @@ -314,6 +314,7 @@ static struct clocksource clocksource_to
> .mult = 1000,
> .shift = 12,
> .flags = CLOCK_SOURCE_IS_CONTINUOUS,
> + .list = LIST_HEAD_INIT(clocksource_tod.list),
> };
Why? This doesn't fix anything...
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 8/9] clocksource: s390 initialize list value
2007-03-30 19:09 ` Heiko Carstens
@ 2007-03-30 19:19 ` Daniel Walker
2007-03-30 19:36 ` Heiko Carstens
0 siblings, 1 reply; 14+ messages in thread
From: Daniel Walker @ 2007-03-30 19:19 UTC (permalink / raw)
To: Heiko Carstens; +Cc: linux-kernel, akpm, johnstul
On Fri, 2007-03-30 at 21:09 +0200, Heiko Carstens wrote:
> On Fri, Mar 30, 2007 at 11:45:49AM -0700, Daniel Walker wrote:
> > Update arch/s390 with list initialization.
> >
> > Signed-Off-By: Daniel Walker <dwalker@mvista.com>
> >
> > ---
> > arch/s390/kernel/time.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > Index: linux-2.6.20/arch/s390/kernel/time.c
> > ===================================================================
> > --- linux-2.6.20.orig/arch/s390/kernel/time.c
> > +++ linux-2.6.20/arch/s390/kernel/time.c
> > @@ -314,6 +314,7 @@ static struct clocksource clocksource_to
> > .mult = 1000,
> > .shift = 12,
> > .flags = CLOCK_SOURCE_IS_CONTINUOUS,
> > + .list = LIST_HEAD_INIT(clocksource_tod.list),
> > };
>
> Why? This doesn't fix anything...
It's needed for patch 9/9 in the series.
Daniel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 8/9] clocksource: s390 initialize list value
2007-03-30 19:19 ` Daniel Walker
@ 2007-03-30 19:36 ` Heiko Carstens
0 siblings, 0 replies; 14+ messages in thread
From: Heiko Carstens @ 2007-03-30 19:36 UTC (permalink / raw)
To: Daniel Walker; +Cc: linux-kernel, akpm, johnstul
On Fri, Mar 30, 2007 at 12:19:18PM -0700, Daniel Walker wrote:
> On Fri, 2007-03-30 at 21:09 +0200, Heiko Carstens wrote:
> > On Fri, Mar 30, 2007 at 11:45:49AM -0700, Daniel Walker wrote:
> > > Update arch/s390 with list initialization.
> > >
> > > Signed-Off-By: Daniel Walker <dwalker@mvista.com>
[...]
> > Why? This doesn't fix anything...
>
> It's needed for patch 9/9 in the series.
Then maybe your patch description should reflect that?
If this gets merged as a seperate patch and somebody checks the git
history of arch/s390 and sees this patch... how could he know what
this is good for?
Does it fix a bug? Needed by any feature? Or...?
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 9/9] clocksource: refactor duplicate registration checking
2007-03-30 18:45 [PATCH 1/9] clocksource: arm initialize list value Daniel Walker
` (6 preceding siblings ...)
2007-03-30 18:45 ` [PATCH 8/9] clocksource: s390 " Daniel Walker
@ 2007-03-30 18:45 ` Daniel Walker
2007-03-31 1:59 ` James Morris
7 siblings, 1 reply; 14+ messages in thread
From: Daniel Walker @ 2007-03-30 18:45 UTC (permalink / raw)
To: linux-kernel; +Cc: akpm, johnstul
[-- Attachment #1: clocksource_use_inited_lists.patch --]
[-- Type: text/plain, Size: 3444 bytes --]
Refactors the duplicate registration checking. This makes it based on the
clocksource structure list state. I was able to drop some if statements
making the registration code path slightly smaller and faster, and remove
some looping which was endemic of the first version of this check.
Signed-Off-By: Daniel Walker <dwalker@mvista.com>
---
include/linux/clocksource.h | 2 +-
kernel/time/clocksource.c | 30 ++++++++++++++++++------------
kernel/time/jiffies.c | 1 +
3 files changed, 20 insertions(+), 13 deletions(-)
Index: linux-2.6.20/include/linux/clocksource.h
===================================================================
--- linux-2.6.20.orig/include/linux/clocksource.h
+++ linux-2.6.20/include/linux/clocksource.h
@@ -26,7 +26,7 @@ struct clocksource;
* Provides mostly state-free accessors to the underlying hardware.
*
* @name: ptr to clocksource name
- * @list: list head for registration
+ * @list: list head for registration, must be initialized.
* @rating: rating value for selection (higher is better)
* To avoid rating inflation the following
* list should give you a guide as to how
Index: linux-2.6.20/kernel/time/clocksource.c
===================================================================
--- linux-2.6.20.orig/kernel/time/clocksource.c
+++ linux-2.6.20/kernel/time/clocksource.c
@@ -232,7 +232,7 @@ static struct clocksource *select_clocks
/*
* Enqueue the clocksource sorted by rating
*/
-static int clocksource_enqueue(struct clocksource *c)
+static void clocksource_enqueue(struct clocksource *c)
{
struct list_head *tmp, *entry = &clocksource_list;
@@ -240,8 +240,7 @@ static int clocksource_enqueue(struct cl
struct clocksource *cs;
cs = list_entry(tmp, struct clocksource, list);
- if (cs == c)
- return -EBUSY;
+
/* Keep track of the place, where to insert */
if (cs->rating >= c->rating)
entry = tmp;
@@ -252,28 +251,35 @@ static int clocksource_enqueue(struct cl
!strcmp(c->name, override_name))
clocksource_override = c;
- return 0;
}
/**
* clocksource_register - Used to install new clocksources
* @t: clocksource to be registered
*
- * Returns -EBUSY if registration fails, zero otherwise.
+ * Always returns zero.
*/
int clocksource_register(struct clocksource *c)
{
unsigned long flags;
- int ret;
+
+ /*
+ * This BUGON indicates that either the clocksource has been
+ * registered already, or the list element hasn't been properly
+ * initialized.
+ */
+ BUG_ON(!list_empty(&c->list));
spin_lock_irqsave(&clocksource_lock, flags);
- ret = clocksource_enqueue(c);
- if (!ret)
- next_clocksource = select_clocksource();
+
+ clocksource_enqueue(c);
+ next_clocksource = select_clocksource();
+
spin_unlock_irqrestore(&clocksource_lock, flags);
- if (!ret)
- clocksource_check_watchdog(c);
- return ret;
+
+ clocksource_check_watchdog(c);
+
+ return 0;
}
EXPORT_SYMBOL(clocksource_register);
Index: linux-2.6.20/kernel/time/jiffies.c
===================================================================
--- linux-2.6.20.orig/kernel/time/jiffies.c
+++ linux-2.6.20/kernel/time/jiffies.c
@@ -62,6 +62,7 @@ struct clocksource clocksource_jiffies =
.mask = 0xffffffff, /*32bits*/
.mult = NSEC_PER_JIFFY << JIFFIES_SHIFT, /* details above */
.shift = JIFFIES_SHIFT,
+ .list = LIST_HEAD_INIT(clocksource_jiffies.list),
};
static int __init init_jiffies_clocksource(void)
--
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH 9/9] clocksource: refactor duplicate registration checking
2007-03-30 18:45 ` [PATCH 9/9] clocksource: refactor duplicate registration checking Daniel Walker
@ 2007-03-31 1:59 ` James Morris
2007-03-31 2:47 ` Daniel Walker
0 siblings, 1 reply; 14+ messages in thread
From: James Morris @ 2007-03-31 1:59 UTC (permalink / raw)
To: Daniel Walker; +Cc: linux-kernel, akpm, johnstul
On Fri, 30 Mar 2007, Daniel Walker wrote:
> /**
> * clocksource_register - Used to install new clocksources
> * @t: clocksource to be registered
> *
> - * Returns -EBUSY if registration fails, zero otherwise.
> + * Always returns zero.
> */
> int clocksource_register(struct clocksource *c)
Return should be void, then.
- James
--
James Morris
<jmorris@namei.org>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 9/9] clocksource: refactor duplicate registration checking
2007-03-31 1:59 ` James Morris
@ 2007-03-31 2:47 ` Daniel Walker
0 siblings, 0 replies; 14+ messages in thread
From: Daniel Walker @ 2007-03-31 2:47 UTC (permalink / raw)
To: James Morris; +Cc: linux-kernel, akpm, johnstul
On Fri, 2007-03-30 at 21:59 -0400, James Morris wrote:
> On Fri, 30 Mar 2007, Daniel Walker wrote:
>
> > /**
> > * clocksource_register - Used to install new clocksources
> > * @t: clocksource to be registered
> > *
> > - * Returns -EBUSY if registration fails, zero otherwise.
> > + * Always returns zero.
> > */
> > int clocksource_register(struct clocksource *c)
>
> Return should be void, then.
Yeah, that's another patch tho ..
Daniel
^ permalink raw reply [flat|nested] 14+ messages in thread