diff options
| author | Osama Muhammad <osmtendev@gmail.com> | 2023-05-15 19:29:38 +0200 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2023-05-16 03:15:00 +0200 |
| commit | 2bf1c45be3b8f3a3f898d0756c1282f09719debd (patch) | |
| tree | c22cd861b74d796b780f59758a01c7e768edf7f9 /drivers/regulator | |
| parent | regulator: pca9450: Fix BUCK2 enable_mask (diff) | |
| download | linux-2bf1c45be3b8f3a3f898d0756c1282f09719debd.tar.xz linux-2bf1c45be3b8f3a3f898d0756c1282f09719debd.zip | |
regulator: Fix error checking for debugfs_create_dir
This patch fixes the error checking in core.c in debugfs_create_dir.
The correct way to check if an error occurred is 'IS_ERR' inline function.
Signed-off-by: Osama Muhammad <osmtendev@gmail.com
Suggested-by: Ivan Orlov <ivan.orlov0322@gmail.com
Link: https://lore.kernel.org/r/20230515172938.13338-1-osmtendev@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org
Diffstat (limited to 'drivers/regulator')
| -rw-r--r-- | drivers/regulator/core.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c index dc741ac156c3..698ab7f5004b 100644 --- a/drivers/regulator/core.c +++ b/drivers/regulator/core.c @@ -5256,7 +5256,7 @@ static void rdev_init_debugfs(struct regulator_dev *rdev) } rdev->debugfs = debugfs_create_dir(rname, debugfs_root); - if (!rdev->debugfs) { + if (IS_ERR(rdev->debugfs)) { rdev_warn(rdev, "Failed to create debugfs directory\n"); return; } @@ -6178,7 +6178,7 @@ static int __init regulator_init(void) ret = class_register(®ulator_class); debugfs_root = debugfs_create_dir("regulator", NULL); - if (!debugfs_root) + if (IS_ERR(debugfs_root)) pr_warn("regulator: Failed to create debugfs directory\n"); #ifdef CONFIG_DEBUG_FS |