Skip to content

De-select CS before resetting#16

Open
deividAlfa wants to merge 1 commit intoFloyd-Fish:masterfrom
deividAlfa:patch-1
Open

De-select CS before resetting#16
deividAlfa wants to merge 1 commit intoFloyd-Fish:masterfrom
deividAlfa:patch-1

Conversation

@deividAlfa
Copy link
Copy Markdown

Not doing so will cause issues if you don't spècifically take care of the CS pin before calling ST7789_Init.
Adjust reset timings as in datasheet.

Not doing so will cause issues if you don't spècifically take care of the CS pin before calling ST7789_Init.
Adjust reset timings as in datasheet.
@Stredoslovak
Copy link
Copy Markdown

Stredoslovak commented Mar 22, 2023

@deividAlfa
Hello, I'm writing this here cuz u didn't provide an Issues tab on your fork of this lib.

I am having some problems regarding the PIN_DEFFINITONs. Mainly the DC and RST pins. I don't know what you meant by "CUBEmx pin names", I've tried defining as PAx and also numbers.
None of this works.
Could you please provide me with a configured LCD.h file? So I can figure out the names?

Here is a part of my current LCD.h setup:

/* Pin connections. Use same names as in CubeMX */
#define LCD_DC 11
#define LCD_RST	12
//#define LCD_CS                LCD_CS  
//#define LCD_BL              LCD_BL

EDIT
I've managed to advance by using this:

/* Pin connections. Use same names as in CubeMX */
#define LCD_DC GPIO_PIN_11
#define LCD_RST	GPIO_PIN_12
//#define LCD_CS                LCD_CS  /* Disable if your display has no CS pin */
//#define LCD_BL              LCD_BL  /* Enable if you need backlight control */

But it still does not compile correctly. What am I missing?

@deividAlfa
Copy link
Copy Markdown
Author

deividAlfa commented Mar 22, 2023

Hi, I enabled the Issues tab, please open one for this.
About cube: simply do right-click on the pin and assign a custom pin name, ex. "LCD_DC”

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.

2 participants