esp-bsp
esp-bsp copied to clipboard
fixes gt911 without reset pin (BSP-642)
ESP-BSP Pull Request checklist
Note: For new BSPs create a PR with this link.
- [ ] Version of modified component bumped
- [ ] CI passing
Change description
Please describe your change here
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
Your Name seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.
I am not sure, if I understand right your changes. But this part of code needs both pins (interrupt + reset). This change is useless from my point of view.
its not. without the patch, you can not use this driver without the reset pin (esp_lcd_touch_gt911->config.rst_gpio_num = GPIO_NUM_NC). but devices like the LilyGo T-Deck Plus have the reset pin not connected. with this change you can use the driver on those devices.
its not. without the patch, you can not use this driver without the reset pin (
esp_lcd_touch_gt911->config.rst_gpio_num = GPIO_NUM_NC). but devices like the LilyGo T-Deck Plus have the reset pin not connected. with this change you can use the driver on those devices.
Sorry for late reply. The code, which you changed, is only for setting I2C address of the controller during initialization. The reset is not necessary for other usage of this driver.
@AkaBkn Please, did you see my latest comment?
@AkaBkn Due to no activity and no issue found - closing this PR. Feel free reopen it.