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.

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

add csChange, pad fields to spiIOCTransfer struct #39

Conversation

bgentry
Copy link

@bgentry bgentry commented Oct 6, 2015

Per the discussion in #38, these two fields are part of the equivalent struct in the kernel SPI driver. Adding the fields allows the SPI driver to work on a Raspberry Pi Model 2 B.

h/t @hbhasker

Fixes #38. Fixes #24.

Per the discussion in kidoman#38, these two fields are part of the equivalent
struct in the kernel SPI driver. Adding the fields allows the SPI driver
to work on a Raspberry Pi Model 2 B.

h/t @hbhasker

Fixes kidoman#38. Fixes kidoman#24.
@luxflux
Copy link

luxflux commented Nov 9, 2015

Any news on this one? The change looks good to me.

@wujiang
Copy link
Contributor

wujiang commented Dec 22, 2015

This is a showstopper for spi on raspberry pi 2. The fix looks good to me. Please review and merge.

@alaingilbert
Copy link

Fix my rpi2 as well.
👍

@kidoman
Copy link
Owner

kidoman commented Dec 26, 2015

Has anyone gotten the chance to test this on the other RPi's ?

@npotts
Copy link

npotts commented Jan 10, 2016

This fixes the problem on mine as well

@anthonylawson
Copy link

@kidoman I have a model B1 running the latest version of Raspbian that was running into this issue, but this fix worked on the B1 version as well.

Copy link

@fhedberg fhedberg left a comment

Choose a reason for hiding this comment

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

Fixes SPI on RPI3

@tve
Copy link
Contributor

tve commented Sep 25, 2016

Closing as this should be fixed by #69, please reopen if I missed something.

@tve tve closed this Sep 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants