When _param is unsigned and the minimum value of range is 0,
CHECK_RANGE macro gives the following warning:

COVERITY NO_EFFECT: This less-than-zero comparison of an unsigned value
is never true.

So I changed the type of hird_threshold from u8 to int to avoid that
warning.

Signed-off-by: Grigor Tovmasyan <tovma...@synopsys.com>
---
 drivers/usb/dwc2/core.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/usb/dwc2/core.h b/drivers/usb/dwc2/core.h
index d83be5651f87..0580de249cd2 100644
--- a/drivers/usb/dwc2/core.h
+++ b/drivers/usb/dwc2/core.h
@@ -509,7 +509,7 @@ struct dwc2_core_params {
        bool lpm_clock_gating;
        bool besl;
        bool hird_threshold_en;
-       u8 hird_threshold;
+       int hird_threshold;
        bool activate_stm_fs_transceiver;
        u16 max_packet_count;
        u32 max_transfer_size;
-- 
2.11.0

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to