Commit 738ac75e authored by Hans de Goede's avatar Hans de Goede Committed by Mauro Carvalho Chehab

media: atomisp: Remove yuvpp_mode

yuvpp_mode is never set, so it always is false remove it.

Link: https://lore.kernel.org/r/20230221145906.8113-10-hdegoede@redhat.comReviewed-by: default avatarAndy Shevchenko <andy@kernel.org>
Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@kernel.org>
parent 9c2f3b8f
...@@ -4813,7 +4813,6 @@ int atomisp_set_fmt(struct video_device *vdev, struct v4l2_format *f) ...@@ -4813,7 +4813,6 @@ int atomisp_set_fmt(struct video_device *vdev, struct v4l2_format *f)
atomisp_csi_lane_config(isp); atomisp_csi_lane_config(isp);
atomisp_check_copy_mode(asd, source_pad, &backup_fmt); atomisp_check_copy_mode(asd, source_pad, &backup_fmt);
asd->yuvpp_mode = false; /* Reset variable */
isp_sink_crop = *atomisp_subdev_get_rect(&asd->subdev, NULL, isp_sink_crop = *atomisp_subdev_get_rect(&asd->subdev, NULL,
V4L2_SUBDEV_FORMAT_ACTIVE, V4L2_SUBDEV_FORMAT_ACTIVE,
......
...@@ -2445,8 +2445,6 @@ static unsigned int atomisp_get_pipe_index(struct atomisp_sub_device *asd, ...@@ -2445,8 +2445,6 @@ static unsigned int atomisp_get_pipe_index(struct atomisp_sub_device *asd,
switch (source_pad) { switch (source_pad) {
case ATOMISP_SUBDEV_PAD_SOURCE_VIDEO: case ATOMISP_SUBDEV_PAD_SOURCE_VIDEO:
if (asd->yuvpp_mode)
return IA_CSS_PIPE_ID_YUVPP;
if (asd->copy_mode) if (asd->copy_mode)
return IA_CSS_PIPE_ID_COPY; return IA_CSS_PIPE_ID_COPY;
if (asd->run_mode->val == ATOMISP_RUN_MODE_VIDEO if (asd->run_mode->val == ATOMISP_RUN_MODE_VIDEO
...@@ -2465,8 +2463,6 @@ static unsigned int atomisp_get_pipe_index(struct atomisp_sub_device *asd, ...@@ -2465,8 +2463,6 @@ static unsigned int atomisp_get_pipe_index(struct atomisp_sub_device *asd,
} }
fallthrough; fallthrough;
case ATOMISP_SUBDEV_PAD_SOURCE_PREVIEW: case ATOMISP_SUBDEV_PAD_SOURCE_PREVIEW:
if (asd->yuvpp_mode)
return IA_CSS_PIPE_ID_YUVPP;
if (asd->copy_mode) if (asd->copy_mode)
return IA_CSS_PIPE_ID_COPY; return IA_CSS_PIPE_ID_COPY;
if (asd->run_mode->val == ATOMISP_RUN_MODE_VIDEO) if (asd->run_mode->val == ATOMISP_RUN_MODE_VIDEO)
......
...@@ -399,15 +399,6 @@ int atomisp_qbuffers_to_css(struct atomisp_sub_device *asd) ...@@ -399,15 +399,6 @@ int atomisp_qbuffers_to_css(struct atomisp_sub_device *asd)
css_video_pipe_id = IA_CSS_PIPE_ID_COPY; css_video_pipe_id = IA_CSS_PIPE_ID_COPY;
} }
if (asd->yuvpp_mode) {
capture_pipe = &asd->video_out_capture;
video_pipe = &asd->video_out_video_capture;
preview_pipe = &asd->video_out_preview;
css_capture_pipe_id = IA_CSS_PIPE_ID_COPY;
css_video_pipe_id = IA_CSS_PIPE_ID_YUVPP;
css_preview_pipe_id = IA_CSS_PIPE_ID_YUVPP;
}
if (capture_pipe) { if (capture_pipe) {
buf_type = atomisp_get_css_buf_type( buf_type = atomisp_get_css_buf_type(
asd, css_capture_pipe_id, asd, css_capture_pipe_id,
...@@ -583,7 +574,6 @@ static void atomisp_subdev_init_struct(struct atomisp_sub_device *asd) ...@@ -583,7 +574,6 @@ static void atomisp_subdev_init_struct(struct atomisp_sub_device *asd)
asd->mipi_frame_size = 0; asd->mipi_frame_size = 0;
asd->copy_mode = false; asd->copy_mode = false;
asd->yuvpp_mode = false;
asd->stream_prepared = false; asd->stream_prepared = false;
asd->high_speed_mode = false; asd->high_speed_mode = false;
......
...@@ -330,7 +330,6 @@ struct atomisp_sub_device { ...@@ -330,7 +330,6 @@ struct atomisp_sub_device {
unsigned int mipi_frame_size; unsigned int mipi_frame_size;
bool copy_mode; /* CSI2+ use copy mode */ bool copy_mode; /* CSI2+ use copy mode */
bool yuvpp_mode; /* CSI2+ yuvpp pipe */
int raw_buffer_bitmap[ATOMISP_MAX_EXP_ID / 32 + int raw_buffer_bitmap[ATOMISP_MAX_EXP_ID / 32 +
1]; /* Record each Raw Buffer lock status */ 1]; /* Record each Raw Buffer lock status */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment