Skip to content

[MAX32630FTHR] Adding new platform #3640

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 1 commit into from Jan 27, 2017
Merged

[MAX32630FTHR] Adding new platform #3640

merged 1 commit into from Jan 27, 2017

Conversation

jessexm
Copy link
Contributor

@jessexm jessexm commented Jan 24, 2017

Description

Adding new platform for Maxim MAX32630FTHR

Status

READY

Migrations

If this PR changes any APIs or behaviors, give a short description of what API users should do when this PR is merged.

NONE

Related PRs

List related PRs against other branches:

NONE

Todos

  • Tests
  • Documentation

Deploy notes

Notes regarding the deployment of this PR. These should note any
required changes in the build environment, tools, compilers, etc.

Steps to test or reproduce

Outline the steps to test or reproduce the PR here.

@bridadan
Copy link
Contributor

/morph test

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1448

All builds and test passed!

@maclobdell
Copy link
Contributor

maclobdell commented Jan 26, 2017

@jessexm

Looks like this test is failing: tests-mbedmicro-rtos-mbed-timer because it expects LED4 to be defined.

For LED4, I recommend you define it to any other LED, such as LED_RED. I made this change, re-ran the test and it passes now.

LED1 = LED_RED,
LED2 = LED_GREEN,
LED3 = LED_BLUE,
LED4 = LED_RED,

Please commit this change and push and it should trigger the CI to run again (or we'll run it again).

@maclobdell
Copy link
Contributor

@adbridge @sg- @0xc0170 Please put high priority on merging this one when it is ready.

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 27, 2017

Code looks good to me, I got already fix on my local branch (I could not push to their remote branch). I'll create a new PR to fix LED4 define.

@0xc0170 0xc0170 merged commit 15cd3da into ARMmbed:master Jan 27, 2017
aisair pushed a commit to aisair/mbed that referenced this pull request Apr 30, 2024
Ports for Upcoming Targets

3571: DISCO_F769NI introduction ARMmbed/mbed-os#3571
3605: Add DELTA_DFCM_NNN50 platform ARMmbed/mbed-os#3605
3640: [MAX32630FTHR] Adding new platform ARMmbed/mbed-os#3640

Fixes and Changes

3397: Add uVisor support for the DISCO_F429ZI ARMmbed/mbed-os#3397
3573: fix failing RTC initialization for MTS_DRAGONFLY_F411RE ARMmbed/mbed-os#3573
3575: Dev stm factorize gpio ARMmbed/mbed-os#3575
3584: STM32: make PeripheralPins.h a common file ARMmbed/mbed-os#3584
3583: STM32F7 Cube FW new release v1.5.1 ARMmbed/mbed-os#3583
3578: Target system - Inherit names from target parents ARMmbed/mbed-os#3578
3599: K22F: Enable TRNG ARMmbed/mbed-os#3599
3614: STM32: make PortNames.h a common file ARMmbed/mbed-os#3614
3617: EFM32GG: Fix GCC_ARM linker script ARMmbed/mbed-os#3617
3618: STM32: Move types definitions to a common file ARMmbed/mbed-os#3618
3631: F3 CUBE update V1.7.0 ARMmbed/mbed-os#3631
3635: STM32 I2C : Fix bug in i2c_byte_read function ARMmbed/mbed-os#3635
3651: Max32630 - fix LED4 ARMmbed/mbed-os#3651
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.

6 participants