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

ESP SPI incorrectly defaults select pin to active=1 #1051

Closed
tve opened this issue Feb 27, 2023 · 2 comments
Closed

ESP SPI incorrectly defaults select pin to active=1 #1051

tve opened this issue Feb 27, 2023 · 2 comments

Comments

@tve
Copy link
Contributor

tve commented Feb 27, 2023

Build environment: Linux
Target device: ESP32

Description
The SPI implementation for ESP incorrectly defaults the select pin active state to 1, see https://github.com/Moddable-OpenSource/moddable/blob/public/modules/io/spi/esp/spi.c#L66
According to ECMA-419: "active: The value to write to the select pin when the SPI instance is active. Must be 1 or 0. This property is optional and defaults to 0."

@phoddie
Copy link
Collaborator

phoddie commented Feb 27, 2023

Thanks for the report.

The SPI implementation for ESP incorrectly defaults the select pin active state to 1

True. The fix is trivial but updating callers that depend on the current implementation is some work.

@phoddie
Copy link
Collaborator

phoddie commented Apr 26, 2023

Fixed with the March 1 commit noted above.

@phoddie phoddie closed this as completed Apr 26, 2023
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

No branches or pull requests

2 participants