MIME-Version: 1.0
In-Reply-To: <CAD6G_RTi4MJ74yWsv3Zdg68mdSHBzszfkLmpz=rGdRVfOOQ2vA@mail.gmail.com>
References: <1502560442-17715-1-git-send-email-jteki@openedev.com>
 <CAGb2v66Scd69iBNxhvYbkqjt7K_5m+S4nYG633O3wYAW_3CyLQ@mail.gmail.com>
 <CAD6G_RRMHMrm1wPGJCxQgmOuwhEqVK_zTZF1Fsv9az4tZ1uvBg@mail.gmail.com>
 <CAGb2v64KJOuS=XfWu=tHyyYbTVjLJKVoTL2cQDyLby=CmnmLWw@mail.gmail.com> <CAD6G_RTi4MJ74yWsv3Zdg68mdSHBzszfkLmpz=rGdRVfOOQ2vA@mail.gmail.com>
From: Chen-Yu Tsai <wens@csie.org>
Date: Mon, 14 Aug 2017 17:13:52 +0800
Message-ID: <CAGb2v663ANZJfhL1OCVVF24fvZtrCS9x7WY=EWEEZpfDmxQkTA@mail.gmail.com>
Subject: Re: [linux-sunxi] [PATCH v2] arm64: allwinner: a64: Add A64-OLinuXino
 initial support
To: Jagan Teki <jagannadh.teki@gmail.com>
Cc: Chen-Yu Tsai <wens@csie.org>,
        Maxime Ripard <maxime.ripard@free-electrons.com>,
        Icenowy Zheng <icenowy@aosc.io>, Rob Herring <robh+dt@kernel.org>,
        Mark Rutland <mark.rutland@arm.com>,
        Catalin Marinas <catalin.marinas@arm.com>,
        Will Deacon <will.deacon@arm.com>,
        Michael Trimarchi <michael@amarulasolutions.com>,
        linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
        devicetree <devicetree@vger.kernel.org>,
        linux-kernel <linux-kernel@vger.kernel.org>,
        linux-sunxi <linux-sunxi@googlegroups.com>,
        Support - OLIMEX Ltd <support@olimex.com>,
        Jagan Teki <jagan@amarulasolutions.com>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1898
Lines: 54

On Mon, Aug 14, 2017 at 5:09 PM, Jagan Teki <jagannadh.teki@gmail.com> wrote:
> On Mon, Aug 14, 2017 at 1:26 PM, Chen-Yu Tsai <wens@csie.org> wrote:
>> On Mon, Aug 14, 2017 at 3:34 PM, Jagan Teki <jagannadh.teki@gmail.com> wrote:
>>> On Mon, Aug 14, 2017 at 12:38 PM, Chen-Yu Tsai <wens@csie.org> wrote:
>>>> Hi,
>>>>
>>>> On Sun, Aug 13, 2017 at 1:54 AM, Jagan Teki <jagannadh.teki@gmail.com> wrote:

[...]

>>>>> +&reg_dcdc1 {
>>>>> +       regulator-always-on;
>>>>> +       regulator-min-microvolt = <3300000>;
>>>>> +       regulator-max-microvolt = <3300000>;
>>>>> +       regulator-name = "vcc-3v3";
>>>>> +};
>>>>> +
>>>>> +&reg_dcdc2 {
>>>>> +       regulator-always-on;
>>>>> +       regulator-min-microvolt = <1100000>;
>>>>> +       regulator-max-microvolt = <1100000>;
>>>>
>>>> The SoC has a wider range than a fixed voltage for the CPUs.
>>>
>>> So, max 1.1v can't enough CPUs? thought similar behavior with pine64?
>>
>> The recommended operating conditions says 1.04 ~ 1.3 V for CPUx,
>> while the typical voltage is 1.1V. The regulator constraints should
>> match the datasheet, especially for power rails that are involved in
>> DVFS.
>
> power rails from datasheet[1] shows 0.5v ~ 1.3v am I not looking
> updated details?

The constraints are there to prevent users from setting a voltage
too high or too low for the connected devices.

You should be looking at the recommended operating conditions
for the A64. The regulator constraints should match the constraints
of the consumer, not the provider. If there are multiple consumers,
it must match the narrowest range supported by _all_ consumers, i.e.
the intersection of all consumer's operating ranges.

ChenYu

>
> [1] http://files.pine64.org/doc/datasheet/pine64/AXP803_Datasheet_V1.0.pdf
>
> thanks!
> --
> Jagan Teki
> Free Software Engineer | www.openedev.com
> U-Boot, Linux | Upstream Maintainer
> Hyderabad, India.