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

Support for stm32duino #156

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

BugerDread
Copy link

I changed types of variables used to specify IO pins such as DC, CS, RST, MISO, CLK from int8_t to int16_t because for example on bluepill PA4 = 196, PB0 = 200, PB1 = 201 and such numbers cant fit into int8_t (that holds +-127 because of sign) and this cause the library doesnt work with stm32duino / bluepill (the RST/DC/CS pins does not output any signal to LCD). It works perfectly with int16_t.

Tested using https://github.com/stm32duino/Arduino_Core_STM32 version 2.1.0

the Adafruit-GFX-Library needs to be modified in similar way, will put another pull request for this in a moment

not tested with other platforms like AVR or ESP because I dont have such HW just now, tested using stm32f103c8 bluepill, also tested that the "graphic test" example compiles wo errors with avr / esp8266 board selected

…this comatible with stm32duino (tested using 2.1.0)
…variables from int8_t to int16_t for this platform
…/ce pin variables from int8_t to int16_t for this platform"

This reverts commit 7857b31.
@BugerDread
Copy link
Author

adafruit/Adafruit-GFX-Library#365 - GFX lib PR

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.

1 participant