Skip to content

Commit d0eedc5

Browse files
ngphibangkartben
authored andcommitted
drivers: video: mcux_csi: Don't set_format() in get_format()
Do not set_format() when doing get_format(). This design seems initially to simplify the sample (just get_format() and everything works out of the box) but it makes thing incomprehensive and error prone. Signed-off-by: Phi Bang Nguyen <phibang.nguyen@nxp.com>
1 parent 0c1e2c9 commit d0eedc5

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

drivers/video/video_mcux_csi.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -184,8 +184,8 @@ static int video_mcux_csi_get_fmt(const struct device *dev, struct video_format
184184
#if defined(CONFIG_VIDEO_MCUX_MIPI_CSI2RX)
185185
video_pix_fmt_convert(fmt, true);
186186
#endif
187-
/* align CSI with source fmt */
188-
return video_mcux_csi_set_fmt(dev, fmt);
187+
188+
return 0;
189189
}
190190

191191
return -EIO;

0 commit comments

Comments
 (0)