Skip to content

spi-overhaul: add limitation for spi async API #8966

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

Conversation

mprse
Copy link
Contributor

@mprse mprse commented Dec 4, 2018

Description

Handling buffers of a different sizes or half-duplex transfer in asyc mode might be very tricky and is a potential place for many problems. This PR changes the documentation to allow only equal buffer sizes and full duplex transfer in async mode.

cc @c1728p9 @ithinuel

Pull request type

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

@c1728p9 c1728p9 requested a review from ithinuel December 4, 2018 16:22
@adbridge adbridge removed the request for review from ithinuel December 4, 2018 17:35
@ciarmcom ciarmcom requested review from a team December 4, 2018 18:44
@ciarmcom
Copy link
Member

ciarmcom commented Dec 4, 2018

@mprse, thank you for your changes.
@ARMmbed/mbed-os-hal @ARMmbed/mbed-os-maintainers please review.

@cmonr cmonr force-pushed the feature-hal-spec-spi branch from a290cfe to 0892667 Compare December 4, 2018 22:23
@mprse mprse force-pushed the feature-hal-spec-spi_doc_fix_async branch from 7e435f2 to ab61114 Compare December 5, 2018 08:44
0xc0170
0xc0170 previously requested changes Dec 5, 2018
Copy link
Contributor

@0xc0170 0xc0170 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add to the commit msg:

Handling buffers of a different sizes or half-duplex transfer in asyc mode might be very tricky and is a potential place for many problems. This PR changes the documentation to allow only equal buffer sizes and full duplex transfer in async mode.

Handling buffers of a different sizes or half-duplex transfer in asyc mode might be very tricky and is a potential place for many problems. This PR changes the documentation to allow only equal buffer sizes and full duplex transfer in async mode.
@mprse mprse force-pushed the feature-hal-spec-spi_doc_fix_async branch from ab61114 to 7eb7df9 Compare December 5, 2018 14:26
@mprse
Copy link
Contributor Author

mprse commented Dec 5, 2018

Add to the commit msg

Done!

@0xc0170 0xc0170 dismissed their stale review December 5, 2018 15:22

Updated

@cmonr cmonr requested review from c1728p9 and ithinuel December 5, 2018 18:48
@cmonr
Copy link
Contributor

cmonr commented Dec 14, 2018

@0xc0170 @ARMmbed/mbed-os-hal @c1728p9 @ithinuel When y'all get a chance.

Copy link
Contributor

@cmonr cmonr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, especially since the commit also contains the reasoning.

Copy link
Member

@ithinuel ithinuel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@0xc0170
Copy link
Contributor

0xc0170 commented Dec 19, 2018

CI started

@cmonr cmonr merged commit eb59056 into ARMmbed:feature-hal-spec-spi Dec 19, 2018
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