-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Adding the level control to light switch and changing to dimmer switch #37448
base: master
Are you sure you want to change the base?
Adding the level control to light switch and changing to dimmer switch #37448
Conversation
PR #37448: Size comparison from 2a77a11 to 284339d Full report (69 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #37448: Size comparison from 2a77a11 to c8a849b Full report (72 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
@@ -4402,31 +4402,31 @@ | |||
}, | |||
{ | |||
"id": 2, | |||
"name": "MA-onofflightswitch", | |||
"name": "MA-dimmerswitch", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just raising attention that this is a common zap file for a few other platform light-switch app. They might be ok with this addition.
If not we can create zap file specific for silab's light-switch app.
c8a849b
to
c0a9ea1
Compare
PR #37448: Size comparison from a373044 to 42e2756 Full report (73 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #37448: Size comparison from b205837 to 873f9d4 Full report (74 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
@chirag-silabs Doesn't button 0 work with these changes? |
Description
Moving the light-switch-app type from onofflightswitch to the dimmerswitch to add the level control client functionality which can be controlled by the light switch app directly.
The button implementation in dimmer switch app by adding level control actions on button sequences. Below are the implementation details :
Default Step direction - Up
Button1 short press - light toggle
Button1 long press - Level up/down as per direction
Button1 shortpress + Button0 shortpress - reverse the direction (down to up or up to down)
Button0 longpress - Factory reset
Testing
Tested on the silabs platform, with the help of shell sending commands of level control to other device
Tested the button sequences on SOC manually for the btn checks