[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20110314113949.GM31153@sortiz-mobl>
Date: Mon, 14 Mar 2011 12:39:50 +0100
From: Samuel Ortiz <sameo@...ux.intel.com>
To: MyungJoo Ham <myungjoo.ham@...sung.com>
Cc: linux-kernel@...r.kernel.org, rtc-linux@...glegroups.com,
Mark Brown <broonie@...nsource.wolfsonmicro.com>,
Liam Girdwood <lrg@...mlogic.co.uk>,
Alessandro Zummo <a.zummo@...ertech.it>,
kyungmin.park@...sung.com, myungjoo.ham@...il.com
Subject: Re: [PATCH v5 2/4] MAX8997/8966 PMIC Regulator Driver Initial
Release
Hi MyungJoo,
On Fri, Mar 11, 2011 at 11:34:44AM +0900, MyungJoo Ham wrote:
> This patch supports PMIC/Regulator part of MAX8997/MAX8966 MFD.
> In this initial release, selecting voltages or current-limit
> and switching on/off the regulators are supported.
>
> Controlling voltages for DVS with GPIOs is not implemented fully
> and requires more considerations: it controls multiple bucks (selection
> of 1, 2, and 5) at the same time with SET1~3 gpios. Thus, when DVS-GPIO
> is activated, we lose the ability to control the voltage of a single
> buck regulator independently; i.e., contolling a buck affects other two
> bucks. Therefore, using the conventional regulator framework directly
> might be problematic. However, in this driver, we try to choose
> a setting without such side effect of affecting other regulators and
> then try to choose a setting with the minimum side effect (the sum of
> voltage changes in other regulators).
>
> On the other hand, controlling all the three bucks simultenously based
> on the voltage set table may help build cpufreq and similar system
> more robust; i.e., all the three voltages are consistent every time
> without glitches during transition.
Patch applied, with Mark's ACK.
Cheers,
Samuel.
--
Intel Open Source Technology Centre
http://oss.intel.com/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists