diff options
| author | Michał Mirosław <mirq-linux@rere.qmqm.pl> | 2023-09-19 01:50:27 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2023-10-25 12:54:23 +0300 |
| commit | f06c3a50ef2bb25a194ca711006004f3ae524af4 (patch) | |
| tree | 5e7e4fca62411cdf1def3b8ab92bc6c16adfbede /drivers/regulator/core.c | |
| parent | 4613414129603fca2a8cb1324eae7101865d03e8 (diff) | |
| download | linux-f06c3a50ef2bb25a194ca711006004f3ae524af4.tar.xz | |
regulator/core: Revert "fix kobject release warning and memory leak in regulator_register()"
[ Upstream commit 6e800968f6a715c0661716d2ec5e1f56ed9f9c08 ]
This reverts commit 5f4b204b6b8153923d5be8002c5f7082985d153f.
Since rdev->dev now has a release() callback, the proper way of freeing
the initialized device can be restored.
Signed-off-by: Michał Mirosław <mirq-linux@rere.qmqm.pl>
Link: https://lore.kernel.org/r/d7f469f3f7b1f0e1d52f9a7ede3f3c5703382090.1695077303.git.mirq-linux@rere.qmqm.pl
Signed-off-by: Mark Brown <broonie@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'drivers/regulator/core.c')
| -rw-r--r-- | drivers/regulator/core.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c index 52b75779dbb7..51c4f604d3b2 100644 --- a/drivers/regulator/core.c +++ b/drivers/regulator/core.c @@ -5483,15 +5483,11 @@ wash: mutex_lock(®ulator_list_mutex); regulator_ena_gpio_free(rdev); mutex_unlock(®ulator_list_mutex); - put_device(&rdev->dev); - rdev = NULL; clean: if (dangling_of_gpiod) gpiod_put(config->ena_gpiod); - if (rdev && rdev->dev.of_node) - of_node_put(rdev->dev.of_node); - kfree(rdev); kfree(config); + put_device(&rdev->dev); rinse: if (dangling_cfg_gpiod) gpiod_put(cfg->ena_gpiod); |