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

[DOC] https://www.arduino.cc/en/Reference/SPI #5273

Closed
olikraus opened this issue Aug 21, 2016 · 3 comments
Closed

[DOC] https://www.arduino.cc/en/Reference/SPI #5273

olikraus opened this issue Aug 21, 2016 · 3 comments
Assignees
Labels
Component: Documentation Related to Arduino's documentation content

Comments

@olikraus
Copy link

olikraus commented Aug 21, 2016

The table about the SPI Modes is wrong for modes 2 and 3. Additionally i suggest to add a column on the clock edge for the data capture (takeover) of the client device.

Existing (wrong) table:

Mode Clock Polarity (CPOL) Clock Phase (CPHA) Output Edge
SPI_MODE0 0 0 Falling
SPI_MODE1 0 1 Rising
SPI_MODE2 1 0 Falling
SPI_MODE3 1 1 Rising

I suggest to replace the table by this (modified cell content in bold):

Mode Clock Polarity (CPOL) Clock Phase (CPHA) Output Edge Data Capture
SPI_MODE0 0 0 Falling Rising
SPI_MODE1 0 1 Rising Falling
SPI_MODE2 1 0 Rising Falling
SPI_MODE3 1 1 Falling Rising

References:
Forum discussion:
http://forum.arduino.cc/index.php?topic=419660.0

Existing issues related to this topic:
olikraus/u8g2#52
esp8266/Arduino#2416

@facchinm facchinm added the Component: Documentation Related to Arduino's documentation content label Aug 22, 2016
@agdl
Copy link
Member

agdl commented Aug 22, 2016

Done thank you!

@agdl agdl closed this as completed Aug 22, 2016
@olikraus
Copy link
Author

Thanks
Unfortunately there is a small spelling mistake on the web page (marked in bold below)

Mode Clock Polarity (CPOL) Clock Phase (CPHA) Output Edge Data Capture
SPI_MODE0 0 0 Falling Rising
SPI_MODE1 0 1 Rising Falling
SPI_MODE2 1 0 Rising Flling
SPI_MODE3 1 1 Falling Rising

@agdl
Copy link
Member

agdl commented Aug 23, 2016

Fixed sorry for that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Documentation Related to Arduino's documentation content
Projects
None yet
Development

No branches or pull requests

3 participants