media: cadence: csi2rx: Fix csi2rx_start error handling
authorSakari Ailus <sakari.ailus@linux.intel.com>
Wed, 9 May 2018 20:24:35 +0000 (16:24 -0400)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Thu, 17 May 2018 10:22:08 +0000 (06:22 -0400)
commit28d42d2fcaad68fa81a328adfb027323796e5f6e
tree09aaf3ea0df607b073cdd6b74b8b93ea408f61f0
parent844ef042491d2824d5ed8dc5f75b5f539487f82c
media: cadence: csi2rx: Fix csi2rx_start error handling

The clocks enabled by csi2rx_start function are intended to be disabled in
an error path but there are two issues:

1) the loop condition is always true and

2) the first clock disabled is the the one enabling of which failed.

Fix these two bugs by changing the loop condition as well as only disabling
the clocks that were actually enabled.

Reported-by: Mauro Chehab <mchehab@kernel.org>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Acked-by: Maxime Ripard <maxime.ripard@bootlin.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
drivers/media/platform/cadence/cdns-csi2rx.c