boards: ambiq: apollo4p_blue_kbr_evb add new board #67
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for the Apollo4 Blue Plus KBR EVB board, including board definition files, test configurations, sample overlays, and updates to the west.yml Ambiq HAL revision.
Key changes:
- Board definition files for new apollo4p_blue_kbr_evb board
- Test overlays and configurations for various drivers (RTC, PWM, GPIO, counter, ADC)
- Sample overlays for LED and counter applications
- Updated west.yml revision for Ambiq HAL module
Reviewed Changes
Copilot reviewed 23 out of 24 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| west.yml | Updates Ambiq HAL revision to newer version |
| boards/ambiq/apollo4p_blue_kbr_evb/* | Complete board definition including DTS, pinctrl, Kconfig, and documentation |
| tests/drivers//boards/apollo4p_blue_kbr_evb. | Test configurations and overlays for the new board |
| samples/*/boards/apollo4p_blue_kbr_evb.overlay | Sample application overlays for the new board |
| boards/ambiq/apollo4p_blue_kxr_evb/apollo4p_blue_kxr_evb-pinctrl.dtsi | Updates existing KXR board to include connector definitions |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
boards/ambiq/apollo4p_blue_kbr_evb/apollo4p_blue_kbr_evb_connector.dtsi
Outdated
Show resolved
Hide resolved
|
The following west manifest projects have changed revision in this Pull Request:
✅ All manifest checks OK Note: This message is automatically posted and updated by the Manifest GitHub Action. |
RichardSWheatley
force-pushed
the
ambiq-kbr-addon
branch
6 times, most recently
from
09dfad1 to
f39b2c4
Compare
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 26 out of 27 changed files in this pull request and generated 4 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
add new board add confs and overlays update some kxr items Signed-off-by: Richard Wheatley <[email protected]>
RichardSWheatley
force-pushed
the
ambiq-kbr-addon
branch
from
d4a31aa to
ddc82b9
Compare
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 26 out of 27 changed files in this pull request and generated 1 comment.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| #if APOLLO4P_BLUE_KBR_EVB | ||
| #undef UART3RX_P3 | ||
| #define UART3RX_P3 APOLLO4_PINMUX(3, 4) | ||
| #endif |
There was a problem hiding this comment.
Using preprocessor conditionals based on board-specific macros in a shared header file creates board-specific dependencies. Consider using device tree overlays or board-specific pinctrl files instead of conditional compilation in shared headers.
| #if APOLLO4P_BLUE_KBR_EVB | |
| #undef UART3RX_P3 | |
| #define UART3RX_P3 APOLLO4_PINMUX(3, 4) | |
| #endif |
add new board
add confs and overlays
update some kxr items