what do you mean with the original programmer? did you disassemble the original software?
No, I did not used the reverse engineering. There are only a number of assumptions, while I looked at PDF manual of the programmer.
this is your code: … /* The AJAWe serial programmer just have DSR off when it is active. */ data_in = sp_get_pin(PIN_DSR);
if(data_in) { have_prog = 0; } if (!have_prog) { msg_perr("No compatible %s programmer detected.\n", ((pony_type == TYPE_AJAWE) ? "AJAWe" : "SI-Prog")); return 1; }i read that as if (DSR == high) then no ajawe attached. correct me please if that is wrong. if it would be if (DSR == low)... then i would agree with you that it is the most plausible configuration, but your code does not make sense to me.
Ouch, you just found a bug in the detection code. :)
The right code (after my last thinking) is:
/* The AJAWe serial programmer just have DSR on when it is active. */ data_in = sp_get_pin(PIN_DSR);
if(!data_in) { have_prog = 0; }
since we can not test or verify it otherwise, i will disable the detection for now as posted. we can add it later if need be and we find out how it really is supposed to work. thanks for looking at the changes and your initial patch!
I totally agree with the disabling of AJAWe auto-detection until we figure out how it really works and IF really we need it.
All the best, Virgil-Adrian.