Opened 13 years ago

Closed 13 years ago

#1184 closed defect (fixed)

Add ability to set port & new channel change code in 6200ch

Reported by: adeffs.mythtv@… Owned by: Isaac Richards
Priority: minor Milestone: 0.20
Component: mythtv Version: head
Severity: low Keywords:
Cc: Ticket locked: no

Description

for users with more than one firewire card, allows the user to set which port 6200ch will use.

patch attached

Attachments (2)

6200ch.diff (1.5 KB) - added by adeffs.mythtv@… 13 years ago.
patch that will add ability to specify port for 6200ch.c
6200ch.c.diff (3.3 KB) - added by Tim Trampedach <tim_trampedach@…> 13 years ago.
Diff which includes Steve's fix

Download all attachments as: .zip

Change History (8)

Changed 13 years ago by adeffs.mythtv@…

Attachment: 6200ch.diff added

patch that will add ability to specify port for 6200ch.c

comment:1 Changed 13 years ago by danielk

Milestone: 0.20
Version: head

comment:2 Changed 13 years ago by Tim Trampedach <tim_trampedach@…>

Milestone: 0.200.19
Severity: mediumlow

Steve (the original submitter) and I talked and agreed to do the following:

1) I added some code that is a different and more reliable method of changing channels, similar to the Scientific Atlanta box. The old way is still available using the "-o" flag for anyone that needs it. 2) I included Steve's previously attached fix in my fix. The diff is against the SVN version which does not contain Steve's changes.

Steve and I have both tested the change and it works very well for us. I'm therefore requesting this to be included in the 0.19 branch if possible. There is no change to the MythTV channel changing for Motorola boxes, just for the 6200ch utility.

Changed 13 years ago by Tim Trampedach <tim_trampedach@…>

Attachment: 6200ch.c.diff added

Diff which includes Steve's fix

comment:3 Changed 13 years ago by Tim Trampedach <tim_trampedach@…>

Summary: Add ability to set port in 6200chAdd ability to set port & new channel change code in 6200ch

Made the headline more descriptive.

comment:4 Changed 13 years ago by danielk

(In [8849]) References #1184.

I'm applying Steve Adeff's patch under a lose definition of "bug". This allows you to specify the port to use when using the external channel changer for a Firewire recorder.

comment:5 Changed 13 years ago by danielk

Milestone: 0.190.20

I've sent an updated version of Tim Trampedach's patch to him and Steve for testing. That version does not change the default, but allows use of the alternate channel changing method which works for both of them. If that patch works for Steve and Tim, I'll apply this for 0.19, otherwise this goes to 0.20.

Neither method appears to work for 100% of firewire tuners, so I think we really need both to be available to users. But I'm not applying the patch as is, because I also don't like changing the default this late in the game.

comment:6 Changed 13 years ago by danielk

Resolution: fixed
Status: newclosed

(In [8855]) Closes #1184, by applying Tim Trampedach's patch with some modifications.

This adds an option to the external Firewire channel changing program that allows you to use a single AVC command to change channels. It is always faster than the default method, and is more reliable for at least two people. From what I understand this used to be the default, but was deamed not as reliable as the current default method. However, Tim has changed the retry count and believes that has made it much more reliable.

Note: See TracTickets for help on using tickets.