On Wed, Feb 3, 2010 at 3:38 PM, Stefan Ringel <stefan.ringel@xxxxxxxx> wrote: > signed-off-by: Stefan Ringel <stefan.ringel@xxxxxxxx> > > --- a/drivers/media/dvb/frontends/zl10353.h > +++ b/drivers/media/dvb/frontends/zl10353.h > @@ -45,6 +45,8 @@ struct zl10353_config > /* clock control registers (0x51-0x54) */ > u8 clock_ctl_1; /* default: 0x46 */ > u8 pll_0; /* default: 0x15 */ > + > + int tm6000:1; > }; Why is this being submitted as its own patch? It is code that is not used by *anything*. If you really did require a new field in the zl10353 config, that field should be added in the same patch as whatever requires it. 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