On Mon, Apr 08, 2019 at 10:48:47AM -0400, Michael Zhivich wrote: > qlcnic driver uses u16 to store SPEED_UKNOWN ethtool constant, > which is defined as -1, resulting in value truncation and > thus incorrect test results against SPEED_UNKNOWN. > > For example, the following test will print "False": > > u16 speed = SPEED_UNKNOWN; > > if (speed == SPEED_UNKNOWN) > printf("True"); > else > printf("False"); > > Change storage of speed to use u32 to avoid this issue. > > Signed-off-by: Michael Zhivich <mzhiv...@akamai.com>
Reviewed-by: Andrew Lunn <and...@lunn.ch> Andrew