Re: dw_apb_timer_of.c: remove parts that were picoxcell-specific

2013-05-10 Thread Pavel Machek
On Tue 2013-05-07 17:30:52, John Stultz wrote: > On 05/07/2013 01:11 PM, Pavel Machek wrote: > >It seems we made a mistake when creating dw_apb_timer_of.c: > >picoxcell sched_clock had parts that were not related to > >dw_apb_timer, yet we moved them to dw_apb_timer_of, and tried to > >use them on

Re: dw_apb_timer_of.c: remove parts that were picoxcell-specific

2013-05-07 Thread John Stultz
On 05/07/2013 01:11 PM, Pavel Machek wrote: It seems we made a mistake when creating dw_apb_timer_of.c: picoxcell sched_clock had parts that were not related to dw_apb_timer, yet we moved them to dw_apb_timer_of, and tried to use them on socfpga. This results in system where user/system time is

Re: dw_apb_timer_of.c: remove parts that were picoxcell-specific

2013-05-07 Thread Pavel Machek
Hi! > >>>So I guess it should go through the timekeeping tree...? > >>I would prefer that, but we can also take it through arm-soc, it doesn't > >>really matter. > >I see no response from timekeeping people... so if you could take it, > >that would be great. > > I don't think the original patch e

Re: dw_apb_timer_of.c: remove parts that were picoxcell-specific

2013-05-07 Thread Pavel Machek
Hi! > >>>So I guess it should go through the timekeeping tree...? > >>I would prefer that, but we can also take it through arm-soc, it doesn't > >>really matter. > >I see no response from timekeeping people... so if you could take it, > >that would be great. > > I don't think the original patch e

Re: dw_apb_timer_of.c: remove parts that were picoxcell-specific

2013-05-07 Thread John Stultz
On 05/07/2013 06:57 AM, Pavel Machek wrote: So I guess it should go through the timekeeping tree...? I would prefer that, but we can also take it through arm-soc, it doesn't really matter. I see no response from timekeeping people... so if you could take it, that would be great. I don't think

Re: dw_apb_timer_of.c: remove parts that were picoxcell-specific

2013-05-07 Thread Pavel Machek
On Mon 2013-05-06 23:24:16, Arnd Bergmann wrote: > On Monday 06 May 2013, Pavel Machek wrote: > > On Mon 2013-05-06 15:45:22, Arnd Bergmann wrote: > > > On Monday 06 May 2013, Pavel Machek wrote: > > > > > > > > Hi! > > > > > > > > Ping? Previous version was tested by Dinh, and this one is based

Re: dw_apb_timer_of.c: remove parts that were picoxcell-specific

2013-05-07 Thread Jamie Iles
Hi Pavel, Apologies for the delay, but this looks good to me. On Fri, Apr 26, 2013 at 11:43:14PM +0200, Pavel Machek wrote: > It seems we made a mistake when creating dw_apb_timer_of.c: > picoxcell sched_clock had parts that were not related to > dw_apb_timer, yet we moved them to dw_apb_timer_of

Re: dw_apb_timer_of.c: remove parts that were picoxcell-specific

2013-05-06 Thread Arnd Bergmann
On Monday 06 May 2013, Pavel Machek wrote: > On Mon 2013-05-06 15:45:22, Arnd Bergmann wrote: > > On Monday 06 May 2013, Pavel Machek wrote: > > > > > > Hi! > > > > > > Ping? Previous version was tested by Dinh, and this one is based on > > > Jamie's updates, so I assume it is acceptable for him,

Re: dw_apb_timer_of.c: remove parts that were picoxcell-specific

2013-05-06 Thread Pavel Machek
On Mon 2013-05-06 15:45:22, Arnd Bergmann wrote: > On Monday 06 May 2013, Pavel Machek wrote: > > > > Hi! > > > > Ping? Previous version was tested by Dinh, and this one is based on > > Jamie's updates, so I assume it is acceptable for him, too. > > > > It brings two platforms closer together, a

Re: dw_apb_timer_of.c: remove parts that were picoxcell-specific

2013-05-06 Thread Arnd Bergmann
On Monday 06 May 2013, Pavel Machek wrote: > > Hi! > > Ping? Previous version was tested by Dinh, and this one is based on > Jamie's updates, so I assume it is acceptable for him, too. > > It brings two platforms closer together, and makes time actually work > on socfpga. It would be good to get

Re: dw_apb_timer_of.c: remove parts that were picoxcell-specific

2013-05-06 Thread Pavel Machek
Hi! Ping? Previous version was tested by Dinh, and this one is based on Jamie's updates, so I assume it is acceptable for him, too. It brings two platforms closer together, and makes time actually work on socfpga. It would be good to get it applied. Thanks,

Re: dw_apb_timer_of.c: remove parts that were picoxcell-specific

2013-04-26 Thread Pavel Machek
It seems we made a mistake when creating dw_apb_timer_of.c: picoxcell sched_clock had parts that were not related to dw_apb_timer, yet we moved them to dw_apb_timer_of, and tried to use them on socfpga. This results in system where user/system time is not measured properly, as demonstrated by

Re: dw_apb_timer_of.c: remove parts that were picoxcell-specific

2013-04-26 Thread Pavel Machek
Hi! > On Fri, Apr 26, 2013 at 02:14:34PM +0200, Pavel Machek wrote: > > diff --git a/arch/arm/mach-picoxcell/common.c > > b/arch/arm/mach-picoxcell/common.c > > index 70b441a..22759f5 100644 > > --- a/arch/arm/mach-picoxcell/common.c > > +++ b/arch/arm/mach-picoxcell/common.c > > @@ -84,11 +84,39

Re: dw_apb_timer_of.c: remove parts that were picoxcell-specific

2013-04-26 Thread Jamie Iles
Hi Pavel, On Fri, Apr 26, 2013 at 02:14:34PM +0200, Pavel Machek wrote: > diff --git a/arch/arm/mach-picoxcell/common.c > b/arch/arm/mach-picoxcell/common.c > index 70b441a..22759f5 100644 > --- a/arch/arm/mach-picoxcell/common.c > +++ b/arch/arm/mach-picoxcell/common.c > @@ -84,11 +84,39 @@ stat

dw_apb_timer_of.c: remove parts that were picoxcell-specific

2013-04-26 Thread Pavel Machek
It seems we made a mistake when creating dw_apb_timer_of.c: picoxcell sched_clock parts were not related to dw_apb_timer, yet we moved them to dw_apb_timer_of, and tried to use them on socfpga. This results in system where user/system time is not measured properly, as demonstrated by