Re: [PATCH v2 02/10] rtc: max77686: Use ARRAY_SIZE() instead of current array length

2016-01-22 Thread Javier Martinez Canillas
Hello Laxman, Thanks a lot for your feedback and acks. On 01/22/2016 06:39 AM, Laxman Dewangan wrote: On Friday 22 January 2016 01:53 AM, Javier Martinez Canillas wrote: It is better to use the ARRAY_SIZE() macro instead of the array length to avoid bugs if the array is later changed and the

Re: [PATCH v2 02/10] rtc: max77686: Use ARRAY_SIZE() instead of current array length

2016-01-22 Thread Laxman Dewangan
On Friday 22 January 2016 01:53 AM, Javier Martinez Canillas wrote: It is better to use the ARRAY_SIZE() macro instead of the array length to avoid bugs if the array is later changed and the length not updated. Signed-off-by: Javier Martinez Canillas Reviewed-by: Krzysztof Kozlowski Acked-

Re: [PATCH v2 02/10] rtc: max77686: Use ARRAY_SIZE() instead of current array length

2016-01-21 Thread Krzysztof Kozlowski
On 22.01.2016 05:23, Javier Martinez Canillas wrote: > It is better to use the ARRAY_SIZE() macro instead of the array length > to avoid bugs if the array is later changed and the length not updated. > > Signed-off-by: Javier Martinez Canillas > Reviewed-by: Krzysztof Kozlowski > > --- Tested

[PATCH v2 02/10] rtc: max77686: Use ARRAY_SIZE() instead of current array length

2016-01-21 Thread Javier Martinez Canillas
It is better to use the ARRAY_SIZE() macro instead of the array length to avoid bugs if the array is later changed and the length not updated. Signed-off-by: Javier Martinez Canillas Reviewed-by: Krzysztof Kozlowski --- Changes in v2: - Add Krzysztof Kozlowski's Reviewed-by tag to patch #2.