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

feature request: add custom SPI pins #113

Closed
hgpt opened this issue Feb 22, 2018 · 1 comment

Comments

Projects
None yet
2 participants
@hgpt
Copy link

commented Feb 22, 2018

GitHub issue state GitHub issue title GitHub issue author GitHub issue label GitHub issue comments GitHub issue age GitHub issue last update

---------------------------- DO NOT DELETE OR EDIT anything above this line ----------------------------

Some uC like the ESP32 allow the user to route the SPI pins (MISO,MOSI,SCK) to any GPIO pins trough the GPIO MUX matrix. It could prove really handy to be able to change the SPI pins in the Arduino IDE sketch as desired.

@Marzogh Marzogh self-assigned this Feb 24, 2018

Marzogh added a commit that referenced this issue Jun 1, 2019

- An alternate version of the constructor
```
SPIFlash flash(int8_t *SPIPinsArray);
```
can be used (only with ESP32 boards as of now) to enable the use of 
custom SPI pins. ```SPIPinsArray``` has to be a 4 element array 
containing the custom SPI pin numbers (as signed integers - int8_t) in 
the following order - sck, miso, mosi, ss. Refer to the wiki for more 
information. Resolves #113
- Created new error code `NOCHIPSELECTDECLARED` for errors with 
declaring custom SPI pins.
- Using other SPI ports (HSPI, VSPI, SPI1, SPI2 etc.) is now also 
supported on ESP32 boards - along with SAMD and STM32 boards (supported 
since v3.0.0). Resolves #117

@Marzogh Marzogh added this to the v3.4.0 milestone Jun 1, 2019

@Marzogh

This comment has been minimized.

Copy link
Owner

commented Jun 1, 2019

Resolved as of commit 444fde0 . Will be released as part of v3.4.0 code. For now, feel free to use the code from the 'stable' branch. 🙂

@Marzogh Marzogh closed this Jun 1, 2019

@Marzogh Marzogh referenced this issue Jun 1, 2019

Merged

Latest stable changes from development #178

5 of 5 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.