On Wed, Feb 3, 2010 at 3:38 PM, Stefan Ringel <stefan.rin...@arcor.de> wrote:
> signed-off-by: Stefan Ringel <stefan.rin...@arcor.de>
>
> --- 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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to