1. 09 Aug, 2011 1 commit
  2. 08 Aug, 2011 1 commit
  3. 03 Aug, 2011 1 commit
  4. 02 Aug, 2011 1 commit
  5. 01 Aug, 2011 1 commit
  6. 29 Jul, 2011 1 commit
  7. 28 Jul, 2011 5 commits
  8. 27 Jul, 2011 2 commits
  9. 26 Jul, 2011 3 commits
  10. 18 Jul, 2011 3 commits
  11. 15 Jul, 2011 1 commit
  12. 14 Jul, 2011 1 commit
  13. 12 Jul, 2011 4 commits
  14. 10 Jul, 2011 1 commit
  15. 04 Jul, 2011 3 commits
    • Drasko DRASKOVIC's avatar
      mips_m4k and arm7_9 : Fix soft bkpt endianess for 16-bit instructions · ac43d7a6
      Drasko DRASKOVIC authored
      The patch fix comparison of target data on the host by using
      target_buffer_get_u16() to transform current_instr to
      _host_ endianess before comparison.
      ac43d7a6
    • Peter Horn's avatar
      cortex_m3: add auto maskisr · d4cd6f03
      Peter Horn authored
      
      
      This patch extends the cortex_m3 maskisr command by a new option 'auto'.
      The 'auto' option handles interrupts during stepping in a way they are
      processed but don't disturb the program flow during debugging.
      
      Before one had to choose to either enable or disable interrupts. The former
      steps into interrupt handlers when they trigger. This disturbs the flow during
      debugging, making it hard to follow some piece of code when interrupts occur
      often.
      
      When interrupts are disabled, the flow isn't disturbed but code relying on
      interrupt handlers to be processed will stop working. For example a delay
      function counting the number of timer interrupts will never complete, RTOS
      task switching will not occur and output I/O queues of interrupt driven
      I/O will stall or overflow.
      
      Using the 'maskisr' command also typically requires gdb hooks to be supplied
      by the user to switch interrupts off during the step and to enable them again
      afterward.
      
      The new 'auto' option of the 'maskisr' command solves the above problems. When
      set, the step command allows pending interrupt handlers to be executed before
      the step, then the step is taken with interrupts disabled and finally interrupts
      are enabled again. This way interrupt processing stays in the background without
      disturbing the flow of debugging. No gdb hooks are required. The 'auto'
      option is the default, since it's believed that handling interrupts in this
      way is suitable for most users.
      
      The principle used for interrupt handling could probably be used for other
      targets too.
      Signed-off-by: default avatarSpencer Oliver <ntfreak@users.sourceforge.net>
      d4cd6f03
    • Spencer Oliver's avatar
      Revert "cortex_m3: add auto maskisr" · 89fa8ce2
      Spencer Oliver authored
      This reverts commit ff640f19.
      
      Original patch reverted as Author's name was incorrectly set.
      89fa8ce2
  16. 01 Jul, 2011 1 commit
    • Drasko DRASKOVIC's avatar
      mips_m4k : Fix soft breakpoint endianess handling · bad3ee87
      Drasko DRASKOVIC authored
      In order to compare data read from the target with some marcros or data
      defined on the host, we must transform this read data from target
      endianess to host endianess.
      target_read_memory() gets bytes from target to the host, but keeps them in _target_
      endianess. This is OK if we just want to temporary keep this data on the
      host, like keeping breakpoint->orig_instr. But if we want to use this
      data for any ispections and comparisons on the host side, we must
      transform it to _host_ endianess, by using target_buffer_get_u32()
      function.
      Currently this transformation is missing, and check current_instr ==
      MIPS32_SDBBP will never pass if target and host endianess differ,
      because current_instr will be kept in _target_ endianess and
      MIPS32_SDBBP will be kept in _host_ endianess,
      
      The patch fix this issue by using target_buffer_get_u32() to transform current_instr to
      _host_ endianess before comparison.
      bad3ee87
  17. 29 Jun, 2011 1 commit
  18. 28 Jun, 2011 5 commits
    • Øyvind Harboe's avatar
      dsp5680xx: disable for now, it generates warnings · d567df02
      Øyvind Harboe authored
      Use "git revert <commit>" to revert this commit, then build and
      repair and post patch to the mailing list.
      
      Warnings generated with:
      
      nios2-elf-gcc (GCC) 3.4.6 (Altera Nios II 9.1 b222)
      
      openocd/src/target/dsp5680xx.c: In function 'eonce_rx_upper_data':
      openocd/src/target/dsp5680xx.c:252: warning: cast increases required
      alignment of target type
      openocd/src/target/dsp5680xx.c: In function 'eonce_rx_lower_data':
      openocd/src/target/dsp5680xx.c:268: warning: cast increases required
      alignment of target type
      openocd/src/target/dsp5680xx.c: In function 'eonce_pc_store':
      openocd/src/target/dsp5680xx.c:508: warning: dereferencing type-punned
      pointer will break strict-aliasing rules
      openocd/src/target/dsp5680xx.c: In function 'dsp5680xx_read':
      openocd/src/target/dsp5680xx.c:736: warning: cast increases required
      alignment of target type
      openocd/src/target/dsp5680xx.c:737: warning: cast increases required
      alignment of target type
      openocd/src/target/dsp5680xx.c: In function 'dsp5680xx_write_8':
      openocd/src/target/dsp5680xx.c:823: warning: cast increases required
      alignment of target type
      openocd/src/target/dsp5680xx.c: In function 'dsp5680xx_write':
      openocd/src/target/dsp5680xx.c:938: warning: cast increases required
      alignment of target type
      openocd/src/target/dsp5680xx.c:941: warning: cast increases required
      alignment of target type
      openocd/src/target/dsp5680xx.c: In function 'dsp5680xx_f_wr':
      openocd/src/target/dsp5680xx.c:1355: warning: cast increases required
      alignment of target type
      d567df02
    • Øyvind Harboe's avatar
      mips4k: fix big-endian hosts and host alignment problems · 2482244b
      Øyvind Harboe authored
      the code was making assumptions about the endianness of the host.
      2482244b
    • Øyvind Harboe's avatar
      dsp5680xx: fix compilation problems · e4c054ca
      Øyvind Harboe authored
      use a more specific global variable name than "context", which
      can easily conflict with other things.
      e4c054ca
    • Spencer Oliver's avatar
      cortex_m3: add auto maskisr · ff640f19
      Spencer Oliver authored
      
      
      This patch extends the cortex_m3 maskisr command by a new option 'auto'.
      The 'auto' option handles interrupts during stepping in a way they are
      processed but don't disturb the program flow during debugging.
      
      Before one had to choose to either enable or disable interrupts. The former
      steps into interrupt handlers when they trigger. This disturbs the flow during
      debugging, making it hard to follow some piece of code when interrupts occur
      often.
      
      When interrupts are disabled, the flow isn't disturbed but code relying on
      interrupt handlers to be processed will stop working. For example a delay
      function counting the number of timer interrupts will never complete, RTOS
      task switching will not occur and output I/O queues of interrupt driven
      I/O will stall or overflow.
      
      Using the 'maskisr' command also typically requires gdb hooks to be supplied
      by the user to switch interrupts off during the step and to enable them again
      afterward.
      
      The new 'auto' option of the 'maskisr' command solves the above problems. When
      set, the step command allows pending interrupt handlers to be executed before
      the step, then the step is taken with interrupts disabled and finally interrupts
      are enabled again. This way interrupt processing stays in the background without
      disturbing the flow of debugging. No gdb hooks are required. The 'auto'
      option is the default, since it's believed that handling interrupts in this
      way is suitable for most users.
      
      The principle used for interrupt handling could probably be used for other
      targets too.
      Signed-off-by: default avatarSpencer Oliver <ntfreak@users.sourceforge.net>
      ff640f19
    • Peter Horn's avatar
      e53f7e5f
  19. 27 Jun, 2011 1 commit
  20. 26 Jun, 2011 3 commits