public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
* Re: [linux-dvb] TDA10086 with Pinnacle 400e tuning broken
       [not found]       ` <Pine.LNX.4.64.0801300047520.23532@pub2.ifh.de>
@ 2008-02-03 22:43         ` Hartmut Hackmann
  2008-02-06  8:17           ` André Weidemann
  0 siblings, 1 reply; 10+ messages in thread
From: Hartmut Hackmann @ 2008-02-03 22:43 UTC (permalink / raw)
  To: Patrick Boettcher; +Cc: linux-dvb

Hi, Patrick

Patrick Boettcher schrieb:
> Hi,
> 
> On Wed, 30 Jan 2008, Hartmut Hackmann wrote:
>>>> If i understand the LNBp21 and isl64xx datasheets correctly, these chips will
>>>> always use their internal 22kHz oscillator, it can just be turned on and off.
>>> I don't see how to turn on 22kHz on LNBP21. How can this be done?
>>>
>>> Patrick.
>>>
>> My understanding is that this needs to be done through the diseqc control:
>> via i2c control of the lnbp21, yo can either turn the tone permanently on
>> or let it be controlled by a pin - through the channel decoder.
> 
> The tda10086 is not generating the tone? Only the lnbp21 can do it? The 
> bit7 only enables that on the lnb21, because it is configured to only do 
> it, when a specific line is 1?
> 
> In that case the lnbp21-driver is not prepared for having the tone forced, 
> it seems. So data sheet reading.
> 
Are we talking about different chips / datashets?
Mine says that if the TEN bit is 0, the 22kHz generator is controlled by the
DSQIN pin. A high at this pin turns the OSCILLATOR on.
And to my understanding this is the only way to do it: the tda10086 has to generate
the ENVELOPE of the 22kHz signal. If it contains the carrier, this will disturb the
on chip oscillator. This might still work in the static case, but not if we
want to generate diseqc messages with bursts of just 11 periods.

> Actually I'm asking myself why it is wrong to not set the bit7 of the 
> tda10086? Could be a config option.
> 
See above. We might make it a config option, but we should only do it if we really
need to and i am not convinced yet.

>> Could it be that you somehow just forgot to choose the upper band in your tuning
>> data / dvb application?
> 
> I'm only using dvbscan and vdr - never bothering about that - it simply 
> worked ;)
> 
> Patrick.
> 
Are you sure that it is a lnbp21 on your board?
What kind of satellite equipment do you have?
- a single LNB, so the 22kHz tone is enough.
- a Multiswitch?
  if yes, which commands does it need / understand?
  - nothing but the tone?
  - a tone burst to switch between satellites and the tone?
  - full diseqc (2?) serial messages?

I got a board with tda10086 and lnbp21 let and started measuring.
voltage switching and static tone work fine with the current
configuration.

Hartmut

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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [linux-dvb] TDA10086 with Pinnacle 400e tuning broken
  2008-02-03 22:43         ` [linux-dvb] TDA10086 with Pinnacle 400e tuning broken Hartmut Hackmann
@ 2008-02-06  8:17           ` André Weidemann
  2008-02-06  8:57             ` hermann pitton
  0 siblings, 1 reply; 10+ messages in thread
From: André Weidemann @ 2008-02-06  8:17 UTC (permalink / raw)
  To: linux-dvb

Hartmut Hackmann wrote:

> Are you sure that it is a lnbp21 on your board?
> What kind of satellite equipment do you have?
> - a single LNB, so the 22kHz tone is enough.
> - a Multiswitch?
>   if yes, which commands does it need / understand?
>   - nothing but the tone?
>   - a tone burst to switch between satellites and the tone?
>   - full diseqc (2?) serial messages?
> 
> I got a board with tda10086 and lnbp21 let and started measuring.
> voltage switching and static tone work fine with the current
> configuration.

Hi Hartmut,
I got the same tuning problems as Patrick. After your patch the Pinnacle 
400e is not working anymore. I can get a signal, but no lock at all.
I took a look at the PCB and there is definitely an LNBP21PD soldered 
onto it.
Your patch disables the modulation of the 22kHz signal inside the demod 
as fas as I understood, but then the LNBP21PD was supposed to generate it.

Unfortunately this is not possible with the Pinnacle 400e. I took a look 
at the LNBP21PD datasheet which states, that DSQIN is supposed to be 
connected to GND if you don't want the 22kHz signal to be generated by 
the LNBP21 itself. Guess what?! Pin 14(DSQIN) is connected to GND.

