diff options
| author | Laurentiu Palcu <laurentiu.palcu@nxp.com> | 2018-11-14 15:01:30 +0200 |
|---|---|---|
| committer | Jason Liu <jason.hui.liu@nxp.com> | 2019-02-12 10:35:18 +0800 |
| commit | f5efa2b652800ab1e6d700cf98e1751bc2b2df7b (patch) | |
| tree | 9050004dda2314f677fa06c4c79734ab6b29b466 /drivers/video | |
| parent | 2a25f920507b764c9b8018d9fdcb6efce6e11b77 (diff) | |
MLK-20216-5: video/fbdev/mxc_dcic: remove needless comparison
This fixes less-then-zero comparison Coverity issue. roi_param->roi_n is
unsigned, hence the comparison is not needed.
Signed-off-by: Laurentiu Palcu <laurentiu.palcu@nxp.com>
CC: Sandor Yu <sandor.yu@nxp.com>
Diffstat (limited to 'drivers/video')
| -rw-r--r-- | drivers/video/fbdev/mxc/mxc_dcic.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/video/fbdev/mxc/mxc_dcic.c b/drivers/video/fbdev/mxc/mxc_dcic.c index cae872a05f7f..3c5a83beeded 100644 --- a/drivers/video/fbdev/mxc/mxc_dcic.c +++ b/drivers/video/fbdev/mxc/mxc_dcic.c @@ -230,7 +230,7 @@ static bool roi_configure(struct dcic_data *dcic, struct roi_params *roi_param) struct roi_regs *roi_reg; u32 val; - if (roi_param->roi_n < 0 || roi_param->roi_n >= 16) { + if (roi_param->roi_n >= 16) { printk(KERN_ERR "Error, Wrong ROI number %d\n", roi_param->roi_n); return false; } |