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 <tovmasya@xxxxxxxxxxxx> --- 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html