Skip to content

Conversation

@ozersa
Copy link
Contributor

@ozersa ozersa commented May 31, 2024

The MAX32675 microcontroller (MCU) is an advanced system-on-chip (SoC) featuring an Arm® Cortex®-M4F.

This PR add MAX32675 and it boards by providing basic functionalities

Tested with:
blinky example
image

button example
image

gpio_basic_api test
image

uart_basic_api test
image

@ozersa ozersa force-pushed the add-MAX32675 branch 4 times, most recently from ece1830 to f6bf7bd Compare June 3, 2024 11:33
@ozersa ozersa force-pushed the add-MAX32675 branch 2 times, most recently from eb6b9e6 to 4dfe6b0 Compare June 13, 2024 06:15
@ozersa ozersa marked this pull request as ready for review June 26, 2024 13:02
@zephyrbot zephyrbot added area: GPIO platform: ADI Analog Devices, Inc. labels Jun 26, 2024
MaureenHelm
MaureenHelm previously approved these changes Jun 28, 2024
@MaureenHelm MaureenHelm added this to the v4.0.0 milestone Jun 28, 2024
ozersa and others added 4 commits July 29, 2024 11:12
Add MAX32675 Kconfig and dts files

Co-authored-by: Maureen Helm <[email protected]>
Signed-off-by: Sadik Ozer <[email protected]>
Added MAX32675EVKit board
For more information about this board please check
https://www.analog.com/

Co-authored-by: Maureen Helm <[email protected]>
Signed-off-by: Sadik Ozer <[email protected]>
Enable gpio driver test for max32675evkit board

Signed-off-by: Sadik Ozer <[email protected]>
TRNG enabled for MAX32675EVKIT board

Signed-off-by: Sadik Ozer <[email protected]>
@ozersa
Copy link
Contributor Author

ozersa commented Jul 29, 2024

Rebased onto main and #74943 enabled for this MCU.

Copy link
Member

@MaureenHelm MaureenHelm left a comment

Choose a reason for hiding this comment

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

The twister failure looks unrelated, but needs to be resolved. It can be reproduced in main and git bisect points to commit 372c718.

commit 372c7183ef082c70ab5a6a5ba3fe9c486eb3d90f (refs/bisect/bad)
Author: Bjarki Arge Andreasen <[email protected]>
Date:   Mon Jun 10 17:51:15 2024 +0200

    modem: pipe: Add explicit timeout to sync APIs

    The modem pipe APIs include synchronous calls to open/close,
    which internally use a fixed timeout of 10 seconds. The timeout
    should be configurable through the APIs, anywhere from K_NO_WAIT
    to K_FOREVER.

    This commit adds timeout parameters to the open/close APIs, and
    updates in-tree usage of the open/close APIs to explicitly
    provide the previously implicit timeout of 10 seconds.

    Signed-off-by: Bjarki Arge Andreasen <[email protected]>

@bjarki-andreasen

@MaureenHelm
Copy link
Member

The twister failure looks unrelated, but needs to be resolved. It can be reproduced in main and git bisect points to commit 372c718.

commit 372c7183ef082c70ab5a6a5ba3fe9c486eb3d90f (refs/bisect/bad)
Author: Bjarki Arge Andreasen <[email protected]>
Date:   Mon Jun 10 17:51:15 2024 +0200

    modem: pipe: Add explicit timeout to sync APIs

    The modem pipe APIs include synchronous calls to open/close,
    which internally use a fixed timeout of 10 seconds. The timeout
    should be configurable through the APIs, anywhere from K_NO_WAIT
    to K_FOREVER.

    This commit adds timeout parameters to the open/close APIs, and
    updates in-tree usage of the open/close APIs to explicitly
    provide the previously implicit timeout of 10 seconds.

    Signed-off-by: Bjarki Arge Andreasen <[email protected]>

@bjarki-andreasen

Fixed in #76461

@MaureenHelm MaureenHelm requested a review from ubieda August 5, 2024 21:12
@nashif nashif merged commit 3240b03 into zephyrproject-rtos:main Aug 6, 2024
@MaureenHelm MaureenHelm deleted the add-MAX32675 branch August 6, 2024 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: GPIO platform: ADI Analog Devices, Inc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants