This is an automated email from Gerrit.

?yvind Harboe (oyvindhar...@gmail.com) just uploaded a new patch set to Gerrit, 
which you can find at http://openocd.zylin.com/32

-- gerrit
commit cc362b6a38d9eaa1d30af13cdf2cb6fe1bb042a5
Author: Matt Reimer <mrei...@sdgsystems.com>
Date:   Mon Sep 19 10:30:13 2011 -0400

    xscale: fix bug in xscale_receive()
    
    The code in xscale_receive() that tries to skip invalid reads (i.e.
    reads that don't have the DBG_SR[0] 'valid' bit set) seems to be
    wrong, as it only looks at the first word's valid flag rather than
    each word's own valid flag. Am I reading the code correctly? If so,
    the attached patch should fix it.
    
    If this looks correct, I'll generate a proper patch and commit message.
    
    Matt
    
    Change-Id: I74ebe2ad7a36d340a9dd3b8487578b6ea7f3cf1e
    Signed-off-by: Øyvind Harboe <oyvind.har...@zylin.com>

diff --git a/src/target/xscale.c b/src/target/xscale.c
index 3b56745..b469b86 100644
--- a/src/target/xscale.c
+++ b/src/target/xscale.c
@@ -317,7 +317,7 @@ static int xscale_receive(struct target *target, uint32_t 
*buffer, int num_words
                /* examine results */
                for (i = words_done; i < num_words; i++)
                {
-                       if (!(field0[0] & 1))
+                       if (!(field0[i] & 1))
                        {
                                /* move backwards if necessary */
                                int j;

-- 
_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to