Ticket #5406 (closed defect: invalid)
Opened 4 years ago
Last modified 3 years ago
Fixed a bug in sa3250ch-BR
| Reported by: | hadees | Owned by: | kormoc |
|---|---|---|---|
| Priority: | minor | Milestone: | unknown |
| Component: | contrib | Version: | unknown |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description
The original sa3250ch script was hard coded to use node 0 however Benton Roberts <benton@…> added support to input your own node and/or port via command line options. He did almost most everything to switch it except actually switching the hard coded zero for the command line node number. He did it for the ports already and it looks like he did some other stuff with different kind of boxes but i'm not sure what. He posted this to the mailing list but doesn't seem to ever have submitted it to trac.
The relevent sections for what I did are in the calls to the function avc1394_transaction_block.
Here is him posting it to the mailing list http://www.gossamer-threads.com/lists/mythtv/users/266768#266768
Direct link to his original patch: http://bentonroberts.com/personal/media-server/code/sa3250ch-BR.c
Attachments
Change History
Changed 4 years ago by hadees
- Attachment multiple_sa3250_box_support.path added
comment:1 Changed 4 years ago by kormoc
- Owner changed from ijr to kormoc
- Status changed from new to accepted
comment:2 Changed 4 years ago by kormoc
- Status changed from accepted to infoneeded
Can you just upload the full file, the patch doesn't apply cleanly to my copy of -BR.c
comment:3 Changed 4 years ago by stuartm
Alternatively please provide a unified diff (svn diff OR diff -U).
comment:4 Changed 3 years ago by danielk
- Status changed from infoneeded to closed
- Resolution set to invalid
broken patch, no response to request for working one for 10 mos...
pathc for sa3250ch.c