Dear Mike,

thank you for your will to merge.

On Thu, 17 Mar 2011, Mike Frysinger wrote:
>
> On Wednesday, March 16, 2011 10:40:39 Marek Peca wrote:
>> (..) I think it is better to wait fot including the code into UrJTAG 
>> repository and then improve the code (so that any Windows users can 
>> post their complaints and/or improvements).
>
> i'd like that sorted out before merging.  i dont expect you to break out
> Windows and test and such, but we should fix up obviously incorrect things.

I understand. Since I have no Windows testing possibility, I would like to 
ask Gaston for help. Please, could you, Gaston, apply our patch, then 
perform your changes and if OK, post it for merging.

If possible, replace body of stapl.c:jam_delay() function by

usleep(microseconds);

and wipe out delay_loop() and calibrate_delay() functions in the same 
file. I have seen, that UrJTAG contains its own usleep(), so this should 
work on all platforms. Thanks.

(Interestingly, I have now no problems with this delay loop, I am daily 
programming FPGAs using the exact patch as published on our URL; perhaps 
the compiler has been in different mood then earlier.)

Then, I think the patch will be fine.


Best regards,
Marek


------------------------------------------------------------------------------
Colocation vs. Managed Hosting
A question and answer guide to determining the best fit
for your organization - today and in the future.
http://p.sf.net/sfu/internap-sfd2d
_______________________________________________
UrJTAG-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/urjtag-development

Reply via email to