Skip to content
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

Add support for Embedded Planet target Atlas #13128

Merged
merged 4 commits into from
Jun 19, 2020

Conversation

farrenv
Copy link
Contributor

@farrenv farrenv commented Jun 16, 2020

Summary of changes

The Embedded Planet target Atlas is a carrier board for the purpose of demonstrating the Embedded Planet Chronos Telit Modem. This PR is adding the Atlas development board support to Mbed OS 6.

Documentation

Depends on #13009
#13009 has been added to master.
My branch has been rebased onto master.


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

Reviewers

@maclobdell @ARMmbed/team-embeddedplanet

@ciarmcom ciarmcom requested review from maclobdell and a team June 16, 2020 23:00
@ciarmcom
Copy link
Member

@farrenv, thank you for your changes.
@maclobdell @ARMmbed/mbed-os-maintainers @ARMmbed/mbed-os-wan please review.

void ONBOARD_TELIT_ME310::press_power_button(int time_ms)
{
gpio_t gpio_CELL_ON_OFF;
//gpio_t gpio_PWR_MON;
Copy link
Contributor

Choose a reason for hiding this comment

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

is this dead code that should not be here? There's lot of code without any further comment.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Dead code removed. Thanks!

// gpio_write(&gpio_CELL_ON_OFF, 0);
// }

gpio_init_out_ex(&gpio_CELL_ON_OFF, P0_31, 1);
Copy link
Contributor

Choose a reason for hiding this comment

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

these lines need alignment

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Aligned in latest commit. Thanks!

A5 = p31,

/**** QSPI pins ****/
// QSPI1_IO0 = P0_20,
Copy link
Contributor

Choose a reason for hiding this comment

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

why are these commented out?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pinnames file borrowed from Agora. Cleaned up in latest commit. Thanks!

@0xc0170 0xc0170 added the release-type: patch Indentifies a PR as containing just a patch label Jun 18, 2020
@mergify mergify bot dismissed 0xc0170’s stale review June 18, 2020 13:00

Pull request has been modified.

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 18, 2020

@farrenv Can you also attached how this was tested (test results as part of PR) ?

I'll start CI meanwhile

@mbed-ci
Copy link

mbed-ci commented Jun 18, 2020

Test run: SUCCESS

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

@0xc0170 0xc0170 merged commit 6646e8e into ARMmbed:master Jun 19, 2020
@mergify mergify bot removed the ready for merge label Jun 19, 2020
@adbridge adbridge added release-version: 6.1.0 Release-pending and removed release-type: patch Indentifies a PR as containing just a patch Release-pending labels Jun 24, 2020
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.

5 participants