Re: [PATCH] ARM: tegra: TN7: relax some regulators

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 06/20/2014 02:56 AM, Mark Brown wrote:
* PGP Signed by an unknown key

On Thu, Jun 19, 2014 at 09:59:04AM -0600, Stephen Warren wrote:
On 06/19/2014 01:49 AM, Alexandre Courbot wrote:

Remove the regulator-always-on property from some regulators that do not
need it. On recent kernels fixed regulators which supply is always on
fail registration.

That sounds like a bug in the regulator core, which should be fixed there.

Please actually describe the problem you believe you are seeing - I've
seen no reports and I can't tell anything from what you've described,
nor can I see any obvious way that a regulator being fixed would have
any effect on its supply.

Here is some more information about what happens.

We have a fixed regulator defined as follows:

vdd_lcd: regulator@2 {
        compatible = "regulator-fixed";
        reg = <2>;
        regulator-name = "VD_LCD_1V8";
        regulator-min-microvolt = <1800000>;
        regulator-max-microvolt = <1800000>;
        gpio = <&palmas_gpio 4 GPIO_ACTIVE_HIGH>;
        enable-active-high;
        vin-supply = <&vdd_1v8>;
        regulator-boot-on;
};

Its vin-supply is part of the palmas device:

vdd_1v8: smps8 {
        regulator-name = "vs-pmu-1v8";
        regulator-min-microvolt = <1800000>;
        regulator-max-microvolt = <1800000>;
        regulator-always-on;
        regulator-boot-on;
};

When vdd_lcd is registered, set_supply() is called, which creates a new regulator for vdd_1v8. In create_regulator(), _regulator_can_change_status() returns false (as it should since the regulator is always_on) and _regulator_is_enabled() *also* returns false, so as a result regulator->always_on remains false for vdd_1v8.

Later in regulator_register(), we try to enable the supply. Since regulator->always_on is false, _regulator_enable() is called on vdd_1v8, and the pair _regulator_is_enabled() / _regulator_can_change_status() is called again with the same result, which causes _regulator_enable() to return -EPERM. This prevents vdd_lcd from being registered.

So I can see three questions here:

1) Why does _regulator_enable() on vdd_1v8 return 0 while everything suggests that it is enabled (this regulator powers lot of devices, like eMMC, which are working fine). This may be an issue with the palmas driver.

2) When an always-on regulator that is not yet enabled is registered, shouldn't it be switched on by the regulator framework?

3) When a boot-on regulator is registered and _regulator_is_enabled() returns contradictory information, what should be done?

Note that whether the regulator-boot-on property is present or not does not change anything.

I tried to find a recent patch that could have introduced a change of behavior, but could not find anything so far. Bisecting is made harder by the fact this happens on a newly-introduced board which requires a bunch of patches of its own, but it we need more information I can try to do it anyway.

Thanks,
Alex.

--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux