Re: [PATCH v3 3/4] regulator: core: Add basic enable/disable support for sync_state() callbacks

2020-08-25 Thread Mark Brown
On Tue, Aug 04, 2020 at 10:10:49PM +0100, Mark Brown wrote: > On Tue, Jul 28, 2020 at 02:14:52PM -0700, Saravana Kannan wrote: > > So, say you have a callback that you get for every single consumer > > binding successfully. What can you do there? For every consumer, you > > have to parse their

Re: [PATCH v3 3/4] regulator: core: Add basic enable/disable support for sync_state() callbacks

2020-08-04 Thread Mark Brown
On Tue, Jul 28, 2020 at 02:14:52PM -0700, Saravana Kannan wrote: > On Tue, Jul 21, 2020 at 1:18 PM Mark Brown wrote: > > On Mon, Jul 20, 2020 at 08:22:15PM -0700, Saravana Kannan wrote: Copying in Stephen having done a bunch of mailing list archive trawling on the fw_devlink stuff since I'm

Re: [PATCH v3 3/4] regulator: core: Add basic enable/disable support for sync_state() callbacks

2020-07-28 Thread Saravana Kannan
On Tue, Jul 21, 2020 at 1:18 PM Mark Brown wrote: > Hi Mark, It *might* be easier if you jump to the bottom and read the reasoning for the current design. The stuff in between is just me trying to clarify some misunderstandings. > On Mon, Jul 20, 2020 at 08:22:15PM -0700, Saravana Kannan wrote:

Re: [PATCH v3 3/4] regulator: core: Add basic enable/disable support for sync_state() callbacks

2020-07-21 Thread Mark Brown
On Mon, Jul 20, 2020 at 08:22:15PM -0700, Saravana Kannan wrote: > On Mon, Jul 20, 2020 at 7:28 AM Mark Brown wrote: > > On Wed, Jul 15, 2020 at 09:20:52PM -0700, Saravana Kannan wrote: > > > There are Android devices that exhibit the issue in the example where > > > regulator-X is an LDO,

Re: [PATCH v3 3/4] regulator: core: Add basic enable/disable support for sync_state() callbacks

2020-07-20 Thread Saravana Kannan
On Mon, Jul 20, 2020 at 7:28 AM Mark Brown wrote: > > On Wed, Jul 15, 2020 at 09:20:52PM -0700, Saravana Kannan wrote: > > > There are Android devices that exhibit the issue in the example where > > regulator-X is an LDO, device-A is a camera device and device-B and > > device-C are UFS and USB.

Re: [PATCH v3 3/4] regulator: core: Add basic enable/disable support for sync_state() callbacks

2020-07-20 Thread Mark Brown
On Wed, Jul 15, 2020 at 09:20:52PM -0700, Saravana Kannan wrote: > There are Android devices that exhibit the issue in the example where > regulator-X is an LDO, device-A is a camera device and device-B and > device-C are UFS and USB. To avoid this, they have their own downstream > changes to the

[PATCH v3 3/4] regulator: core: Add basic enable/disable support for sync_state() callbacks

2020-07-15 Thread Saravana Kannan
Consider the following example: - regulator-X is provided by device-X. - regulator-X is a supplier to device-A, device-B and device-C. - device-A is off/inactive from boot. - device-B and device-C are left on/active by the bootloader - regulator-X is left on boot by the bootloader at 2000 mV to