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.

Sign up for GitHub

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

Jump to bottom

Renesas : Modify LPTicker driver #8234

Merged
merged 2 commits into from
Oct 12, 2018
Merged

Renesas : Modify LPTicker driver #8234

merged 2 commits into from
Oct 12, 2018

Conversation

TomoYamanaka
Copy link
Contributor

Description

I modified the LPTicker driver for Renesas boards.

  • Change the channel number of MTU2 used at LPTicker from 3 to 2. [GR-LYCHEE]
  • Modify typo of macro that use at return value of mbed_get_a9_tick_irqn(). [GR-PEACH, GR-LYCHEE]

Test result

Ticker and sleep test results are here.

Test_GR-PEACH_GCC.txt
Test_GR-LYCHEE_GCC.txt

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Breaking change

@TomoYamanaka
Change MTU2 channel number for LPTicker of GR-LYCHEE
48038e4 
MTU2(Multi function Timer pulse Unit 2) can create the low freaquency.
Currently GR-LYCHEE uses MTU2 channel 3 for LPTicker, but I noticed that a part of it is used by another function.
Thus, I changed MTU2 channel number for LPTicker to 2.
@TomoYamanaka
Modify typo of return value at mbed_get_a9_tick_irqn().
b01cdcb
@TomoYamanaka
Copy link
Contributor Author

@0xc0170 I'd appreciate if you could trigger a CI test.

@NirSonnenschein
Copy link
Contributor

/morph build

@mbed-ci
Copy link

mbed-ci commented Oct 11, 2018

Build : SUCCESS

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

Triggering tests

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

@mbed-ci
Copy link

mbed-ci commented Oct 11, 2018

Exporter Build : SUCCESS

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

@mbed-ci
Copy link

mbed-ci commented Oct 11, 2018

Test : SUCCESS

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

@cmonr cmonr merged commit 285de88 into ARMmbed:master Oct 12, 2018
@TomoYamanaka TomoYamanaka deleted the modify_LPTicker branch October 15, 2018 01:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0xc0170 0xc0170 0xc0170 approved these changes

Assignees
No one assigned
Labels
release-version: 5.10.2
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@TomoYamanaka @NirSonnenschein @mbed-ci @0xc0170 @cmonr