diff options
| author | Laurentiu Palcu <laurentiu.palcu@nxp.com> | 2018-11-14 14:50:43 +0200 |
|---|---|---|
| committer | Jason Liu <jason.hui.liu@nxp.com> | 2019-02-12 10:35:18 +0800 |
| commit | e2608916d4c92f645f277355e30fdcd3482fe34f (patch) | |
| tree | e2aacd8ad3a644e4e2f18a9a92cd6c5dd6c0549f /drivers/video | |
| parent | 3b42f038cc0170146a31652c02c84f2fcaf6b8fa (diff) | |
MLK-20216-3: video/fbdev/imx_dcss: remove unnecessary BUG_ONs
This fixes Coverity issue since num_pix_x and num_pix_y are unsigned and
the conditions are never true.
Signed-off-by: Laurentiu Palcu <laurentiu.palcu@nxp.com>
CC: Fancy Fang <chen.fang@nxp.com>
Diffstat (limited to 'drivers/video')
| -rw-r--r-- | drivers/video/fbdev/mxc/imx_dcss.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/video/fbdev/mxc/imx_dcss.c b/drivers/video/fbdev/mxc/imx_dcss.c index 2e279232864d..0da21d458211 100644 --- a/drivers/video/fbdev/mxc/imx_dcss.c +++ b/drivers/video/fbdev/mxc/imx_dcss.c @@ -1490,13 +1490,11 @@ static int dcss_dpr_config(uint32_t dpr_ch, struct dcss_info *info) pix_size = ilog2(input->bits_per_pixel >> 3); num_pix_x = dpr_pix_x_calc(pix_size, input->width, input->tile_type); - BUG_ON(num_pix_x < 0); fill_sb(cb, chan_info->dpr_addr + 0xa0, num_pix_x); switch (fmt_is_yuv(input->format)) { case 0: /* RGB */ num_pix_y = dpr_pix_y_calc(1, input->height, input->tile_type); - BUG_ON(num_pix_y < 0); fill_sb(cb, chan_info->dpr_addr + 0xb0, num_pix_y); if (!need_resolve) @@ -1513,7 +1511,6 @@ static int dcss_dpr_config(uint32_t dpr_ch, struct dcss_info *info) case 2: /* YUV 2P */ /* Two planes YUV format */ num_pix_y = dpr_pix_y_calc(0, input->height, input->tile_type); - BUG_ON(num_pix_y < 0); fill_sb(cb, chan_info->dpr_addr + 0xb0, num_pix_y); fill_sb(cb, chan_info->dpr_addr + 0x50, 0xc1); @@ -1532,7 +1529,6 @@ static int dcss_dpr_config(uint32_t dpr_ch, struct dcss_info *info) * UV height is 1/2 height of Luma. */ num_pix_y = dpr_pix_y_calc(0, input->height >> 1, input->tile_type); - BUG_ON(num_pix_y < 0); fill_sb(cb, chan_info->dpr_addr + 0x100, num_pix_y); break; default: |