Skip to content

Conversation

@maclobdell
Copy link
Contributor

Summary of changes

Adds support for Cypress PSoC 62 Prototyping board (CY8CPROTO-062-4343W).

This configuration adds usage of Quad SPI Flash for storage. This frees up internal memory.

[Yes] I confirm this contribution is my own and I agree to license it with Apache 2.0.

[Yes] I confirm the moderators may change the PR before merging it in.

For new board enablements only:

[Yes] I confirm the board is Mbed Enabled and passes the Mbed Enabled test set.

[Yes] I confirm the contribution has been tested properly and the tests results for TESTS are attached.
pytest-cy8cproto.txt

[Yes] I confirm mbed-os.lib and mbed-cloud-client.lib hashes or the content in folders mbed-os and mbed-cloud-client were not modified in order to pass the tests.

Details

  • I could not get file system for external kvstore working (plus I heard that file system for kvstore is deprecated anyway). So I configured it for tbd_external. This seems to work except for the known issue below.
  • To build the bootloader, I had to use a slightly modified version of mbed-os and mbed-bootloader. This is pushed to my repo at https://github.com/maclobdell/mbed-bootloader,
    on branch cy8cproto-bootloader-build-4.1.3. This is a temporary location and I will work to get it supported in the official mbed-bootloader repo.
  • I included the bootloader in binary form instead of hex to avoid gaps in the bootloader memory footprint which can occur on the platform due to dual core layout. This avoids any potential issues with the build tools not merging the bootloader correctly.

Known Issues

  • Currently the device gets a new ID every time the application is re-flashed. This needs to be debugged.
  • I know that the configuration is a bit messy. I used absolute addresses and sizes instead of formulas. If that is a problem, I can go back and change them to formulas where possible

@iot-pdmc
Copy link
Collaborator

Thanks for your changes.
@ARMmbed/pdmc-integrators please review.

@maclobdell
Copy link
Contributor Author

See attached log related to the known issue I mentioned. Please let me know if you have any suggestion. I can make a change and push a new branch. This is with trace set to 'TRACE_LEVEL_INFO_WARN'
cy8cproto_trace_log.txt

This is after a reset. There are several errors, but I don't know if they are recoverable errors (normal), or not (real problem).

Copy link
Contributor

@teetak01 teetak01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tdb_external is more optimal over filesystem as it does not pull in the filesystem driver.

@teetak01
Copy link
Contributor

Can you rebase the PR? It seems to be out-of-date?

OTherwise the PR looks fine. The new device ID with flashing is known issue in our backlog, probably related on Device key generation. Something we can not do explicitly before Mbed OS 6.

@maclobdell maclobdell force-pushed the add-cy8cproto-062-4343w branch from 75dce9c to 505546a Compare May 14, 2020 11:53
@maclobdell
Copy link
Contributor Author

I just rebased the PR.

@teetak01
Copy link
Contributor

run-ci

Copy link
Contributor

@teetak01 teetak01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you still update the README?

@teetak01
Copy link
Contributor

@maclobdell can you check the linker issues? There is quite a bit of warnings and some crypto-errors.

@maclobdell maclobdell force-pushed the add-cy8cproto-062-4343w branch from 505546a to 3661077 Compare May 18, 2020 18:24
@maclobdell
Copy link
Contributor Author

maclobdell commented May 18, 2020

I just updated the branch with fix for linker issues, updates to readme, and slight re-ordering of the config section for this board. I squashed the commits down to one and force pushed. I tested firmware update after building with the Arm compiler and it worked fine. Please re-test!

@teetak01
Copy link
Contributor

build-only

@iot-pdmc
Copy link
Collaborator

@teetak01 teetak01 merged commit 46c8b0a into master May 19, 2020
@maclobdell maclobdell deleted the add-cy8cproto-062-4343w branch September 4, 2020 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants