Re: [linux-dvb] [PATCH] experimental support for C-1501

[Sigmund Augdal <sigmund@snap.tv>]

On Mon, 2008-06-09 at 08:21 +0300, Arthur Konovalov wrote:
> Oliver Endriss wrote:
> > Sigmund Augdal wrote:
> >> Here is a new version. This one passes checkpatch without warnings. I
> >> removed the read_pwm function, as it always uses the fallback path for
> >> my card (and frankly I have no idea wether it is actually relevant at
> >> all for this kind of card). Furthermore the tda10023 driver doesn't seem
> >> to use this value for anything.
> > 
> > Any issues with this patch? If not I will commit it next weekend.
> 
> At the first glance it works fine, thanks Sigmund.
Good to hear.
> Although for multiproto drivers I slightly modified patches and replaced some 
> files from linux-dvb tree (tda10021.c, tda10023.c, tda1002x.h).
This i was only to get the patch to apply, because the context had
changed slightly (because of the definitions for the s2 cards), right?
No changes needed to the applied code? I tried this with a multiproto
tree also, and it works fine (but patch from one does not apply to the
other)

> 
> However, I don't know how necessary is tda827x-oops patch with multiproto.
I think it is necessary.
> 
> At compile have only one warning:
>    CC [M]  /usr/local/src/multiproto-2008-05-14/v4l/budget-av.o
> /usr/local/src/multiproto-2008-05-14/v4l/budget-av.c: In function 'frontend_init':
> /usr/local/src/multiproto-2008-05-14/v4l/budget-av.c:1306: warning: passing 
> argument 1 of '__a' from incompatible pointer type
This line is, to the best of what I can see not added by my patch,
neither in my version, or the version you provided for the multiproto
tree. If it is in fact part of my code I'd appreciate if you posted the
line in question, or the equivalent warning from an otherwise unchanged
hg master clone.



Best regards

Sigmund Augdal
> 
> 
> Regards,
> AK
> plain text document attachment (tda827x-oops-multiproto.patch)
> --- linux/drivers/media/dvb/frontends/tda827x.c.old	2008-06-06 13:57:57.000000000 +0300
> +++ linux/drivers/media/dvb/frontends/tda827x.c	2008-06-06 13:59:52.000000000 +0300
> @@ -554,7 +554,7 @@
>  	struct tda827x_priv *priv = fe->tuner_priv;
>  	unsigned char buf[] = {0x22, 0x01};
>  	int arg;
> -	struct i2c_msg msg = { .addr = priv->i2c_addr, .flags = 0,
> +	struct i2c_msg msg = { .flags = 0,
>  			       .buf = buf, .len = sizeof(buf) };
>  
>  	if (NULL == priv->cfg) {
> @@ -562,6 +562,7 @@
>  		return;
>  	}
>  
> +	msg.addr = priv->i2c_addr;
>  	if (priv->cfg->config) {
>  		if (high)
>  			dprintk("setting LNA to high gain\n");
> plain text document attachment (c-1501-multiproto.patch)
> --- linux/drivers/media/dvb/ttpci/budget-ci.c.old	2008-06-06 13:39:23.000000000 +0300
> +++ linux/drivers/media/dvb/ttpci/budget-ci.c	2008-06-06 13:50:18.000000000 +0300
> @@ -51,6 +51,8 @@
>  #include "lnbp21.h"
>  #include "bsbe1.h"
>  #include "bsru6.h"
> +#include "tda1002x.h"
> +#include "tda827x.h"
>  
>  /*
>   * Regarding DEBIADDR_IR:
> @@ -1337,6 +1339,16 @@
>  	.refclock	= 27000000,
>  };
>  
> +static struct tda10023_config tda10023_config = {
> +	.demod_address = 0xc,
> +	.invert = 0,
> +	.xtal = 16000000,
> +	.pll_m = 11,
> +	.pll_p = 3,
> +	.pll_n = 1,
> +	.deltaf = 0xA511,
> +};
> +
>  static void frontend_init(struct budget_ci *budget_ci)
>  {
>  	switch (budget_ci->budget.dev->pci->subsystem_device) {
> @@ -1404,7 +1416,20 @@
>  				budget_ci->budget.dvb_frontend = NULL;
>  			}
>  		}
> -
> +		break;
> +	case 0x101a: /* TT Budget-C-1501 (philips tda10023/philips tda8274A) */
> +		budget_ci->budget.dvb_frontend =
> +			dvb_attach(tda10023_attach, &tda10023_config,
> +				   &budget_ci->budget.i2c_adap, 0x48);
> +		if (budget_ci->budget.dvb_frontend) {
> +			if (dvb_attach(tda827x_attach,
> +				       budget_ci->budget.dvb_frontend, 0x61,
> +				       &budget_ci->budget.i2c_adap, NULL)
> +			    == NULL)
> +				printk(KERN_ERR "%s: No tda827x found!\n",
> +				       __func__);
> +			break;
> +		}
>  		break;
>  
>  	case 0x1019:		// TT S2-3200 PCI
> @@ -1535,6 +1560,7 @@
>  MAKE_BUDGET_INFO(ttbtci, "TT-Budget-T-CI PCI", BUDGET_TT);
>  MAKE_BUDGET_INFO(ttbcci, "TT-Budget-C-CI PCI", BUDGET_TT);
>  MAKE_BUDGET_INFO(tt3200, "TT-Budget S2-3200 PCI", BUDGET_TT);
> +MAKE_BUDGET_INFO(ttc1501, "TT-Budget C-1501 PCI", BUDGET_TT);
>  
>  static struct pci_device_id pci_tbl[] = {
>  	MAKE_EXTENSION_PCI(ttbci, 0x13c2, 0x100c),
> @@ -1544,6 +1570,7 @@
>  	MAKE_EXTENSION_PCI(ttbtci, 0x13c2, 0x1012),
>  	MAKE_EXTENSION_PCI(ttbs2, 0x13c2, 0x1017),
>  	MAKE_EXTENSION_PCI(tt3200, 0x13c2, 0x1019),
> +	MAKE_EXTENSION_PCI(ttc1501, 0x13c2, 0x101A),
>  	{
>  	 .vendor = 0,
>  	 }
> _______________________________________________
> linux-dvb mailing list
> linux-dvb@linuxtv.org
> http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb


_______________________________________________
linux-dvb mailing list
linux-dvb@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb