Hi,
the following changesets breaks Tuning to Vertical Transponders :
http://mercurial.intuxication.org/hg/s2-liplianin/rev/1ca67881d96a
http://linuxtv.org/hg/v4l-dvb/rev/2cd7efb4cc19
For example :
DMAX;BetaDigital:12246:vC34M2O0S0:S19.2E:27500:511:512=deu:32:0:10101:1:1092:0
cu
Edgar ( gimli ) Hucek
_______________________________________________
linux-dvb users mailing list
For V4L/DVB development, please use instead [email protected] [email protected]
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

Mika Laitio

2009-01-28 23:08 UTC #2 lore raw

Quoted 2 lines

I also can confirm this. Reverting to a previous revision restores proper tuning to vertical transponders.

I can also confirm it with hvr-4000 and latest drivers v4l-dvb drivers from linuxtv org. Before I reverted this patch, I could not get lock with vdr for example to:

Sky News;BT:11597:vC56:S19.2E:22000:305+131:306=eng:579:0:28707:1:1026:0 Al Jazeera;CANALSATELLITE:11567:vC56:S19.2E:22000:55:56=ara:0:0:9021:1:1024:0

while this worked

Eurosport;SES Astra:12226:hC34:S19.2E:27500:101:103=deu:102:0:31200:1:1091:0

Once I reverted that patch, all channels showed up nicely.

Mika

Quoted 29 lines · gimli

Quoted 14 lines
Hi,
the following changesets breaks Tuning to Vertical Transponders :
http://mercurial.intuxication.org/hg/s2-liplianin/rev/1ca67881d96a
http://linuxtv.org/hg/v4l-dvb/rev/2cd7efb4cc19
For example :
DMAX;BetaDigital:12246:vC34M2O0S0:S19.2E:27500:511:512=deu:32:0:10101:1:1092:0
cu
Quoted 8 lines
Edgar ( gimli ) Hucek
_______________________________________________
linux-dvb users mailing list
For V4L/DVB development, please use instead [email protected] [email protected]
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

Igor M. Liplianin

2009-01-29 16:10 UTC #3 lore raw

In reply to gimli

В сообщении от 27 January 2009 22:39:51 gimli написал(а):

Quoted 15 lines

Hi,
the following changesets breaks Tuning to Vertical Transponders :
http://mercurial.intuxication.org/hg/s2-liplianin/rev/1ca67881d96a
http://linuxtv.org/hg/v4l-dvb/rev/2cd7efb4cc19
For example :
DMAX;BetaDigital:12246:vC34M2O0S0:S19.2E:27500:511:512=deu:32:0:10101:1:109 2:0
cu

Quoted 7 lines

Edgar ( gimli ) Hucek
_______________________________________________
linux-dvb users mailing list
For V4L/DVB development, please use instead [email protected] [email protected]
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

More likely not polarization, but hi band may broken. Anyway, please, try attached patch.

Signature · 2 lines

Igor M. Liplianin
Microsoft Windows Free Zone - Linux used for all Computing Tasks

# HG changeset patch
# User Igor M. Liplianin <[email protected]> # Date 1233189531 -7200
# Node ID d09924ba6c75233d8212ef79076739d9400d79a8 # Parent 43dbc8ebb5a21c8991df5e5ead54b724c0dc18f4 HVR-4000 Test

linux/drivers/media/dvb/frontends/cx24116.c +7 -2

diff -r 43dbc8ebb5a2 -r d09924ba6c75 linux/drivers/media/dvb/frontends/cx24116.c
@@ -861,6 +861,7 @@
 static int cx24116_set_tone(struct dvb_frontend *fe,
 	fe_sec_tone_mode_t tone)
 {
+	struct cx24116_state *state = fe->demodulator_priv;
 	struct cx24116_cmd cmd;
 	int ret;

@@ -870,8 +871,12 @@
 		return -EINVAL;
 	}
 
-	/* Wait for LNB ready */
-	ret = cx24116_wait_for_lnb(fe);
+	if (state->config->use_lnb_dc != 1)
+		ret = cx24116_set_voltage(fe, SEC_VOLTAGE_13);
+	else
+		/* Wait for LNB ready */
+		ret = cx24116_wait_for_lnb(fe);
+
 	if (ret != 0)
 		return ret;

linux/drivers/media/dvb/frontends/cx24116.h +3 -0

