Re: [PATCH 3/5] [media] s5p-fimc: remove stop_streaming() callback return

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Monday, April 04, 2011 01:51:08 Pawel Osciak wrote:
> The stop_streaming() callback does not return a value anymore.
> 
> Signed-off-by: Pawel Osciak <pawel@xxxxxxxxxx>
> ---
>  drivers/media/video/s5p-fimc/fimc-capture.c |    4 ++--
>  drivers/media/video/s5p-fimc/fimc-core.c    |    4 +---
>  2 files changed, 3 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/media/video/s5p-fimc/fimc-capture.c b/drivers/media/video/s5p-fimc/fimc-capture.c
> index 95f8b4e1..34e55a4 100644
> --- a/drivers/media/video/s5p-fimc/fimc-capture.c
> +++ b/drivers/media/video/s5p-fimc/fimc-capture.c
> @@ -247,7 +247,7 @@ static int start_streaming(struct vb2_queue *q)
>  	return 0;
>  }
>  
> -static int stop_streaming(struct vb2_queue *q)
> +static void stop_streaming(struct vb2_queue *q)
>  {
>  	struct fimc_ctx *ctx = q->drv_priv;
>  	struct fimc_dev *fimc = ctx->fimc_dev;
> @@ -255,7 +255,7 @@ static int stop_streaming(struct vb2_queue *q)
>  	if (!fimc_capture_active(fimc))
>  		return -EINVAL;

Return in a void function?

Regards,

	Hans

>  
> -	return fimc_stop_capture(fimc);
> +	fimc_stop_capture(fimc);
>  }
>  
>  static unsigned int get_plane_size(struct fimc_frame *fr, unsigned int plane)
> diff --git a/drivers/media/video/s5p-fimc/fimc-core.c b/drivers/media/video/s5p-fimc/fimc-core.c
> index 6c919b3..66571d7 100644
> --- a/drivers/media/video/s5p-fimc/fimc-core.c
> +++ b/drivers/media/video/s5p-fimc/fimc-core.c
> @@ -348,13 +348,11 @@ static void fimc_m2m_shutdown(struct fimc_ctx *ctx)
>  		fimc_m2m_job_finish(ctx, VB2_BUF_STATE_ERROR);
>  }
>  
> -static int stop_streaming(struct vb2_queue *q)
> +static void stop_streaming(struct vb2_queue *q)
>  {
>  	struct fimc_ctx *ctx = q->drv_priv;
>  
>  	fimc_m2m_shutdown(ctx);
> -
> -	return 0;
>  }
>  
>  static void fimc_capture_irq_handler(struct fimc_dev *fimc)
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux