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

fix: IKEA SYMFONISK gen2 does not need to divide the batteryPercentageRemaining #5844

Merged
merged 1 commit into from Jun 5, 2023

Conversation

sjorge
Copy link
Sponsor Contributor

@sjorge sjorge commented Jun 5, 2023

We have a ikea.fz.battery that tries to be smart based on the fwVersion if we need to divide or not, IKEA fixed there implementation to be in spec a while ago.

The SYMFONISK gen2 remote never had the problem and our fwVersion check checkes for a version that is high enough, but this remote has an good firmware even at major 1.

We can simply switch to the regular fz.battery to get the correct result here.

Fixes Koenkk/zigbee2mqtt#17616

…eRemaining

We have a `ikea.fz.battery` that tries to be smart based on the fwVersion if we need to divide or not, IKEA fixed there implementation to be in spec a while ago.

The SYMFONISK gen2 remote never had the problem and our fwVersion check checkes for a version that is high enough, but this remote has an good firmware even at major 1.

We can simply switch to the regular `fz.battery` to get the correct result here.

Fixes Koenkk/zigbee2mqtt#17616
@Koenkk Koenkk merged commit 582f76f into Koenkk:master Jun 5, 2023
1 check passed
@Koenkk
Copy link
Owner

Koenkk commented Jun 5, 2023

Thanks!

@sjorge sjorge deleted the patch-17 branch June 5, 2023 08:39
@MattWestb
Copy link

@sjorge The first shipped version was needing the doubling but Dirigera was updating it one week after it was released and TF is do it also now. I think it good not supporting the first shipped firmware version and the user shall updating the firmware then this is the first device with the new firmware structure and have one old Zigbee stack that is updated to next latest certificated one.

Rodret series is in the pipe and is the new version of OnOff / Dimmer, OpenClose and Shortcut remotes and they shall having the same firmware status then its using the same program structure as Symfonisk 2 (Thes is ZCL R9 = Matter compatible).

@sjorge
Copy link
Sponsor Contributor Author

sjorge commented Jun 5, 2023

OK so for those we will also need to use the regular fz.battery, or can we somehow detect the newer stack based on zclVersion/appVersion? If so, that might be worth adding to the ikea.fz.battery as a different conditional branch.

@sjorge
Copy link
Sponsor Contributor Author

sjorge commented Jun 5, 2023

OK so for those we will also need to use the regular fz.battery, or can we somehow detect the newer stack based on zclVersion/appVersion? If so, that might be worth adding to the ikea.fz.battery as a different conditional branch.

Edit: having a new rev come out for the open/close, on/off makes sense. I noticed one of my new fyrtur blinds came with an open/close with super old firmware that still supports groups, yet it was package with the new style box, slightly different battery casing and better mounting brackets, so they might be flushing all older stock.

@MattWestb
Copy link

I have only seen the 1.0.012 as shipped firmware and was needing battery doubling in ZHA and Z2M all newer is fixed so only need filtering on that and all shall working OK if IKEA is not having more version shipped with there new devices.

All IKEA controllers can being very old its depends witch way there was shipped and stored and the lock downs in china was braking the supply change completely but it shall start working more normal now.

RODRET serie is looking like thits https://www.ifun.de/ikea-rodret-neuer-mehrfach-dimmer-im-anmarsch-209671/ and only different is data settings in user data and the laser graved buttons if reading the FCC papers https://device.report/ikea-of-sweden-ab/e2201.

@sjorge
Copy link
Sponsor Contributor Author

sjorge commented Jun 5, 2023

ooh the single AAA battery is a pretty good improvement, I hate the silly coincells, not sure it's worth replacing my 8 remotes though, unless they also have group binding, but that seems unlikely, as I think ikea stopped doing it to avoid issues with broadcasts in bigger networks.

@MattWestb
Copy link

The problem is normally you cant baying the OpenClose remotes then its only shipped with the blinds but shall being possible ordering as spare parts.

Itss possible converting the old OnOff dimmer to one OpenClose remote by writing the user data from one OpenClose then they is using the same firmware in the main flash (i have one running for test OTA files) but the new devices is 100% debug locked so cant reading and writing the flash but we have hacking the Somfonisk 2 and cave cloning it with one old bootloader plus one new OTA file and its working great.

Group binding is 100% not implanted in new devices :-(((
If you like you can flashing your old OpenClose (or OnOff) with SWD.probe and the last working group binding file that many ZHA and Z2M user have doing and its working.

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.

IKEA E2123 (SYMFONISK sound remote gen2) reports incorrect battery level
3 participants