diff -r 43dbc8ebb5a2 -r d09924ba6c75 linux/drivers/media/dvb/frontends/cx24116.h
@@ -35,6 +35,9 @@
 
 	/* Need to set MPEG parameters */
 	u8 mpg_clk_pos_pol:0x02;
+
+	/* Need to set LNB power control */
+	u8 use_lnb_dc;
 };
 
 #if defined(CONFIG_DVB_CX24116) || \

linux/drivers/media/video/cx23885/cx23885-dvb.c +1 -0

diff -r 43dbc8ebb5a2 -r d09924ba6c75 linux/drivers/media/video/cx23885/cx23885-dvb.c
@@ -334,6 +334,7 @@
 
 static struct cx24116_config dvbworld_cx24116_config = {
 	.demod_address = 0x05,
+	.use_lnb_dc = 1,
 };
 
 static int dvb_register(struct cx23885_tsport *port)

gimli

author

2009-01-29 18:07 UTC #4 lore raw

Hi,

your patch seems to work.

Edgar (gimli) Hucek

Quoted 28 lines · Igor M. Liplianin

В сообщении от 27 January 2009 22:39:51 gimli написал(а):
Quoted 15 lines
Hi,
the following changesets breaks Tuning to Vertical Transponders :
http://mercurial.intuxication.org/hg/s2-liplianin/rev/1ca67881d96a
http://linuxtv.org/hg/v4l-dvb/rev/2cd7efb4cc19
For example :
DMAX;BetaDigital:12246:vC34M2O0S0:S19.2E:27500:511:512=deu:32:0:10101:1:109 2:0
cu
Quoted 7 lines
Edgar ( gimli ) Hucek
_______________________________________________
linux-dvb users mailing list
For V4L/DVB development, please use instead [email protected] [email protected]
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb
More likely not polarization, but hi band may broken. Anyway, please, try attached patch.

Igor M. Liplianin

2009-01-29 19:12 UTC #5 lore raw

В сообщении от 29 January 2009 20:07:32 gimli написал(а):

Quoted 3 lines

Hi,
your patch seems to work.

If it works, then I prepare more simple patch.

# HG changeset patch
# User Igor M. Liplianin <[email protected]> # Date 1233253267 -7200
# Node ID 3542d1c1e03add577ce85175327701c552d14856 # Parent 4086371cea7b7f8b461e1a77513274aa43583c8c Bug fix: Restore HVR-4000 tuning.

From: Igor M. Liplianin <[email protected]>

Some cards uses cx24116 LNB_DC pin for LNB power control, some not uses, some uses it different way, like HVR-4000.

Signed-off-by Igor M. Liplianin <[email protected]>

linux/drivers/media/dvb/frontends/cx24116.c +6 -1

diff -r 4086371cea7b -r 3542d1c1e03a linux/drivers/media/dvb/frontends/cx24116.c
@@ -1184,7 +1184,12 @@
 	if (ret != 0)
 		return ret;
 
