PSA: PSoc 6 configurable secure memory size

[evgenibo]

Description

On M0 target the secure flash and ram section sizes should be configurable.
Flash secure ROM  size can be configured to 128K, 256K or 512K. 
Changed the defalut size to 256K.

SRAM size can be configured to 64K or 128K. 64K is a default value.

Pull request type

[ ] Fix
[ ] Refactor
[x] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

@ARMmbed/team-cypress @ARMmbed/mbed-os-psa

Release Notes

[0xc0170]

@ARMmbed/mbed-os-psa Is this targeting 5.12rc2 ?

[orenc17]

@ARMmbed/mbed-os-psa Is this targeting 5.12rc2 ?

rc3

[orenc17]

if one of the PSA macros is missing this is an error

[orenc17]

Also please add alignment check with static assert

[orenc17]

@ARMmbed/team-cypress does the PSoC6 has any alignment requirement for flash and ram split between the cores?

[morser499]

Flash must be on a 512k boundary, not overlapping between cores. No specific requirements for SRAM.

[OlegKapshii]

Flash and RAM start addresses of protected areas have to be aligned to a number dividable by the protected area size. To use unaligned regions - subregion field is used. It is quite complex algo to do it automatically.
Will check in docs tomorrow, just to be 150% sure.

[orenc17]

Flash and RAM start addresses of protected areas have to be aligned to a number dividable by the protected area size. To use unaligned regions - subregion field is used. It is quite complex algo to do it automatically.
Will check in docs tomorrow, just to be 150% sure.

@OlegKapshii
i'm asking because i wanted @evgenibo to add a static assert like this:

static_assert( PSA_SECURE_ROM_START % PSA_SECURE_RAM_SIZE == 0)

[OlegKapshii]

@orenc17 It would be great to have such assert.

[OlegKapshii]

Is it reasonable to add all possible region sizes conversion in the if sequence?

[orenc17]

@evgenibo i think @OlegKapshii request is acceptable,
for the RAM please create if sections for the following sizes: 32KB - 256KB ( 4 options)
for the FLASH please create if sections for the following sizes: 64KB - 1MB ( 5 options)

[cmonr]

@evgenibo Would anything bad happen if this was retargeted to 5.12.1?

As is, this PR doesn't seem critical enough for RC3.

[cmonr]

Or does it fall in this bucket? #10055 (comment)

[evgenibo]

This is critical to run client application on this board and has the same significance as #10055

[cmonr]

Still waiting on #10034 (comment) to be addressed before starting CI

[0xc0170]

Still waiting on #10034 (comment) to be addressed before starting CI

Was this addressed by latest push?

[orenc17]

there is another change required for kvstore config i've showed @evgenibo where the fix should be

[orenc17]

I can confirm the PSA tests pass on the board with the changes

[0xc0170]

CI started

[OlegKapshii]

LGTM

[mbed-ci]

Test run: SUCCESS

Summary: 13 of 13 test jobs passed
Build number : 1
Build artifacts