For the time being I reverted your patch on my local system and the 
Pinnacle 400e is working flawlessly again.

  André

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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [linux-dvb] TDA10086 with Pinnacle 400e tuning broken
  2008-02-06  8:17           ` André Weidemann
@ 2008-02-06  8:57             ` hermann pitton
  2008-02-06 14:12               ` Patrick Boettcher
                                 ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: hermann pitton @ 2008-02-06  8:57 UTC (permalink / raw)
  To: André Weidemann, Hartmut Hackmann; +Cc: linux-dvb

Am Mittwoch, den 06.02.2008, 09:17 +0100 schrieb André Weidemann:
> Hartmut Hackmann wrote:
> 
> > Are you sure that it is a lnbp21 on your board?
> > What kind of satellite equipment do you have?
> > - a single LNB, so the 22kHz tone is enough.
> > - a Multiswitch?
> >   if yes, which commands does it need / understand?
> >   - nothing but the tone?
> >   - a tone burst to switch between satellites and the tone?
> >   - full diseqc (2?) serial messages?
> > 
> > I got a board with tda10086 and lnbp21 let and started measuring.
> > voltage switching and static tone work fine with the current
> > configuration.
> 
> Hi Hartmut,
> I got the same tuning problems as Patrick. After your patch the Pinnacle 
> 400e is not working anymore. I can get a signal, but no lock at all.
> I took a look at the PCB and there is definitely an LNBP21PD soldered 
> onto it.
> Your patch disables the modulation of the 22kHz signal inside the demod 
> as fas as I understood, but then the LNBP21PD was supposed to generate it.
> 
> Unfortunately this is not possible with the Pinnacle 400e. I took a look 
> at the LNBP21PD datasheet which states, that DSQIN is supposed to be 
> connected to GND if you don't want the 22kHz signal to be generated by 
> the LNBP21 itself. Guess what?! Pin 14(DSQIN) is connected to GND.
> 
> For the time being I reverted your patch on my local system and the 
> Pinnacle 400e is working flawlessly again.
> 
>   André
> 


Hi,

we should try to get this sorted.

With the prior state, working for Andre and others, it does not work on
the LifeView Trio (PCI and cardbus) and the saa7134 driver. Three guys I
think reported it and Hartmut did wait with the patch for long and
allowed about half a year for testing ...

So, if we can't fix it soon, prior state of course counts and those
later will have to use the patches further. Such a "fix" can always be
committed prior to 2.6.25 release.

Hermann







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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [linux-dvb] TDA10086 with Pinnacle 400e tuning broken
  2008-02-06  8:57             ` hermann pitton
