Opened 13 years ago

Closed 12 years ago

#5406 closed defect (invalid)

Fixed a bug in sa3250ch-BR

Reported by: hadees Owned by: Rob Smith
Priority: minor Milestone: unknown
Component: contrib Version: unknown
Severity: medium Keywords:
Cc: Ticket locked: no


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

Direct link to his original patch:

Attachments (1)

multiple_sa3250_box_support.path (3.5 KB) - added by hadees 13 years ago.
pathc for sa3250ch.c

Download all attachments as: .zip

Change History (5)

Changed 13 years ago by hadees

pathc for sa3250ch.c

comment:1 Changed 13 years ago by Rob Smith

Owner: changed from Isaac Richards to Rob Smith
Status: newaccepted

comment:2 Changed 13 years ago by Rob Smith

Status: acceptedinfoneeded

Can you just upload the full file, the patch doesn't apply cleanly to my copy of -BR.c

comment:3 Changed 13 years ago by stuartm

Alternatively please provide a unified diff (svn diff OR diff -U).

comment:4 Changed 12 years ago by danielk

Resolution: invalid
Status: infoneededclosed

broken patch, no response to request for working one for 10 mos...

Note: See TracTickets for help on using tickets.