lk/linux-kernel-test
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

[media] fimc-lite: Fix the variable type to avoid possible crash

Browse Source 
Changing the variable type to 'int' from 'unsigned int'. Driver
logic expects the variable type to be 'int'.

Signed-off-by: Shaik Ameer Basha <shaik.ameer@samsung.com>
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
This commit is contained in:
Shaik Ameer Basha
2013-02-21 07:54:17 -03:00
committed by Mauro Carvalho Chehab
parent 4cec1893d8
commit 6a5360966a
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
drivers/media/platform/s5p-fimc/fimc-lite-reg.c
View File

@@ -128,10 +128,10 @@ static const u32 src_pixfmt_map[8][3] = {
void flite_hw_set_source_format(struct fimc_lite *dev, struct flite_frame *f) void flite_hw_set_source_format(struct fimc_lite *dev, struct flite_frame *f)
{ {
enum v4l2_mbus_pixelcode pixelcode = dev->fmt->mbus_code; enum v4l2_mbus_pixelcode pixelcode = dev->fmt->mbus_code;
unsigned int i = ARRAY_SIZE(src_pixfmt_map); int i = ARRAY_SIZE(src_pixfmt_map);
u32 cfg; u32 cfg;
while (i-- >= 0) { while (--i >= 0) {
if (src_pixfmt_map[i][0] == pixelcode) if (src_pixfmt_map[i][0] == pixelcode)
break; break;
} }
@@ -224,9 +224,9 @@ static void flite_hw_set_out_order(struct fimc_lite *dev, struct flite_frame *f)
{ V4L2_MBUS_FMT_VYUY8_2X8, FLITE_REG_CIODMAFMT_CRYCBY }, { V4L2_MBUS_FMT_VYUY8_2X8, FLITE_REG_CIODMAFMT_CRYCBY },
}; };
u32 cfg = readl(dev->regs + FLITE_REG_CIODMAFMT); u32 cfg = readl(dev->regs + FLITE_REG_CIODMAFMT);
unsigned int i = ARRAY_SIZE(pixcode); int i = ARRAY_SIZE(pixcode);
while (i-- >= 0) while (--i >= 0)
if (pixcode[i][0] == dev->fmt->mbus_code) if (pixcode[i][0] == dev->fmt->mbus_code)
break; break;
cfg &= ~FLITE_REG_CIODMAFMT_YCBCR_ORDER_MASK; cfg &= ~FLITE_REG_CIODMAFMT_YCBCR_ORDER_MASK;