@ 2008-02-06 14:12               ` Patrick Boettcher
  2008-02-06 21:54               ` Hartmut Hackmann
  2008-02-07  0:41               ` Hartmut Hackmann
  2 siblings, 0 replies; 10+ messages in thread
From: Patrick Boettcher @ 2008-02-06 14:12 UTC (permalink / raw)
  To: hermann pitton; +Cc: Hartmut Hackmann, linux-dvb

[-- Attachment #1: Type: TEXT/PLAIN, Size: 2501 bytes --]

Seeing all that, the fix is easy:

make the 22khz tone generation a field in the tda10086-config-struct and 
enable it for the 400e and disable it for other (default).

Patrick.

On Wed, 6 Feb 2008, hermann pitton wrote:

> Am Mittwoch, den 06.02.2008, 09:17 +0100 schrieb André Weidemann:
> > Hartmut Hackmann wrote:
> > 
> > > Are you sure that it is a lnbp21 on your board?
> > > What kind of satellite equipment do you have?
> > > - a single LNB, so the 22kHz tone is enough.
> > > - a Multiswitch?
> > >   if yes, which commands does it need / understand?
> > >   - nothing but the tone?
> > >   - a tone burst to switch between satellites and the tone?
> > >   - full diseqc (2?) serial messages?
> > > 
> > > I got a board with tda10086 and lnbp21 let and started measuring.
> > > voltage switching and static tone work fine with the current
> > > configuration.
> > 
> > Hi Hartmut,
> > I got the same tuning problems as Patrick. After your patch the Pinnacle 
> > 400e is not working anymore. I can get a signal, but no lock at all.
> > I took a look at the PCB and there is definitely an LNBP21PD soldered 
> > onto it.
> > Your patch disables the modulation of the 22kHz signal inside the demod 
> > as fas as I understood, but then the LNBP21PD was supposed to generate it.
> > 
> > Unfortunately this is not possible with the Pinnacle 400e. I took a look 
> > at the LNBP21PD datasheet which states, that DSQIN is supposed to be 
> > connected to GND if you don't want the 22kHz signal to be generated by 
> > the LNBP21 itself. Guess what?! Pin 14(DSQIN) is connected to GND.
> > 
> > For the time being I reverted your patch on my local system and the 
> > Pinnacle 400e is working flawlessly again.
> > 
> >   André
> > 
> 
> 
> Hi,
> 
> we should try to get this sorted.
> 
> With the prior state, working for Andre and others, it does not work on
> the LifeView Trio (PCI and cardbus) and the saa7134 driver. Three guys I
> think reported it and Hartmut did wait with the patch for long and
> allowed about half a year for testing ...
> 
> So, if we can't fix it soon, prior state of course counts and those
> later will have to use the patches further. Such a "fix" can always be
> committed prior to 2.6.25 release.
> 
> Hermann
> 
> 
> 
> 
> 
> 
> 
> _______________________________________________
> linux-dvb mailing list
> linux-dvb@linuxtv.org
> http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb
> 

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

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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [linux-dvb] TDA10086 with Pinnacle 400e tuning broken
  2008-02-06  8:57             ` hermann pitton
  2008-02-06 14:12               ` Patrick Boettcher
@ 2008-02-06 21:54               ` Hartmut Hackmann
  2008-02-07  0:41               ` Hartmut Hackmann
  2 siblings, 0 replies; 10+ messages in thread
From: Hartmut Hackmann @ 2008-02-06 21:54 UTC (permalink / raw)
  To: hermann pitton; +Cc: linux-dvb

Hi,

hermann pitton schrieb:
> Am Mittwoch, den 06.02.2008, 09:17 +0100 schrieb André Weidemann:
>> Hartmut Hackmann wrote:
>>
>>> Are you sure that it is a lnbp21 on your board?
>>> What kind of satellite equipment do you have?
>>> - a single LNB, so the 22kHz tone is enough.
>>> - a Multiswitch?
>>>   if yes, which commands does it need / understand?
>>>   - nothing but the tone?
>>>   - a tone burst to switch between satellites and the tone?
>>>   - full diseqc (2?) serial messages?
>>>
>>> I got a board with tda10086 and lnbp21 let and started measuring.
>>> voltage switching and static tone work fine with the current
>>> configuration.
>> Hi Hartmut,
>> I got the same tuning problems as Patrick. After your patch the Pinnacle 
>> 400e is not working anymore. I can get a signal, but no lock at all.
>> I took a look at the PCB and there is definitely an LNBP21PD soldered 
>> onto it.
>> Your patch disables the modulation of the 22kHz signal inside the demod 
>> as fas as I understood, but then the LNBP21PD was supposed to generate it.
>>
>> Unfortunately this is not possible with the Pinnacle 400e. I took a look 
>> at the LNBP21PD datasheet which states, that DSQIN is supposed to be 
>> connected to GND if you don't want the 22kHz signal to be generated by 
>> the LNBP21 itself. Guess what?! Pin 14(DSQIN) is connected to GND.
>>
>> For the time being I reverted your patch on my local system and the 
>> Pinnacle 400e is working flawlessly again.
>>
>>   André
>>
> 
> 
> Hi,
> 
> we should try to get this sorted.
> 
> With the prior state, working for Andre and others, it does not work on
> the LifeView Trio (PCI and cardbus) and the saa7134 driver. Three guys I
> think reported it and Hartmut did wait with the patch for long and
> allowed about half a year for testing ...
> 
> So, if we can't fix it soon, prior state of course counts and those
> later will have to use the patches further. Such a "fix" can always be
> committed prior to 2.6.25 release.
> 
> Hermann
> 
I think Andre gave the important hint: Obviously there are boards with
a separate modulator for the 22kHz signal. I must say i haven't expected that.
But now we know what needs to be done:
We have to make the tone generation a config option for the tda10086.
Default state should be off and we need to find the boards which need it on.
I will post a patch here asap.

Hartmut

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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [linux-dvb] TDA10086 with Pinnacle 400e tuning broken
  2008-02-06  8:57             ` hermann pitton
  2008-02-06 14:12               ` Patrick Boettcher
  2008-02-06 21:54               ` Hartmut Hackmann
