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

Missing TFT_CS signal in st7735 driver #583

Open
ivoszz opened this issue Jun 16, 2023 · 1 comment
Open

Missing TFT_CS signal in st7735 driver #583

ivoszz opened this issue Jun 16, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@ivoszz
Copy link
Contributor

ivoszz commented Jun 16, 2023

I have some problems with the ST7735 driver setup. I started to study the source code for this driver and I came across one thing I don't understand. In commit #89 @aykevl removed the code to work with csPin (TFT_CS). In my opinion this can work in simple cases with a single SPI device, but should not be in a generic driver. At the same time, csPin was left as a variable in the Device struct and Configuration method, even though it is no longer used anywhere, which is quite confusing. It's possible I'm missing something. What was the reason for removing the code to activate the TFT_CS signal?

@aykevl
Copy link
Member

aykevl commented Jun 20, 2023

To be honest, I think I made #89 without fully understanding CS. It indeed appears to be somewhat broken: it should at least have kept CS at the start and end of the transaction like is done for the ST7789 (see #549).

There are a number of displays however that don't even expose CS.

@deadprogram deadprogram added the bug Something isn't working label Nov 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants