Re: [PATCH] regulator: arizona-ldo1: Only enable status change if we have LDOENA

2016-04-25 Thread Richard Fitzgerald
On 25/04/16 00:23, Mark Brown wrote: On Fri, Apr 22, 2016 at 05:38:02PM +0100, Richard Fitzgerald wrote: Please fix your mailer to leave blank lines between paragraphs (and not delete them in quotes), it makes your mails harder to read. On 22/04/16 16:04, Mark Brown wrote: On Fri, Apr 22,

Re: [PATCH] regulator: arizona-ldo1: Only enable status change if we have LDOENA

2016-04-25 Thread Richard Fitzgerald
On 25/04/16 00:23, Mark Brown wrote: On Fri, Apr 22, 2016 at 05:38:02PM +0100, Richard Fitzgerald wrote: Please fix your mailer to leave blank lines between paragraphs (and not delete them in quotes), it makes your mails harder to read. On 22/04/16 16:04, Mark Brown wrote: On Fri, Apr 22,

Re: [PATCH] regulator: arizona-ldo1: Only enable status change if we have LDOENA

2016-04-24 Thread Mark Brown
On Fri, Apr 22, 2016 at 05:38:02PM +0100, Richard Fitzgerald wrote: Please fix your mailer to leave blank lines between paragraphs (and not delete them in quotes), it makes your mails harder to read. > On 22/04/16 16:04, Mark Brown wrote: > >On Fri, Apr 22, 2016 at 02:43:28PM +0100, Richard

Re: [PATCH] regulator: arizona-ldo1: Only enable status change if we have LDOENA

2016-04-24 Thread Mark Brown
On Fri, Apr 22, 2016 at 05:38:02PM +0100, Richard Fitzgerald wrote: Please fix your mailer to leave blank lines between paragraphs (and not delete them in quotes), it makes your mails harder to read. > On 22/04/16 16:04, Mark Brown wrote: > >On Fri, Apr 22, 2016 at 02:43:28PM +0100, Richard

Re: [PATCH] regulator: arizona-ldo1: Only enable status change if we have LDOENA

2016-04-22 Thread Richard Fitzgerald
On 22/04/16 16:04, Mark Brown wrote: On Fri, Apr 22, 2016 at 02:43:28PM +0100, Richard Fitzgerald wrote: The driver was hardcoding REGULATOR_CHANGE_STATUS on the regulator which made the regulator core assume that it can be powered off. The power state of the regulator is controlled by the

Re: [PATCH] regulator: arizona-ldo1: Only enable status change if we have LDOENA

2016-04-22 Thread Richard Fitzgerald
On 22/04/16 16:04, Mark Brown wrote: On Fri, Apr 22, 2016 at 02:43:28PM +0100, Richard Fitzgerald wrote: The driver was hardcoding REGULATOR_CHANGE_STATUS on the regulator which made the regulator core assume that it can be powered off. The power state of the regulator is controlled by the

Re: [PATCH] regulator: arizona-ldo1: Only enable status change if we have LDOENA

2016-04-22 Thread Mark Brown
On Fri, Apr 22, 2016 at 02:43:28PM +0100, Richard Fitzgerald wrote: > The driver was hardcoding REGULATOR_CHANGE_STATUS on the regulator > which made the regulator core assume that it can be powered off. > > The power state of the regulator is controlled by the LDOENA pin so > this patch changes

Re: [PATCH] regulator: arizona-ldo1: Only enable status change if we have LDOENA

2016-04-22 Thread Mark Brown
On Fri, Apr 22, 2016 at 02:43:28PM +0100, Richard Fitzgerald wrote: > The driver was hardcoding REGULATOR_CHANGE_STATUS on the regulator > which made the regulator core assume that it can be powered off. > > The power state of the regulator is controlled by the LDOENA pin so > this patch changes

[PATCH] regulator: arizona-ldo1: Only enable status change if we have LDOENA

2016-04-22 Thread Richard Fitzgerald
The driver was hardcoding REGULATOR_CHANGE_STATUS on the regulator which made the regulator core assume that it can be powered off. The power state of the regulator is controlled by the LDOENA pin so this patch changes to only setting the REGULATOR_CHANGE_STATUS flag if we have a valid gpio for

[PATCH] regulator: arizona-ldo1: Only enable status change if we have LDOENA

2016-04-22 Thread Richard Fitzgerald
The driver was hardcoding REGULATOR_CHANGE_STATUS on the regulator which made the regulator core assume that it can be powered off. The power state of the regulator is controlled by the LDOENA pin so this patch changes to only setting the REGULATOR_CHANGE_STATUS flag if we have a valid gpio for