All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kevin Hilman <khilman@ti.com>
To: t-kristo@ti.com
Cc: linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCHv10 3/4] omap3: add common twl configurations for vdd1 and vdd2
Date: Wed, 22 Feb 2012 11:16:42 -0800	[thread overview]
Message-ID: <8739a21xwl.fsf@ti.com> (raw)
In-Reply-To: <1329907675.4102.488.camel@sokoban> (Tero Kristo's message of "Wed, 22 Feb 2012 12:47:55 +0200")

Tero Kristo <t-kristo@ti.com> writes:

> On Tue, 2012-02-21 at 16:00 -0800, Kevin Hilman wrote:
>> Tero Kristo <t-kristo@ti.com> writes:
>> 
>> > VDD1 and VDD2 are the core voltage regulators on OMAP3. VDD1 is used
>> > to control MPU/IVA voltage, and VDD2 is used for CORE. These regulators
>> > are needed by DVFS.
>> >
>> > Voltage ranges for VDD1 and VDD2 are taken from twl4030/twl5030 data manuals:
>> > - SWCS019L : TWL4030 ES3.1 Data Manual rev L
>> > - SWCS030E : TWL5030 ES1.2 Data Manual rev E
>> >
>> > Signed-off-by: Tero Kristo <t-kristo@ti.com>
>> 
>> Do you have a similar patch for OMAP4 support?
>
> It looks like OMAP4 support requires some changes to the twl-regulator /
> twl-core in addition to the twl-common.c. I have these available (just
> created them), should I post these out? 

Yes please.

> Attached here for reference in case you need to test something
> quickly.

Thanks, it's easier for me to validate actual voltage scaling on OMAP4
since I have a setup to easily measure voltage rails.

Thanks for these, I verified they work using my v2 CPUfreq driver on
OMAP4430.

Kevin


WARNING: multiple messages have this Message-ID (diff)
From: khilman@ti.com (Kevin Hilman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv10 3/4] omap3: add common twl configurations for vdd1 and vdd2
Date: Wed, 22 Feb 2012 11:16:42 -0800	[thread overview]
Message-ID: <8739a21xwl.fsf@ti.com> (raw)
In-Reply-To: <1329907675.4102.488.camel@sokoban> (Tero Kristo's message of "Wed, 22 Feb 2012 12:47:55 +0200")

Tero Kristo <t-kristo@ti.com> writes:

> On Tue, 2012-02-21 at 16:00 -0800, Kevin Hilman wrote:
>> Tero Kristo <t-kristo@ti.com> writes:
>> 
>> > VDD1 and VDD2 are the core voltage regulators on OMAP3. VDD1 is used
>> > to control MPU/IVA voltage, and VDD2 is used for CORE. These regulators
>> > are needed by DVFS.
>> >
>> > Voltage ranges for VDD1 and VDD2 are taken from twl4030/twl5030 data manuals:
>> > - SWCS019L : TWL4030 ES3.1 Data Manual rev L
>> > - SWCS030E : TWL5030 ES1.2 Data Manual rev E
>> >
>> > Signed-off-by: Tero Kristo <t-kristo@ti.com>
>> 
>> Do you have a similar patch for OMAP4 support?
>
> It looks like OMAP4 support requires some changes to the twl-regulator /
> twl-core in addition to the twl-common.c. I have these available (just
> created them), should I post these out? 

Yes please.

> Attached here for reference in case you need to test something
> quickly.

Thanks, it's easier for me to validate actual voltage scaling on OMAP4
since I have a setup to easily measure voltage rails.

Thanks for these, I verified they work using my v2 CPUfreq driver on
OMAP4430.

Kevin

  reply	other threads:[~2012-02-22 19:16 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-20 10:26 [PATCHv10 0/4] omap smps regulator support Tero Kristo
2012-02-20 10:26 ` Tero Kristo
2012-02-20 10:26 ` [PATCHv10 1/4] TEMP: OMAP3: beagle rev-c4: enable OPP6 Tero Kristo
2012-02-20 10:26   ` Tero Kristo
2012-02-20 10:26 ` [PATCHv10 2/4] omap3: voltage: fix channel configuration Tero Kristo
2012-02-20 10:26   ` Tero Kristo
2012-02-20 10:26 ` [PATCHv10 3/4] omap3: add common twl configurations for vdd1 and vdd2 Tero Kristo
2012-02-20 10:26   ` Tero Kristo
2012-02-22  0:00   ` Kevin Hilman
2012-02-22  0:00     ` Kevin Hilman
2012-02-22 10:47     ` Tero Kristo
2012-02-22 10:47       ` Tero Kristo
2012-02-22 19:16       ` Kevin Hilman [this message]
2012-02-22 19:16         ` Kevin Hilman
2012-02-20 10:26 ` [PATCHv10 4/4] omap3: twl: add external controllers for core voltage regulators Tero Kristo
2012-02-20 10:26   ` Tero Kristo

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=8739a21xwl.fsf@ti.com \
    --to=khilman@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=t-kristo@ti.com \
    /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 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.