Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 435ec8af authored by Daeseok Youn's avatar Daeseok Youn Committed by Greg Kroah-Hartman
Browse files

staging: atomisp: remove useless condition in if-statements 



The css_pipe_id was checked with 'CSS_PIPE_ID_COPY' in previous if-
statement. In this case, if the css_pipe_id equals to 'CSS_PIPE_ID_COPY',
it could not enter the next if-statement. But the "next" if-statement
has the condition to check whether the css_pipe_id equals to
'CSS_PIPE_ID_COPY' or not. It should be removed.

Signed-off-by: default avatarDaeseok Youn <daeseok.youn@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 00987f50

drivers/staging/media/atomisp/pci/atomisp2/atomisp_cmd.c

+2 −4
Original line number Diff line number Diff line
@@ -903,8 +903,7 @@ static struct atomisp_video_pipe *__atomisp_get_pipe( 
	} else if (asd->run_mode->val == ATOMISP_RUN_MODE_VIDEO) {

		/* For online video or SDV video pipe. */

		if (css_pipe_id == CSS_PIPE_ID_VIDEO ||

		    css_pipe_id == CSS_PIPE_ID_COPY ||

		    css_pipe_id == CSS_PIPE_ID_YUVPP) {

		    css_pipe_id == CSS_PIPE_ID_COPY) {

			if (buf_type == CSS_BUFFER_TYPE_OUTPUT_FRAME)

				return &asd->video_out_video_capture;

			return &asd->video_out_preview;
@@ -912,8 +911,7 @@ static struct atomisp_video_pipe *__atomisp_get_pipe( 
	} else if (asd->run_mode->val == ATOMISP_RUN_MODE_PREVIEW) {

		/* For online preview or ZSL preview pipe. */

		if (css_pipe_id == CSS_PIPE_ID_PREVIEW ||

		    css_pipe_id == CSS_PIPE_ID_COPY ||

		    css_pipe_id == CSS_PIPE_ID_YUVPP)

		    css_pipe_id == CSS_PIPE_ID_COPY)

			return &asd->video_out_preview;

	}

	/* For capture pipe. */