* Re: [PATCH 6/8] clocksource: m86knommu: Convert to clocksource_register_hz/khz [not found] <4ea6fbe8.43310e0a.592c.5416SMTPIN_ADDED@mx.google.com> @ 2011-10-25 18:35 ` Geert Uytterhoeven 2011-10-25 18:39 ` john stultz 2011-10-25 18:46 ` [PATCH 6/8] clocksource: m86k: " john stultz 0 siblings, 2 replies; 6+ messages in thread From: Geert Uytterhoeven @ 2011-10-25 18:35 UTC (permalink / raw) To: y; +Cc: lkml, John Stultz, Greg Ungerer, Greg Ungerer, uClinux development list On Tue, Oct 25, 2011 at 20:08, <y@kernel.beaverton.ibm.com> wrote: > From: John Stultz <johnstul@us.ibm.com> > > This converts the m86knommu clocksources to use clocksource_register_hz/khz > > This is untested, so any assistance in testing would be appreciated! > > CC: Geert Uytterhoeven <geert@linux-m68k.org> > CC: Greg Ungerer <gerg@uclinux.org> > Signed-off-by: John Stultz <johnstul@us.ibm.com> > --- > arch/m68k/platform/68328/timers.c | 4 +- > arch/m68k/platform/coldfire/dma_timer.c | 5 +- > arch/m68knommu/platform/coldfire/pit.c | 167 ++++++++++++++++++++++++++++ > arch/m68knommu/platform/coldfire/timers.c | 172 +++++++++++++++++++++++++++++ > 4 files changed, 341 insertions(+), 7 deletions(-) > create mode 100644 arch/m68knommu/platform/coldfire/pit.c > create mode 100644 arch/m68knommu/platform/coldfire/timers.c arch/m68knommu no longer exists. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 6/8] clocksource: m86knommu: Convert to clocksource_register_hz/khz 2011-10-25 18:35 ` [PATCH 6/8] clocksource: m86knommu: Convert to clocksource_register_hz/khz Geert Uytterhoeven @ 2011-10-25 18:39 ` john stultz 2011-10-25 18:46 ` [PATCH 6/8] clocksource: m86k: " john stultz 1 sibling, 0 replies; 6+ messages in thread From: john stultz @ 2011-10-25 18:39 UTC (permalink / raw) To: Geert Uytterhoeven Cc: lkml, Greg Ungerer, Greg Ungerer, uClinux development list On Tue, 2011-10-25 at 20:35 +0200, Geert Uytterhoeven wrote: > On Tue, Oct 25, 2011 at 20:08, <y@kernel.beaverton.ibm.com> wrote: > > From: John Stultz <johnstul@us.ibm.com> > > > > This converts the m86knommu clocksources to use clocksource_register_hz/khz > > > > This is untested, so any assistance in testing would be appreciated! > > > > CC: Geert Uytterhoeven <geert@linux-m68k.org> > > CC: Greg Ungerer <gerg@uclinux.org> > > Signed-off-by: John Stultz <johnstul@us.ibm.com> > > --- > > arch/m68k/platform/68328/timers.c | 4 +- > > arch/m68k/platform/coldfire/dma_timer.c | 5 +- > > arch/m68knommu/platform/coldfire/pit.c | 167 ++++++++++++++++++++++++++++ > > arch/m68knommu/platform/coldfire/timers.c | 172 +++++++++++++++++++++++++++++ > > 4 files changed, 341 insertions(+), 7 deletions(-) > > create mode 100644 arch/m68knommu/platform/coldfire/pit.c > > create mode 100644 arch/m68knommu/platform/coldfire/timers.c > > arch/m68knommu no longer exists. Ah. My apologies. These patches have been around forever. Thanks for the heads up. thanks -john ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 6/8] clocksource: m86k: Convert to clocksource_register_hz/khz 2011-10-25 18:35 ` [PATCH 6/8] clocksource: m86knommu: Convert to clocksource_register_hz/khz Geert Uytterhoeven 2011-10-25 18:39 ` john stultz @ 2011-10-25 18:46 ` john stultz 2011-10-26 5:10 ` Greg Ungerer 1 sibling, 1 reply; 6+ messages in thread From: john stultz @ 2011-10-25 18:46 UTC (permalink / raw) To: Geert Uytterhoeven Cc: lkml, Greg Ungerer, Greg Ungerer, uClinux development list Updated to merge the valid bits of the two m68k patches. This converts the m86k clocksources to use clocksource_register_hz/khz This is untested, so any assistance in testing would be appreciated! CC: Geert Uytterhoeven <geert@linux-m68k.org> CC: Greg Ungerer <gerg@uclinux.org> Signed-off-by: John Stultz <johnstul@us.ibm.com> --- arch/m68k/platform/68328/timers.c | 4 +--- arch/m68k/platform/coldfire/dma_timer.c | 5 +---- arch/m68k/platform/coldfire/pit.c | 4 +--- arch/m68k/platform/coldfire/sltimers.c | 4 +--- arch/m68k/platform/coldfire/timers.c | 4 +--- 5 files changed, 5 insertions(+), 16 deletions(-) diff --git a/arch/m68k/platform/68328/timers.c b/arch/m68k/platform/68328/timers.c index 309f725..f267886 100644 --- a/arch/m68k/platform/68328/timers.c +++ b/arch/m68k/platform/68328/timers.c @@ -93,7 +93,6 @@ static struct clocksource m68328_clk = { .name = "timer", .rating = 250, .read = m68328_read_clk, - .shift = 20, .mask = CLOCKSOURCE_MASK(32), .flags = CLOCK_SOURCE_IS_CONTINUOUS, }; @@ -115,8 +114,7 @@ void hw_timer_init(void) /* Enable timer 1 */ TCTL |= TCTL_TEN; - m68328_clk.mult = clocksource_hz2mult(TICKS_PER_JIFFY*HZ, m68328_clk.shift); - clocksource_register(&m68328_clk); + clocksource_register_hz(&m68328_clk, TICKS_PER_JIFFY*HZ); } /***************************************************************************/ diff --git a/arch/m68k/platform/coldfire/dma_timer.c b/arch/m68k/platform/coldfire/dma_timer.c index a5f5628..235ad57 100644 --- a/arch/m68k/platform/coldfire/dma_timer.c +++ b/arch/m68k/platform/coldfire/dma_timer.c @@ -44,7 +44,6 @@ static struct clocksource clocksource_cf_dt = { .rating = 200, .read = cf_dt_get_cycles, .mask = CLOCKSOURCE_MASK(32), - .shift = 20, .flags = CLOCK_SOURCE_IS_CONTINUOUS, }; @@ -60,9 +59,7 @@ static int __init init_cf_dt_clocksource(void) __raw_writeb(0x00, DTER0); __raw_writel(0x00000000, DTRR0); __raw_writew(DMA_DTMR_CLK_DIV_16 | DMA_DTMR_ENABLE, DTMR0); - clocksource_cf_dt.mult = clocksource_hz2mult(DMA_FREQ, - clocksource_cf_dt.shift); - return clocksource_register(&clocksource_cf_dt); + return clocksource_register_hz(&clocksource_cf_dt, DMA_FREQ); } arch_initcall(init_cf_dt_clocksource); diff --git a/arch/m68k/platform/coldfire/pit.c b/arch/m68k/platform/coldfire/pit.c index c2b9809..02663d2 100644 --- a/arch/m68k/platform/coldfire/pit.c +++ b/arch/m68k/platform/coldfire/pit.c @@ -144,7 +144,6 @@ static struct clocksource pit_clk = { .name = "pit", .rating = 100, .read = pit_read_clk, - .shift = 20, .mask = CLOCKSOURCE_MASK(32), }; @@ -162,8 +161,7 @@ void hw_timer_init(void) setup_irq(MCFINT_VECBASE + MCFINT_PIT1, &pit_irq); - pit_clk.mult = clocksource_hz2mult(FREQ, pit_clk.shift); - clocksource_register(&pit_clk); + clocksource_register_hz(&pit_clk, FREQ); } /***************************************************************************/ diff --git a/arch/m68k/platform/coldfire/sltimers.c b/arch/m68k/platform/coldfire/sltimers.c index 6a85daf..b7f822b 100644 --- a/arch/m68k/platform/coldfire/sltimers.c +++ b/arch/m68k/platform/coldfire/sltimers.c @@ -114,7 +114,6 @@ static struct clocksource mcfslt_clk = { .name = "slt", .rating = 250, .read = mcfslt_read_clk, - .shift = 20, .mask = CLOCKSOURCE_MASK(32), .flags = CLOCK_SOURCE_IS_CONTINUOUS, }; @@ -136,8 +135,7 @@ void hw_timer_init(void) setup_irq(MCF_IRQ_TIMER, &mcfslt_timer_irq); - mcfslt_clk.mult = clocksource_hz2mult(MCF_BUSCLK, mcfslt_clk.shift); - clocksource_register(&mcfslt_clk); + clocksource_register_hz(&mcfslt_clk, MCF_BUSCLK); #ifdef CONFIG_HIGHPROFILE mcfslt_profile_init(); diff --git a/arch/m68k/platform/coldfire/timers.c b/arch/m68k/platform/coldfire/timers.c index 60242f6..0d90da3 100644 --- a/arch/m68k/platform/coldfire/timers.c +++ b/arch/m68k/platform/coldfire/timers.c @@ -88,7 +88,6 @@ static struct clocksource mcftmr_clk = { .name = "tmr", .rating = 250, .read = mcftmr_read_clk, - .shift = 20, .mask = CLOCKSOURCE_MASK(32), .flags = CLOCK_SOURCE_IS_CONTINUOUS, }; @@ -109,8 +108,7 @@ void hw_timer_init(void) __raw_writew(MCFTIMER_TMR_ENORI | MCFTIMER_TMR_CLK16 | MCFTIMER_TMR_RESTART | MCFTIMER_TMR_ENABLE, TA(MCFTIMER_TMR)); - mcftmr_clk.mult = clocksource_hz2mult(FREQ, mcftmr_clk.shift); - clocksource_register(&mcftmr_clk); + clocksource_register_hz(&mcftmr_clk, FREQ); setup_irq(MCF_IRQ_TIMER, &mcftmr_timer_irq); -- 1.7.3.2.146.gca209 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 6/8] clocksource: m86k: Convert to clocksource_register_hz/khz 2011-10-25 18:46 ` [PATCH 6/8] clocksource: m86k: " john stultz @ 2011-10-26 5:10 ` Greg Ungerer 2011-10-26 17:57 ` john stultz 0 siblings, 1 reply; 6+ messages in thread From: Greg Ungerer @ 2011-10-26 5:10 UTC (permalink / raw) To: john stultz Cc: Geert Uytterhoeven, lkml, Greg Ungerer, uClinux development list Hi John, On 26/10/11 04:46, john stultz wrote: > Updated to merge the valid bits of the two m68k patches. > > This converts the m86k clocksources to use clocksource_register_hz/khz > > This is untested, so any assistance in testing would be appreciated! This looks good. I test compiled across all targets these files affect. And run tested on ColdFire/5208 (which uses pit.c). > CC: Geert Uytterhoeven<geert@linux-m68k.org> > CC: Greg Ungerer<gerg@uclinux.org> > Signed-off-by: John Stultz<johnstul@us.ibm.com> Acked-by: Greg Ungerer <gerg@uclinux.org> Do you want me to push these via the m68knommu git tree? Regards Greg > arch/m68k/platform/68328/timers.c | 4 +--- > arch/m68k/platform/coldfire/dma_timer.c | 5 +---- > arch/m68k/platform/coldfire/pit.c | 4 +--- > arch/m68k/platform/coldfire/sltimers.c | 4 +--- > arch/m68k/platform/coldfire/timers.c | 4 +--- > 5 files changed, 5 insertions(+), 16 deletions(-) > > diff --git a/arch/m68k/platform/68328/timers.c b/arch/m68k/platform/68328/timers.c > index 309f725..f267886 100644 > --- a/arch/m68k/platform/68328/timers.c > +++ b/arch/m68k/platform/68328/timers.c > @@ -93,7 +93,6 @@ static struct clocksource m68328_clk = { > .name = "timer", > .rating = 250, > .read = m68328_read_clk, > - .shift = 20, > .mask = CLOCKSOURCE_MASK(32), > .flags = CLOCK_SOURCE_IS_CONTINUOUS, > }; > @@ -115,8 +114,7 @@ void hw_timer_init(void) > > /* Enable timer 1 */ > TCTL |= TCTL_TEN; > - m68328_clk.mult = clocksource_hz2mult(TICKS_PER_JIFFY*HZ, m68328_clk.shift); > - clocksource_register(&m68328_clk); > + clocksource_register_hz(&m68328_clk, TICKS_PER_JIFFY*HZ); > } > > /***************************************************************************/ > diff --git a/arch/m68k/platform/coldfire/dma_timer.c b/arch/m68k/platform/coldfire/dma_timer.c > index a5f5628..235ad57 100644 > --- a/arch/m68k/platform/coldfire/dma_timer.c > +++ b/arch/m68k/platform/coldfire/dma_timer.c > @@ -44,7 +44,6 @@ static struct clocksource clocksource_cf_dt = { > .rating = 200, > .read = cf_dt_get_cycles, > .mask = CLOCKSOURCE_MASK(32), > - .shift = 20, > .flags = CLOCK_SOURCE_IS_CONTINUOUS, > }; > > @@ -60,9 +59,7 @@ static int __init init_cf_dt_clocksource(void) > __raw_writeb(0x00, DTER0); > __raw_writel(0x00000000, DTRR0); > __raw_writew(DMA_DTMR_CLK_DIV_16 | DMA_DTMR_ENABLE, DTMR0); > - clocksource_cf_dt.mult = clocksource_hz2mult(DMA_FREQ, > - clocksource_cf_dt.shift); > - return clocksource_register(&clocksource_cf_dt); > + return clocksource_register_hz(&clocksource_cf_dt, DMA_FREQ); > } > > arch_initcall(init_cf_dt_clocksource); > diff --git a/arch/m68k/platform/coldfire/pit.c b/arch/m68k/platform/coldfire/pit.c > index c2b9809..02663d2 100644 > --- a/arch/m68k/platform/coldfire/pit.c > +++ b/arch/m68k/platform/coldfire/pit.c > @@ -144,7 +144,6 @@ static struct clocksource pit_clk = { > .name = "pit", > .rating = 100, > .read = pit_read_clk, > - .shift = 20, > .mask = CLOCKSOURCE_MASK(32), > }; > > @@ -162,8 +161,7 @@ void hw_timer_init(void) > > setup_irq(MCFINT_VECBASE + MCFINT_PIT1,&pit_irq); > > - pit_clk.mult = clocksource_hz2mult(FREQ, pit_clk.shift); > - clocksource_register(&pit_clk); > + clocksource_register_hz(&pit_clk, FREQ); > } > > /***************************************************************************/ > diff --git a/arch/m68k/platform/coldfire/sltimers.c b/arch/m68k/platform/coldfire/sltimers.c > index 6a85daf..b7f822b 100644 > --- a/arch/m68k/platform/coldfire/sltimers.c > +++ b/arch/m68k/platform/coldfire/sltimers.c > @@ -114,7 +114,6 @@ static struct clocksource mcfslt_clk = { > .name = "slt", > .rating = 250, > .read = mcfslt_read_clk, > - .shift = 20, > .mask = CLOCKSOURCE_MASK(32), > .flags = CLOCK_SOURCE_IS_CONTINUOUS, > }; > @@ -136,8 +135,7 @@ void hw_timer_init(void) > > setup_irq(MCF_IRQ_TIMER,&mcfslt_timer_irq); > > - mcfslt_clk.mult = clocksource_hz2mult(MCF_BUSCLK, mcfslt_clk.shift); > - clocksource_register(&mcfslt_clk); > + clocksource_register_hz(&mcfslt_clk, MCF_BUSCLK); > > #ifdef CONFIG_HIGHPROFILE > mcfslt_profile_init(); > diff --git a/arch/m68k/platform/coldfire/timers.c b/arch/m68k/platform/coldfire/timers.c > index 60242f6..0d90da3 100644 > --- a/arch/m68k/platform/coldfire/timers.c > +++ b/arch/m68k/platform/coldfire/timers.c > @@ -88,7 +88,6 @@ static struct clocksource mcftmr_clk = { > .name = "tmr", > .rating = 250, > .read = mcftmr_read_clk, > - .shift = 20, > .mask = CLOCKSOURCE_MASK(32), > .flags = CLOCK_SOURCE_IS_CONTINUOUS, > }; > @@ -109,8 +108,7 @@ void hw_timer_init(void) > __raw_writew(MCFTIMER_TMR_ENORI | MCFTIMER_TMR_CLK16 | > MCFTIMER_TMR_RESTART | MCFTIMER_TMR_ENABLE, TA(MCFTIMER_TMR)); > > - mcftmr_clk.mult = clocksource_hz2mult(FREQ, mcftmr_clk.shift); > - clocksource_register(&mcftmr_clk); > + clocksource_register_hz(&mcftmr_clk, FREQ); > > setup_irq(MCF_IRQ_TIMER,&mcftmr_timer_irq); > -- ------------------------------------------------------------------------ Greg Ungerer -- Principal Engineer EMAIL: gerg@snapgear.com SnapGear Group, McAfee PHONE: +61 7 3435 2888 8 Gardner Close FAX: +61 7 3217 5323 Milton, QLD, 4064, Australia WEB: http://www.SnapGear.com ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 6/8] clocksource: m86k: Convert to clocksource_register_hz/khz 2011-10-26 5:10 ` Greg Ungerer @ 2011-10-26 17:57 ` john stultz 2011-10-27 4:15 ` Greg Ungerer 0 siblings, 1 reply; 6+ messages in thread From: john stultz @ 2011-10-26 17:57 UTC (permalink / raw) To: Greg Ungerer Cc: Geert Uytterhoeven, lkml, Greg Ungerer, uClinux development list On Wed, 2011-10-26 at 15:10 +1000, Greg Ungerer wrote: > Hi John, > > On 26/10/11 04:46, john stultz wrote: > > Updated to merge the valid bits of the two m68k patches. > > > > This converts the m86k clocksources to use clocksource_register_hz/khz > > > > This is untested, so any assistance in testing would be appreciated! > > This looks good. I test compiled across all targets these files affect. > And run tested on ColdFire/5208 (which uses pit.c). Thanks for testing! I really appreciate it! > > > CC: Geert Uytterhoeven<geert@linux-m68k.org> > > CC: Greg Ungerer<gerg@uclinux.org> > > Signed-off-by: John Stultz<johnstul@us.ibm.com> > > Acked-by: Greg Ungerer <gerg@uclinux.org> > > Do you want me to push these via the m68knommu git tree? I'd prefer it go through either one of the m68k trees. thanks -john ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 6/8] clocksource: m86k: Convert to clocksource_register_hz/khz 2011-10-26 17:57 ` john stultz @ 2011-10-27 4:15 ` Greg Ungerer 0 siblings, 0 replies; 6+ messages in thread From: Greg Ungerer @ 2011-10-27 4:15 UTC (permalink / raw) To: john stultz Cc: Geert Uytterhoeven, lkml, Greg Ungerer, uClinux development list Hi John, On 27/10/11 03:57, john stultz wrote: > On Wed, 2011-10-26 at 15:10 +1000, Greg Ungerer wrote: >> Hi John, >> >> On 26/10/11 04:46, john stultz wrote: >>> Updated to merge the valid bits of the two m68k patches. >>> >>> This converts the m86k clocksources to use clocksource_register_hz/khz >>> >>> This is untested, so any assistance in testing would be appreciated! >> >> This looks good. I test compiled across all targets these files affect. >> And run tested on ColdFire/5208 (which uses pit.c). > > Thanks for testing! I really appreciate it! > >> >>> CC: Geert Uytterhoeven<geert@linux-m68k.org> >>> CC: Greg Ungerer<gerg@uclinux.org> >>> Signed-off-by: John Stultz<johnstul@us.ibm.com> >> >> Acked-by: Greg Ungerer<gerg@uclinux.org> >> >> Do you want me to push these via the m68knommu git tree? > > I'd prefer it go through either one of the m68k trees. Ok, I have put it int the m68knommu git tree (since all the changes apply to non-mmu versions of the m68k). Thanks Greg ------------------------------------------------------------------------ Greg Ungerer -- Principal Engineer EMAIL: gerg@snapgear.com SnapGear Group, McAfee PHONE: +61 7 3435 2888 8 Gardner Close FAX: +61 7 3217 5323 Milton, QLD, 4064, Australia WEB: http://www.SnapGear.com ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-10-27 4:15 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <4ea6fbe8.43310e0a.592c.5416SMTPIN_ADDED@mx.google.com>
2011-10-25 18:35 ` [PATCH 6/8] clocksource: m86knommu: Convert to clocksource_register_hz/khz Geert Uytterhoeven
2011-10-25 18:39 ` john stultz
2011-10-25 18:46 ` [PATCH 6/8] clocksource: m86k: " john stultz
2011-10-26 5:10 ` Greg Ungerer
2011-10-26 17:57 ` john stultz
2011-10-27 4:15 ` Greg Ungerer
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).