diff options
| author | Grigor Tovmasyan <Grigor.Tovmasyan@synopsys.com> | 2018-04-03 14:22:25 +0300 |
|---|---|---|
| committer | Felipe Balbi <felipe.balbi@linux.intel.com> | 2018-05-21 10:40:16 +0300 |
| commit | 47265c067c0d129f3a0e94bc221293a780af9d78 (patch) | |
| tree | 0b283bad258f5009f43bc3061c57305655dae467 /drivers/usb | |
| parent | 8223b2f89ca63e203dcb54148e30d94979f17b0b (diff) | |
| download | linux-47265c067c0d129f3a0e94bc221293a780af9d78.tar.xz | |
usb: dwc2: gadget: Fix coverity issue
When _param is unsigned and the minimum value of range is 0, it gives
the following warning:
COVERITY NO_EFFECT: This less-than-zero comparison of an unsigned value
is never true.
Converting ._param to int to avoid this warning.
Signed-off-by: Grigor Tovmasyan <tovmasya@synopsys.com>
Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
Diffstat (limited to 'drivers/usb')
| -rw-r--r-- | drivers/usb/dwc2/params.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/dwc2/params.c b/drivers/usb/dwc2/params.c index a24ba13a2f15..af075d4da895 100644 --- a/drivers/usb/dwc2/params.c +++ b/drivers/usb/dwc2/params.c @@ -556,7 +556,7 @@ static void dwc2_check_param_tx_fifo_sizes(struct dwc2_hsotg *hsotg) } #define CHECK_RANGE(_param, _min, _max, _def) do { \ - if ((hsotg->params._param) < (_min) || \ + if ((int)(hsotg->params._param) < (_min) || \ (hsotg->params._param) > (_max)) { \ dev_warn(hsotg->dev, "%s: Invalid parameter %s=%d\n", \ __func__, #_param, hsotg->params._param); \ |