ipdbg: fix double free of virtual-ir data
[openocd.git] / src / jtag / drivers / imx_gpio.c
index d677e8a7e550162f4529d587d1514e9c057e4340..d44b1278c0c104fb6904916ccd92539004fbc19f 100644 (file)
@@ -1,20 +1,9 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+
 /***************************************************************************
  *   Copyright (C) 2017 by Grzegorz Kostka, kostka.grzegorz@gmail.com      *
  *                                                                         *
  *   Based on bcm2835gpio.c                                                 *
- *                                                                         *
- *   This program is free software; you can redistribute it and/or modify  *
- *   it under the terms of the GNU General Public License as published by  *
- *   the Free Software Foundation; either version 2 of the License, or     *
- *   (at your option) any later version.                                   *
- *                                                                         *
- *   This program is distributed in the hope that it will be useful,       *
- *   but WITHOUT ANY WARRANTY; without even the implied warranty of        *
- *   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the         *
- *   GNU General Public License for more details.                          *
- *                                                                         *
- *   You should have received a copy of the GNU General Public License     *
- *   along with this program.  If not, see <http://www.gnu.org/licenses/>. *
  ***************************************************************************/
 
 #ifdef HAVE_CONFIG_H
@@ -22,6 +11,7 @@
 #endif
 
 #include <jtag/interface.h>
+#include <transport/transport.h>
 #include "bitbang.h"
 
 #include <sys/mman.h>
@@ -87,6 +77,7 @@ static int imx_gpio_write(int tck, int tms, int tdi);
 
 static int imx_gpio_swdio_read(void);
 static void imx_gpio_swdio_drive(bool is_output);
+static int imx_gpio_swd_write(int swclk, int swdio);
 
 static int imx_gpio_init(void);
 static int imx_gpio_quit(void);
@@ -96,6 +87,7 @@ static struct bitbang_interface imx_gpio_bitbang = {
        .write = imx_gpio_write,
        .swdio_read = imx_gpio_swdio_read,
        .swdio_drive = imx_gpio_swdio_drive,
+       .swd_write = imx_gpio_swd_write,
        .blink = NULL
 };
 
@@ -143,10 +135,10 @@ static int imx_gpio_write(int tck, int tms, int tdi)
        return ERROR_OK;
 }
 
