Closed ashok-rao closed 6 years ago
@ashok-rao overall looks good.
@dhwalters423 please have a look - hopefully this is all we need when porting to a new platform.
@dhwalters423 @MarceloSalazar : 2 sections ONLY, or it is minimum 2 or more (depending on flash size available on the MCU)?? Can update the docs accordingly..
@teetak01 or @JanneKiiskila Can you please clarify @ashok-rao question
From: https://cloud.mbed.com/docs/current/porting/changing-a-customized-porting-layer.html
PAL_INTERNAL_FLASHNUM#_SECTIONS (default value: 2, max value: 2): The number of sections allocated for security use. Set to 0 if no available sections exist on the internal flash.
PAL_INTERNAL_FLASHSECTION#_ADDRESS: The starting address for each section number (either 1 or 2 depending on how many sections you have defined) for security use. Set to 0 if no available section exists on the internal flash.
So, in principle 1 section would work also, but few things to consider:
I think we need to update the documentation to be more explicit about the requirement to have 2 sections. This is stability and durability requirement from product.
@teetak01 @JanneKiiskila .. thanks!
@MarceloSalazar , @dhwalters423 : added a note about placing SOTP section in flash & also based on feedback from @teetak01 and @JanneKiiskila , clarified minimum SOTP sections= 2 as the requirement. Could you please re-review..thanks.
For this part: "For the Firmware update over the air (FOTA), you need the following:
FlashIAP - Flash In-Application Programming (IAP) Mbed Bootloader or a bootloader compatible with Mbed Cloud Client. " currently, TCP connectivity is also needed. Right now, Firmware download supports only HTTP download which requires TCP (@evedon , @GaryThomsonARM )
-> new PR made for that - https://github.com/ARMmbed/simple-mbed-cloud-client-example-restricted/pull/9
Please set someone from docs team also as reviewer, either @iriark01 or @peknis01 .
Added a section to README for sotp specific changes required to mbed_app.json.
@MarceloSalazar , could you please review? thanks!