Re: [PATCH v2] serial: vt8500_serial: Fix a parameter of find_first_zero_bit.

2016-08-24 Thread Arnd Bergmann
On Wednesday, August 24, 2016 7:06:58 AM CEST Christophe JAILLET wrote: > The 2nd parameter of 'find_first_zero_bit' is the number of bits to search. > In this case, we are passing 'sizeof(vt8500_ports_in_use)'. > 'vt8500_ports_in_use' is an 'unsigned long'. So the sizeof is likely to > return 4

Re: [PATCH v2] serial: vt8500_serial: Fix a parameter of find_first_zero_bit.

2016-08-24 Thread Arnd Bergmann
On Wednesday, August 24, 2016 7:06:58 AM CEST Christophe JAILLET wrote: > The 2nd parameter of 'find_first_zero_bit' is the number of bits to search. > In this case, we are passing 'sizeof(vt8500_ports_in_use)'. > 'vt8500_ports_in_use' is an 'unsigned long'. So the sizeof is likely to > return 4

[PATCH v2] serial: vt8500_serial: Fix a parameter of find_first_zero_bit.

2016-08-23 Thread Christophe JAILLET
The 2nd parameter of 'find_first_zero_bit' is the number of bits to search. In this case, we are passing 'sizeof(vt8500_ports_in_use)'. 'vt8500_ports_in_use' is an 'unsigned long'. So the sizeof is likely to return 4 on a 32 bits kernel. A few lines below, we check if it is below

[PATCH v2] serial: vt8500_serial: Fix a parameter of find_first_zero_bit.

2016-08-23 Thread Christophe JAILLET
The 2nd parameter of 'find_first_zero_bit' is the number of bits to search. In this case, we are passing 'sizeof(vt8500_ports_in_use)'. 'vt8500_ports_in_use' is an 'unsigned long'. So the sizeof is likely to return 4 on a 32 bits kernel. A few lines below, we check if it is below