Patchwork [linux-dvb] Broken Tuning on Wintv Nova HD S2

login
register
mail settings
Submitter Igor M. Liplianin
Date Jan. 29, 2009, 7:12 p.m.
Message ID <200901292112.15587.liplianin@tut.by>
Download mbox | patch
Permalink /patch/4637/
State Not Applicable
Headers show

Comments

Igor M. Liplianin - Jan. 29, 2009, 7:12 p.m.
В сообщении от 29 January 2009 20:07:32 gimli написал(а):
> Hi,
>
> your patch seems to work.
If it works, then I prepare more simple patch.
gimli - Jan. 29, 2009, 7:30 p.m.
Igor M. Liplianin schrieb:
> В сообщении от 29 January 2009 20:07:32 gimli написал(а):
>> 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 <gimli@dark-green.com>

to the list.

cu

Edgar (gimli) Hucek


--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Igor M. Liplianin - Jan. 29, 2009, 8:42 p.m.
On 29 января 2009, "Igor M. Liplianin" <liplianin@tut.by> wrote:
> Igor M. Liplianin schrieb:
> > п▓ я│п╬п╬п╠я┴п╣п╫п╦п╦ п╬я┌ 29 January 2009 20:07:32 gimli п╫п╟п©п╦я│п╟п╩(п╟):
> >> 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 <gimli@dark-green.com>
>
> to the list.
>
> cu
>
> Edgar (gimli) Hucek

Does simple patch work ?
I need your Acked-by :)
Mika Laitio - Jan. 29, 2009, 9:36 p.m.
>> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
gimli - Jan. 30, 2009, 12:10 a.m.
Igor M. Liplianin schrieb:
> On 29 января 2009, "Igor M. Liplianin" <liplianin@tut.by> wrote:
>> Igor M. Liplianin schrieb:
>>> п▓ я│п╬п╬п╠я┴п╣п╫п╦п╦ п╬я┌ 29 January 2009 20:07:32 gimli п╫п╟п©п╦я│п╟п╩(п╟):
>>>> 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 <gimli@dark-green.com>
>>
>> to the list.
>>
>> cu
>>
>> Edgar (gimli) Hucek
> 
> Does simple patch work ?
> I need your Acked-by :)
> 

Acked-by : Edgar Hucek <gimli@dark-green.com>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Igor M. Liplianin - Jan. 30, 2009, 12:43 a.m.
В сообщении от 30 January 2009 02:10:18 gimli написал(а):
> Igor M. Liplianin schrieb:
> > On 29 января 2009, "Igor M. Liplianin" <liplianin@tut.by> wrote:
> >> Igor M. Liplianin schrieb:
> >>> п▓ я│п╬п╬п╠я┴п╣п╫п╦п╦ п╬я┌ 29 January 2009 20:07:32 gimli п╫п╟п©п╦я│п╟п╩(п╟):
> >>>> 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 <gimli@dark-green.com>
> >>
> >> to the list.
> >>
> >> cu
> >>
> >> Edgar (gimli) Hucek
> >
> > Does simple patch work ?
> > I need your Acked-by :)
>
> Acked-by : Edgar Hucek <gimli@dark-green.com>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

OK

Igor
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Igor M. Liplianin - Jan. 30, 2009, 1:07 a.m.
В сообщении от 30 January 2009 02:10:18 gimli написал(а):
> Acked-by : Edgar Hucek <gimli@dark-green.com>
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.

Patch

# HG changeset patch
# User Igor M. Liplianin <liplianin@me.by>
# Date 1233253267 -7200
# Node ID 3542d1c1e03add577ce85175327701c552d14856
# Parent  4086371cea7b7f8b461e1a77513274aa43583c8c
Bug fix: Restore HVR-4000 tuning.

From: Igor M. Liplianin <liplianin@me.by>

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 <liplianin@me.by>

diff -r 4086371cea7b -r 3542d1c1e03a linux/drivers/media/dvb/frontends/cx24116.c
--- a/linux/drivers/media/dvb/frontends/cx24116.c	Sat Jan 17 17:23:31 2009 +0200
+++ b/linux/drivers/media/dvb/frontends/cx24116.c	Thu Jan 29 20:21:07 2009 +0200
@@ -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);
 }
 
 /*