public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] arm: pxa: config option for PXA270 turbo mode
Date: Tue, 21 May 2013 13:22:07 +0200	[thread overview]
Message-ID: <201305211322.08127.marex@denx.de> (raw)
In-Reply-To: <1369134856.20319.15.camel@host5.omatika.ru>

Dear Sergey Yanovich,

> Dear Marek Vasut,
> 
> On Tue, 2013-05-21 at 12:55 +0200, Marek Vasut wrote:
> > > Difference -- approx. 2.5 times faster system.
> > > 
> > > Who uses it -- ex. LP-8x4x board, support for which I am trying to
> > > merge in a separate patch. Any PXA27X board can use it.
> > 
> > Why don't you enable it globally then ?
> 
> It increases power consumption proportionally. Not everyone would agree
> with this. Especially those with portable devices may rightfully object.
> 
> It is up to you to enable it globally. I could alter the patch to switch
> turbo mode on by default and provide an option to disable it.

Actually, I wonder if Linux's cpufreq still does depend on bootloader speed 
settings. Especially the turbo bit. Can you please check that? It'd be 
interesting to know ...

I'd say enable it by default ... I probably have all the PXA devices supported 
in U-Boot and I'm quite sure none of them will mind ;-)

Best regards,
Marek Vasut

  reply	other threads:[~2013-05-21 11:22 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-20 21:44 [U-Boot] [PATCH] arm: pxa: config option for PXA270 turbo mode Sergey Yanovich
2013-05-21 10:35 ` Marek Vasut
2013-05-21 10:51   ` Sergey Yanovich
2013-05-21 10:55     ` Marek Vasut
2013-05-21 11:14       ` Sergey Yanovich
2013-05-21 11:22         ` Marek Vasut [this message]
2013-05-21 11:22           ` Marek Vasut
2013-05-21 16:23             ` Sergey Yanovich
2013-05-21 19:25               ` Marek Vasut
2013-05-21 19:45                 ` Sergey Yanovich
2013-05-21 19:49                 ` Sergey Yanovich
2013-05-21 19:58                   ` Marek Vasut
2013-05-21 20:00                     ` Sergey Yanovich
2013-05-21 20:09                       ` Marek Vasut
2013-05-21 11:42           ` Sergey Yanovich

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201305211322.08127.marex@denx.de \
    --to=marex@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox