On Friday 02 July 2010 12:15:06 Øyvind Harboe wrote:
> I'm going to leave it to you to follow up and make the decision on whether
> to revert here.
Hi all!
It seems that 2/2 has introduced problems.
It seemed trivial to me, but finally it isn't!
I think we should revert it.
Best regards
Marc
_
I'm going to leave it to you to follow up and make the decision on whether
to revert here.
I was just committing stuff based upon a consensus in the list that this
was the right direction to go, I don't know this code very well...
--
Øyvind Harboe
US toll free 1-866-980-3434 / International +4
On 17/06/2010 09:16, Marc Pignat wrote:
---
src/jtag/drivers/ft2232.c | 19 ++-
1 files changed, 10 insertions(+), 9 deletions(-)
diff --git a/src/jtag/drivers/ft2232.c b/src/jtag/drivers/ft2232.c
index bc8463e..9d40b1c 100644
--- a/src/jtag/drivers/ft2232.c
+++ b/src/jtag/
On 01/07/2010 08:34, Øyvind Harboe wrote:
merged both patches.
Thanks!
Just got back from various travels.
I have committed a build fix for this patchset.
patch2 also breaks when using the ftdi driver, not looked into why yet.
Cheers
Spen
___
Ope
merged both patches.
Thanks!
--
Øyvind Harboe
US toll free 1-866-980-3434 / International +47 51 63 25 00
http://www.zylin.com/zy1000.html
ARM7 ARM9 ARM11 XScale Cortex
JTAG debugger and flash programmer
___
Openocd-development mailing list
Openocd-de
Would you mind posting a link to both patches?
I'm not really following this interface development, just doing
the mechanics of merging patches as approperiate.
--
Øyvind Harboe
US toll free 1-866-980-3434 / International +47 51 63 25 00
http://www.zylin.com/zy1000.html
ARM7 ARM9 ARM11 XScale
David Brownell wrote:
>> It has gotten positive testing feedback so far,
>
> I'd like clarification on that: there were two
> patches. This #2/2 was more invasive, and is
> presumably what was tested. But both cleanups
> should likely get merged.
For my part, I applied both patches and tested.
> I'll let this cool off for a few days
> and then I'll commit it.
Great; I wish I could do that but my current
patches-from-email situation is malfunctioning.
> It has gotten positive testing feedback so far,
I'd like clarification on that: there were two
patches. This #2/2 was more invasi
I'll let this cool off for a few days and then I'll commit it.
It has gotten positive testing feedback so far, so anyone with
objections should speak now.
--
Øyvind Harboe
US toll free 1-866-980-3434 / International +47 51 63 25 00
http://www.zylin.com/zy1000.html
ARM7 ARM9 ARM11 XScale Cortex
>/ These read OK, I thought, but I'm not in a
/>/ position right now to review (in detail) or
/>/ verify either patch.
/>/
/>/ Could someone with an FTDI-based adapter
/>/ please try these out and verify them? Then
/>/ maybe have a committer commit them, if they
/>/ ideed check out?
/
Works for
David Brownell wrote:
> These read OK, I thought, but I'm not in a
> position right now to review (in detail) or
> verify either patch.
>
> Could someone with an FTDI-based adapter
> please try these out and verify them? Then
> maybe have a committer commit them, if they
> ideed check out?
Works
These read OK, I thought, but I'm not in a
position right now to review (in detail) or
verify either patch.
Could someone with an FTDI-based adapter
please try these out and verify them? Then
maybe have a committer commit them, if they
ideed check out?
The first one looked pretty obvious (cleani
---
src/jtag/drivers/ft2232.c | 19 ++-
1 files changed, 10 insertions(+), 9 deletions(-)
diff --git a/src/jtag/drivers/ft2232.c b/src/jtag/drivers/ft2232.c
index bc8463e..9d40b1c 100644
--- a/src/jtag/drivers/ft2232.c
+++ b/src/jtag/drivers/ft2232.c
@@ -714,23 +714,24 @@ stati
13 matches
Mail list logo
