On Mon, Oct 12, 2015 at 4:31 PM, Maxime Ripard
<[email protected]> wrote:
> Hi,
>
> On Sat, Oct 10, 2015 at 10:48:56PM +0800, Chen-Yu Tsai wrote:
>> The VDD-CPU and VDD-GPU regulators were incorrectly swapped.
>>
>> Fixes: bab03561224ba ("ARM: dts: sun6i: hummingbird: Add AXP221 regulator
>>        nodes")
>>
>> Signed-off-by: Chen-Yu Tsai <[email protected]>
>> ---
>>  arch/arm/boot/dts/sun6i-a31-hummingbird.dts | 8 ++++----
>>  1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/sun6i-a31-hummingbird.dts 
>> b/arch/arm/boot/dts/sun6i-a31-hummingbird.dts
>> index f9d9fc07bc63..e25d4924dbe7 100644
>> --- a/arch/arm/boot/dts/sun6i-a31-hummingbird.dts
>> +++ b/arch/arm/boot/dts/sun6i-a31-hummingbird.dts
>> @@ -191,18 +191,18 @@
>>                               regulator-name = "vcc-3v0";
>>                       };
>>
>> -                     vdd_cpu: dcdc2 {
>> +                     vdd_gpu: dcdc2 {
>>                               regulator-always-on;
>>                               regulator-min-microvolt = <700000>;
>>                               regulator-max-microvolt = <1320000>;
>> -                             regulator-name = "vdd-cpu";
>> +                             regulator-name = "vdd-gpu";
>>                       };
>
> Do we need to leave the GPU regulator on ?

No. But that is addressed in patch 3, which converts to using axp22x.dtsi.

I wanted to keep this fix as simple as possible.

ChenYu

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to