Re: [PATCH][next] drm/bridge: sii902x: fix comparision of u32 with less than zero

2019-06-07 Thread Andrzej Hajda
On 03.06.2019 16:21, Colin King wrote: > From: Colin Ian King > > The less than check for the variable num_lanes is always going to be > false because the variable is a u32. Fix this by making num_lanes an > int and also make loop index i an int too. > > Addresses-Coverity: ("Unsigned compared

Re: [PATCH][next] drm/bridge: sii902x: fix comparision of u32 with less than zero

2019-06-07 Thread Colin Ian King
On 07/06/2019 10:41, Andrzej Hajda wrote: > On 03.06.2019 16:21, Colin King wrote: >> From: Colin Ian King >> >> The less than check for the variable num_lanes is always going to be >> false because the variable is a u32. Fix this by making num_lanes an >> int and also make loop index i an int

Re: [PATCH][next] drm/bridge: sii902x: fix comparision of u32 with less than zero

2019-06-07 Thread Andrzej Hajda
On 03.06.2019 16:21, Colin King wrote: > From: Colin Ian King > > The less than check for the variable num_lanes is always going to be > false because the variable is a u32. Fix this by making num_lanes an > int and also make loop index i an int too. > > Addresses-Coverity: ("Unsigned compared

Re: [PATCH][next] drm/bridge: sii902x: fix comparision of u32 with less than zero

2019-06-03 Thread Jyri Sarha
On 03/06/2019 17:21, Colin King wrote: > From: Colin Ian King > > The less than check for the variable num_lanes is always going to be > false because the variable is a u32. Fix this by making num_lanes an > int and also make loop index i an int too. > > Addresses-Coverity: ("Unsigned compared

[PATCH][next] drm/bridge: sii902x: fix comparision of u32 with less than zero

2019-06-03 Thread Colin King
From: Colin Ian King The less than check for the variable num_lanes is always going to be false because the variable is a u32. Fix this by making num_lanes an int and also make loop index i an int too. Addresses-Coverity: ("Unsigned compared against 0") Fixes: ff5781634c41 ("drm/bridge: