diff options
author | Michał Mirosław <mirq-linux@rere.qmqm.pl> | 2020-08-10 06:33:32 +0200 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2020-09-07 18:35:24 +0100 |
commit | 3bca239d6184df61a619d78764e0481242d844b4 (patch) | |
tree | abe6e3d8718a283def01812befae6af95eaf4e40 /drivers/regulator | |
parent | 6656d4462c7ac5ca0bc5d5a8a91a8c98f0fd1409 (diff) | |
download | linux-3bca239d6184df61a619d78764e0481242d844b4.tar.gz linux-3bca239d6184df61a619d78764e0481242d844b4.tar.bz2 linux-3bca239d6184df61a619d78764e0481242d844b4.zip |
regulator: don't require mutex for regulator_notifier_call_chain()
Since 3801b86aa482 ("regulator: Refactor supply implementation
to work as regular consumers") we no longer cascade notifications
and so notifier head's built-in rwsem is enough to protect the
notifier chain. Remove the requirement to fix one case where
rdev->mutex might be forced to be taken recursively.
Signed-off-by: Michał Mirosław <mirq-linux@rere.qmqm.pl>
Reviewed-by: Dmitry Osipenko <digetx@gmail.com>
Link: https://lore.kernel.org/r/5a0da9017c69a4dbc3f9b50f44476fce80a73387.1597032945.git.mirq-linux@rere.qmqm.pl
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/regulator')
-rw-r--r-- | drivers/regulator/core.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c index 07edbd8ebb98..2ed035114ee8 100644 --- a/drivers/regulator/core.c +++ b/drivers/regulator/core.c @@ -4725,14 +4725,11 @@ EXPORT_SYMBOL_GPL(regulator_bulk_free); * @data: callback-specific data. * * Called by regulator drivers to notify clients a regulator event has - * occurred. We also notify regulator clients downstream. - * Note lock must be held by caller. + * occurred. */ int regulator_notifier_call_chain(struct regulator_dev *rdev, unsigned long event, void *data) { - lockdep_assert_held_once(&rdev->mutex.base); - _notifier_call_chain(rdev, event, data); return NOTIFY_DONE; |