diff options
| author | Jacopo Mondi <jacopo+renesas@jmondi.org> | 2018-08-03 12:40:58 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab+samsung@kernel.org> | 2018-09-10 12:13:41 -0400 |
| commit | 422e91cc48f8661273311e96cee9d8b1115aa788 (patch) | |
| tree | 2dec937bdda1fb46742552638a60b86fe58b3247 /tools/spi | |
| parent | d842a7cf938b6e0f8a1aa9f1aec0476c9a599310 (diff) | |
media: i2c: mt9v111: Fix v4l2-ctrl error handling
Fix error handling of v4l2_ctrl creation by inspecting the ctrl.error flag
instead of testing for each returned value correctness.
As reported by Dan Carpenter returning PTR_ERR() on the v4l2_ctrl_new_std()
return value is also wrong, as that function return NULL on error.
While at there re-order the cleanup path to respect the operation inverse
order.
Fixes: aab7ed1c "media: i2c: Add driver for Aptina MT9V111"
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Diffstat (limited to 'tools/spi')
0 files changed, 0 insertions, 0 deletions