Re: [PATCH] DVB: lgdt330x frontend: some bug fixes & add lgdt3303 support

[Mauro Carvalho Chehab <mchehab@brturbo.com.br>]

	This should't be applied to 2.6.13. It does contain a hack at V4L code,
since mute_tda9887 is implemented outside tda9887.c module and could
potentially cause troubles since there are some work to provide it on a
correct way.
	 It should be applied to -mm and go to mainstream only after provided a
correct implementation.

Mkrufky,
	Please avoid trying to submit yet experimental patches to mainstream.

Mauro.

Em Dom, 2005-08-07 às 20:08 -0400, Michael Krufky escreveu:
> For 2.6.13, if possible.  Patch generated against 2.6.13-rc6
> 
> anexo documento em texto simples (lgdt330x-structure-update.patch)

> diff -u linux-2.6.13/drivers/media/dvb/frontends/lgdt330x.c linux/drivers/media/dvb/frontends/lgdt330x.c
> +
> +#ifdef MUTE_TDA9887
> +static int i2c_write_ntsc_demod (struct lgdt330x_state* state, u8 buf[2])
> +{
> +	struct i2c_msg msg =
> +		{ .addr = 0x43,
> +		  .flags = 0, 
> +		  .buf = buf,
> +		  .len = 2 };
> +	int err;
> +
> +	if ((err = i2c_transfer(state->i2c, &msg, 1)) != 1) {
> +			printk(KERN_WARNING "lgdt330x: %s error (addr %02x <- %02x, err = %i)\n", __FUNCTION__, msg.buf[0], msg.buf[1], err);
> +		if (err < 0)
> +			return err;
> +		else
> +			return -EREMOTEIO;
> +	}
> +	return 0;
> +}
> +
> +static void fiddle_with_ntsc_if_demod(struct lgdt330x_state* state)
> +{
> +	// Experimental code
> +	u8 buf0[] = {0x00, 0x20};
> +	u8 buf1[] = {0x01, 0x00};
> +	u8 buf2[] = {0x02, 0x00};
> +
> +	i2c_write_ntsc_demod(state, buf0);
> +	i2c_write_ntsc_demod(state, buf1);
> +	i2c_write_ntsc_demod(state, buf2);
> +}
> +#endif