Skip to content

add csChange, pad fields to spiIOCTransfer struct#39

Closed
bgentry wants to merge 1 commit intokidoman:masterfrom
bgentry:add-missing-spiioctransfer-struct-fields
Closed

add csChange, pad fields to spiIOCTransfer struct#39
bgentry wants to merge 1 commit intokidoman:masterfrom
bgentry:add-missing-spiioctransfer-struct-fields

Conversation

@bgentry
Copy link
Copy Markdown

@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
Copy Markdown

luxflux commented Nov 9, 2015

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

@wujiang
Copy link
Copy Markdown
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
Copy Markdown

Fix my rpi2 as well.
👍

@kidoman
Copy link
Copy Markdown
Owner

kidoman commented Dec 26, 2015

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

@npotts
Copy link
Copy Markdown

npotts commented Jan 10, 2016

This fixes the problem on mine as well

@anthonylawson
Copy link
Copy Markdown

@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
Copy Markdown

@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
Copy Markdown
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