diff options
| author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2022-11-01 17:09:47 +0200 |
|---|---|---|
| committer | Helge Deller <deller@gmx.de> | 2022-12-14 20:01:48 +0100 |
| commit | 6273c43769cbd4451b03d239dc16d5c54bb7279a (patch) | |
| tree | eb182a8bacf1870fda93e937718bf0faa44e6475 /drivers/video/fbdev/ssd1307fb.c | |
| parent | 025e3b507a3a8e1ee96a3112bb67495c77d6cdb6 (diff) | |
| download | lwn-6273c43769cbd4451b03d239dc16d5c54bb7279a.tar.gz lwn-6273c43769cbd4451b03d239dc16d5c54bb7279a.zip | |
fbdev: ssd1307fb: Drop duplicate NULL checks for PWM APIs
pwm_disable() and pwm_put() are NULL-aware, no need to
duplicate the check in the caller.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Helge Deller <deller@gmx.de>
Diffstat (limited to 'drivers/video/fbdev/ssd1307fb.c')
| -rw-r--r-- | drivers/video/fbdev/ssd1307fb.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/drivers/video/fbdev/ssd1307fb.c b/drivers/video/fbdev/ssd1307fb.c index 5c891aa00d59..046b9990d27c 100644 --- a/drivers/video/fbdev/ssd1307fb.c +++ b/drivers/video/fbdev/ssd1307fb.c @@ -803,10 +803,8 @@ static int ssd1307fb_probe(struct i2c_client *client) bl_init_error: unregister_framebuffer(info); panel_init_error: - if (par->device_info->need_pwm) { - pwm_disable(par->pwm); - pwm_put(par->pwm); - } + pwm_disable(par->pwm); + pwm_put(par->pwm); regulator_enable_error: if (par->vbat_reg) regulator_disable(par->vbat_reg); @@ -827,10 +825,8 @@ static void ssd1307fb_remove(struct i2c_client *client) backlight_device_unregister(info->bl_dev); unregister_framebuffer(info); - if (par->device_info->need_pwm) { - pwm_disable(par->pwm); - pwm_put(par->pwm); - } + pwm_disable(par->pwm); + pwm_put(par->pwm); if (par->vbat_reg) regulator_disable(par->vbat_reg); fb_deferred_io_cleanup(info); |