@ 2008-02-07  0:41               ` Hartmut Hackmann
  2008-02-07 20:45                 ` Patrick Boettcher
                                   ` (2 more replies)
  2 siblings, 3 replies; 10+ messages in thread
From: Hartmut Hackmann @ 2008-02-07  0:41 UTC (permalink / raw)
  To: hermann pitton; +Cc: linux-dvb

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

Hi

hermann pitton schrieb:
> Am Mittwoch, den 06.02.2008, 09:17 +0100 schrieb André Weidemann:
>> Hartmut Hackmann wrote:
>>
>>> Are you sure that it is a lnbp21 on your board?
>>> What kind of satellite equipment do you have?
>>> - a single LNB, so the 22kHz tone is enough.
>>> - a Multiswitch?
>>>   if yes, which commands does it need / understand?
>>>   - nothing but the tone?
>>>   - a tone burst to switch between satellites and the tone?
>>>   - full diseqc (2?) serial messages?
>>>
>>> I got a board with tda10086 and lnbp21 let and started measuring.
>>> voltage switching and static tone work fine with the current
>>> configuration.
>> Hi Hartmut,
>> I got the same tuning problems as Patrick. After your patch the Pinnacle 
>> 400e is not working anymore. I can get a signal, but no lock at all.
>> I took a look at the PCB and there is definitely an LNBP21PD soldered 
>> onto it.
>> Your patch disables the modulation of the 22kHz signal inside the demod 
>> as fas as I understood, but then the LNBP21PD was supposed to generate it.
>>
>> Unfortunately this is not possible with the Pinnacle 400e. I took a look 
>> at the LNBP21PD datasheet which states, that DSQIN is supposed to be 
>> connected to GND if you don't want the 22kHz signal to be generated by 
>> the LNBP21 itself. Guess what?! Pin 14(DSQIN) is connected to GND.
>>
>> For the time being I reverted your patch on my local system and the 
>> Pinnacle 400e is working flawlessly again.
>>
>>   André
>>
> 
> 
> Hi,
> 
> we should try to get this sorted.
> 
> With the prior state, working for Andre and others, it does not work on
> the LifeView Trio (PCI and cardbus) and the saa7134 driver. Three guys I
> think reported it and Hartmut did wait with the patch for long and
> allowed about half a year for testing ...
> 
> So, if we can't fix it soon, prior state of course counts and those
> later will have to use the patches further. Such a "fix" can always be
> committed prior to 2.6.25 release.
> 
> Hermann
> 
> 
So here is the patch that make the the 22kHz tone a config option.
Please be aware that i have no means to test it, so please report

Signed-off-by: Hartmut Hackmann <hartmut.hackmann@t-online.de


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

diff -r 7564c110491e linux/drivers/media/dvb/dvb-usb/ttusb2.c
--- a/linux/drivers/media/dvb/dvb-usb/ttusb2.c	Sun Jan 20 09:13:44 2008 -0200
+++ b/linux/drivers/media/dvb/dvb-usb/ttusb2.c	Thu Feb 07 01:20:19 2008 +0100
@@ -150,6 +150,7 @@ static struct tda10086_config tda10086_c
 static struct tda10086_config tda10086_config = {
 	.demod_address = 0x0e,
 	.invert = 0,
+	.diseqc_tone = 1,
 };
 
 static int ttusb2_frontend_attach(struct dvb_usb_adapter *adap)
diff -r 7564c110491e linux/drivers/media/dvb/frontends/tda10086.c
--- a/linux/drivers/media/dvb/frontends/tda10086.c	Sun Jan 20 09:13:44 2008 -0200
+++ b/linux/drivers/media/dvb/frontends/tda10086.c	Thu Feb 07 01:29:37 2008 +0100
@@ -107,9 +107,12 @@ static int tda10086_init(struct dvb_fron
 static int tda10086_init(struct dvb_frontend* fe)
 {
 	struct tda10086_state* state = fe->demodulator_priv;
-
-	dprintk ("%s\n", __FUNCTION__);
-
+	u8 t22k_off = 0x80;
+        
+	dprintk ("%s\n", __FUNCTION__);
+
+	if (state->config->diseqc_tone)
+        	t22k_off = 0;
 	// reset
 	tda10086_write_byte(state, 0x00, 0x00);
 	msleep(10);
@@ -159,7 +162,7 @@ static int tda10086_init(struct dvb_fron
 	tda10086_write_byte(state, 0x3d, 0x80);
 
 	// setup SEC
-	tda10086_write_byte(state, 0x36, 0x80); // all SEC off, no 22k tone
+	tda10086_write_byte(state, 0x36, t22k_off); // all SEC off, 22k tone
 	tda10086_write_byte(state, 0x34, (((1<<19) * (22000/1000)) / (SACLK/1000)));      // } tone frequency
 	tda10086_write_byte(state, 0x35, (((1<<19) * (22000/1000)) / (SACLK/1000)) >> 8); // }
 
@@ -181,16 +184,20 @@ static int tda10086_set_tone (struct dvb
 static int tda10086_set_tone (struct dvb_frontend* fe, fe_sec_tone_mode_t tone)
 {
 	struct tda10086_state* state = fe->demodulator_priv;
-
-	dprintk ("%s\n", __FUNCTION__);
+	u8 t22k_off = 0x80;
+        
+	dprintk ("%s\n", __FUNCTION__);
+
+	if (state->config->diseqc_tone)
+        	t22k_off = 0;
 
 	switch (tone) {
 	case SEC_TONE_OFF:
-		tda10086_write_byte(state, 0x36, 0x80);
+		tda10086_write_byte(state, 0x36, t22k_off);
 		break;
 
 	case SEC_TONE_ON:
-		tda10086_write_byte(state, 0x36, 0x81);
+		tda10086_write_byte(state, 0x36, 0x01 + t22k_off);
 		break;
 	}
 
@@ -203,8 +210,12 @@ static int tda10086_send_master_cmd (str
 	struct tda10086_state* state = fe->demodulator_priv;
 	int i;
 	u8 oldval;
-
-	dprintk ("%s\n", __FUNCTION__);
+	u8 t22k_off = 0x80;
+        
+	dprintk ("%s\n", __FUNCTION__);
+
+	if (state->config->diseqc_tone)
+        	t22k_off = 0;
 
 	if (cmd->msg_len > 6)
 		return -EINVAL;
@@ -213,7 +224,8 @@ static int tda10086_send_master_cmd (str
 	for(i=0; i< cmd->msg_len; i++) {
 		tda10086_write_byte(state, 0x48+i, cmd->msg[i]);
 	}
-	tda10086_write_byte(state, 0x36, 0x88 | ((cmd->msg_len - 1) << 4));
+	tda10086_write_byte(state, 0x36, (0x08 + t22k_off)
+					| ((cmd->msg_len - 1) << 4));
 
 	tda10086_diseqc_wait(state);
 
@@ -226,16 +238,20 @@ static int tda10086_send_burst (struct d
 {
 	struct tda10086_state* state = fe->demodulator_priv;
 	u8 oldval = tda10086_read_byte(state, 0x36);
-
-	dprintk ("%s\n", __FUNCTION__);
+	u8 t22k_off = 0x80;
+        
+	dprintk ("%s\n", __FUNCTION__);
+
+	if (state->config->diseqc_tone)
+        	t22k_off = 0;
 
 	switch(minicmd) {
 	case SEC_MINI_A:
-		tda10086_write_byte(state, 0x36, 0x84);
+		tda10086_write_byte(state, 0x36, 0x04 + t22k_off);
 		break;
 
 	case SEC_MINI_B:
-		tda10086_write_byte(state, 0x36, 0x86);
+		tda10086_write_byte(state, 0x36, 0x06 + t22k_off);
 		break;
 	}
 
diff -r 7564c110491e linux/drivers/media/dvb/frontends/tda10086.h
--- a/linux/drivers/media/dvb/frontends/tda10086.h	Sun Jan 20 09:13:44 2008 -0200
+++ b/linux/drivers/media/dvb/frontends/tda10086.h	Thu Feb 07 01:16:45 2008 +0100
@@ -33,6 +33,9 @@ struct tda10086_config
 
 	/* does the "inversion" need inverted? */
 	u8 invert;
+
+        /* do we need the diseqc signal with carrier? */
+        u8 diseqc_tone;
 };
 
 #if defined(CONFIG_DVB_TDA10086) || (defined(CONFIG_DVB_TDA10086_MODULE) && defined(MODULE))
diff -r 7564c110491e linux/drivers/media/dvb/ttpci/budget.c
--- a/linux/drivers/media/dvb/ttpci/budget.c	Sun Jan 20 09:13:44 2008 -0200
+++ b/linux/drivers/media/dvb/ttpci/budget.c	Thu Feb 07 01:19:53 2008 +0100
@@ -351,6 +351,7 @@ static struct tda10086_config tda10086_c
 static struct tda10086_config tda10086_config = {
 	.demod_address = 0x0e,
 	.invert = 0,
+	.diseqc_tone = 1,
 };
 
 static u8 read_pwm(struct budget* budget)
diff -r 7564c110491e linux/drivers/media/video/saa7134/saa7134-dvb.c
--- a/linux/drivers/media/video/saa7134/saa7134-dvb.c	Sun Jan 20 09:13:44 2008 -0200
+++ b/linux/drivers/media/video/saa7134/saa7134-dvb.c	Thu Feb 07 01:21:14 2008 +0100
@@ -826,6 +826,7 @@ static struct tda10086_config flydvbs = 
 static struct tda10086_config flydvbs = {
 	.demod_address = 0x0e,
 	.invert = 0,
+	.diseqc_tone = 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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [linux-dvb] TDA10086 with Pinnacle 400e tuning broken
  2008-02-07  0:41               ` Hartmut Hackmann
