Skip to content

M263: Support custom board #12642

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Mar 24, 2020

Conversation

ccli8
Copy link
Contributor

@ccli8 ccli8 commented Mar 18, 2020

Summary of changes

Continuing #12576, this PR tries to support custom board based on M261 series chips by enabling capability of being configurable for the following items:

  1. Pin definitions
  2. UART for USB VCOM and STDIO
  3. Flash/SRAM specification (start/size)

Pull request type

[x] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results

[] No Tests required for this change (E.g docs only update)
[x] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

@ciarmcom ciarmcom requested review from Ronny-Liu and a team March 18, 2020 04:00
@ciarmcom
Copy link
Member

@ccli8, thank you for your changes.
@Ronny-Liu @ARMmbed/mbed-os-maintainers please review.

0xc0170
0xc0170 previously approved these changes Mar 19, 2020
@0xc0170
Copy link
Contributor

0xc0170 commented Mar 19, 2020

CI started

@mbed-ci
Copy link

mbed-ci commented Mar 19, 2020

Test run: FAILED

Summary: 3 of 4 test jobs failed
Build number : 1
Build artifacts

Failed test jobs:

  • jenkins-ci/mbed-os-ci_build-GCC_ARM
  • jenkins-ci/mbed-os-ci_build-ARM
  • jenkins-ci/mbed-os-ci_build-IAR

@0xc0170
Copy link
Contributor

0xc0170 commented Mar 19, 2020

Looks like new MCU target is getting build (not set to private?)

@@ -14712,13 +14712,12 @@
"5"
]
},
"NUMAKER_IOT_M263A": {
"MCU_M261": {
Copy link
Contributor

Choose a reason for hiding this comment

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

As noted in my comment, if this is base MCU, needs to be private

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@0xc0170 Fixed it. Thanks for your explanation.

ccli8 added 5 commits March 20, 2020 09:16
This is to support M261 series based targets.
Don't set SERIAL_TX/SERIAL_RX to USBTX/USBRX which should be dedicated for USB VCOM and not for other uses.
This is to support custom boards based on M261 series chips.
-   STDIO_UART and STDIO_UART_TX/STDIO_UART_RX
-   USB_UART and USBTX/USBRX
This is to support custom boards based on M261 series chips.
@ccli8 ccli8 force-pushed the nuvoton_m263_custom_board branch from 6901277 to f5cb899 Compare March 20, 2020 01:28
@mergify mergify bot dismissed 0xc0170’s stale review March 20, 2020 01:29

Pull request has been modified.

@0xc0170
Copy link
Contributor

0xc0170 commented Mar 23, 2020

CI restarted

@mbed-ci
Copy link

mbed-ci commented Mar 23, 2020

Test run: SUCCESS

Summary: 6 of 6 test jobs passed
Build number : 2
Build artifacts

@0xc0170 0xc0170 merged commit 2c5660a into ARMmbed:master Mar 24, 2020
@mergify mergify bot removed the ready for merge label Mar 24, 2020
@ccli8 ccli8 deleted the nuvoton_m263_custom_board branch March 24, 2020 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants