Commit b72e4495 authored by Helen Fornazier's avatar Helen Fornazier Committed by Mauro Carvalho Chehab
Browse files

media: vimc: stream: cleanup frame field from struct vimc_stream 



There is no need to have the frame field in the vimc_stream struct.

Signed-off-by: default avatarHelen Koike <helen.koike@collabora.com>
Tested-by: default avatarAndré Almeida <andre.almeida@collabora.com>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+samsung@kernel.org>
parent 5efbc65f

drivers/media/platform/vimc/vimc-streamer.c

+4 −6
Original line number Diff line number Diff line
@@ -117,6 +117,7 @@ static int vimc_streamer_pipeline_init(struct vimc_stream *stream, 
static int vimc_streamer_thread(void *data)

{

	struct vimc_stream *stream = data;

	u8 *frame = NULL;

	int i;



	set_freezable();
@@ -127,12 +128,9 @@ static int vimc_streamer_thread(void *data) 
			break;



		for (i = stream->pipe_size - 1; i >= 0; i--) {

			stream->frame = stream->ved_pipeline[i]->process_frame(

					stream->ved_pipeline[i],

					stream->frame);

			if (!stream->frame)

				break;

			if (IS_ERR(stream->frame))

			frame = stream->ved_pipeline[i]->process_frame(

					stream->ved_pipeline[i], frame);

			if (!frame || IS_ERR(frame))

				break;

		}

		//wait for 60hz

drivers/media/platform/vimc/vimc-streamer.h

+0 −1
Original line number Diff line number Diff line
@@ -19,7 +19,6 @@ struct vimc_stream { 
	struct media_pipeline pipe;

	struct vimc_ent_device *ved_pipeline[VIMC_STREAMER_PIPELINE_MAX_SIZE];

	unsigned int pipe_size;

	u8 *frame;

	struct task_struct *kthread;

};