Re: [PATCH v5 07/23] regulator: core: Remove regulator_list

2015-09-21 Thread Tomeu Vizoso
On 19 September 2015 at 17:01, Mark Brown wrote: > On Thu, Sep 17, 2015 at 02:57:01PM +0200, Tomeu Vizoso wrote: >> As we are already registering a device with regulator_class for each >> regulator device, regulator_list is redundant and can be replaced with >> calls to class_find_device() and cla

Re: [PATCH v5 07/23] regulator: core: Remove regulator_list

2015-09-21 Thread Tomeu Vizoso
On 20 September 2015 at 22:32, Russell King - ARM Linux wrote: > On Sat, Sep 19, 2015 at 08:01:29AM -0700, Mark Brown wrote: >> On Thu, Sep 17, 2015 at 02:57:01PM +0200, Tomeu Vizoso wrote: >> > As we are already registering a device with regulator_class for each >> > regulator device, regulator_l

Re: [PATCH v5 07/23] regulator: core: Remove regulator_list

2015-09-20 Thread Russell King - ARM Linux
On Sat, Sep 19, 2015 at 08:01:29AM -0700, Mark Brown wrote: > On Thu, Sep 17, 2015 at 02:57:01PM +0200, Tomeu Vizoso wrote: > > As we are already registering a device with regulator_class for each > > regulator device, regulator_list is redundant and can be replaced with > > calls to class_find_dev

Re: [PATCH v5 07/23] regulator: core: Remove regulator_list

2015-09-19 Thread Mark Brown
On Thu, Sep 17, 2015 at 02:57:01PM +0200, Tomeu Vizoso wrote: > As we are already registering a device with regulator_class for each > regulator device, regulator_list is redundant and can be replaced with > calls to class_find_device() and class_for_each_device(). This appears to leak references

[PATCH v5 07/23] regulator: core: Remove regulator_list

2015-09-17 Thread Tomeu Vizoso
As we are already registering a device with regulator_class for each regulator device, regulator_list is redundant and can be replaced with calls to class_find_device() and class_for_each_device(). Signed-off-by: Tomeu Vizoso --- Changes in v5: None Changes in v4: None Changes in v3: None Change