On Thu, Mar 24, 2011 at 11:31:08PM +0300, Dan Carpenter wrote: > On Thu, Mar 24, 2011 at 04:07:00PM -0400, Jarod Wilson wrote: > > Signed-off-by: Jarod Wilson <jarod@xxxxxxxxxx> > > Jarod, there is a lot of information missing from your change log... :/ Hrm, I'm building the media stack with all warnings fatal, so this was just a quick fix to silence the compiler warning, didn't really look into it at all. > > CC: devel@xxxxxxxxxxxxxxxxxxxx > > --- > > drivers/staging/tm6000/tm6000-video.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/drivers/staging/tm6000/tm6000-video.c b/drivers/staging/tm6000/tm6000-video.c > > index c80a316..bfebedd 100644 > > --- a/drivers/staging/tm6000/tm6000-video.c > > +++ b/drivers/staging/tm6000/tm6000-video.c > > @@ -228,7 +228,7 @@ static int copy_streams(u8 *data, unsigned long len, > > unsigned long header = 0; > > int rc = 0; > > unsigned int cmd, cpysize, pktsize, size, field, block, line, pos = 0; > > - struct tm6000_buffer *vbuf; > > + struct tm6000_buffer *vbuf = NULL; > > char *voutp = NULL; > > unsigned int linewidth; > > > > This looks like a real bug versus just a GCC warning. It was introduced > in 8aff8ba95155df "[media] tm6000: add radio support to the driver". > I've added Dmitri to the CC list. Thanks much, will try to pay more attention next time. ;) -- Jarod Wilson jarod@xxxxxxxxxx -- 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
