target: declare local symbols as static
[openocd.git] / src / target / dsp5680xx.c
index 1563b6458583e161ad0a76dd785cfda0af4006e2..ee26d24acfc75ceebc349c466d169deb2a8b317b 100644 (file)
@@ -16,9 +16,7 @@
  *  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, write to the                         *
- *  Free Software Foundation, Inc.,                                       *
- *  51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.           *
+ *  along with this program.  If not, see <http://www.gnu.org/licenses/>. *
  ***************************************************************************/
 
 #ifdef HAVE_CONFIG_H
@@ -42,7 +40,7 @@ struct dsp5680xx_common dsp5680xx_context;
 #define CHECK_HALT(target) if (target->state != TARGET_HALTED) HALT_FAIL
 #define check_halt_and_debug(target) { CHECK_HALT(target); CHECK_DBG; }
 
-int dsp5680xx_execute_queue(void)
+static int dsp5680xx_execute_queue(void)
 {
        int retval;
 
@@ -95,7 +93,7 @@ static int dsp5680xx_drscan(struct target *target, uint8_t *d_in,
        if (len > 32) {
                retval = ERROR_FAIL;
                err_check(retval, DSP5680XX_ERROR_JTAG_DR_LEN_OVERFLOW,
-                         "dr_len overflow, maxium is 32");
+                         "dr_len overflow, maximum is 32");
        }
        /* TODO what values of len are valid for jtag_add_plain_dr_scan? */
        /* can i send as many bits as i want? */
@@ -640,7 +638,7 @@ static int switch_tap(struct target *target, struct jtag_tap *master_tap,
  * more complicated routine, which is guaranteed to work, but requires
  * a reset. This will complicate comm with the flash module, since
  * after a reset clock divisors must be set again.
- * This implementation works most of the time, and is not accesible to the
+ * This implementation works most of the time, and is not accessible to the
  * user.
  *
  * @param target
@@ -894,12 +892,6 @@ static int dsp5680xx_arch_state(struct target *target)
        return ERROR_OK;
 }
 
-int dsp5680xx_target_status(struct target *target, uint8_t *jtag_st,
-                           uint16_t *eonce_st)
-{
-       return target->state;
-}
-
 static int dsp5680xx_assert_reset(struct target *target)
 {
        target->state = TARGET_RESET;
@@ -1003,17 +995,17 @@ static int dsp5680xx_poll(struct target *target)
                         __func__);
                target->state = TARGET_UNKNOWN;
                return ERROR_TARGET_FAILURE;
-       };
+       }
        if (target->state == TARGET_UNKNOWN) {
                LOG_ERROR("%s: Target status invalid - communication failure",
                          __func__);
                return ERROR_TARGET_FAILURE;
-       };
+       }
        return ERROR_OK;
 }
 
 static int dsp5680xx_resume(struct target *target, int current,
-                           uint32_t address, int hb, int d)
+                           target_addr_t address, int hb, int d)
 {
        if (target->state == TARGET_RUNNING) {
                LOG_USER("Target already running.");
@@ -1125,7 +1117,7 @@ static int dsp5680xx_read_16_single(struct target *t, uint32_t a,
        /* at this point the data i want is at the reg eonce can read */
        retval = core_rx_lower_data(target, data_read);
        err_check_propagate(retval);
-       LOG_DEBUG("%s:Data read from 0x%06X: 0x%02X%02X", __func__, address,
+       LOG_DEBUG("%s:Data read from 0x%06" PRIX32 ": 0x%02X%02X", __func__, address,
                  data_read[1], data_read[0]);
        return retval;
 }
@@ -1170,8 +1162,8 @@ static int dsp5680xx_read_32_single(struct target *t, uint32_t a,
        return retval;
 }
 
-static int dsp5680xx_read(struct target *t, uint32_t a, unsigned size,
-                         unsigned count, uint8_t *buf)
+static int dsp5680xx_read(struct target *t, target_addr_t a, uint32_t size,
+                         uint32_t count, uint8_t *buf)
 {
        struct target *target = t;
 
@@ -1307,7 +1299,7 @@ static int dsp5680xx_write_8(struct target *t, uint32_t a, uint32_t c,
                        dsp5680xx_write_16_single(target, address + iter, data_16,
                                                  pmem);
                if (retval != ERROR_OK) {
-                       LOG_ERROR("%s: Could not write to p:0x%04X", __func__,
+                       LOG_ERROR("%s: Could not write to p:0x%04" PRIX32, __func__,
                                  address);
                        dsp5680xx_context.flush = 1;
                        return retval;
@@ -1364,7 +1356,7 @@ static int dsp5680xx_write_16(struct target *t, uint32_t a, uint32_t c,
                        dsp5680xx_write_16_single(target, address + iter,
                                                  data[iter], pmem);
                if (retval != ERROR_OK) {
-                       LOG_ERROR("%s: Could not write to p:0x%04X", __func__,
+                       LOG_ERROR("%s: Could not write to p:0x%04" PRIX32, __func__,
                                  address);
                        dsp5680xx_context.flush = 1;
                        return retval;
@@ -1401,7 +1393,7 @@ static int dsp5680xx_write_32(struct target *t, uint32_t a, uint32_t c,
                        dsp5680xx_write_32_single(target, address + (iter << 1),
                                                  data[iter], pmem);
                if (retval != ERROR_OK) {
-                       LOG_ERROR("%s: Could not write to p:0x%04X", __func__,
+                       LOG_ERROR("%s: Could not write to p:0x%04" PRIX32, __func__,
                                  address);
                        dsp5680xx_context.flush = 1;
                        return retval;
@@ -1425,7 +1417,7 @@ static int dsp5680xx_write_32(struct target *t, uint32_t a, uint32_t c,
  *
  * @return
  */
-static int dsp5680xx_write(struct target *t, uint32_t a, uint32_t s, uint32_t c,
+static int dsp5680xx_write(struct target *t, target_addr_t a, uint32_t s, uint32_t c,
                           const uint8_t *b)
 {
        /* TODO Cannot write 32bit to odd address, will write 0x12345678  as 0x5678 0x0012 */
@@ -1470,7 +1462,7 @@ static int dsp5680xx_write(struct target *t, uint32_t a, uint32_t s, uint32_t c,
        return retval;
 }
 
-static int dsp5680xx_write_buffer(struct target *t, uint32_t a, uint32_t size,
+static int dsp5680xx_write_buffer(struct target *t, target_addr_t a, uint32_t size,
                                  const uint8_t *b)
 {
        check_halt_and_debug(t);
@@ -1487,7 +1479,7 @@ static int dsp5680xx_write_buffer(struct target *t, uint32_t a, uint32_t size,
  *
  * @return
  */
-static int dsp5680xx_read_buffer(struct target *t, uint32_t a, uint32_t size,
+static int dsp5680xx_read_buffer(struct target *t, target_addr_t a, uint32_t size,
                                 uint8_t *buf)
 {
        check_halt_and_debug(t);
@@ -1507,7 +1499,7 @@ static int dsp5680xx_read_buffer(struct target *t, uint32_t a, uint32_t size,
  *
  * @return
  */
-static int dsp5680xx_checksum_memory(struct target *t, uint32_t a, uint32_t s,
+static int dsp5680xx_checksum_memory(struct target *t, target_addr_t a, uint32_t s,
                                     uint32_t *checksum)
 {
        return ERROR_FAIL;
@@ -1524,7 +1516,7 @@ static int dsp5680xx_checksum_memory(struct target *t, uint32_t a, uint32_t s,
  *
  * @return
  */
-static int perl_crc(uint8_t *buff8, uint32_t word_count)
+static int perl_crc(const uint8_t *buff8, uint32_t word_count)
 {
        uint16_t checksum = 0xffff;
 
@@ -1557,7 +1549,7 @@ static int perl_crc(uint8_t *buff8, uint32_t word_count)
  *
  * @return
  */
-int dsp5680xx_f_SIM_reset(struct target *target)
+static int dsp5680xx_f_SIM_reset(struct target *target)
 {
        int retval = ERROR_OK;
 
@@ -1733,7 +1725,12 @@ static int dsp5680xx_f_ex(struct target *t, uint16_t c, uint32_t a, uint32_t d,
 }
 
 /**
- * Prior to the execution of any Flash module command, the Flash module Clock Divider (CLKDIV) register must be initialized. The values of this register determine the speed of the internal Flash Clock (FCLK). FCLK must be in the range of 150kHz ≤ FCLK ≤ 200kHz for proper operation of the Flash module. (Running FCLK too slowly wears out the module, while running it too fast under programs Flash leading to bit errors.)
+ * Prior to the execution of any Flash module command, the Flash module Clock
+ * Divider (CLKDIV) register must be initialized. The values of this register
+ * determine the speed of the internal Flash Clock (FCLK). FCLK must be in the
+ * range of 150kHz ≤ FCLK ≤ 200kHz for proper operation of the Flash module.
+ * (Running FCLK too slowly wears out the module, while running it too fast
+ * under programs Flash leading to bit errors.)
  *
  * @param target
  *
@@ -1789,7 +1786,11 @@ static int set_fm_ck_div(struct target *target)
 }
 
 /**
- * Executes the FM calculate signature command. The FM will calculate over the data from @address to @address + @words -1. The result is written to a register, then read out by this function and returned in @signature. The value @signature may be compared to the the one returned by perl_crc to verify the flash was written correctly.
+ * Executes the FM calculate signature command. The FM will calculate over the
+ * data from @address to @address + @words -1. The result is written to a
+ * register, then read out by this function and returned in @signature. The
+ * value @signature may be compared to the one returned by perl_crc to
+ * verify the flash was written correctly.
  *
  * @param target
  * @param address Start of flash array where the signature should be calculated.
@@ -1971,7 +1972,8 @@ int dsp5680xx_f_erase(struct target *target, int first, int last)
  * 0x0000001E  0xA961           bra     *-30
  */
 
-const uint16_t pgm_write_pflash[] = { 0x8A46, 0x0013, 0x807D, 0xE700,
+static const uint16_t pgm_write_pflash[] = {
+               0x8A46, 0x0013, 0x807D, 0xE700,
                0xE700, 0x8A44, 0xFFFE, 0x017B,
                0xE700, 0xF514, 0x8563, 0x8646,
                0x0020, 0x0014, 0x8646, 0x0080,
@@ -1981,16 +1983,16 @@ const uint16_t pgm_write_pflash[] = { 0x8A46, 0x0013, 0x807D, 0xE700,
                0x0013, 0x0010, 0xA961
 };
 
-const uint32_t pgm_write_pflash_length = 31;
+static const uint32_t pgm_write_pflash_length = 31;
 
-int dsp5680xx_f_wr(struct target *t, uint8_t *b, uint32_t a, uint32_t count,
+int dsp5680xx_f_wr(struct target *t, const uint8_t *b, uint32_t a, uint32_t count,
                   int is_flash_lock)
 {
        struct target *target = t;
 
        uint32_t address = a;
 
-       uint8_t *buffer = b;
+       const uint8_t *buffer = b;
 
        int retval = ERROR_OK;
 
@@ -2264,7 +2266,7 @@ int dsp5680xx_f_lock(struct target *target)
        return retval;
 }
 
-static int dsp5680xx_step(struct target *target, int current, uint32_t address,
+static int dsp5680xx_step(struct target *target, int current, target_addr_t address,
                          int handle_breakpoints)
 {
        err_check(ERROR_FAIL, DSP5680XX_ERROR_NOT_IMPLEMENTED_STEP,

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)