Skip to content

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