LPC55S69: Update Flash driver to set clock frequency

[mmahadevan108]

Description

This is to ensure the flash access time is set correctly

Pull request type

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

[mmahadevan108]

cc @orenc17 @mikisch81 @0xc0170

[orenc17]

Just discard the binary change

[cmonr]

@orenc17 Why is the binary change not needed?

[orenc17]

It has been agreed that binary updates will occur only on releases

The CI is already compiling the secure binaries for each PR

[cmonr]

Making sure I understand then (completely possible I missed this), is there no guarantee that binaries on master are up to date?

CC @ARMmbed/mbed-os-maintainers @ARMmbed/mbed-os-psa?

[mmahadevan108]

@orenc17 . If the binary is not updated then developers working on the master branch will not get this change, am I understanding this correctly?

[mmahadevan108]

@orenc17 . If the binary is not updated then developers working on the master branch will not get this change, am I understanding this correctly?

[orenc17]

@alzix can update on this better

[alzix]

There are prebuilt secure image binaries committed to mbed-os master, which require manual rebuild and commit.
I'm not fond of this approach, but it is here for providing backward compatibility (in term of build instructions) for application developers working with default secure image.
This technique has its own drawbacks, request for rebuild&commit secure images upon any change is one of them.
We will work to provide better user experience in the upcoming releases.

[NirSonnenschein]

note that when running CI, the CI will build updated secure images prior to testing (but these images are not pushed into master). This means that any developer developing on a device that requires secure images must build these images before running as a the default ones provided as part of the OS tree are usually out of date (they will get the change if they rebuild the secure image). This approach is not the most friendly as Alex said above but it does prevent issues with changes being overrun when edited by multiple PRs at once(secure images are non-mergable). before creating a release the secure images will be updated, so that at least official releases should always have updated binaries,

[0xc0170]

@mmahadevan108 tfm.bin should be removed as requested above

[mmahadevan108]

PR updated to remove binary

Binary removed.

[cmonr]

CI started

[mbed-ci]

Test run: SUCCESS

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