@ 2008-02-07 20:45                 ` Patrick Boettcher
       [not found]                   ` <47AE432A.3070007@t-online.de>
  2008-02-07 20:58                 ` Dirk Brenken
  2008-02-07 22:29                 ` Oliver Endriss
  2 siblings, 1 reply; 10+ messages in thread
From: Patrick Boettcher @ 2008-02-07 20:45 UTC (permalink / raw)
  To: Hartmut Hackmann; +Cc: linux-dvb

Acked-By: Patrick Boettcher <pb@linuxtv.org>

Patrick.

On Thu, 7 Feb 2008, Hartmut Hackmann wrote:
> So here is the patch that make the the 22kHz tone a config option.
> Please be aware that i have no means to test it, so please report
>
> Signed-off-by: Hartmut Hackmann <hartmut.hackmann@t-online.de
>
>

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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [linux-dvb] TDA10086 with Pinnacle 400e tuning broken
  2008-02-07  0:41               ` Hartmut Hackmann
  2008-02-07 20:45                 ` Patrick Boettcher
@ 2008-02-07 20:58                 ` Dirk Brenken
  2008-02-07 22:29                 ` Oliver Endriss
  2 siblings, 0 replies; 10+ messages in thread
From: Dirk Brenken @ 2008-02-07 20:58 UTC (permalink / raw)
  To: Hartmut Hackmann; +Cc: linux-dvb

