stlink-common: Fix STM32L4 loader write count to reflect 64bits granularity

The stm32l4 loader expects a count of 32 bits words while its granularity is
really 64 bits.

This patch fixes this to simplify count calculation in run_flash_loader().

Signed-off-by: Maxime Coquelin <mcoquelin.stm32@gmail.com>

stlink-common: Update STM32L0/1 loaders to return remaining count in r2

All the loaders returns remaining work count in r2, except stm32l0/1.
Make these loaders behaving as the others to simplify run_flash_loader() code.

Signed-off-by: Maxime Coquelin <mcoquelin.stm32@gmail.com>

stlink-common: Update STM32L0 and STM32L1 loader ABI

This patch invert source and destination registers in the stm32l0 and stm32l1
loaders, so that it follows the same ABI as other stm32 loaders.

Doing that, the run_flash_loader() function can be simplified a little.

Signed-off-by: Maxime Coquelin <mcoquelin.stm32@gmail.com>

stlink-common: Simplify run_flash_loader()

This patch simplifies run_flash_loader() function in preparation for error
propagation from backends.

Doing this, we have less call sites for stlink API.

Signed-off-by: Maxime Coquelin <mcoquelin.stm32@gmail.com>

stlink-common: Introduce type of flash controller enum

Having a type of flash information has some advantages:
 - Make the code easier to read
 - Make adding family derivatives easier (only add a new entry in header file)
 - Make the backends error propagation easier to implement (less places to fix)

Signed-off-by: Maxime Coquelin <mcoquelin.stm32@gmail.com>

st-flash: Improve error handling

Now that libusb errors are propagated up to stlink API, we can handle these
errors.

Signed-off-by: Maxime Coquelin <mcoquelin.stm32@gmail.com>

stlink-common: Make stlink API propagate backend errors

Signed-off-by: Maxime Coquelin <mcoquelin.stm32@gmail.com>

Make the backends propagate errors

As the libusb returns errors, make the backends propagates them so that
callers can decide to continue or stop task execution.

Signed-off-by: Maxime Coquelin <mcoquelin.stm32@gmail.com>

stlink_target_voltage: Check for libusb error

_stlink_usb_target_voltage already returns an error value.
If value return is positive, this is a voltage, if negative this is an error.
Check the return on callers side to inform there is an error in reading the
voltage, instead of notifying of a too low voltage.

Signed-off-by: Maxime Coquelin <mcoquelin.stm32@gmail.com>

stlink_core_id: No need to return core_id value

This rework is done in order to prepare for propagating errors returned by
libusb.

Signed-off-by: Maxime Coquelin <mcoquelin.stm32@gmail.com>

read_debug32: Use a pointer instead of returning the value

This rework is done in order to prepare for propagating errors returned by
libusb.

Signed-off-by: Maxime Coquelin <mcoquelin.stm32@gmail.com>

Merge pull request #383 from xor-gate/cmake-add-st-info

Add st-info to CMakeLists.txt

Add st-info to CMakeLists.txt

be quicker about determining erased byte pattern when flashing, more readable code

Merge pull request #377 from gtalusan/master

be quicker about determining erased byte pattern when flashing

be quicker about determining erased byte pattern when flashing

Merge pull request #376 from gtalusan/master

make st-flash clean up on sigint/term/segv, make gdb-server clean up on sigsegv.

make st-flash clean up on sigint/term/segv, make gdb-server clean up on sigsegv.

Merge pull request #375 from gtalusan/master

make usb backend status check work, remove some dead code

make usb backend status check work, remove some dead code

Merge pull request #374 from gtalusan/master

use libusb synchronous api

use libusb synchronous api

Merge pull request #371 from gtalusan/master

remove empty byte truncation from stlink_fread

Merge pull request #372 from gtalusan/gdb-bigread

clamp gdb memory reads to 0x1800

clamp gdb memory reads to 0x1800

remove empty byte truncation from stlink_fread

Added STM32F051R8T6

Merge pull request #370 from brunowonka/STM32L4_flash_fix

Stm32 l4 flash fix

clean SR flags before programming CR

fix on stm32l4 to clear flash mass erase flags on CR

