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

drivers: touch: esp32: update driver support #79941

Conversation

wmrsouza
Copy link
Collaborator

@wmrsouza wmrsouza commented Oct 16, 2024

This PR updates the hal_espressif revision to restore touch_sensor support lost on hal side.
The PR also adds some .conf files to the input_dump sample to enable INPUT_ESP32_TOUCH_SENSOR parameter for Espressif boards.

Fixes: #79947

@zephyrbot
Copy link
Collaborator

zephyrbot commented Oct 16, 2024

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
hal_espressif zephyrproject-rtos/hal_espressif@5e7220b zephyrproject-rtos/hal_espressif@09676fc (zephyr) zephyrproject-rtos/hal_espressif@5e7220b4..09676fc3

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@zephyrbot zephyrbot added manifest manifest-hal_espressif DNM This PR should not be merged (Do Not Merge) labels Oct 16, 2024
@wmrsouza wmrsouza force-pushed the bugfix/touch_sensor_support_restoration branch from d7e57dc to 64ff1e9 Compare October 16, 2024 22:02
@zephyrbot zephyrbot removed the DNM This PR should not be merged (Do Not Merge) label Oct 16, 2024
@wmrsouza wmrsouza changed the title input_esp32_touch_sensor support restoration drivers: touch: esp32: update driver support Oct 16, 2024
@fabiobaltieri fabiobaltieri added the backport v3.7-branch Request backport to the v3.7-branch label Oct 16, 2024
fabiobaltieri
fabiobaltieri previously approved these changes Oct 16, 2024
@wmrsouza
Copy link
Collaborator Author

wmrsouza commented Oct 16, 2024

@fabiobaltieri
The hal_espressif revision from this PR points to a different commit (in a different branch) that pointed by #79888, so I'll reopen #79888, remove backport v3.7-branch label from here and add this label there

@wmrsouza wmrsouza removed the backport v3.7-branch Request backport to the v3.7-branch label Oct 16, 2024
raffarost
raffarost previously approved these changes Oct 17, 2024
sylvioalves
sylvioalves previously approved these changes Oct 17, 2024
@wmrsouza wmrsouza force-pushed the bugfix/touch_sensor_support_restoration branch from 64ff1e9 to fb3709d Compare October 17, 2024 12:44
@zephyrbot zephyrbot added the platform: ESP32 Espressif ESP32 label Oct 17, 2024
@zephyrbot zephyrbot requested a review from uLipe October 17, 2024 12:44
@wmrsouza wmrsouza force-pushed the bugfix/touch_sensor_support_restoration branch from fb3709d to 2870db5 Compare October 17, 2024 13:45
raffarost
raffarost previously approved these changes Oct 17, 2024
LucasTambor
LucasTambor previously approved these changes Oct 17, 2024
Update hal_espressif revision to restore support to touch_sensor.

Signed-off-by: Marcio Ribeiro <marcio.ribeiro@espressif.com>
Adds intc_esp32.h missing header file and remove unused code

Signed-off-by: Marcio Ribeiro <marcio.ribeiro@espressif.com>
.conf files added to enable INPUT_ESP32_TOUCH_SENSOR during sample build.
The absence of these files incorrectly prevented the input_dump sample
build from failing due to missing required hal files.

Signed-off-by: Marcio Ribeiro <marcio.ribeiro@espressif.com>
@wmrsouza wmrsouza dismissed stale reviews from LucasTambor and raffarost via 4152ab3 October 17, 2024 14:55
@wmrsouza wmrsouza force-pushed the bugfix/touch_sensor_support_restoration branch from 2870db5 to 4152ab3 Compare October 17, 2024 14:55
@dleach02 dleach02 merged commit 98e0e65 into zephyrproject-rtos:main Oct 21, 2024
26 checks passed
@wmrsouza wmrsouza deleted the bugfix/touch_sensor_support_restoration branch October 29, 2024 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ESP32 touch_sensor not working
7 participants