From mboxrd@z Thu Jan 1 00:00:00 1970 From: josh.wu@atmel.com (Josh Wu) Date: Wed, 14 Oct 2015 14:44:01 +0800 Subject: [PATCH 4/5] media: atmel-isi: setup YCC_SWAP correctly when using preview path In-Reply-To: References: <1442898875-7147-1-git-send-email-josh.wu@atmel.com> <1442898875-7147-5-git-send-email-josh.wu@atmel.com> Message-ID: <561DF9B1.6010009@atmel.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, Dear Guennadi On 10/5/2015 12:50 AM, Guennadi Liakhovetski wrote: > On Tue, 22 Sep 2015, Josh Wu wrote: > >> The preview path only can convert UYVY format to RGB data. >> >> To make preview path work correctly, we need to set up YCC_SWAP >> according to sensor output and convert them to UYVY. >> >> Signed-off-by: Josh Wu >> --- >> >> drivers/media/platform/soc_camera/atmel-isi.c | 16 ++++++++++++++++ >> 1 file changed, 16 insertions(+) >> >> diff --git a/drivers/media/platform/soc_camera/atmel-isi.c b/drivers/media/platform/soc_camera/atmel-isi.c >> index bbf6449..e87d354 100644 >> --- a/drivers/media/platform/soc_camera/atmel-isi.c >> +++ b/drivers/media/platform/soc_camera/atmel-isi.c >> @@ -127,6 +127,22 @@ static u32 setup_cfg2_yuv_swap(struct atmel_isi *isi, >> cfg2_yuv_swap = ISI_CFG2_YCC_SWAP_MODE_1; >> break; >> } >> + } else if (xlate->host_fmt->fourcc == V4L2_PIX_FMT_RGB565) { >> + /* Preview path is enabled, it will convert UYVY to RGB format. >> + * But if sensor output format is not UYVY, we need to set >> + * YCC_SWAP_MODE to convert it as UYVY. >> + */ > Please, fix multiline comment style: > > /* > * ... > * ... > */ oh, yes, Sure. I'll fix this. Thanks. Best Regards, Josh Wu >> + switch (xlate->code) { >> + case MEDIA_BUS_FMT_VYUY8_2X8: >> + cfg2_yuv_swap = ISI_CFG2_YCC_SWAP_MODE_1; >> + break; >> + case MEDIA_BUS_FMT_YUYV8_2X8: >> + cfg2_yuv_swap = ISI_CFG2_YCC_SWAP_MODE_2; >> + break; >> + case MEDIA_BUS_FMT_YVYU8_2X8: >> + cfg2_yuv_swap = ISI_CFG2_YCC_SWAP_MODE_3; >> + break; >> + } >> } >> >> return cfg2_yuv_swap; >> -- >> 1.9.1 >>