Re: [PATCH] mpis: cavium-octeon/executive/cvmx-l2c.c: fix uninitialized variable

2013-01-16 Thread Geert Uytterhoeven
On Wed, Jan 16, 2013 at 11:57 AM, Ralf Baechle wrote: > On Mon, Jan 14, 2013 at 10:47:03PM +0100, Cong Ding wrote: > >> the variable dummy is used without initialization. > > Interesting - I wonder how you found this one. My compiler (gcc 4.7) > doesn't warn about this one. Probably older gcc do

Re: [PATCH] mpis: cavium-octeon/executive/cvmx-l2c.c: fix uninitialized variable

2013-01-16 Thread David Daney
On 01/16/2013 02:57 AM, Ralf Baechle wrote: On Mon, Jan 14, 2013 at 10:47:03PM +0100, Cong Ding wrote: the variable dummy is used without initialization. Interesting - I wonder how you found this one. My compiler (gcc 4.7) doesn't warn about this one. I get no warnings either. So I wonde

Re: [PATCH] mpis: cavium-octeon/executive/cvmx-l2c.c: fix uninitialized variable

2013-01-16 Thread Ralf Baechle
On Mon, Jan 14, 2013 at 10:47:03PM +0100, Cong Ding wrote: > the variable dummy is used without initialization. Interesting - I wonder how you found this one. My compiler (gcc 4.7) doesn't warn about this one. Nor does gcc notice that the whole summing up business is wasted efford. So here's m