Re: Palmas regulator broken (was Re: [PATCH] ARM: tegra: TN7: relax some regulators)

2014-06-20 Thread Nishanth Menon
On 06/20/2014 09:14 AM, Alexandre Courbot wrote: [...] > Tried this v2 and it seems to do the trick! My panel switches on as expected. Thanks. I posted a formal patch https://patchwork.kernel.org/patch/4391271/ -> might want to add tested/reviewed/ack on that thread. I hope I got your email ID

Re: Palmas regulator broken (was Re: [PATCH] ARM: tegra: TN7: relax some regulators)

2014-06-20 Thread Alexandre Courbot
On Fri, Jun 20, 2014 at 10:54 PM, Nishanth Menon wrote: > On 08:23-20140620, Nishanth Menon wrote: >> + l-o, >> http://marc.info/?t=14031642754=1=2 full thread >> >> Minor change in subject to indicate palmas regulator fail >> >> On 18:49-20140620, Alexandre Courbot wrote: >> > On

Re: Palmas regulator broken (was Re: [PATCH] ARM: tegra: TN7: relax some regulators)

2014-06-20 Thread Nishanth Menon
On 08:23-20140620, Nishanth Menon wrote: > + l-o, > http://marc.info/?t=14031642754=1=2 full thread > > Minor change in subject to indicate palmas regulator fail > > On 18:49-20140620, Alexandre Courbot wrote: > > On 06/20/2014 06:41 PM, Mark Brown wrote: > > >* PGP Signed by an

Palmas regulator broken (was Re: [PATCH] ARM: tegra: TN7: relax some regulators)

2014-06-20 Thread Nishanth Menon
+ l-o, http://marc.info/?t=14031642754=1=2 full thread Minor change in subject to indicate palmas regulator fail On 18:49-20140620, Alexandre Courbot wrote: > On 06/20/2014 06:41 PM, Mark Brown wrote: > >* PGP Signed by an unknown key > > > >On Fri, Jun 20, 2014 at 03:44:46PM +0900,

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

2014-06-20 Thread Alexandre Courbot
On 06/20/2014 07:16 PM, Mark Brown wrote: * PGP Signed by an unknown key On Fri, Jun 20, 2014 at 02:26:43PM +0900, Alexandre Courbot wrote: 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

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

2014-06-20 Thread Mark Brown
On Fri, Jun 20, 2014 at 02:26:43PM +0900, Alexandre Courbot wrote: > 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

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

2014-06-20 Thread Alexandre Courbot
On 06/20/2014 06:41 PM, Mark Brown wrote: * PGP Signed by an unknown key On Fri, Jun 20, 2014 at 03:44:46PM +0900, Alexandre Courbot wrote: dbabd624d regulator: palmas: Reemove open coded functions with helper functions Keerthy, Nishanth, could it be that there is still something wrong

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

2014-06-20 Thread Mark Brown
On Fri, Jun 20, 2014 at 03:44:46PM +0900, Alexandre Courbot wrote: > dbabd624d > regulator: palmas: Reemove open coded functions with helper functions > Keerthy, Nishanth, could it be that there is still something wrong with the > REGULATOR_LINEAR_RANGE() definitions? > This seems to be the

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

2014-06-20 Thread Alexandre Courbot
On 06/20/2014 02:26 PM, Alexandre Courbot wrote: 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

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

2014-06-20 Thread Alexandre Courbot
On 06/20/2014 02:26 PM, Alexandre Courbot wrote: 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

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

2014-06-20 Thread Mark Brown
On Fri, Jun 20, 2014 at 03:44:46PM +0900, Alexandre Courbot wrote: dbabd624d regulator: palmas: Reemove open coded functions with helper functions Keerthy, Nishanth, could it be that there is still something wrong with the REGULATOR_LINEAR_RANGE() definitions? This seems to be the cause

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

2014-06-20 Thread Alexandre Courbot
On 06/20/2014 06:41 PM, Mark Brown wrote: * PGP Signed by an unknown key On Fri, Jun 20, 2014 at 03:44:46PM +0900, Alexandre Courbot wrote: dbabd624d regulator: palmas: Reemove open coded functions with helper functions Keerthy, Nishanth, could it be that there is still something wrong

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

2014-06-20 Thread Mark Brown
On Fri, Jun 20, 2014 at 02:26:43PM +0900, Alexandre Courbot wrote: 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

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

2014-06-20 Thread Alexandre Courbot
On 06/20/2014 07:16 PM, Mark Brown wrote: * PGP Signed by an unknown key On Fri, Jun 20, 2014 at 02:26:43PM +0900, Alexandre Courbot wrote: 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

Palmas regulator broken (was Re: [PATCH] ARM: tegra: TN7: relax some regulators)

2014-06-20 Thread Nishanth Menon
+ l-o, http://marc.info/?t=14031642754r=1w=2 full thread Minor change in subject to indicate palmas regulator fail On 18:49-20140620, Alexandre Courbot wrote: On 06/20/2014 06:41 PM, Mark Brown wrote: * PGP Signed by an unknown key On Fri, Jun 20, 2014 at 03:44:46PM +0900,

Re: Palmas regulator broken (was Re: [PATCH] ARM: tegra: TN7: relax some regulators)

2014-06-20 Thread Nishanth Menon
On 08:23-20140620, Nishanth Menon wrote: + l-o, http://marc.info/?t=14031642754r=1w=2 full thread Minor change in subject to indicate palmas regulator fail On 18:49-20140620, Alexandre Courbot wrote: On 06/20/2014 06:41 PM, Mark Brown wrote: * PGP Signed by an unknown key

Re: Palmas regulator broken (was Re: [PATCH] ARM: tegra: TN7: relax some regulators)

2014-06-20 Thread Alexandre Courbot
On Fri, Jun 20, 2014 at 10:54 PM, Nishanth Menon n...@ti.com wrote: On 08:23-20140620, Nishanth Menon wrote: + l-o, http://marc.info/?t=14031642754r=1w=2 full thread Minor change in subject to indicate palmas regulator fail On 18:49-20140620, Alexandre Courbot wrote: On

Re: Palmas regulator broken (was Re: [PATCH] ARM: tegra: TN7: relax some regulators)

2014-06-20 Thread Nishanth Menon
On 06/20/2014 09:14 AM, Alexandre Courbot wrote: [...] Tried this v2 and it seems to do the trick! My panel switches on as expected. Thanks. I posted a formal patch https://patchwork.kernel.org/patch/4391271/ - might want to add tested/reviewed/ack on that thread. I hope I got your email ID

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

2014-06-19 Thread Alexandre Courbot
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

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

2014-06-19 Thread Mark Brown
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

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

2014-06-19 Thread Stephen Warren
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.

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

2014-06-19 Thread Alexandre Courbot
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. Signed-off-by: Alexandre Courbot --- Stephen, do you think you could queue this for 3.16-rc2 or 3? Without this the TN7 panel does not

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

2014-06-19 Thread Alexandre Courbot
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. Signed-off-by: Alexandre Courbot acour...@nvidia.com --- Stephen, do you think you could queue this for 3.16-rc2 or 3? Without this the

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

2014-06-19 Thread Stephen Warren
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. After

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

2014-06-19 Thread Mark Brown
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

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

2014-06-19 Thread Alexandre Courbot
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