-	return cx24116_diseqc_init(fe);
+	ret = cx24116_diseqc_init(fe);
+	if (ret != 0)
+		return ret;
+
+	/* HVR-4000 needs this */
+	return cx24116_set_voltage(fe, SEC_VOLTAGE_13);
 }
 
 /*

gimli

author

2009-01-29 19:30 UTC #6 lore raw

Quoted 7 lines · Igor M. Liplianin

В сообщении от 29 January 2009 20:07:32 gimli написал(а):
Quoted 3 lines
Hi,
your patch seems to work.
If it works, then I prepare more simple patch.

Hi,

you can also put my :

Signed-off-by Edgar Hucek <[email protected]>

to the list.

Edgar (gimli) Hucek

Igor M. Liplianin

2009-01-29 20:49 UTC #7 lore raw

Quoted 12 lines · Igor M. Liplianin

Quoted 6 lines · Igor M. Liplianin
п▓ я│п╬п╬п╠я┴п╣п╫п╦п╦ п╬я┌ 29 January 2009 20:07:32 gimli п╫п╟п©п╦я│п╟п╩(п╟):
Quoted 3 lines
Hi,
your patch seems to work.
If it works, then I prepare more simple patch.
Hi,
you can also put my :

Quoted 7 lines

Signed-off-by Edgar Hucek <[email protected]>
to the list.
cu
Edgar (gimli) Hucek

Does simple patch work ?
I need your Acked-by :)

Signature · 2 lines

Igor M. Liplianin
Microsoft Windows Free Zone - Linux used for all Computing Tasks

Mika Laitio

2009-01-29 21:36 UTC #8 lore raw

Quoted 4 lines

Quoted 1 line
Edgar (gimli) Hucek
Does simple patch work ?
I need your Acked-by :)

Hi, I have only saw one version of your patch in mailing list, did you send the simpler version somewhere?

Mika

Igor M. Liplianin

2009-01-29 23:42 UTC #9 lore raw

В сообщении от 29 January 2009 23:36:43 Mika Laitio написал(а):

Quoted 9 lines

Quoted 4 lines
Quoted 1 line
Edgar (gimli) Hucek
Does simple patch work ?
I need your Acked-by :)
Hi, I have only saw one version of your patch in mailing list, did you send the simpler version somewhere?
Mika

Sorry, send it to Edgar only.
But it is unintentionally.

Signature · 2 lines

Igor M. Liplianin
Microsoft Windows Free Zone - Linux used for all Computing Tasks

From: Igor M. Liplianin <[email protected]>

Some cards uses cx24116 LNB_DC pin for LNB power control, some not uses, some uses it different way, like HVR-4000.

Signed-off-by Igor M. Liplianin <[email protected]>

linux/drivers/media/dvb/frontends/cx24116.c +6 -1

diff -r 4086371cea7b -r 3542d1c1e03a linux/drivers/media/dvb/frontends/cx24116.c
@@ -1184,7 +1184,12 @@
 	if (ret != 0)
 		return ret;
 
-	return cx24116_diseqc_init(fe);
+	ret = cx24116_diseqc_init(fe);
+	if (ret != 0)
+		return ret;
+
+	/* HVR-4000 needs this */
+	return cx24116_set_voltage(fe, SEC_VOLTAGE_13);
 }
 
 /*

gimli

author Acked-by

2009-01-30 00:10 UTC #10 lore raw

In reply to Igor M. Liplianin

Quoted 13 lines · Igor M. Liplianin

Quoted 12 lines · Igor M. Liplianin
Quoted 5 lines · Igor M. Liplianin
п▓ я│п╬п╬п╠я┴п╣п╫п╦п╦ п╬я┌ 29 January 2009 20:07:32 gimli п╫п╟п©п╦я│п╟п╩(п╟):
Quoted 3 lines
Hi,
your patch seems to work.
If it works, then I prepare more simple patch.
Hi,
you can also put my :
Signed-off-by Edgar Hucek <[email protected]>

Quoted 9 lines

Quoted 5 lines
to the list.
cu
Edgar (gimli) Hucek
Does simple patch work ?
I need your Acked-by :)

Acked-by Edgar Hucek <[email protected]>

Igor M. Liplianin

2009-01-30 04:48 UTC #11 lore raw

В сообщении от 30 January 2009 02:10:18 gimli написал(а):

Quoted 22 lines

Quoted 21 lines · Igor M. Liplianin
Quoted 19 lines · Igor M. Liplianin
Quoted 6 lines · Igor M. Liplianin
п▓ я│п╬п╬п╠я┴п╣п╫п╦п╦ п╬я┌ 29 January 2009 20:07:32 gimli п╫п╟п©п╦я│п╟п╩(п╟):
Quoted 3 lines
Hi,
your patch seems to work.
If it works, then I prepare more simple patch.
Hi,
you can also put my :
Signed-off-by Edgar Hucek <[email protected]>
to the list.
cu
Edgar (gimli) Hucek

Quoted 8 lines

Quoted 2 lines
Does simple patch work ?
I need your Acked-by :)
Acked-by Edgar Hucek <[email protected]>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

Igor

Igor M. Liplianin

2009-01-30 04:48 UTC #12 lore raw

In reply to gimli

В сообщении от 30 January 2009 02:10:18 gimli написал(а):

Quoted 1 line

Acked-by Edgar Hucek <[email protected]>

Explanation of using Signed-off-by, Acked-by and Tested-by.

	http://kerneltrap.org/node/8329

Sorry yet, but it seems Tested-by :)
Anyway, I already included your Tested-by clause.

Signature · 2 lines

Igor M. Liplianin
Microsoft Windows Free Zone - Linux used for all Computing Tasks