On Tue, Mar 23, 2010 at 4:42 PM, Ricardo Maraschini <xrmarsx@xxxxxxxxx> wrote: > --- a/linux/drivers/media/video/cx88/cx88-dvb.c Tue Mar 23 16:17:11 2010 -0300 > +++ b/linux/drivers/media/video/cx88/cx88-dvb.c Tue Mar 23 17:29:29 2010 -0300 > @@ -1401,7 +1401,8 @@ > case CX88_BOARD_SAMSUNG_SMT_7020: > dev->ts_gen_cntrl = 0x08; > > - struct cx88_core *core = dev->core; > + struct cx88_core *core; > + core = dev->core; > > cx_set(MO_GP0_IO, 0x0101); > > > > Signed-off-by: Ricardo Maraschini <ricardo.maraschini@xxxxxxxxx> How do you think this actually addresses the warning in question? You still have the declaration of the variable in the middle of the switch statement. Devin -- Devin J. Heitmueller - Kernel Labs http://www.kernellabs.com -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html