Re: [PATCH] [POWERPC] 8xx: timebase frequency should not depend on bus-frequency

2008-02-22 Thread Bryan O'Donoghue
On Fri, 22 Feb 2008 02:41:36 +0300 Anton Vorontsov <[EMAIL PROTECTED]> wrote: > On Thu, Feb 21, 2008 at 03:20:10PM -0600, Scott Wood wrote: > > Anton Vorontsov wrote: > > > On Thu, Feb 21, 2008 at 02:06:58PM -0600, Scott Wood wrote: > > >> Current u-boots don't support device trees at all on 8xx.

Re: [PATCH] [POWERPC] 8xx: timebase frequency should not depend on bus-frequency

2008-02-21 Thread Anton Vorontsov
On Thu, Feb 21, 2008 at 03:20:10PM -0600, Scott Wood wrote: > Anton Vorontsov wrote: > > On Thu, Feb 21, 2008 at 02:06:58PM -0600, Scott Wood wrote: > >> Current u-boots don't support device trees at all on 8xx. > > > > Yes, vanilla u-boots. I assume many of us use some u-boot hacks to > > actuall

Re: [PATCH] [POWERPC] 8xx: timebase frequency should not depend on bus-frequency

2008-02-21 Thread Scott Wood
Anton Vorontsov wrote: > On Thu, Feb 21, 2008 at 02:06:58PM -0600, Scott Wood wrote: >> Current u-boots don't support device trees at all on 8xx. > > Yes, vanilla u-boots. I assume many of us use some u-boot hacks to > actually boot with the device tree (no, not cuboots)... ;-) Fine, but don't ex

Re: [PATCH] [POWERPC] 8xx: timebase frequency should not depend on bus-frequency

2008-02-21 Thread Anton Vorontsov
On Thu, Feb 21, 2008 at 02:06:58PM -0600, Scott Wood wrote: [...] > >>>+++ b/arch/powerpc/platforms/8xx/m8xx_setup.c > >>>@@ -111,17 +111,12 @@ void __init mpc8xx_calibrate_decr(void) > >>> > >>> /* Processor frequency is MHz. > >>>*/ > >>>- ppc_tb_freq = 5000; > >>>- if (!get_freq("bus

Re: [PATCH] [POWERPC] 8xx: timebase frequency should not depend on bus-frequency

2008-02-21 Thread Scott Wood
Anton Vorontsov wrote: > On Thu, Feb 21, 2008 at 01:48:23PM -0600, Scott Wood wrote: >> Anton Vorontsov wrote: >>> diff --git a/arch/powerpc/platforms/8xx/m8xx_setup.c >>> b/arch/powerpc/platforms/8xx/m8xx_setup.c >>> index 184f998..0d9f75c 100644 >>> --- a/arch/powerpc/platforms/8xx/m8xx_setup.c

Re: [PATCH] [POWERPC] 8xx: timebase frequency should not depend on bus-frequency

2008-02-21 Thread Anton Vorontsov
On Thu, Feb 21, 2008 at 01:48:23PM -0600, Scott Wood wrote: > Anton Vorontsov wrote: > >diff --git a/arch/powerpc/platforms/8xx/m8xx_setup.c > >b/arch/powerpc/platforms/8xx/m8xx_setup.c > >index 184f998..0d9f75c 100644 > >--- a/arch/powerpc/platforms/8xx/m8xx_setup.c > >+++ b/arch/powerpc/platform

Re: [PATCH] [POWERPC] 8xx: timebase frequency should not depend on bus-frequency

2008-02-21 Thread Scott Wood
Anton Vorontsov wrote: > diff --git a/arch/powerpc/platforms/8xx/m8xx_setup.c > b/arch/powerpc/platforms/8xx/m8xx_setup.c > index 184f998..0d9f75c 100644 > --- a/arch/powerpc/platforms/8xx/m8xx_setup.c > +++ b/arch/powerpc/platforms/8xx/m8xx_setup.c > @@ -111,17 +111,12 @@ void __init mpc8xx_calib

[PATCH] [POWERPC] 8xx: timebase frequency should not depend on bus-frequency

2008-02-21 Thread Anton Vorontsov
m8xx_setup.c says: /* Force all 8xx processors to use divide by 16 processor clock. */ And at the same time it is using bus-frequency for calculating timebase. It is okay for most setups because bus-frequency is equal to clock-frequency. The problem emerges when cpu frequency is > 66MHz, quoti