Skip to content

Conversation

asyncbridge
Copy link

Add configuration for the TP-Link Tapo C120 v1.

@viktorxda viktorxda force-pushed the master branch 12 times, most recently from debe03d to 9bbe14e Compare June 6, 2025 14:46
@NicksonYap
Copy link

Thanks for the PR, @asyncbridge did you successfully get the Tapo C120 with SC340AI to work with OpenIPC?

I noticed a few GitHub issues relating to it as well
OpenIPC/firmware#1654
OpenIPC/firmware#1766

@asyncbridge
Copy link
Author

Thanks for the PR, @asyncbridge did you successfully get the Tapo C120 with SC340AI to work with OpenIPC?

I noticed a few GitHub issues relating to it as well OpenIPC/firmware#1654 OpenIPC/firmware#1766

No, I haven't been successful either, so I reached out to OpenIPC tech support for assistance.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need clarification on what has changed relative to the base file.
Perhaps it would be correct to make changes to the Firmware repository

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only add sc430ai to set_sensor() (no other changes) in this file.

...

set_sensor() {
case $SENSOR in
gc4653|imx335|imx415|os04a10|sc4336p|sc401ai|sc430ai|sc501ai|sc830ai|sc850sl)
...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the purpose of using this file?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I copied this file from the previous builder’s PR for consistency. There isn’t a particular new purpose on my side, so please let me know if it should be removed.

There may be an error calling the driver package.
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.

3 participants