This is an automated email from Gerrit.

Antonio Borneo (borneo.anto...@gmail.com) just uploaded a new patch set to 
Gerrit, which you can find at http://openocd.zylin.com/85

-- gerrit

commit e6e3105aebd13fe1cf7312af51d375e24cde8a92
Author: Antonio Borneo <borneo.anto...@gmail.com>
Date:   Sun Oct 23 12:02:57 2011 +0800

    NAND/CORE: fix clang warning
    
    The fix is inline with the Linux coding style that forbids
    assignment in if condition
    
    Change-Id: I42a371d6adfdf3b3fb867705211c47d89776ee2a
    Signed-off-by: Antonio Borneo <borneo.anto...@gmail.com>

diff --git a/src/flash/nand/core.c b/src/flash/nand/core.c
index 03deabd..d9bb7de 100644
--- a/src/flash/nand/core.c
+++ b/src/flash/nand/core.c
@@ -598,7 +598,8 @@ int nand_erase(struct nand_device *nand, int first_block, 
int last_block)
                        return ERROR_NAND_OPERATION_TIMEOUT;
                }
 
-               if ((retval = nand_read_status(nand, &status)) != ERROR_OK)
+               retval = nand_read_status(nand, &status);
+               if (retval != ERROR_OK)
                {
                        LOG_ERROR("couldn't read status");
                        return ERROR_NAND_OPERATION_FAILED;

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

Reply via email to