This is an automated email from Gerrit.

"Name of user not set <chris.whee...@narfindustries.com>" just uploaded a new 
patch set to Gerrit, which you can find at 
https://review.openocd.org/c/openocd/+/8141

-- gerrit

commit 3ec7f9fedea17375712e50fa0816f9b9f2b4eea7
Author: WheelNarf <chris.whee...@narfindustries.com>
Date:   Mon Feb 12 10:26:56 2024 -0800

    jtag/drivers/imx_gpio: add configurable gpio address space size to 
accommodate other imx chips.  Fix style/formatting errors
    
    Change-Id: Ie88ecbffe3187e3f9e348e864a32bccf9d5c31ac
    Signed-off-by: WheelNarf <chris.whee...@narfindustries.com>

diff --git a/src/jtag/drivers/imx_gpio.c b/src/jtag/drivers/imx_gpio.c
index e9f2b76982..ca446503df 100644
--- a/src/jtag/drivers/imx_gpio.c
+++ b/src/jtag/drivers/imx_gpio.c
@@ -212,9 +212,14 @@ COMMAND_HANDLER(imx_gpio_handle_jtag_gpionums)
                return ERROR_COMMAND_SYNTAX_ERROR;
        }
 
-       if( tck_gpio / 32 >= IMX_GPIO_REGS_COUNT || tms_gpio / 32 >= 
IMX_GPIO_REGS_COUNT || tdi_gpio / 32 >= IMX_GPIO_REGS_COUNT || tdo_gpio / 32 >= 
IMX_GPIO_REGS_COUNT) {
+       if(tck_gpio / 32 >= IMX_GPIO_REGS_COUNT)
+               return ERROR_JTAG_INIT_FAILED;
+       if(tms_gpio / 32 >= IMX_GPIO_REGS_COUNT)
+               return ERROR_JTAG_INIT_FAILED;
+       if(tdi_gpio / 32 >= IMX_GPIO_REGS_COUNT)
+               return ERROR_JTAG_INIT_FAILED;
+       if(tdo_gpio / 32 >= IMX_GPIO_REGS_COUNT)
                return ERROR_JTAG_INIT_FAILED;
-       }
 
        command_print(CMD,
                        "imx_gpio GPIO config: tck = %d, tms = %d, tdi = %d, 
tdo = %d",

-- 

Reply via email to