search

AGlass0fMilk / mbed-mcuboot-demo

Demo of mcuboot with Mbed
11 stars 15 forks source link

Improve support for DISCO_L475VG_IOT01A #13

Open LDong-Arm opened 3 years ago

LDong-Arm commented 3 years ago

Mbed OS builds .bin images by default for most targets, but many targets (e.g. ST targets running the latest ST-LINK firmware) also support .hex files, making this demo much easier to prepare. In case a target doesn't support .hex files, README.md already contains steps for merging .hex files into a single one and converting it to a .bin file.

The majority of flash devices support reading single bytes without alignment concerns. The config mcuboot.read-granularity enables additional code to handle read alignment, and it comes with a slight code size and performance overhead. So disable it on DISCO_L475VG_IOT01A.

Also update README for https://github.com/AGlass0fMilk/mbed-mcuboot-blinky/pull/4.

@AGlass0fMilk

noonfom commented 3 years ago

Unrelated to the PR: why exclude CC310 from the NRF52840_DK and EP_AGORA configs?

LDong-Arm commented 3 years ago

Unrelated to the PR: why exclude CC310 from the NRF52840_DK and EP_AGORA configs?

@AGlass0fMilk Presumably this is to reduce code size? MCUboot needs crypto APIs from mbedtls (which is in the "requires" field of mbed_app.json), but the bootloader doesn't need the complete hardware support.

AGlass0fMilk commented 3 years ago

Unrelated to the PR: why exclude CC310 from the NRF52840_DK and EP_AGORA configs?

@noonfom @LDong-Arm

The nRF52840 CC310 driver has never worked properly for me and causes hard faults. If you can test it with CC310 enabled and do not encounter hard faults we can re-enable it.