-static int imx_gpio_swd_write(int tck, int tms, int tdi)
+static int imx_gpio_swd_write(int swclk, int swdio)
 {
-       tdi ? gpio_set(swdio_gpio) : gpio_clear(swdio_gpio);
-       tck ? gpio_set(swclk_gpio) : gpio_clear(swclk_gpio);
+       swdio ? gpio_set(swdio_gpio) : gpio_clear(swdio_gpio);
+       swclk ? gpio_set(swclk_gpio) : gpio_clear(swclk_gpio);
 
        for (unsigned int i = 0; i < jtag_delay; i++)
                asm volatile ("");
@@ -427,19 +419,25 @@ static const struct command_registration imx_gpio_command_handlers[] = {
 
 static const char * const imx_gpio_transports[] = { "jtag", "swd", NULL };
 
-struct jtag_interface imx_gpio_interface = {
-       .name = "imx_gpio",
+static struct jtag_interface imx_gpio_interface = {
        .supported = DEBUG_CAP_TMS_SEQ,
        .execute_queue = bitbang_execute_queue,
+};
+
+struct adapter_driver imx_gpio_adapter_driver = {
+       .name = "imx_gpio",
        .transports = imx_gpio_transports,
-       .swd = &bitbang_swd,
-       .speed = imx_gpio_speed,
-       .khz = imx_gpio_khz,
-       .speed_div = imx_gpio_speed_div,
        .commands = imx_gpio_command_handlers,
+
        .init = imx_gpio_init,
        .quit = imx_gpio_quit,
        .reset = imx_gpio_reset,
+       .speed = imx_gpio_speed,
+       .khz = imx_gpio_khz,
+       .speed_div = imx_gpio_speed_div,
+
+       .jtag_ops = &imx_gpio_interface,
+       .swd_ops = &bitbang_swd,
 };
 
 static bool imx_gpio_jtag_mode_possible(void)
@@ -470,25 +468,22 @@ static int imx_gpio_init(void)
 
        LOG_INFO("imx_gpio GPIO JTAG/SWD bitbang driver");
 
-       if (imx_gpio_jtag_mode_possible()) {
-               if (imx_gpio_swd_mode_possible())
-                       LOG_INFO("JTAG and SWD modes enabled");
-               else
-                       LOG_INFO("JTAG only mode enabled (specify swclk and swdio gpio to add SWD mode)");
-       } else if (imx_gpio_swd_mode_possible()) {
-               LOG_INFO("SWD only mode enabled (specify tck, tms, tdi and tdo gpios to add JTAG mode)");
-       } else {
-               LOG_ERROR("Require tck, tms, tdi and tdo gpios for JTAG mode and/or swclk and swdio gpio for SWD mode");
+       if (transport_is_jtag() && !imx_gpio_jtag_mode_possible()) {
+               LOG_ERROR("Require tck, tms, tdi and tdo gpios for JTAG mode");
+               return ERROR_JTAG_INIT_FAILED;
+       }
+
+       if (transport_is_swd() && !imx_gpio_swd_mode_possible()) {
+               LOG_ERROR("Require swclk and swdio gpio for SWD mode");
                return ERROR_JTAG_INIT_FAILED;
        }
 
        dev_mem_fd = open("/dev/mem", O_RDWR | O_SYNC);
        if (dev_mem_fd < 0) {
-               perror("open");
+               LOG_ERROR("open: %s", strerror(errno));
                return ERROR_JTAG_INIT_FAILED;
        }
 
-
        LOG_INFO("imx_gpio mmap: pagesize: %u, regionsize: %u",
                        (unsigned int) sysconf(_SC_PAGE_SIZE), IMX_GPIO_REGS_COUNT * IMX_GPIO_SIZE);
        pio_base = mmap(NULL, IMX_GPIO_REGS_COUNT * IMX_GPIO_SIZE,
@@ -496,7 +491,7 @@ static int imx_gpio_init(void)
                                MAP_SHARED, dev_mem_fd, imx_gpio_peri_base);
 
        if (pio_base == MAP_FAILED) {
-               perror("mmap");
+               LOG_ERROR("mmap: %s", strerror(errno));
                close(dev_mem_fd);
                return ERROR_JTAG_INIT_FAILED;
        }
@@ -505,7 +500,7 @@ static int imx_gpio_init(void)
         * Configure TDO as an input, and TDI, TCK, TMS, TRST, SRST
         * as outputs.  Drive TDI and TCK low, and TMS/TRST/SRST high.
         */
-       if (imx_gpio_jtag_mode_possible()) {
+       if (transport_is_jtag()) {
                tdo_gpio_mode = gpio_mode_get(tdo_gpio);
                tdi_gpio_mode = gpio_mode_get(tdi_gpio);
                tck_gpio_mode = gpio_mode_get(tck_gpio);
@@ -519,8 +514,15 @@ static int imx_gpio_init(void)
                gpio_mode_output_set(tdi_gpio);
                gpio_mode_output_set(tck_gpio);
                gpio_mode_output_set(tms_gpio);
+
+               if (trst_gpio != -1) {
+                       trst_gpio_mode = gpio_mode_get(trst_gpio);
+                       gpio_set(trst_gpio);
+                       gpio_mode_output_set(trst_gpio);
+               }
        }
-       if (imx_gpio_swd_mode_possible()) {
+
+       if (transport_is_swd()) {
                swclk_gpio_mode = gpio_mode_get(swclk_gpio);
                swdio_gpio_mode = gpio_mode_get(swdio_gpio);
 
@@ -529,11 +531,7 @@ static int imx_gpio_init(void)
                gpio_mode_output_set(swclk_gpio);
                gpio_mode_output_set(swdio_gpio);
        }
-       if (trst_gpio != -1) {
-               trst_gpio_mode = gpio_mode_get(trst_gpio);
-               gpio_set(trst_gpio);
-               gpio_mode_output_set(trst_gpio);
-       }
+
        if (srst_gpio != -1) {
                srst_gpio_mode = gpio_mode_get(srst_gpio);
                gpio_set(srst_gpio);
@@ -544,28 +542,26 @@ static int imx_gpio_init(void)
                  "tdo %d trst %d srst %d", tck_gpio_mode, tms_gpio_mode,
                  tdi_gpio_mode, tdo_gpio_mode, trst_gpio_mode, srst_gpio_mode);
 
-       if (swd_mode) {
-               imx_gpio_bitbang.write = imx_gpio_swd_write;
-               bitbang_switch_to_swd();
-       }
-
        return ERROR_OK;
 }
 
 static int imx_gpio_quit(void)
 {
-       if (imx_gpio_jtag_mode_possible()) {
+       if (transport_is_jtag()) {
                gpio_mode_set(tdo_gpio, tdo_gpio_mode);
                gpio_mode_set(tdi_gpio, tdi_gpio_mode);
                gpio_mode_set(tck_gpio, tck_gpio_mode);
                gpio_mode_set(tms_gpio, tms_gpio_mode);
+
+               if (trst_gpio != -1)
+                       gpio_mode_set(trst_gpio, trst_gpio_mode);
        }
-       if (imx_gpio_swd_mode_possible()) {
+
+       if (transport_is_swd()) {
                gpio_mode_set(swclk_gpio, swclk_gpio_mode);
                gpio_mode_set(swdio_gpio, swdio_gpio_mode);
        }
-       if (trst_gpio != -1)
-               gpio_mode_set(trst_gpio, trst_gpio_mode);
+
        if (srst_gpio != -1)
                gpio_mode_set(srst_gpio, srst_gpio_mode);
 

Linking to existing account procedure

If you already have an account and want to add another login method you MUST first sign in with your existing account and then change URL to read https://review.openocd.org/login/?link to get to this page again but this time it'll work for linking. Thank you.

SSH host keys fingerprints

1024 SHA256:YKx8b7u5ZWdcbp7/4AeXNaqElP49m6QrwfXaqQGJAOk gerrit-code-review@openocd.zylin.com (DSA)
384 SHA256:jHIbSQa4REvwCFG4cq5LBlBLxmxSqelQPem/EXIrxjk gerrit-code-review@openocd.org (ECDSA)
521 SHA256:UAOPYkU9Fjtcao0Ul/Rrlnj/OsQvt+pgdYSZ4jOYdgs gerrit-code-review@openocd.org (ECDSA)
256 SHA256:A13M5QlnozFOvTllybRZH6vm7iSt0XLxbA48yfc2yfY gerrit-code-review@openocd.org (ECDSA)
256 SHA256:spYMBqEYoAOtK7yZBrcwE8ZpYt6b68Cfh9yEVetvbXg gerrit-code-review@openocd.org (ED25519)
+--[ED25519 256]--+
|=..              |
|+o..   .         |
|*.o   . .        |
|+B . . .         |
|Bo. = o S        |
|Oo.+ + =         |
|oB=.* = . o      |
| =+=.+   + E     |
|. .=o   . o      |
+----[SHA256]-----+
2048 SHA256:0Onrb7/PHjpo6iVZ7xQX2riKN83FJ3KGU0TvI0TaFG4 gerrit-code-review@openocd.zylin.com (RSA)