Skip to contentSkip to conversation

Cablestar DVB-C (Flexcop + stv0297) broken since changeset 7fb12d754061 (7469)

linux-dvb Oliver Endriss 2 participants 3 messages Aug 24, 2008 – Sep 6, 2008 1 message not in the archive Media (V4L/DVB)

lore
Share & tools
lore
#0 lore raw

Hi,

according to the thread
http://www.vdr-portal.de/board/thread.php?threadid=76932
in vdr-portal, the modifications of the flexcop i2c handling in changeset 7fb12d754061 (7469) broke support for the Cablestar card.

Note that stv0297 does not support repeated start conditions (same problem as with s5h1420).

CU
Oliver

Signature · 8 lines
----------------------------------------------------------------
VDR Remote Plugin 0.4.0: http://www.escape-edv.de/endriss/vdr/
----------------------------------------------------------------

_______________________________________________
linux-dvb mailing list
[email protected]
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

A message that is not in the archive (sent off-list or not yet received): [email protected]

#2 lore raw
Quoted 9 lines · Martin Dauskardt

So we have broken support for at least three cards now. (Technotrend 2400 and Hauppauge Noba-S-SE with s5h1420 frontend doesn`t work any longer ,too. See
http://linuxtv.org/pipermail/linux-dvb/2008-August/028249.html)

In my opinion it is not acceptable to break support for three cards in the v4l-dvb main tree to get one other card supported.

I strongly request to revert the changeset until a working solution for all cards has been tested.

We need to separate the things:

1) breakage of stv0297-based cards

This was a mistake - I'm still trying to see which atomic modification causes that.... Should be a 3-liner to fix it, but where...

2) s5h1420-based cards

I modified it to support a new card - right. I published the patch and ask for owners of the NOVA-S SE to test it. After not receiving a comment from anyone Mauro agreed to integrate it and wait if it breaks for someone. I agree this is not the smoothest/nicest way, but what else could I have done to move forward?

Blindly reverting now would push me back where I was 6 months ago :(. But I totally agree with this idea when someone of you NOVA-S SE owners gives me a hand in order to test a possibly fixed patch. ;)

Patrick.

_______________________________________________
linux-dvb mailing list
[email protected]
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

#3 lore raw

Hi again,

I just pushed patch to my hg repository which fixes the s5h1420-support for other cards than the flexcop-based ones.

I will ask Mauro to pull right now.

Patrick.

Quoted 30 lines · Patrick Boettcher
Quoted 9 lines · Martin Dauskardt

So we have broken support for at least three cards now. (Technotrend 2400 and Hauppauge Noba-S-SE with s5h1420 frontend doesn`t work any longer ,too. See
http://linuxtv.org/pipermail/linux-dvb/2008-August/028249.html)

In my opinion it is not acceptable to break support for three cards in the v4l-dvb main tree to get one other card supported.

I strongly request to revert the changeset until a working solution for all cards has been tested.

We need to separate the things:

1) breakage of stv0297-based cards

This was a mistake - I'm still trying to see which atomic modification causes that.... Should be a 3-liner to fix it, but where...

2) s5h1420-based cards

I modified it to support a new card - right. I published the patch and ask for owners of the NOVA-S SE to test it. After not receiving a comment from anyone Mauro agreed to integrate it and wait if it breaks for someone. I agree this is not the smoothest/nicest way, but what else could I have done to move forward?

Blindly reverting now would push me back where I was 6 months ago :(. But I totally agree with this idea when someone of you NOVA-S SE owners gives me a hand in order to test a possibly fixed patch. ;)

Quoted 7 lines

Patrick.

_______________________________________________
linux-dvb mailing list
[email protected]
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

_______________________________________________
linux-dvb mailing list
[email protected]
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb