On Thu, Jun 24, 2010 at 08:51:27PM +0200, Marek Vasut wrote:
> Dne St 23. Ĩervna 2010 19:49:59 Robert Jarzmik napsal(a):
> > Marek Vasut <marek.va...@gmail.com> writes:
> > > The MCxx values must be based off memory clock, not CPU core clock.
> > > 
> > > This also fixes the bug where on some machines the LCD went crazy while
> > > using PCMCIA.
> > 
> > That looks correct to me.
> > 
> > Reviewed-by: Robert Jarzmik <robert.jarz...@free.fr>
> > 
> > Cheers.
> > 
> > --
> > Robert
> 
> Actually lemme rethink this ... it might still be incorrect.

As Nicolas was the one who originally wrote that code, while he was
working for Montavista and interacting with Intel, I'd be surprised
if he didn't try finding out what the right answer is to that fixme.

I suspect as the fixme remained that there was no clear answer.

Nicolas, can you shed any light on this:

static int
pxa2xx_pcmcia_frequency_change(struct soc_pcmcia_socket *skt,
                               unsigned long val,
                               struct cpufreq_freqs *freqs)
{
#warning "it's not clear if this is right since the core CPU (N) clock has no 
effect on the memory (L) clock"

_______________________________________________
Zaurus-devel mailing list
Zaurus-devel@lists.linuxtogo.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/zaurus-devel

Reply via email to