Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-03-01 Thread Michael Ellerman
Mathieu Malaterre writes: > On Thu, Mar 1, 2018 at 3:55 AM, Michael Ellerman wrote: >> Mathieu Malaterre writes: >> >>> When neither CONFIG_ALTIVEC, nor CONFIG_VSX or CONFIG_PPC64 is defined, the >>> array feature_properties is defined

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-03-01 Thread Michael Ellerman
Mathieu Malaterre writes: > On Thu, Mar 1, 2018 at 3:55 AM, Michael Ellerman wrote: >> Mathieu Malaterre writes: >> >>> When neither CONFIG_ALTIVEC, nor CONFIG_VSX or CONFIG_PPC64 is defined, the >>> array feature_properties is defined as an empty array, which in turn >>> triggers the

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-28 Thread Mathieu Malaterre
On Thu, Mar 1, 2018 at 3:55 AM, Michael Ellerman wrote: > Mathieu Malaterre writes: > >> When neither CONFIG_ALTIVEC, nor CONFIG_VSX or CONFIG_PPC64 is defined, the >> array feature_properties is defined as an empty array, which in turn >> triggers the

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-28 Thread Mathieu Malaterre
On Thu, Mar 1, 2018 at 3:55 AM, Michael Ellerman wrote: > Mathieu Malaterre writes: > >> When neither CONFIG_ALTIVEC, nor CONFIG_VSX or CONFIG_PPC64 is defined, the >> array feature_properties is defined as an empty array, which in turn >> triggers the following warning (treated as error on

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-28 Thread Michael Ellerman
Mathieu Malaterre writes: > When neither CONFIG_ALTIVEC, nor CONFIG_VSX or CONFIG_PPC64 is defined, the > array feature_properties is defined as an empty array, which in turn > triggers the following warning (treated as error on W=1): > > CC arch/powerpc/kernel/prom.o >

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-28 Thread Michael Ellerman
Mathieu Malaterre writes: > When neither CONFIG_ALTIVEC, nor CONFIG_VSX or CONFIG_PPC64 is defined, the > array feature_properties is defined as an empty array, which in turn > triggers the following warning (treated as error on W=1): > > CC arch/powerpc/kernel/prom.o >

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-28 Thread Andy Shevchenko
On Tue, Feb 27, 2018 at 10:42 PM, Segher Boessenkool wrote: > On Tue, Feb 27, 2018 at 05:52:06PM +0200, Andy Shevchenko wrote: >> On Tue, Feb 27, 2018 at 9:44 AM, Mathieu Malaterre wrote: >> > On Tue, Feb 27, 2018 at 8:33 AM, Christophe LEROY >> >

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-28 Thread Andy Shevchenko
On Tue, Feb 27, 2018 at 10:42 PM, Segher Boessenkool wrote: > On Tue, Feb 27, 2018 at 05:52:06PM +0200, Andy Shevchenko wrote: >> On Tue, Feb 27, 2018 at 9:44 AM, Mathieu Malaterre wrote: >> > On Tue, Feb 27, 2018 at 8:33 AM, Christophe LEROY >> > wrote: >> >> > Much simpler is just add >>

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-27 Thread Segher Boessenkool
On Tue, Feb 27, 2018 at 05:52:06PM +0200, Andy Shevchenko wrote: > On Tue, Feb 27, 2018 at 9:44 AM, Mathieu Malaterre wrote: > > On Tue, Feb 27, 2018 at 8:33 AM, Christophe LEROY > > wrote: > > > Much simpler is just add > > > > if

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-27 Thread Segher Boessenkool
On Tue, Feb 27, 2018 at 05:52:06PM +0200, Andy Shevchenko wrote: > On Tue, Feb 27, 2018 at 9:44 AM, Mathieu Malaterre wrote: > > On Tue, Feb 27, 2018 at 8:33 AM, Christophe LEROY > > wrote: > > > Much simpler is just add > > > > if (ARRAY_SIZE() == 0) > > return; > > >> Or

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-27 Thread Mathieu Malaterre
On Tue, Feb 27, 2018 at 4:52 PM, Andy Shevchenko wrote: > On Tue, Feb 27, 2018 at 9:44 AM, Mathieu Malaterre wrote: >> On Tue, Feb 27, 2018 at 8:33 AM, Christophe LEROY >> wrote: > >> Much simpler is just add >>

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-27 Thread Mathieu Malaterre
On Tue, Feb 27, 2018 at 4:52 PM, Andy Shevchenko wrote: > On Tue, Feb 27, 2018 at 9:44 AM, Mathieu Malaterre wrote: >> On Tue, Feb 27, 2018 at 8:33 AM, Christophe LEROY >> wrote: > >> Much simpler is just add >> >> if (ARRAY_SIZE() == 0) >> return; > >>> Or add in front: >>>

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-27 Thread Andy Shevchenko
On Tue, Feb 27, 2018 at 9:44 AM, Mathieu Malaterre wrote: > On Tue, Feb 27, 2018 at 8:33 AM, Christophe LEROY > wrote: > Much simpler is just add > > if (ARRAY_SIZE() == 0) > return; >> Or add in front: >> if

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-27 Thread Andy Shevchenko
On Tue, Feb 27, 2018 at 9:44 AM, Mathieu Malaterre wrote: > On Tue, Feb 27, 2018 at 8:33 AM, Christophe LEROY > wrote: > Much simpler is just add > > if (ARRAY_SIZE() == 0) > return; >> Or add in front: >> if (!ARRAY_SIZE(feature_properties)) >> return; > > (not

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-26 Thread Mathieu Malaterre
On Tue, Feb 27, 2018 at 8:33 AM, Christophe LEROY wrote: > > > Le 27/02/2018 à 08:25, Mathieu Malaterre a écrit : >> >> On Mon, Feb 26, 2018 at 3:45 PM, Andy Shevchenko >> wrote: >>> >>> On Mon, Feb 26, 2018 at 4:44 PM, Andy Shevchenko >>>

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-26 Thread Mathieu Malaterre
On Tue, Feb 27, 2018 at 8:33 AM, Christophe LEROY wrote: > > > Le 27/02/2018 à 08:25, Mathieu Malaterre a écrit : >> >> On Mon, Feb 26, 2018 at 3:45 PM, Andy Shevchenko >> wrote: >>> >>> On Mon, Feb 26, 2018 at 4:44 PM, Andy Shevchenko >>> wrote: On Sun, Feb 25, 2018 at 7:22 PM,

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-26 Thread Christophe LEROY
Le 27/02/2018 à 08:25, Mathieu Malaterre a écrit : On Mon, Feb 26, 2018 at 3:45 PM, Andy Shevchenko wrote: On Mon, Feb 26, 2018 at 4:44 PM, Andy Shevchenko wrote: On Sun, Feb 25, 2018 at 7:22 PM, Mathieu Malaterre

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-26 Thread Christophe LEROY
Le 27/02/2018 à 08:25, Mathieu Malaterre a écrit : On Mon, Feb 26, 2018 at 3:45 PM, Andy Shevchenko wrote: On Mon, Feb 26, 2018 at 4:44 PM, Andy Shevchenko wrote: On Sun, Feb 25, 2018 at 7:22 PM, Mathieu Malaterre wrote: static void __init check_cpu_feature_properties(unsigned long

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-26 Thread Mathieu Malaterre
On Mon, Feb 26, 2018 at 3:45 PM, Andy Shevchenko wrote: > On Mon, Feb 26, 2018 at 4:44 PM, Andy Shevchenko > wrote: >> On Sun, Feb 25, 2018 at 7:22 PM, Mathieu Malaterre wrote: > >>> static void __init

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-26 Thread Mathieu Malaterre
On Mon, Feb 26, 2018 at 3:45 PM, Andy Shevchenko wrote: > On Mon, Feb 26, 2018 at 4:44 PM, Andy Shevchenko > wrote: >> On Sun, Feb 25, 2018 at 7:22 PM, Mathieu Malaterre wrote: > >>> static void __init check_cpu_feature_properties(unsigned long node) >>> { >>> - unsigned long i; >>>

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-26 Thread Andy Shevchenko
On Mon, Feb 26, 2018 at 4:44 PM, Andy Shevchenko wrote: > On Sun, Feb 25, 2018 at 7:22 PM, Mathieu Malaterre wrote: >> static void __init check_cpu_feature_properties(unsigned long node) >> { >> - unsigned long i; >> struct

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-26 Thread Andy Shevchenko
On Mon, Feb 26, 2018 at 4:44 PM, Andy Shevchenko wrote: > On Sun, Feb 25, 2018 at 7:22 PM, Mathieu Malaterre wrote: >> static void __init check_cpu_feature_properties(unsigned long node) >> { >> - unsigned long i; >> struct feature_property *fp = feature_properties; >>

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-26 Thread Andy Shevchenko
On Sun, Feb 25, 2018 at 7:22 PM, Mathieu Malaterre wrote: > When neither CONFIG_ALTIVEC, nor CONFIG_VSX or CONFIG_PPC64 is defined, the > array feature_properties is defined as an empty array, which in turn > triggers the following warning (treated as error on W=1): > > CC

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-26 Thread Andy Shevchenko
On Sun, Feb 25, 2018 at 7:22 PM, Mathieu Malaterre wrote: > When neither CONFIG_ALTIVEC, nor CONFIG_VSX or CONFIG_PPC64 is defined, the > array feature_properties is defined as an empty array, which in turn > triggers the following warning (treated as error on W=1): > > CC

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-25 Thread Segher Boessenkool
Hi! On Sun, Feb 25, 2018 at 06:22:16PM +0100, Mathieu Malaterre wrote: > When neither CONFIG_ALTIVEC, nor CONFIG_VSX or CONFIG_PPC64 is defined, the > array feature_properties is defined as an empty array, which in turn > triggers the following warning (treated as error on W=1): > - for (i =

Re: [PATCH 01/21] powerpc: Remove warning on array size when empty

2018-02-25 Thread Segher Boessenkool
Hi! On Sun, Feb 25, 2018 at 06:22:16PM +0100, Mathieu Malaterre wrote: > When neither CONFIG_ALTIVEC, nor CONFIG_VSX or CONFIG_PPC64 is defined, the > array feature_properties is defined as an empty array, which in turn > triggers the following warning (treated as error on W=1): > - for (i =