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

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

[Sigmund Augdal]

[-- Attachment #1: Type: text/plain, Size: 281 bytes --]

The following experimental patch adds support for the technotrend budget
C-1501 dvb-c card. The parameters used to configure the tda10023 demod
chip are largely determined experimentally, but works quite for me in my
initial tests.

Signed-Off-By: Sigmund Augdal <sigmund@snap.tv>

[-- Attachment #2: c-1501.patch --]
[-- Type: text/x-patch, Size: 2202 bytes --]

diff -r 6541620a09b7 linux/drivers/media/dvb/ttpci/budget-ci.c
--- a/linux/drivers/media/dvb/ttpci/budget-ci.c	Tue Jun 03 10:32:16 2008 -0300
+++ b/linux/drivers/media/dvb/ttpci/budget-ci.c	Wed Jun 04 15:03:45 2008 +0200
@@ -46,6 +46,8 @@
 #include "lnbp21.h"
 #include "bsbe1.h"
 #include "bsru6.h"
+#include "tda1002x.h"
+#include "tda827x.h"
 
 /*
  * Regarding DEBIADDR_IR:
@@ -1069,6 +1071,32 @@
 
 
 
+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 u8 read_pwm(struct budget_ci *budget_ci)
+{
+	u8 b = 0xff;
+	u8 pwm;
+	struct i2c_msg msg[] = { {.addr = 0x50,.flags = 0,.buf = &b,.len = 1},
+	{.addr = 0x50,.flags = I2C_M_RD,.buf = &pwm,.len = 1}
+	};
+
+	if ((i2c_transfer(&budget_ci->budget.i2c_adap, msg, 2) != 2)
+	    || (pwm == 0xff))
+		pwm = 0x48;
+
+	return pwm;
+}
+
+
 static void frontend_init(struct budget_ci *budget_ci)
 {
 	switch (budget_ci->budget.dev->pci->subsystem_device) {
@@ -1138,6 +1166,17 @@
 		}
 
 		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, read_pwm(budget_ci));
+		if (budget_ci->budget.dvb_frontend) {
+			if (dvb_attach(tda827x_attach, budget_ci->budget.dvb_frontend, 0x61,
+				       &budget_ci->budget.i2c_adap, 0) == NULL)
+				printk("%s: No tda827x found!\n", __FUNCTION__);
+			break;
+		}
+		break;
+         
 	}
 
 	if (budget_ci->budget.dvb_frontend == NULL) {
@@ -1226,6 +1265,7 @@
 MAKE_BUDGET_INFO(ttbt2, "TT-Budget/WinTV-NOVA-T	 PCI", BUDGET_TT);
 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(ttc1501, "TT-Budget C-1501 PCI", BUDGET_TT);
 
 static struct pci_device_id pci_tbl[] = {
 	MAKE_EXTENSION_PCI(ttbci, 0x13c2, 0x100c),
@@ -1234,6 +1274,7 @@
 	MAKE_EXTENSION_PCI(ttbt2, 0x13c2, 0x1011),
 	MAKE_EXTENSION_PCI(ttbtci, 0x13c2, 0x1012),
 	MAKE_EXTENSION_PCI(ttbs2, 0x13c2, 0x1017),
+	MAKE_EXTENSION_PCI(ttc1501, 0x13c2, 0x101A),
 	{
 	 .vendor = 0,
 	 }

[-- Attachment #3: Type: text/plain, Size: 150 bytes --]

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

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

[Antti Palosaari]

Sigmund Augdal wrote:
> The following experimental patch adds support for the technotrend budget
> C-1501 dvb-c card. The parameters used to configure the tda10023 demod
> chip are largely determined experimentally, but works quite for me in my
> initial tests.

You finally found correct values :) I see that it uses same clock 
settings than Anysee. Also deltaf could be correct because I remember 
from my tests that it cannot set wrong otherwise it will not work at 
all. How did you find defltaf?

Your patch has at least coding style issues, please run make checkpatch 
fix errors and resend patch.

regards
Antti

> 
> Signed-Off-By: Sigmund Augdal <sigmund@snap.tv>
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> linux-dvb mailing list
> linux-dvb@linuxtv.org
> http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb


-- 
http://palosaari.fi/

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

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

[Sigmund Augdal]

ons, 04.06.2008 kl. 16.58 +0300, skrev Antti Palosaari:
> Sigmund Augdal wrote:
> > The following experimental patch adds support for the technotrend budget
> > C-1501 dvb-c card. The parameters used to configure the tda10023 demod
> > chip are largely determined experimentally, but works quite for me in my
> > initial tests.
> 
> You finally found correct values :) I see that it uses same clock 
> settings than Anysee. Also deltaf could be correct because I remember 
> from my tests that it cannot set wrong otherwise it will not work at 
> all. How did you find defltaf?
I guessed the clock settings based on how the same tuner is used by a
different demod. The deltaf value was found by trial and error (helped
by some scripting). deltaf values slightly off will also work, but
tuning will be very slow. I also think the deltaf value will depend on
the bandwidth of the transponder tuned. All transponders I've tested
with are 8MHz, but I think other values are possible, and these will
most likely not work. I submitted the patch anyway in the hope that some
broader testing might help uncover any remaining problems.
> 
> Your patch has at least coding style issues, please run make checkpatch 
> fix errors and resend patch.
I was trying to follow the guidelines, but I guess I wasn't doing that
good enough. Will try to clean that up and resend soon.

regards
Sigmund
> 
> regards
> Antti
> 
> > 
> > Signed-Off-By: Sigmund Augdal <sigmund@snap.tv>
> > 
> > 
> > ------------------------------------------------------------------------
> > 
> > _______________________________________________
> > 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

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

[Sigmund Augdal]

[-- Attachment #1: Type: text/plain, Size: 2383 bytes --]

On Wed, 2008-06-04 at 16:37 +0200, Sigmund Augdal wrote:
> ons, 04.06.2008 kl. 16.58 +0300, skrev Antti Palosaari:
> > Sigmund Augdal wrote:
> > > The following experimental patch adds support for the technotrend budget
> > > C-1501 dvb-c card. The parameters used to configure the tda10023 demod
> > > chip are largely determined experimentally, but works quite for me in my
> > > initial tests.
> > 
> > You finally found correct values :) I see that it uses same clock 
> > settings than Anysee. Also deltaf could be correct because I remember 
> > from my tests that it cannot set wrong otherwise it will not work at 
> > all. How did you find defltaf?
> I guessed the clock settings based on how the same tuner is used by a
> different demod. The deltaf value was found by trial and error (helped
> by some scripting). deltaf values slightly off will also work, but
> tuning will be very slow. I also think the deltaf value will depend on
> the bandwidth of the transponder tuned. All transponders I've tested
> with are 8MHz, but I think other values are possible, and these will
> most likely not work. I submitted the patch anyway in the hope that some
> broader testing might help uncover any remaining problems.
> > 
> > Your patch has at least coding style issues, please run make checkpatch 
> > fix errors and resend patch.
> I was trying to follow the guidelines, but I guess I wasn't doing that
> good enough. Will try to clean that up and resend soon.
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.

Best regards

Sigmund Augdal
> 
> regards
> Sigmund
> > 
> > regards
> > Antti
> > 
> > > 
> > > Signed-Off-By: Sigmund Augdal <sigmund@snap.tv>
> > > 
> > > 
> > > ------------------------------------------------------------------------
> > > 
> > > _______________________________________________
> > > 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
> 

[-- Attachment #2: c-1501_try2.patch --]
[-- Type: text/x-patch, Size: 1887 bytes --]

Signed-off-by: Sigmund Augdal <sigmund@snap.tv>
diff -r 6541620a09b7 linux/drivers/media/dvb/ttpci/budget-ci.c
--- a/linux/drivers/media/dvb/ttpci/budget-ci.c	Tue Jun 03 10:32:16 2008 -0300
+++ b/linux/drivers/media/dvb/ttpci/budget-ci.c	Thu Jun 05 11:02:28 2008 +0200
@@ -46,6 +46,8 @@
 #include "lnbp21.h"
 #include "bsbe1.h"
 #include "bsru6.h"
+#include "tda1002x.h"
+#include "tda827x.h"
 
 /*
  * Regarding DEBIADDR_IR:
@@ -1069,6 +1071,16 @@
 
 
 
+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) {
@@ -1137,6 +1149,20 @@
 			}
 		}
 
+		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;
 	}
 
@@ -1226,6 +1252,7 @@
 MAKE_BUDGET_INFO(ttbt2, "TT-Budget/WinTV-NOVA-T	 PCI", BUDGET_TT);
 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(ttc1501, "TT-Budget C-1501 PCI", BUDGET_TT);
 
 static struct pci_device_id pci_tbl[] = {
 	MAKE_EXTENSION_PCI(ttbci, 0x13c2, 0x100c),
@@ -1234,6 +1261,7 @@
 	MAKE_EXTENSION_PCI(ttbt2, 0x13c2, 0x1011),
 	MAKE_EXTENSION_PCI(ttbtci, 0x13c2, 0x1012),
 	MAKE_EXTENSION_PCI(ttbs2, 0x13c2, 0x1017),
+	MAKE_EXTENSION_PCI(ttc1501, 0x13c2, 0x101A),
 	{
 	 .vendor = 0,
 	 }

[-- Attachment #3: Type: text/plain, Size: 150 bytes --]

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

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

[Oliver Endriss]

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.

CU
Oliver

-- 
----------------------------------------------------------------
VDR Remote Plugin 0.4.0: http://www.escape-edv.de/endriss/vdr/
----------------------------------------------------------------

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

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

[Arthur Konovalov]

[-- Attachment #1: Type: text/plain, Size: 1043 bytes --]

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.
Although for multiproto drivers I slightly modified patches and replaced some 
files from linux-dvb tree (tda10021.c, tda10023.c, tda1002x.h).

However, I don't know how necessary is tda827x-oops patch with multiproto.

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


Regards,
AK

[-- Attachment #2: tda827x-oops-multiproto.patch --]
[-- Type: text/plain, Size: 608 bytes --]

--- 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");

[-- Attachment #3: c-1501-multiproto.patch --]
[-- Type: text/plain, Size: 1880 bytes --]

--- 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,
 	 }

[-- Attachment #4: Type: text/plain, Size: 150 bytes --]

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

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

[Sigmund Augdal]

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

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

[Arthur Konovalov]

Sigmund Augdal wrote:

> 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?
You're right.
Code not changed, in principle. Changes needed only for compile with multiproto 
due dvbs2 additions.

> 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.
You're right again. Sorry for misunderstanding.

Regards,
AK


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

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

[Simon Baxter]

Hi

What am I doing wrong - sorry for the potentially idiot error...

I got what I thought was the latest, which would include these c-1501 
patches yes??
hg clone http://linuxtv.org/hg/v4l-dvb


But there's no "linux/drivers/media/dvb/frontends/tda827x.c"   and the 
"linux/drivers/media/dvb/ttpci/budget-ci.c"   is not the same as the patches 
references.

[root@freddy v4l-dvb]# find . | grep tda827x.c
./.hg/store/data/linux/drivers/media/common/tuners/tda827x.c.i
./.hg/store/data/linux/drivers/media/dvb/frontends/tda827x.c.i
./linux/drivers/media/common/tuners/tda827x.c


Have I got the wrong repository??




----- Original Message ----- 
From: "Arthur Konovalov" <artlov@gmail.com>
To: <linux-dvb@linuxtv.org>
Sent: Monday, June 09, 2008 5:21 PM
Subject: Re: [linux-dvb] [PATCH] experimental support for C-1501


> 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.
> Although for multiproto drivers I slightly modified patches and replaced 
> some
> files from linux-dvb tree (tda10021.c, tda10023.c, tda1002x.h).
>
> However, I don't know how necessary is tda827x-oops patch with multiproto.
>
> 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
>
>
> Regards,
> AK
>


--------------------------------------------------------------------------------


> --- 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");
>


--------------------------------------------------------------------------------


> --- 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

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

[Arthur Konovalov]

Simon Baxter wrote:
> Hi
> 
> What am I doing wrong - sorry for the potentially idiot error...
> 
> I got what I thought was the latest, which would include these c-1501 
> patches yes??
> hg clone http://linuxtv.org/hg/v4l-dvb
> 
> 
> But there's no "linux/drivers/media/dvb/frontends/tda827x.c"   and the 
> "linux/drivers/media/dvb/ttpci/budget-ci.c"   is not the same as the patches 
> references.
> 
> [root@freddy v4l-dvb]# find . | grep tda827x.c
> ./.hg/store/data/linux/drivers/media/common/tuners/tda827x.c.i
> ./.hg/store/data/linux/drivers/media/dvb/frontends/tda827x.c.i
> ./linux/drivers/media/common/tuners/tda827x.c
> 
> 
> Have I got the wrong repository??
> 
Mentioned patches adopted to multiproto tree. For v4l-dvb please use 
Sigmund's original patches if these not committed yet.

Regards,
AK


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

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

[SG]

The patch works quite well and nearly all channels seem to work.

But when tuning to some radio channels I'll get this kernel message:

saa7146 (0) saa7146_i2c_writeout [irq]: timed out waiting for end of xfer

Also I'm not able to tune to 'transponder 386000000 6900000 0 3' which works
smoothly when using Win32.

initial transponder 386000000 6900000 0 3
 >>> tune to: 386:M64:C:6900:
WARNING: >>> tuning failed!!!
 >>> tune to: 386:M64:C:6900: (tuning failed)
WARNING: >>> tuning failed!!!
ERROR: initial tuning failed
dumping lists (0 services)
Done.

Any hints ?

Thanks and regards.
Martin

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.
>
> Best regards
>
> Sigmund Augdal

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

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

[Arthur Konovalov]

SG wrote:
> The patch works quite well and nearly all channels seem to work.
> 
> But when tuning to some radio channels I'll get this kernel message:
> 
> saa7146 (0) saa7146_i2c_writeout [irq]: timed out waiting for end of xfer
> 
> Also I'm not able to tune to 'transponder 386000000 6900000 0 3' which works
> smoothly when using Win32.
> 
> initial transponder 386000000 6900000 0 3
>  >>> tune to: 386:M64:C:6900:
> WARNING: >>> tuning failed!!!
>  >>> tune to: 386:M64:C:6900: (tuning failed)
> WARNING: >>> tuning failed!!!
> ERROR: initial tuning failed
> dumping lists (0 services)
> Done.

Yes, I discovered too that tuning to frequency 386MHz has no lock.
VDR channels.conf: TV3:386000:C0M64:C:6875:703:803:0:0:1003:16:1:0

At same time, 394MHz (and others) works.

Regards,
AK



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

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

[Sigmund Augdal]

On Mon, 2008-06-16 at 08:14 +0300, Arthur Konovalov wrote:
> SG wrote:
> > The patch works quite well and nearly all channels seem to work.
> > 
> > But when tuning to some radio channels I'll get this kernel message:
> > 
> > saa7146 (0) saa7146_i2c_writeout [irq]: timed out waiting for end of xfer
> > 
> > Also I'm not able to tune to 'transponder 386000000 6900000 0 3' which works
> > smoothly when using Win32.
> > 
> > initial transponder 386000000 6900000 0 3
> >  >>> tune to: 386:M64:C:6900:
> > WARNING: >>> tuning failed!!!
> >  >>> tune to: 386:M64:C:6900: (tuning failed)
> > WARNING: >>> tuning failed!!!
> > ERROR: initial tuning failed
> > dumping lists (0 services)
> > Done.
> 
> Yes, I discovered too that tuning to frequency 386MHz has no lock.
> VDR channels.conf: TV3:386000:C0M64:C:6875:703:803:0:0:1003:16:1:0
> 
> At same time, 394MHz (and others) works.
Hi.

Both transponders reported to not tune here has different symbolrates
from what I used for my testing. Maybe this is relevant in some way.
Could you please compare this with the channels that did tune to see if
there is a pattern?

About the i2c message, I get that every now and then here as well, but I
have not seen any ill effect from it. I also see that on some other TT
cards so I think that might be unrelated to the demod/tuner.

Regards

Sigmund Augdal
> 
> Regards,
> AK
> 
> 
> 
> _______________________________________________
> 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

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

[Sigmund Augdal]

On Mon, 2008-06-16 at 16:26 +0200, siestagomez@web.de wrote:
> Sigmund Augdal schrieb: 
> 
> > On Mon, 2008-06-16 at 08:14 +0300, Arthur Konovalov wrote:
> >> SG wrote:
> >> > The patch works quite well and nearly all channels seem to work.
> >> > 
> >> > But when tuning to some radio channels I'll get this kernel message:
> >> > 
> >> > saa7146 (0) saa7146_i2c_writeout [irq]: timed out waiting for end of xfer
> >> > 
> >> > Also I'm not able to tune to 'transponder 386000000 6900000 0 3' which works
> >> > smoothly when using Win32.
> >> > 
> >> > initial transponder 386000000 6900000 0 3
> >> >  >>> tune to: 386:M64:C:6900:
> >> > WARNING: >>> tuning failed!!!
> >> >  >>> tune to: 386:M64:C:6900: (tuning failed)
> >> > WARNING: >>> tuning failed!!!
> >> > ERROR: initial tuning failed
> >> > dumping lists (0 services)
> >> > Done. 
> >> 
> >> Yes, I discovered too that tuning to frequency 386MHz has no lock.
> >> VDR channels.conf: TV3:386000:C0M64:C:6875:703:803:0:0:1003:16:1:0 
> >> 
> >> At same time, 394MHz (and others) works.
> > Hi. 
> > 
> > Both transponders reported to not tune here has different symbolrates
> > from what I used for my testing. Maybe this is relevant in some way.
> > Could you please compare this with the channels that did tune to see if
> > there is a pattern? 
> > 
> > About the i2c message, I get that every now and then here as well, but I
> > have not seen any ill effect from it. I also see that on some other TT
> > cards so I think that might be unrelated to the demod/tuner. 
> > 
> > Regards 
> > 
> > Sigmund Augdal
> 
> The symbolrate is the same on all other working channels. 
> 
> Regarding the i2c message when watching video I'll get this only once but 
> when tuning to a radio channel my log gets flooded and it seems to hangup. 
How does radio or not make a difference? As far as I know the card is
DVB only and should not care whether the tuned transponder contains
radio or video or whatever. Could you send a list of transponders that
work (with tuning parameters) and ones that doesn't?

Regards

Sigmund Augdal
> 
> Martin 
> 


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

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

[SG]

On Mon, 16 Jun 2008 16:33:52 +0200
Sigmund Augdal <sigmund@snap.tv> wrote:

> On Mon, 2008-06-16 at 16:26 +0200, siestagomez@web.de wrote:
> > Sigmund Augdal schrieb: 
> > 
> > > On Mon, 2008-06-16 at 08:14 +0300, Arthur Konovalov wrote:
> > >> SG wrote:
> > >> > The patch works quite well and nearly all channels seem to work.
> > >> > 
> > >> > But when tuning to some radio channels I'll get this kernel message:
> > >> > 
> > >> > saa7146 (0) saa7146_i2c_writeout [irq]: timed out waiting for end of xfer
> > >> > 
> > >> > Also I'm not able to tune to 'transponder 386000000 6900000 0 3' which works
> > >> > smoothly when using Win32.
> > >> > 
> > >> > initial transponder 386000000 6900000 0 3
> > >> >  >>> tune to: 386:M64:C:6900:
> > >> > WARNING: >>> tuning failed!!!
> > >> >  >>> tune to: 386:M64:C:6900: (tuning failed)
> > >> > WARNING: >>> tuning failed!!!
> > >> > ERROR: initial tuning failed
> > >> > dumping lists (0 services)
> > >> > Done. 
> > >> 
> > >> Yes, I discovered too that tuning to frequency 386MHz has no lock.
> > >> VDR channels.conf: TV3:386000:C0M64:C:6875:703:803:0:0:1003:16:1:0 
> > >> 
> > >> At same time, 394MHz (and others) works.
> > > Hi. 
> > > 
> > > Both transponders reported to not tune here has different symbolrates
> > > from what I used for my testing. Maybe this is relevant in some way.
> > > Could you please compare this with the channels that did tune to see if
> > > there is a pattern? 
> > > 
> > > About the i2c message, I get that every now and then here as well, but I
> > > have not seen any ill effect from it. I also see that on some other TT
> > > cards so I think that might be unrelated to the demod/tuner. 
> > > 
> > > Regards 
> > > 
> > > Sigmund Augdal
> > 
> > The symbolrate is the same on all other working channels. 
> > 
> > Regarding the i2c message when watching video I'll get this only once but 
> > when tuning to a radio channel my log gets flooded and it seems to hangup. 
> How does radio or not make a difference? As far as I know the card is
> DVB only and should not care whether the tuned transponder contains
> radio or video or whatever. Could you send a list of transponders that
> work (with tuning parameters) and ones that doesn't?
> 

Odd today no problem with radio and kernel log.
Anyway here are a few entries from transponder.ini which I use for dvbscan:
C 362000000 6900000 NONE QAM64		OK
C 370000000 6900000 NONE QAM64		OK
C 378000000 6900000 NONE QAM64		OK
C 386000000 6900000 NONE QAM64		NOT OK
C 394000000 6900000 NONE QAM64		OK
C 402000000 6900000 NONE QAM256		NOT OK
C 410000000 6900000 NONE QAM64		OK
C 426000000 6900000 NONE QAM64		OK

I noticed when using Win32 the signal strenght is very poor on the non working transponders for linux-dvb.
Perhaps it's enough for Win32 but not for the linux driver.

Regards
Martin

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

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

[SG]

[-- Attachment #1: Type: text/plain, Size: 3648 bytes --]

On Mon, 16 Jun 2008 19:42:56 +0200
SG <SiestaGomez@web.de> wrote:

> On Mon, 16 Jun 2008 16:33:52 +0200
> Sigmund Augdal <sigmund@snap.tv> wrote:
> 
> > On Mon, 2008-06-16 at 16:26 +0200, siestagomez@web.de wrote:
> > > Sigmund Augdal schrieb: 
> > > 
> > > > On Mon, 2008-06-16 at 08:14 +0300, Arthur Konovalov wrote:
> > > >> SG wrote:
> > > >> > The patch works quite well and nearly all channels seem to work.
> > > >> > 
> > > >> > But when tuning to some radio channels I'll get this kernel message:
> > > >> > 
> > > >> > saa7146 (0) saa7146_i2c_writeout [irq]: timed out waiting for end of xfer
> > > >> > 
> > > >> > Also I'm not able to tune to 'transponder 386000000 6900000 0 3' which works
> > > >> > smoothly when using Win32.
> > > >> > 
> > > >> > initial transponder 386000000 6900000 0 3
> > > >> >  >>> tune to: 386:M64:C:6900:
> > > >> > WARNING: >>> tuning failed!!!
> > > >> >  >>> tune to: 386:M64:C:6900: (tuning failed)
> > > >> > WARNING: >>> tuning failed!!!
> > > >> > ERROR: initial tuning failed
> > > >> > dumping lists (0 services)
> > > >> > Done. 
> > > >> 
> > > >> Yes, I discovered too that tuning to frequency 386MHz has no lock.
> > > >> VDR channels.conf: TV3:386000:C0M64:C:6875:703:803:0:0:1003:16:1:0 
> > > >> 
> > > >> At same time, 394MHz (and others) works.
> > > > Hi. 
> > > > 
> > > > Both transponders reported to not tune here has different symbolrates
> > > > from what I used for my testing. Maybe this is relevant in some way.
> > > > Could you please compare this with the channels that did tune to see if
> > > > there is a pattern? 
> > > > 
> > > > About the i2c message, I get that every now and then here as well, but I
> > > > have not seen any ill effect from it. I also see that on some other TT
> > > > cards so I think that might be unrelated to the demod/tuner. 
> > > > 
> > > > Regards 
> > > > 
> > > > Sigmund Augdal
> > > 
> > > The symbolrate is the same on all other working channels. 
> > > 
> > > Regarding the i2c message when watching video I'll get this only once but 
> > > when tuning to a radio channel my log gets flooded and it seems to hangup. 
> > How does radio or not make a difference? As far as I know the card is
> > DVB only and should not care whether the tuned transponder contains
> > radio or video or whatever. Could you send a list of transponders that
> > work (with tuning parameters) and ones that doesn't?
> > 
> 
> Odd today no problem with radio and kernel log.
> Anyway here are a few entries from transponder.ini which I use for dvbscan:
> C 362000000 6900000 NONE QAM64		OK
> C 370000000 6900000 NONE QAM64		OK
> C 378000000 6900000 NONE QAM64		OK
> C 386000000 6900000 NONE QAM64		NOT OK
> C 394000000 6900000 NONE QAM64		OK
> C 402000000 6900000 NONE QAM256		NOT OK
> C 410000000 6900000 NONE QAM64		OK
> C 426000000 6900000 NONE QAM64		OK
> 
> I noticed when using Win32 the signal strenght is very poor on the non working transponders for linux-dvb.
> Perhaps it's enough for Win32 but not for the linux driver.
> 
> Regards
> Martin
> 

Hi 

I just added a line to get the bundled IR-remote to work using lirc.
Patch appended.

Regards
Martin

-- 
<todays fortune>
The number of UNIX installations has grown to 10, with more expected.
		-- The Unix Programmer's Manual, 2nd Edition, June 1972
</todays fortune>
---------------------------------------------------------------------------
   Martin Waldorf              email:  martin@waldorf-net.com
                               mobil:  +49 178 3341205
                               fon  :  +49 211 17832343
---------------------------------------------------------------------------

[-- Attachment #2: ir-patch.diff --]
[-- Type: text/x-diff, Size: 484 bytes --]

diff -Naur a/linux/drivers/media/dvb/ttpci/budget-ci.c b/linux/drivers/media/dvb/ttpci/budget-ci.c
--- a/linux/drivers/media/dvb/ttpci/budget-ci.c	2008-06-16 19:58:04.000000000 +0200
+++ b/linux/drivers/media/dvb/ttpci/budget-ci.c	2008-06-08 17:19:39.000000000 +0200
@@ -237,6 +237,7 @@
 		break;
 	case 0x1010:
 	case 0x1017:
+	case 0x101a:
 		/* for the Technotrend 1500 bundled remote */
 		ir_input_init(input_dev, &budget_ci->ir.state,
 			      IR_TYPE_RC5, ir_codes_tt_1500);


[-- Attachment #3: Type: text/plain, Size: 150 bytes --]

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

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

[SG]

On Mon, 16 Jun 2008 19:42:56 +0200
SG <SiestaGomez@web.de> wrote:

> On Mon, 16 Jun 2008 16:33:52 +0200
> Sigmund Augdal <sigmund@snap.tv> wrote:
> 
> > On Mon, 2008-06-16 at 16:26 +0200, siestagomez@web.de wrote:
> > > Sigmund Augdal schrieb: 
> > > 
> > > > On Mon, 2008-06-16 at 08:14 +0300, Arthur Konovalov wrote:
> > > >> SG wrote:
> > > >> > The patch works quite well and nearly all channels seem to work.
> > > >> > 
> > > >> > But when tuning to some radio channels I'll get this kernel message:
> > > >> > 
> > > >> > saa7146 (0) saa7146_i2c_writeout [irq]: timed out waiting for end of xfer
> > > >> > 
> > > >> > Also I'm not able to tune to 'transponder 386000000 6900000 0 3' which works
> > > >> > smoothly when using Win32.
> > > >> > 
> > > >> > initial transponder 386000000 6900000 0 3
> > > >> >  >>> tune to: 386:M64:C:6900:
> > > >> > WARNING: >>> tuning failed!!!
> > > >> >  >>> tune to: 386:M64:C:6900: (tuning failed)
> > > >> > WARNING: >>> tuning failed!!!
> > > >> > ERROR: initial tuning failed
> > > >> > dumping lists (0 services)
> > > >> > Done. 
> > > >> 
> > > >> Yes, I discovered too that tuning to frequency 386MHz has no lock.
> > > >> VDR channels.conf: TV3:386000:C0M64:C:6875:703:803:0:0:1003:16:1:0 
> > > >> 
> > > >> At same time, 394MHz (and others) works.
> > > > Hi. 
> > > > 
> > > > Both transponders reported to not tune here has different symbolrates
> > > > from what I used for my testing. Maybe this is relevant in some way.
> > > > Could you please compare this with the channels that did tune to see if
> > > > there is a pattern? 
> > > > 
> > > > About the i2c message, I get that every now and then here as well, but I
> > > > have not seen any ill effect from it. I also see that on some other TT
> > > > cards so I think that might be unrelated to the demod/tuner. 
> > > > 
> > > > Regards 
> > > > 
> > > > Sigmund Augdal
> > > 
> > > The symbolrate is the same on all other working channels. 
> > > 
> > > Regarding the i2c message when watching video I'll get this only once but 
> > > when tuning to a radio channel my log gets flooded and it seems to hangup. 
> > How does radio or not make a difference? As far as I know the card is
> > DVB only and should not care whether the tuned transponder contains
> > radio or video or whatever. Could you send a list of transponders that
> > work (with tuning parameters) and ones that doesn't?
> > 
> 
> Odd today no problem with radio and kernel log.
> Anyway here are a few entries from transponder.ini which I use for dvbscan:
> C 362000000 6900000 NONE QAM64		OK
> C 370000000 6900000 NONE QAM64		OK
> C 378000000 6900000 NONE QAM64		OK
> C 386000000 6900000 NONE QAM64		NOT OK
> C 394000000 6900000 NONE QAM64		OK
> C 402000000 6900000 NONE QAM256		NOT OK
> C 410000000 6900000 NONE QAM64		OK
> C 426000000 6900000 NONE QAM64		OK
> 
> I noticed when using Win32 the signal strenght is very poor on the non working transponders for linux-dvb.
> Perhaps it's enough for Win32 but not for the linux driver.
> 
> Regards
> Martin
> 

Hi 

I just added a line to get the bundled IR-remote to work using lirc.


--- a/linux/drivers/media/dvb/ttpci/budget-ci.c 2008-06-16 19:58:04.000000000 +0200
+++ b/linux/drivers/media/dvb/ttpci/budget-ci.c 2008-06-08 17:19:39.000000000 +0200
@@ -237,6 +237,7 @@
        break;
    case 0x1010:
    case 0x1017:
+   case 0x101a:
        /* for the Technotrend 1500 bundled remote */
        ir_input_init(input_dev, &budget_ci->ir.state,
                  IR_TYPE_RC5, ir_codes_tt_1500);


Regards
Martin




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

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

[Arthur Konovalov]

Sigmund Augdal wrote:
> Both transponders reported to not tune here has different symbolrates
> from what I used for my testing. Maybe this is relevant in some way.
> Could you please compare this with the channels that did tune to see if
> there is a pattern? 

 From my side i can add that all frequency from local cable provider's 
works with c-1501 except one:
274
282
290
298
306
314
386 NO LOCK
394
402
410
All channels QAM64 and SR 6875.

In the same PC I have second DVB-C card (KNC One DVB-C), which sharing 
same cable with c-1501 and there no problem with reception and signal's 
strength. This is reason why I did not discover current problem earlier. :(

Arthur






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

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

[Antti Palosaari]

SG wrote:
> I noticed when using Win32 the signal strenght is very poor on the non working transponders for linux-dvb.
> Perhaps it's enough for Win32 but not for the linux driver.

Cold you test different deltaf and output_mode setting for demodulator? 
Those settings could provide better performance when signal is not so good.

Antti
-- 
http://palosaari.fi/

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

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

[Oliver Endriss]

Sigmund Augdal wrote:
> On Wed, 2008-06-04 at 16:37 +0200, Sigmund Augdal wrote:
> > ons, 04.06.2008 kl. 16.58 +0300, skrev Antti Palosaari:
> > > Sigmund Augdal wrote:
> > > > The following experimental patch adds support for the technotrend budget
> > > > C-1501 dvb-c card. The parameters used to configure the tda10023 demod
> > > > chip are largely determined experimentally, but works quite for me in my
> > > > initial tests.
> > > 
> > > You finally found correct values :) I see that it uses same clock 
> > > settings than Anysee. Also deltaf could be correct because I remember 
> > > from my tests that it cannot set wrong otherwise it will not work at 
> > > all. How did you find defltaf?
> > I guessed the clock settings based on how the same tuner is used by a
> > different demod. The deltaf value was found by trial and error (helped
> > by some scripting). deltaf values slightly off will also work, but
> > tuning will be very slow. I also think the deltaf value will depend on
> > the bandwidth of the transponder tuned. All transponders I've tested
> > with are 8MHz, but I think other values are possible, and these will
> > most likely not work. I submitted the patch anyway in the hope that some
> > broader testing might help uncover any remaining problems.
> > > 
> > > Your patch has at least coding style issues, please run make checkpatch 
> > > fix errors and resend patch.
> > I was trying to follow the guidelines, but I guess I wasn't doing that
> > good enough. Will try to clean that up and resend soon.
> 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.

Patch applied with minor modifications:
- error handling: detach frontend if tda827x_attach fails
- use lower case characters for hex constants
- make the code more readable (better ignore 'line length warnings'
  if the resulting code would look too bad)

CU
Oliver

-- 
----------------------------------------------------------------
VDR Remote Plugin 0.4.0: http://www.escape-edv.de/endriss/vdr/
----------------------------------------------------------------

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

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

[Simon Baxter]

> Sigmund Augdal wrote:
>> Both transponders reported to not tune here has different symbolrates
>> from what I used for my testing. Maybe this is relevant in some way.
>> Could you please compare this with the channels that did tune to see if
>> there is a pattern?
>
> From my side i can add that all frequency from local cable provider's
> works with c-1501 except one:
> 274
> 282
> 290
> 298
> 306
> 314
> 386 NO LOCK
> 394
> 402
> 410
> All channels QAM64 and SR 6875.
>
> In the same PC I have second DVB-C card (KNC One DVB-C), which sharing
> same cable with c-1501 and there no problem with reception and signal's
> strength. This is reason why I did not discover current problem earlier. 
> :(
>
> Arthur

Did you have any luck tying this down?

I have a few channels which show no signal strength and won't lock, but it 
works on my other C-2300 and C-1500 cards.

Also, have a problem getting stable sound.  I switch to a new channel (with 
VDR) and the sound cuts in/out/in/out/in - and occasionally just in/out (ie 
no sound)

Have the current/latest changes been committed?  Or is there a latest patch 
floating around for v4l-dvb?? 


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

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

[Arthur Konovalov]

Simon Baxter wrote:
> Did you have any luck tying this down?
> 
No, 322 and 386 MHz not working.

AK

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

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

[Simon Baxter]

Are you using multiproto or v4l-dvb?

any difference??


----- Original Message ----- 
From: "Arthur Konovalov" <artlov@gmail.com>
To: "Simon Baxter" <linuxtv@nzbaxters.com>
Cc: "linux-dvb" <linux-dvb@linuxtv.org>
Sent: Tuesday, July 15, 2008 4:56 AM
Subject: Re: [PATCH] experimental support for C-1501


> Simon Baxter wrote:
>> Did you have any luck tying this down?
>> 
> No, 322 and 386 MHz not working.
> 
> AK
>

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