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

New Feature: BS814A-2 Touch Driver #10447

Merged
merged 5 commits into from
Jan 8, 2021
Merged

Conversation

vic42
Copy link

@vic42 vic42 commented Jan 7, 2021

Description:

This code implements a touch key input driver for the BS814A-2 touch controller of Holtek-Semicon.
The driver was tested on a Teekar brand three-gang touch switch with an ESP82xx module.
I don't have access to hardware including an ESP32 and the touch chip but assume it will work there, too, because all the timing is left to OS functions.

Related issue (if applicable): fixes #
This PR "fixes" #10195

Checklist:

  • The pull request is done against the latest development branch
  • Only relevant files were touched
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • The code change is tested and works on Tasmota core ESP8266 V.2.7.4.9
  • The code change is tested and works on Tasmota core ESP32 V.1.0.5-rc4
  • I accept the CLA.

NOTE: The code change must pass CI tests. Your PR cannot be merged unless tests pass

@vic42 vic42 closed this Jan 7, 2021
@vic42 vic42 reopened this Jan 7, 2021
@arendst
Copy link
Owner

arendst commented Jan 7, 2021

I've now seen zillions of these PR's.... Is this the final one?

If not just update this one instead of closing/opening a new one.

@vic42
Copy link
Author

vic42 commented Jan 7, 2021

I have to find out how change an existing one.
A bit hasty because i don't want to lose my chosen numbers again ;-)
Anyway, this is the one.

@vic42
Copy link
Author

vic42 commented Jan 8, 2021

This is final.

@arendst arendst merged commit 0e148ab into arendst:development Jan 8, 2021
@vic42 vic42 deleted the BS814_NG_2 branch January 8, 2021 16:42
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