fix signed unsigned comparison

Merge pull request #369 from meisteg/stutil_reset

st-util: Fix for connecting to running devices

Merge pull request #368 from meisteg/l4_memory_map

Add memory map for STM32L4

Merge pull request #367 from meisteg/fix_every_other

Fix issue where "unknown chip id!" is seen every other time

st-util: Fix for connecting to running devices

Previously, even with the reset enabled, st-util would not connect
to a target if it was running. This change will reset the chip
when opening the connection, which allows st-util to connect
successfully.

Signed-off-by: Greg Meiste <w30289@motorola.com>

Add memory map for STM32L4

Signed-off-by: Greg Meiste <w30289@motorola.com>

Fix issue where "unknown chip id!" is seen every other time

Previously when running the program, reading the chip version
would fail. Running the program a second time worked. This
sequence is repeated the next time st-flash/st-util is run.

Giving reset 10ms to complete before trying to read the chip
version resolves the issue.

Signed-off-by: Greg Meiste <w30289@motorola.com>

Merge pull request #366 from meisteg/l4_voltage

Allow flashing of STM32L4 down to 1.71 V

Allow flashing of STM32L4 down to 1.71 V

The STM32L4 specification specifies that VDD can be 1.71 - 3.6 V.

Signed-off-by: Greg Meiste <w30289@motorola.com>

Merge pull request #363 from gingold-adacore/cache

st-util: synchronize cache for stm32f7

st-util: synchronize cache for stm32f7

This patch handles the cache on the cortex-m7 stm32f7 cpu.
When gdb inserts a soft breakpoint (which is the default for
code in RAM), it replaces an instruction with a breakpoint
instruction.  But if the caches are enabled, the replacement
may be made only in the D-cache.  To reach the I-cache, the
D-cache must be flushed and the I-cache invalidated.

This implementation is coarse: it cleans the whole D-cache
and invalidate the whole I-cache.  It is possible to
track which cache lines have to be cleaned and invalidated.

Merge pull request #360 from gingold-adacore/stm32f7-regs

stm32f7: also send features to gdb.

Merge pull request #359 from gingold-adacore/no-trailing-nul

Do not send a NUL at end of packets to gdb.

stm32f7: also send features to gdb.

So that 'extra' registers (msp, psp, ...) are visible to user.

Do not send a NUL at end of packets to gdb.

GDB doesn't need it and see the NUL as a junk character
(visible with 'set debug remote 1').

Merge pull request #357 from jlbirccyn/patch-1

Added STM32L053R8

Added STM32L053R8

Hello, checked this board with stink. It communicates without flaw with the board:

```
2015-12-15T14:47:03 INFO src/stlink-common.c: Loading device parameters....
2015-12-15T14:47:03 INFO src/stlink-common.c: Device connected is: L0x3 device, id 0x10086417
2015-12-15T14:47:03 INFO src/stlink-common.c: SRAM size: 0x2000 bytes (8 KiB), Flash: 0x10000 bytes (64 KiB) in pages of 128 bytes
2015-12-15T14:47:03 INFO gdbserver/gdb-server.c: Chip ID is 00000417, Core ID is  0bc11477.
2015-12-15T14:47:03 INFO gdbserver/gdb-server.c: Target voltage is 3264 mV.
2015-12-15T14:47:03 INFO gdbserver/gdb-server.c: Listening at *:4242...
```

When I launch GDB and connect to the server, I get :

```
2015-12-15T14:51:41 ERROR gdbserver/gdb-server.c: [FP_CTRL] = 0x00000041 expecting 0x00000261
2015-12-15T14:51:41 INFO gdbserver/gdb-server.c: GDB connected.
```

on the st-util side and :

```
Cannot access memory at address 0xffffffff
```

on the GDB side

Reading and writing to memory and registers works however I get

```
Cannot access memory at address 0xffffffff
```

Each time I write to the memory or to a register

Merge pull request #354 from gvz/master

Added parameter to specify one stlink v2 of many

Added parameter to specify one stlink v2 of many

This adds a parameter to the function stlink_open_usb and to the binary
st-flash to specify one of multiple connected stlinks.
As the identifier the iSerial of the stlink is used.
If no serial is given the function and binary behave as before.

