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

SPI pins are fixed #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

SPI pins are fixed #2

wants to merge 1 commit into from

Conversation

elral
Copy link
Collaborator

@elral elral commented Jan 15, 2024

Description of changes

This display uses the hardware SPI interface, so two pins are fixed and cannot be chosen within the configuration as before.
CLK, MOSI and MISO are deleted from the board.json files.
CLK and Data are deleted from the device.json as they don't need to be configured anymore.
Readme file is updated accordingly.

Copy link

Community folder for this pull request:
Community.zip

@DocMoebiuz
Copy link
Collaborator

Why is this? I think when I started with this, all pins were freely assignable.

@elral
Copy link
Collaborator Author

elral commented Jan 21, 2024

Why is this? I think when I started with this, all pins were freely assignable.

Nearly from the beginning on this device uses the HW SPI. Maybe you remember the slow refresh rate if using SW SPI. There too much data to do it this way. SInce then the readme stated to use fixed pins (the HW SPI ones), I think this makes no sense.

Better to exclude them from the board.json and device.json file, it's also an example how to handle this.

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.

None yet

2 participants