Skip to content

Add new target USI WM-BN-BM-22 #6511

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 2 commits into from
Apr 9, 2018
Merged

Add new target USI WM-BN-BM-22 #6511

merged 2 commits into from
Apr 9, 2018

Conversation

ashok-rao
Copy link
Contributor

Description

Adding USI WM-BN-BM22 (MTB) as a new target.

Pull request type

[ ] Fix
[ ] Refactor
[X] New target
[ ] Feature
[ ] Breaking change

GT logs are attached below. From the logs:

  1. RTC is timing out on GCC and IAR and
  2. There are lp_timer / lp_ticker timing out on ARMCC similar to that reported here: Add support for target MTB_ADV_WISE_1530. #6423 (comment) . Please note that these 2 modules are using the exact same CPU.

GT_Log_ARMCC.txt
GT_Log_GCC.txt
GT_Log_IAR.txt

@0xc0170 @cmonr .. could you please review? Thanks.

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 3, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Apr 3, 2018

Build : SUCCESS

Build number : 1643
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/6511/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 3, 2018

Looking at the test results and issues there. Should lp ticker and rtc be disabled as part of this addition and reenabled once fixed?

@ashok-rao
Copy link
Contributor Author

Thanks @0xc0170 ..this may have to fixed from the MCU implementation side for both this and the #6423 as both use the same MCU.. I can remove those 2 features from targets.json for this module.. and need to add back once fixed..
I'll push the changes to targets.json soon.

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 3, 2018

@ashok-rao If this is affecting some other MCU, please create a new PR for this disabling (this would stay as it is and would just be rebased to include the inclusion).

@mbed-ci
Copy link

mbed-ci commented Apr 3, 2018

@mbed-ci
Copy link

mbed-ci commented Apr 3, 2018

Test : SUCCESS

Build number : 1432
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/6511/1432

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 6, 2018

@ashok-rao Is this ready? I haven't seen the PR disabling features that are failing (report with failures) ?

@0xc0170 0xc0170 changed the title Adding USI WM-BN-BM-22 as a new target Add new target USI WM-BN-BM-22 Apr 9, 2018
@0xc0170 0xc0170 merged commit a3faf58 into ARMmbed:master Apr 9, 2018
@0xc0170
Copy link
Contributor

0xc0170 commented Apr 9, 2018

@ashok-rao Lets discuss the issues you saw with the parent target

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