Skip to content

MIMXRT1050_EVK: Update SPI HAL driver #7904

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 1 commit into from
Sep 5, 2018

Conversation

mmahadevan108
Copy link
Contributor

Description

Use a different SDK API to write to the SPI Bus

Pull request type

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

Use a different SDK API to write to the SPI Bus

Signed-off-by: Mahesh Mahadevan <[email protected]>
@mmahadevan108
Copy link
Contributor Author

cc @cmonr @0xc0170

@mmahadevan108
Copy link
Contributor Author

cc @maclobdell

@cmonr
Copy link
Contributor

cmonr commented Sep 4, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Sep 4, 2018

Build : SUCCESS

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

Triggering tests

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

@mbed-ci
Copy link

mbed-ci commented Sep 4, 2018

@cmonr
Copy link
Contributor

cmonr commented Sep 4, 2018

/morph test

@mbed-ci
Copy link

mbed-ci commented Sep 4, 2018

@cmonr
Copy link
Contributor

cmonr commented Sep 4, 2018

Export node got stuck.
/morph export-build

@mbed-ci
Copy link

mbed-ci commented Sep 4, 2018

@mbed-ci
Copy link

mbed-ci commented Sep 5, 2018

@cmonr
Copy link
Contributor

cmonr commented Sep 5, 2018

Duplicate ASM symbols issue.

/morph export-build

@cmonr
Copy link
Contributor

cmonr commented Sep 5, 2018

Test errors don't appear to be PR related.

/morph test

@mbed-ci
Copy link

mbed-ci commented Sep 5, 2018

@mbed-ci
Copy link

mbed-ci commented Sep 5, 2018

@cmonr cmonr merged commit dc45990 into ARMmbed:master Sep 5, 2018
@mmahadevan108 mmahadevan108 deleted the MIMXRT1050_Fix_Spi branch September 5, 2018 16:49
@maclobdell
Copy link
Contributor

maclobdell commented Sep 6, 2018

@cmonr - which release will this be in? There are multiple conflicting tags. (earlier release is preferred, please).

@cmonr
Copy link
Contributor

cmonr commented Sep 6, 2018

@maclobdell The timing that this PR got merged has put it in a weird, but known, spot.

This will be going into 5.9.7 early tomorrow, but since 5.10 has already been made, we also need this to enter 5.10.1 to make sure we don't lose it.

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