Skip to content

cmake: Add MAX32657 platform and board config - #3272

Open
adi-ramonimbao wants to merge 1 commit into
staging/capi-max32657from
staging/max32657-cmake
Open

cmake: Add MAX32657 platform and board config#3272
adi-ramonimbao wants to merge 1 commit into
staging/capi-max32657from
staging/max32657-cmake

Conversation

@adi-ramonimbao

@adi-ramonimbao adi-ramonimbao commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

Pull Request Description

This PR adds build system support for the MAX32657, and board config files for the MAX32657EVKIT:

  • Board preset and defconfig:
    • board_configs/maxim/CMakePresets.json: new max32657evkit preset
    • board_configs/maxim/max32657evkit_defconfig: default configuration enabling CAPI only
  • MAX32657 platform build files:
    • drivers/platform/maxim/max32657/CMakeLists.txt: pulls in the MSDK PeriphDriver sources, and memory layout definitions
    • drivers/platform/maxim/max32657/memory_layout.cmake: memory layout file
    • drivers/platform/maxim/CMakeLists.txt: do not include maxim_init.c and maxim_delay.c when using CAPI
    • drivers/platform/maxim/toolchain.cmake: add an if() guard to allow overriding of OPENOCD_INTERFACE since you can use also OpenOCD with JLink interface on the MAX32657EVKIT.
  • CAPI CMake wiring:
    • capi/platform/maxim/max32657/CMakeLists.txt: pulls the necessary Maxim CAPI drivers
    • capi/platform/maxim/CMakeLists.txt: selects the max32657 backend when TARGET == max32657

Must be merged after #3271.

PR Type

  • Bug fix (change that fixes an issue)
  • New feature (change that adds new functionality)
  • Breaking change (has dependencies in other repos or will cause CI to fail)

PR Checklist

  • I have followed the Coding style guidelines
  • I have complied with the Submission Checklist
  • I have performed a self-review of the changes
  • I have commented my code, at least hard-to-understand parts
  • I have build all projects affected by the changes in this PR
  • I have tested in hardware affected projects, at the relevant boards
  • I have signed off all commits from this PR
  • I have updated the documentation (wiki pages, ReadMe etc), if applies

@adi-ramonimbao

Copy link
Copy Markdown
Collaborator Author

V2:

  • Change if() guard around non-CAPI maxim_init and maxim_delay

@adi-ramonimbao
adi-ramonimbao force-pushed the staging/max32657-cmake branch from add0fbe to 3ca7a71 Compare August 19, 2026 01:07
@adi-ramonimbao adi-ramonimbao changed the title cmake: Add MAX32657EVKIT board config board_configs: Add MAX32657EVKIT board config Aug 19, 2026
@adi-ramonimbao
adi-ramonimbao force-pushed the staging/max32657-cmake branch from 3ca7a71 to 0eace5b Compare August 19, 2026 07:04
@adi-ramonimbao

Copy link
Copy Markdown
Collaborator Author

V3:

  • Add an if() guard for setting the OpenOCD interface in Maxim toolchain.cmake
  • Set interface/jlink.cfg as the OpenOCD interface for the MAX32657EVKIT.

@adi-ramonimbao

adi-ramonimbao commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator Author

V4:

  • Add capi_wdt.c source if CAPI_WDT is enabled in capi/CMakeLists.txt
  • Set CONFIG_CAPI_WDT=y on MAX32657EVKIT board definition
  • Rebase

EDIT: I previously added a new entry for CAPI_WDT to capi/Kconfig, but realized it's not part of the main branch yet. It still depends on getting #3206 merged. I will update the PR once merged.

@adi-ramonimbao
adi-ramonimbao force-pushed the staging/max32657-cmake branch 2 times, most recently from ab55f9b to de8c022 Compare August 20, 2026 03:46
@adi-ramonimbao

Copy link
Copy Markdown
Collaborator Author

V5:

  • Add drivers/platforms/maxim/max32657/CMakeLists.txt

@adi-ramonimbao
adi-ramonimbao force-pushed the staging/max32657-cmake branch 2 times, most recently from c913452 to 718b11b Compare August 20, 2026 07:16
@adi-ramonimbao

Copy link
Copy Markdown
Collaborator Author

V6:

  • Fix indentation issues in drivers/platform/maxim/CMakeLists.txt causing CMake errors.

@adi-ramonimbao
adi-ramonimbao force-pushed the staging/max32657-cmake branch from 718b11b to 8e1df30 Compare August 24, 2026 06:29
@adi-ramonimbao
adi-ramonimbao force-pushed the staging/max32657-cmake branch from 72ed555 to 882addd Compare August 25, 2026 02:00
@adi-ramonimbao

Copy link
Copy Markdown
Collaborator Author

V9:

  • Rebased from main to staging/capi-max32657 and changed to stacked PR.

@adi-ramonimbao
adi-ramonimbao force-pushed the staging/max32657-cmake branch from 882addd to 522b1bf Compare August 25, 2026 06:37
@adi-ramonimbao

Copy link
Copy Markdown
Collaborator Author

V10:

  • Move memory layout definitions from CMakeLists.txt to memory_layout.cmake.

@adi-ramonimbao
adi-ramonimbao force-pushed the staging/max32657-cmake branch from 522b1bf to b514237 Compare August 25, 2026 06:42
@adi-ramonimbao
adi-ramonimbao force-pushed the staging/max32657-cmake branch 3 times, most recently from 3789592 to 37a4207 Compare August 25, 2026 07:50
@adi-ramonimbao

Copy link
Copy Markdown
Collaborator Author

V11:

  • Removed capi/CMakeLists.txt again.
  • For some reason drivers/platform/maxim/max32657/CMakeLists.txt and drivers/platform/maxim/max32657/memory_layout.cmake were missing in the latest push. Added them again.

@adi-ramonimbao
adi-ramonimbao force-pushed the staging/max32657-cmake branch 3 times, most recently from 1fd6d44 to 8e3013c Compare August 28, 2026 03:43
@adi-ramonimbao adi-ramonimbao changed the title board_configs: Add MAX32657EVKIT board config cmake: Add MAX32657 platform and board config Aug 28, 2026
@adi-ramonimbao

Copy link
Copy Markdown
Collaborator Author

V12:

  • Updated commit message and changed title

@adi-ramonimbao
adi-ramonimbao force-pushed the staging/max32657-cmake branch from 8e3013c to 5517d08 Compare August 28, 2026 06:40
@adi-ramonimbao
adi-ramonimbao force-pushed the staging/max32657-cmake branch 2 times, most recently from 8642c61 to befdde6 Compare August 31, 2026 23:56
@adi-ramonimbao

Copy link
Copy Markdown
Collaborator Author

V13:

  • Add PINT drivers to max32657evkit_defconfig and CMake wiring

Signed-off-by: Ramon Miguel Imbao <ramonmiguel.imbao@analog.com>
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.

2 participants