Merge pull request #350 from cheshirekow/cmake_build

Cmake build

Merge branch 'master' into cmake_build

fix MINGW test in cmake

Merge pull request #349 from ibobalo/master

mingw compilation error (sleep) fixed

mingw compilation error (sleep) fixed

Merge pull request #348 from cheshirekow/cmake_build

first stab at a cmake build

first stab at a cmake build

Merge pull request #347 from jauler/master

Implement simple non-blocking ring buffer for communication over stlink

Implement simple non-blocking ring buffer for communication over stlink

Merge pull request #345 from mcoquelin-stm32/stm32f469

Add support to STM32F469/STM32F479

Add support to STM32F469/STM32F479

STM32F469 is a STM32F429 MCU with Dual-QSPI NOR support and DSI screen
support.

Signed-off-by: Maxime Coquelin <mcoquelin.stm32@gmail.com>

README, add working target

Merge pull request #341 from mlu/master

Breakpoint handling for Cortex M7

Breakpoint handling for Cortex M7

Merge pull request #339 from b1y1k/master

correct error message is given upon missing pkg-config package.

correct error message is given upon missing pkg-config package. fixes texane/stlink#338

Merge pull request #336 from mlu/revert-328-master

Revert "Make _stlink_usb_reset use hardreset"

Revert "Make _stlink_usb_reset use hardreset"

Merge pull request #330 from brunowonka/length_fix

segmentation fault length fix

Merge pull request #329 from dj3vande/gdbserver-segfault-fix

Fix segfault in gdb-server

segmentation fault length fix

Correctly compute flash write size for partial pages

Merge pull request #328 from mlu/master

Make _stlink_usb_reset use hardreset

Make _stlink_usb_reset use hardreset

Merge pull request #327 from mlu/master

Make sure MCU is halted before running RAM based flashloaders.

Make sure MCU is halted before running RAM based flashloaders.

Merge pull request #326 from mlu/master

Correct flash sector numbers and sizes for STM32F7

Correct flash sector numbers and sizes for STM32F7

Merge pull request #325 from trlsmax/master

fix can not flash STM32_F3_SMALL

Merge pull request #324 from mlu/master

Added STM32F7 Support

Added STM32F7 Support

fix can not flash STM32_F3_SMALL

FIX: "unaligned addr or size" in attempt of write the program in the RAM (#323)

Merge pull request #321 from dj3vande/stm32l4

STM32L4 support

Clear STM32L4 PER,MER,PG bits

The controller doesn't clear them automatically when the operation
finishes, so to avoid getting errors we need to clear them manually
before we request another operation.

Flash loader for STM32L4

Add supprt for STM32L4 flash driver

No loader yet, but we can erase!

Add STM32L4 to CHIPID #defines and devices[]

Merge pull request #320 from pavel-kirienko/master

STM32F446 support

GDN server: F446 support

Basic support for F446

Merge pull request #316 from radii/stm32f4-low-voltage

STM32F4 8-bit support for 1.8v operation

STM32F4 8-bit support for 1.8v operation

Per ST doc DM00035129.pdf "DocID022063 Rev 5", the STM32F415xx data sheet,
table 40 on page 110 of the PDF, 32-bit program operation is only possible
above 2.7 Volts.

In order to support programming on devices running at lower voltages, this
commit adds an 8-bit programming mode and the necessary tests to enable it
when the reported voltage is below 2.7 Volts.

Merge pull request #315 from schodet/l1xx-cat2-support

Add STM32L1xx Cat.2 devices support

Add STM32L1xx Cat.2 devices support

Merge pull request #313 from schodet/f2-memory

Fix F2 memory map

Fix F2 memory map

This enables GCC to only erase needed pages.

Merge pull request #311 from naedanger/master

Updated the memory map for stm32f42xxx and stm32f43xxx devices.

Updated the memory map for stm32f42xxx and stm32f43xxx devices.

* These devices have up to 256K of sram.
* External memory accessed via the FMC controller is now accessible
  in GDB.

Merge pull request #310 from rkreis/add-working-targets

add some known working targets

add some known working targets