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

Improved the reader loop of IoTPy and fix weioSPI and weioSmb #242

Merged
merged 2 commits into from
Oct 2, 2015
Merged

Improved the reader loop of IoTPy and fix weioSPI and weioSmb #242

merged 2 commits into from
Oct 2, 2015

Conversation

ks156
Copy link
Contributor

@ks156 ks156 commented Oct 2, 2015

IoTPy :

With large SFP frame, sometime the SFP frame was decoded even if it was not complete.
This PR fix the reader loop and ensure that the SFP frame is complete before sending it to the decoder

weioSPI :

Fixed a problem with the block_read function
Added function to get and set the block size configuration

weioSmbus :

Added function to get and set the block size configuration

@ks156
Ensure that the SFP frame is completly received before sending it to the
a6b69d0 
decoder
@ks156
Fixed SPILib.read_block_data (struct.pack exception)
7c8ff7d 
Added set/get function to change SPI_FUNC_READ_BLOCK_DATA
and I2C_FUNC_SMBUS_READ_BLOCK_DATA (get_block_size and
set_block_size)
drasko added a commit that referenced this pull request Oct 2, 2015
@drasko
Merge pull request #242 from ks156/next
a0e47fd 
Improved the reader loop of IoTPy and fix weioSPI and weioSmb
@drasko drasko merged commit a0e47fd into nodesign:next Oct 2, 2015
@drasko
Copy link
Contributor

drasko commented Oct 2, 2015

Merged, thanks!

drasko added a commit that referenced this pull request Oct 18, 2015
@drasko
Merge pull request #249 from ks156/next
81fcfdb 
Added some protections (related to #242)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ks156 @drasko