Re: [PATCH 2/3] regulator: remove locking around regulator_notifier_call_chain()

2020-09-06 Thread Dmitry Osipenko
10.08.2020 07:33, Michał Mirosław пишет:
> regulator_notifier_call_chain() doesn't need rdev lock and rdev's
> existence is assumed in the code anyway. Remove the locks from drivers.
> 
> Signed-off-by: Michał Mirosław 
> ---
>  drivers/regulator/da9055-regulator.c   |  2 --
>  drivers/regulator/da9062-regulator.c   |  2 --
>  drivers/regulator/da9063-regulator.c   |  2 --
>  drivers/regulator/da9210-regulator.c   |  4 
>  drivers/regulator/da9211-regulator.c   |  4 
>  drivers/regulator/lp8755.c |  6 --
>  drivers/regulator/ltc3589.c| 10 ++
>  drivers/regulator/ltc3676.c| 10 ++
>  drivers/regulator/pv88060-regulator.c  | 10 ++
>  drivers/regulator/pv88080-regulator.c  | 10 ++
>  drivers/regulator/pv88090-regulator.c  | 10 ++
>  drivers/regulator/slg51000-regulator.c |  4 
>  drivers/regulator/stpmic1_regulator.c  |  4 
>  drivers/regulator/wm831x-dcdc.c|  4 
>  drivers/regulator/wm831x-isink.c   |  2 --
>  drivers/regulator/wm831x-ldo.c |  2 --
>  drivers/regulator/wm8350-regulator.c   |  2 --
>  17 files changed, 10 insertions(+), 78 deletions(-)

Reviewed-by: Dmitry Osipenko 


RE: [PATCH 2/3] regulator: remove locking around regulator_notifier_call_chain()

2020-08-10 Thread Adam Thomson
On 10 August 2020 05:34, Michał Mirosław wrote:

> regulator_notifier_call_chain() doesn't need rdev lock and rdev's
> existence is assumed in the code anyway. Remove the locks from drivers.
> 
> Signed-off-by: Michał Mirosław 

For da9*-regulator.c, pv88*-regulator.c and slg51000-regulator.c:

Acked-by: Adam Thomson 

> ---
>  drivers/regulator/da9055-regulator.c   |  2 --
>  drivers/regulator/da9062-regulator.c   |  2 --
>  drivers/regulator/da9063-regulator.c   |  2 --
>  drivers/regulator/da9210-regulator.c   |  4 
>  drivers/regulator/da9211-regulator.c   |  4 
>  drivers/regulator/lp8755.c |  6 --
>  drivers/regulator/ltc3589.c| 10 ++
>  drivers/regulator/ltc3676.c| 10 ++
>  drivers/regulator/pv88060-regulator.c  | 10 ++
>  drivers/regulator/pv88080-regulator.c  | 10 ++
>  drivers/regulator/pv88090-regulator.c  | 10 ++
>  drivers/regulator/slg51000-regulator.c |  4 
>  drivers/regulator/stpmic1_regulator.c  |  4 
>  drivers/regulator/wm831x-dcdc.c|  4 
>  drivers/regulator/wm831x-isink.c   |  2 --
>  drivers/regulator/wm831x-ldo.c |  2 --
>  drivers/regulator/wm8350-regulator.c   |  2 --
>  17 files changed, 10 insertions(+), 78 deletions(-)