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

Fix lockup issue for REALTEK_RTL8195AM #5464

Merged
merged 4 commits into from Nov 13, 2017

Conversation

Projects
None yet
6 participants
@Archcady
Contributor

Archcady commented Nov 9, 2017

This PR includes PR #5418 and updates APIs in HAL level.

tung7970 and others added some commits Oct 25, 2017

rtl8195am - fix us_ticker interrupt handling
TIMER2_7_IRQ is shared among several timer sources, including
us_ticker. Raising TIMER2_7_IRQ pending bit will trigger the timer
interrupt, but the timer interrupt handler will not know which timer
source this interrupt is for. This patch sets timer load value to one
tick and force us_ticker to fire almost "immediately".

TIMER2_7_IRQ is handled through a common interrupt handler, and
is automatically cleared. Therefore, there is no need to clear IRQ.

The underlying timer HAL treats load value as micro-seconds and does
conversion internally. Therefore, simply pass micro-seconds to timer
HAL without converting to tick first.

Signed-off-by: Tony Wu <tung7970@gmail.com>
@@ -22,7 +22,7 @@ LR_IRAM 0x10007000 (0x70000 - 0x7000) {
*(i.mbedtls*)
*libc.a (+RO)
*rtx_*.o (+RO)
*lib_peripheral_mbed_arm.ar (+RO)
;*lib_peripheral_mbed_arm.ar (+RO)

This comment has been minimized.

@bulislaw

bulislaw Nov 9, 2017

Member

Could we remove this line rather than comment it out for readability?

@tung7970

This comment has been minimized.

Contributor

tung7970 commented Nov 9, 2017

@0xc0170 The first patch is cherry-picked from my previous PR, so I closed mine. And let the whole thing merge in one PR.

@0xc0170

0xc0170 approved these changes Nov 9, 2017

@0xc0170 0xc0170 added needs: CI and removed needs: review labels Nov 9, 2017

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Nov 9, 2017

/morph build

@mbed-ci

This comment has been minimized.

mbed-ci commented Nov 9, 2017

Build : SUCCESS

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

Triggering tests

/morph test
/morph uvisor-test

@mbed-ci

This comment has been minimized.

@0xc0170 0xc0170 added ready for merge and removed needs: CI labels Nov 10, 2017

@theotherjimmy theotherjimmy merged commit 757a4e3 into ARMmbed:master Nov 13, 2017

5 checks passed

AWS-CI uVisor Build & Test Success
Details
ci-morph-build build completed
Details
ci-morph-test test completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment