* [PATCH] clocksource: vf_pit_timer: fix pit_read_sched_clock
@ 2014-04-15 10:48 Xiubo Li
2014-04-15 12:35 ` Daniel Lezcano
0 siblings, 1 reply; 5+ messages in thread
From: Xiubo Li @ 2014-04-15 10:48 UTC (permalink / raw)
To: daniel.lezcano, tglx, shawn.guo; +Cc: linux-kernel, Xiubo Li
The pit_read_sched_clock mean to getting the current counter value,
and the vf pit timer is a down count timer, this should return
the counter value or the cycles elapsed.
Signed-off-by: Xiubo Li <Li.Xiubo@freescale.com>
---
drivers/clocksource/vf_pit_timer.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/clocksource/vf_pit_timer.c b/drivers/clocksource/vf_pit_timer.c
index 02821b0..a918bc4 100644
--- a/drivers/clocksource/vf_pit_timer.c
+++ b/drivers/clocksource/vf_pit_timer.c
@@ -54,7 +54,7 @@ static inline void pit_irq_acknowledge(void)
static u64 pit_read_sched_clock(void)
{
- return __raw_readl(clksrc_base + PITCVAL);
+ return ~__raw_readl(clksrc_base + PITCVAL);
}
static int __init pit_clocksource_init(unsigned long rate)
--
1.8.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] clocksource: vf_pit_timer: fix pit_read_sched_clock
2014-04-15 10:48 [PATCH] clocksource: vf_pit_timer: fix pit_read_sched_clock Xiubo Li
@ 2014-04-15 12:35 ` Daniel Lezcano
2014-04-15 21:53 ` Thomas Gleixner
2014-04-16 0:33 ` Li.Xiubo
0 siblings, 2 replies; 5+ messages in thread
From: Daniel Lezcano @ 2014-04-15 12:35 UTC (permalink / raw)
To: Xiubo Li, tglx, shawn.guo; +Cc: linux-kernel
On 04/15/2014 12:48 PM, Xiubo Li wrote:
> The pit_read_sched_clock mean to getting the current counter value,
> and the vf pit timer is a down count timer, this should return
> the counter value or the cycles elapsed.
>
> Signed-off-by: Xiubo Li <Li.Xiubo@freescale.com>
> ---
> drivers/clocksource/vf_pit_timer.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/clocksource/vf_pit_timer.c b/drivers/clocksource/vf_pit_timer.c
> index 02821b0..a918bc4 100644
> --- a/drivers/clocksource/vf_pit_timer.c
> +++ b/drivers/clocksource/vf_pit_timer.c
> @@ -54,7 +54,7 @@ static inline void pit_irq_acknowledge(void)
>
> static u64 pit_read_sched_clock(void)
> {
> - return __raw_readl(clksrc_base + PITCVAL);
> + return ~__raw_readl(clksrc_base + PITCVAL);
> }
>
> static int __init pit_clocksource_init(unsigned long rate)
>
Hi Xiubo,
thanks for the patch but this has already been fixed [1] and is
available in the timers/urgent branch.
Thanks
-- Daniel
[1]
https://git.kernel.org/cgit/linux/kernel/git/tip/tip.git/commit/?h=timers/urgent&id=224aa3ed45c8735ae02bb2ecca002409fa6aa772
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] clocksource: vf_pit_timer: fix pit_read_sched_clock
2014-04-15 12:35 ` Daniel Lezcano
@ 2014-04-15 21:53 ` Thomas Gleixner
2014-04-16 0:51 ` Li.Xiubo
2014-04-16 0:33 ` Li.Xiubo
1 sibling, 1 reply; 5+ messages in thread
From: Thomas Gleixner @ 2014-04-15 21:53 UTC (permalink / raw)
To: Daniel Lezcano; +Cc: Xiubo Li, shawn.guo, linux-kernel
On Tue, 15 Apr 2014, Daniel Lezcano wrote:
> On 04/15/2014 12:48 PM, Xiubo Li wrote:
> > The pit_read_sched_clock mean to getting the current counter value,
> > and the vf pit timer is a down count timer, this should return
> > the counter value or the cycles elapsed.
> >
> > Signed-off-by: Xiubo Li <Li.Xiubo@freescale.com>
> > ---
> > drivers/clocksource/vf_pit_timer.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/clocksource/vf_pit_timer.c
> > b/drivers/clocksource/vf_pit_timer.c
> > index 02821b0..a918bc4 100644
> > --- a/drivers/clocksource/vf_pit_timer.c
> > +++ b/drivers/clocksource/vf_pit_timer.c
> > @@ -54,7 +54,7 @@ static inline void pit_irq_acknowledge(void)
> >
> > static u64 pit_read_sched_clock(void)
> > {
> > - return __raw_readl(clksrc_base + PITCVAL);
> > + return ~__raw_readl(clksrc_base + PITCVAL);
> > }
> >
> > static int __init pit_clocksource_init(unsigned long rate)
> >
>
> Hi Xiubo,
>
> thanks for the patch but this has already been fixed [1] and is available in
> the timers/urgent branch.
It's in Linus tree already and tagged for stable.
Thanks,
tglx
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] clocksource: vf_pit_timer: fix pit_read_sched_clock
2014-04-15 12:35 ` Daniel Lezcano
2014-04-15 21:53 ` Thomas Gleixner
@ 2014-04-16 0:33 ` Li.Xiubo
1 sibling, 0 replies; 5+ messages in thread
From: Li.Xiubo @ 2014-04-16 0:33 UTC (permalink / raw)
To: Daniel Lezcano, tglx, shawn.guo; +Cc: linux-kernel
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 858 bytes --]
> > diff --git a/drivers/clocksource/vf_pit_timer.c
> b/drivers/clocksource/vf_pit_timer.c
> > index 02821b0..a918bc4 100644
> > --- a/drivers/clocksource/vf_pit_timer.c
> > +++ b/drivers/clocksource/vf_pit_timer.c
> > @@ -54,7 +54,7 @@ static inline void pit_irq_acknowledge(void)
> >
> > static u64 pit_read_sched_clock(void)
> > {
> > - return __raw_readl(clksrc_base + PITCVAL);
> > + return ~__raw_readl(clksrc_base + PITCVAL);
> > }
> >
> > static int __init pit_clocksource_init(unsigned long rate)
> >
>
> Hi Xiubo,
>
> thanks for the patch but this has already been fixed [1] and is
> available in the timers/urgent branch.
>
Good news and I will pull it.
Thanks,
BRs
Xiubo
ÿôèº{.nÇ+·®+%Ëÿ±éݶ\x17¥wÿº{.nÇ+·¥{±þG«éÿ{ayº\x1dÊÚë,j\a¢f£¢·hïêÿêçz_è®\x03(éÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?¨èÚ&£ø§~á¶iOæ¬z·vØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?I¥
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] clocksource: vf_pit_timer: fix pit_read_sched_clock
2014-04-15 21:53 ` Thomas Gleixner
@ 2014-04-16 0:51 ` Li.Xiubo
0 siblings, 0 replies; 5+ messages in thread
From: Li.Xiubo @ 2014-04-16 0:51 UTC (permalink / raw)
To: Thomas Gleixner, Daniel Lezcano; +Cc: shawn.guo, linux-kernel
> >
> > thanks for the patch but this has already been fixed [1] and is available in
> > the timers/urgent branch.
>
> It's in Linus tree already and tagged for stable.
>
Yes, I have found it, I was developing this in the old tree...
:)
Thanks,
BRs
Xiubo
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-04-16 0:52 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-04-15 10:48 [PATCH] clocksource: vf_pit_timer: fix pit_read_sched_clock Xiubo Li
2014-04-15 12:35 ` Daniel Lezcano
2014-04-15 21:53 ` Thomas Gleixner
2014-04-16 0:51 ` Li.Xiubo
2014-04-16 0:33 ` Li.Xiubo
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.