Hi Hartmut,
I've applied your patch against latest v4l-dvb tree and the tuning 
problems with my Technotrend S-1401 (thread "TT-1401 budget card support 
broken since 2.6.24-rc6" on the list) are gone. Thanks!

Best Regards
Dirk

Hartmut Hackmann schrieb:
> Hi
> 
> hermann pitton schrieb:
>> Am Mittwoch, den 06.02.2008, 09:17 +0100 schrieb André Weidemann:
>>> Hartmut Hackmann wrote:
>>>
>>>> Are you sure that it is a lnbp21 on your board?
>>>> What kind of satellite equipment do you have?
>>>> - a single LNB, so the 22kHz tone is enough.
>>>> - a Multiswitch?
>>>>   if yes, which commands does it need / understand?
>>>>   - nothing but the tone?
>>>>   - a tone burst to switch between satellites and the tone?
>>>>   - full diseqc (2?) serial messages?
>>>>
>>>> I got a board with tda10086 and lnbp21 let and started measuring.
>>>> voltage switching and static tone work fine with the current
>>>> configuration.
>>> Hi Hartmut,
>>> I got the same tuning problems as Patrick. After your patch the Pinnacle 
>>> 400e is not working anymore. I can get a signal, but no lock at all.
>>> I took a look at the PCB and there is definitely an LNBP21PD soldered 
>>> onto it.
>>> Your patch disables the modulation of the 22kHz signal inside the demod 
>>> as fas as I understood, but then the LNBP21PD was supposed to generate it.
>>>
>>> Unfortunately this is not possible with the Pinnacle 400e. I took a look 
>>> at the LNBP21PD datasheet which states, that DSQIN is supposed to be 
>>> connected to GND if you don't want the 22kHz signal to be generated by 
>>> the LNBP21 itself. Guess what?! Pin 14(DSQIN) is connected to GND.
>>>
>>> For the time being I reverted your patch on my local system and the 
>>> Pinnacle 400e is working flawlessly again.
>>>
>>>   André
>>>
>>
>> Hi,
>>
>> we should try to get this sorted.
>>
>> With the prior state, working for Andre and others, it does not work on
>> the LifeView Trio (PCI and cardbus) and the saa7134 driver. Three guys I
>> think reported it and Hartmut did wait with the patch for long and
>> allowed about half a year for testing ...
>>
>> So, if we can't fix it soon, prior state of course counts and those
>> later will have to use the patches further. Such a "fix" can always be
>> committed prior to 2.6.25 release.
>>
>> Hermann
>>
>>
> So here is the patch that make the the 22kHz tone a config option.
> Please be aware that i have no means to test it, so please report
> 
> Signed-off-by: Hartmut Hackmann <hartmut.hackmann@t-online.de
> 
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> 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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [linux-dvb] TDA10086 with Pinnacle 400e tuning broken
  2008-02-07  0:41               ` Hartmut Hackmann
  2008-02-07 20:45                 ` Patrick Boettcher
  2008-02-07 20:58                 ` Dirk Brenken
@ 2008-02-07 22:29                 ` Oliver Endriss
  2 siblings, 0 replies; 10+ messages in thread
From: Oliver Endriss @ 2008-02-07 22:29 UTC (permalink / raw)
  To: linux-dvb

Hi,

Hartmut Hackmann wrote:
> ...
> So here is the patch that make the the 22kHz tone a config option.
> Please be aware that i have no means to test it, so please report
> 
> Signed-off-by: Hartmut Hackmann <hartmut.hackmann@t-online.de

Acked-by: Oliver Endriss <o.endriss@gmx.de>

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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [linux-dvb] TDA10086 with Pinnacle 400e tuning broken
       [not found]                   ` <47AE432A.3070007@t-online.de>
@ 2008-02-10  7:09                     ` Dirk Brenken
  0 siblings, 0 replies; 10+ messages in thread
From: Dirk Brenken @ 2008-02-10  7:09 UTC (permalink / raw)
  To: linux-dvb; +Cc: Hartmut Hackmann

Hi,
please do that - it's a fix for the following bugzilla entry ...

http://bugzilla.kernel.org/show_bug.cgi?id=9887

Best regards
Dirk

Hartmut Hackmann schrieb:
> Hi, all
> 
> I pushed the patch to my personal repository at
> 
>   http://linuxtv.org/hg/~hhackmann/v4l-dvb/
> 
> Do you agree that we should ask Linus to pull it as a bug fix?
> 
> Btw: tda10086 contains a number of coding style violations. I did not
> fix them to keep the patch as small as possible.
> 
> Best regards
>   Hartmut
> 
> _______________________________________________
> 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

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2008-02-10  7:10 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <Pine.LNX.4.64.0801271922040.21518@pub2.ifh.de>
     [not found] ` <479D1632.4010006@t-online.de>
     [not found]   ` <Pine.LNX.4.64.0801292211380.23532@pub2.ifh.de>
     [not found]     ` <479FB52A.6010401@t-online.de>
     [not found]       ` <Pine.LNX.4.64.0801300047520.23532@pub2.ifh.de>
2008-02-03 22:43         ` [linux-dvb] TDA10086 with Pinnacle 400e tuning broken Hartmut Hackmann
2008-02-06  8:17           ` André Weidemann
2008-02-06  8:57             ` hermann pitton
2008-02-06 14:12               ` Patrick Boettcher
2008-02-06 21:54               ` Hartmut Hackmann
2008-02-07  0:41               ` Hartmut Hackmann
2008-02-07 20:45                 ` Patrick Boettcher
     [not found]                   ` <47AE432A.3070007@t-online.de>
2008-02-10  7:09                     ` Dirk Brenken
2008-02-07 20:58                 ` Dirk Brenken
2008-02-07 22:29                 ` Oliver Endriss

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox