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

Where does the genPollCtrl binding come from ? #154

Open
bernard-dandrea opened this issue Jan 26, 2025 · 6 comments
Open

Where does the genPollCtrl binding come from ? #154

bernard-dandrea opened this issue Jan 26, 2025 · 6 comments

Comments

@bernard-dandrea
Copy link

Image

I have looked in zigbeetlc.js and cannot find bindings only for 'msTemperatureMeasurement', 'msRelativeHumidity', 'genPowerCfg'

Image

with the devbis firmware, there is no binding on genPollCrl

I have the feeling that the TABLE FULL error when you want to update the existing reporting is directly linked to the binding of genPollCtrl

when you manage to remove that binding (or with devbis version), you have no issue to update the reporting or add ones

thanks for your answer

@pvvx
Copy link
Owner

pvvx commented Jan 26, 2025

Judging by the name "FULL TABLE" your software makes the device create more than 8 bindings (reports). I don't understand what the point of this is if only four are used.

The only point of this is to drain the device's battery faster by requiring additional transmissions.

@pvvx
Copy link
Owner

pvvx commented Jan 26, 2025

binding on genPollCrl

Why does your program "binding on genPollCrl" at all?

It is enough to receive temperature, humidity and battery data. That is, only 3 bindings, and the device has a buffer of 8.


Poll_Control is a separate story.
When they fix ZHA's behavior with this cluster according to the Zigbee 3.0 standard, then we'll talk.
Or maybe we won't talk, since it usually takes more than a decade to fix bugs in Z2M or ZHA...
I am not going to create special workarounds in the firmware to work with bugs in Z2M or ZHA that are not in accordance with the Zigbee 3.0 standard.

I already had to remove (patch) in the firmware from the constant forced installation of a long polling interval of 6 seconds, which requires ZHA for all devices.

@bernard-dandrea
Copy link
Author

Many thanks for your fast answer

Just want to clarify that I don't want to add any reporting but modify the existing one. After a clean installation of the device with telink utility and integration in Zigbee2mqtt, it is not possible to update the reporting of temperature due the TABLE FILL error.

I point out that disabling genPollCrl in binding make the updates in reporting work so I suspect that corresponding entries full the binding table

So, I was wondering what has generated the genPollCrl entries in reporting as I cannot find anything in z2m configuration file zigbeetlc.js (I notice that there was a binding in [zigbeetlc_v0121.js](https://github.com/pvvx/ZigbeeTLc/blob/master/z2m/zigbeetlc_v0121.js but I guess that file is not be used anymore).

I don't understand your explanations concerning Poll_Control story as I am not aware of the initial problem with ZHA / Z2M and what needed to be done to cope with it

@pvvx
Copy link
Owner

pvvx commented Jan 26, 2025

I don't understand your explanations concerning Poll_Control story

#128 (comment)

@robvanoostenrijk
Copy link

@bernard-dandrea Remove any custom extensions you have installed for ZigbeeTLc, Zigbee2MQTT supports these devices natively. The custom extensions were never updated, only the native Zigbee2MQTT support was.

After restarting Zigbee2MQTT without the custom extension, remove/unpair your ZigbeeTLc device and repair it.
This should resolve the TABLE_FULL errors you are seeing.

@pvvx Maybe remove the custom extension code from the repository, as support there in Z2M out of the box.

@pvvx
Copy link
Owner

pvvx commented Jan 28, 2025

@pvvx Maybe remove the custom extension code from the repository, as support there in Z2M out of the box.

ok

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

No branches or pull requests

3 participants