Commit 9a57d72b authored by Sakari Ailus's avatar Sakari Ailus Committed by Mauro Carvalho Chehab

media: mt9m111: No need to check for the regulator

The regulator_get() function returns a regulator when it succeeds. There's
no need to check whether the regulator is NULL later on.
Signed-off-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
Reviewed-by: default avatarMarco Felsch <m.felsch@pengutronix.de>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+samsung@kernel.org>
parent 3a959dcd
...@@ -984,11 +984,9 @@ static int mt9m111_power_on(struct mt9m111 *mt9m111) ...@@ -984,11 +984,9 @@ static int mt9m111_power_on(struct mt9m111 *mt9m111)
if (ret < 0) if (ret < 0)
return ret; return ret;
if (mt9m111->regulator) { ret = regulator_enable(mt9m111->regulator);
ret = regulator_enable(mt9m111->regulator); if (ret < 0)
if (ret < 0) return ret;
return ret;
}
ret = mt9m111_resume(mt9m111); ret = mt9m111_resume(mt9m111);
if (ret < 0) { if (ret < 0) {
...@@ -1002,8 +1000,7 @@ static int mt9m111_power_on(struct mt9m111 *mt9m111) ...@@ -1002,8 +1000,7 @@ static int mt9m111_power_on(struct mt9m111 *mt9m111)
static void mt9m111_power_off(struct mt9m111 *mt9m111) static void mt9m111_power_off(struct mt9m111 *mt9m111)
{ {
mt9m111_suspend(mt9m111); mt9m111_suspend(mt9m111);
if (mt9m111->regulator) regulator_disable(mt9m111->regulator);
regulator_disable(mt9m111->regulator);
v4l2_clk_disable(mt9m111->clk); v4l2_clk_disable(mt9m111->clk);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment