Add building, flashing, and debugging support for CY8CPROTO_062S3_4343W #501
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary of changes
I picked up an Infineon PSoC 62 dev kit a while back, and finally got around to trying it with Mbed. Getting it working was a little bit of an adventure (as some new features had to be added to support multicore CPUs), but I have building, flashing, and debugging working now!
Made the following changes:
monitor program
instead ofload
with OpenOCD, as the former works with this target while the latter does not.MCU_PSOC62_CM4
andMCU_PSOC64_CM4
their own target families to better reflect how the devices are organizedOpenOCD_SCRIPT_DIR
entirely. I added this a long time ago, before I realized that you could just pass OpenOCD scripts by relative path. This variable made it hard to switch the version of OpenOCD in use, because you had to set bothOpenOCD
andOpenOCD_SCRIPT_DIR
on the command line, so removing it improves usability by a lot.mbed_adjust_upload_debug_commands()
. This function, I found out, had a significant issue, which is that the way it used cache variables made it impossible to modify MBED_UPLOAD_LAUNCH_COMMANDS after the first time CMake was run! Looking at it, it seemed a lot simpler to make this function into a getter that doesn't modify the global variable, as it's only used in two places.Note that I did not run the test suite yet as there still seem to be a few issues to fix there. But I will do that next.
Impact of changes
Migration actions required
Documentation
Updated the doc site with the new upload method variables!
Pull request type
Test results
I have verified that I can flash and debug using VS Code and the command line (sadly not CLion as it doesn't allow the same level of customization of the GDB init sequence...). Also verified that the wifi firmware does boot (though the wifi tests are still hardfaulting, need to investigate why in a subsequent PR).