diff options
| author | Narcisa Ana Maria Vasile <narcisaanamaria12@gmail.com> | 2017-03-01 17:37:27 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-03-06 13:57:17 +0100 |
| commit | 3302603e14838b19e41c46cbc400debfe95451db (patch) | |
| tree | 5a722c5458cd878cb488ed58b1f283b1355d0bf5 | |
| parent | 91b0faa50cc12b343a6e728a90f03517d9dc6740 (diff) | |
staging: vc04_services: bcm2835-camera: Remove unnecessary variable
Instead of storing the result, make the function return directly
Signed-off-by: Narcisa Ana Maria Vasile <narcisaanamaria12@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c b/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c index 53e275e70274..84650d9a8d06 100644 --- a/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c +++ b/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c @@ -671,7 +671,6 @@ static struct vb2_ops bm2835_mmal_video_qops = { static int set_overlay_params(struct bm2835_mmal_dev *dev, struct vchiq_mmal_port *port) { - int ret; struct mmal_parameter_displayregion prev_config = { .set = MMAL_DISPLAY_SET_LAYER | MMAL_DISPLAY_SET_ALPHA | MMAL_DISPLAY_SET_DEST_RECT | MMAL_DISPLAY_SET_FULLSCREEN, @@ -685,11 +684,9 @@ static int set_overlay_params(struct bm2835_mmal_dev *dev, .height = dev->overlay.w.height, }, }; - ret = vchiq_mmal_port_parameter_set(dev->instance, port, - MMAL_PARAMETER_DISPLAYREGION, - &prev_config, sizeof(prev_config)); - - return ret; + return vchiq_mmal_port_parameter_set(dev->instance, port, + MMAL_PARAMETER_DISPLAYREGION, + &prev_config, sizeof(prev_config)); } /* overlay ioctl */ |