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

Binding table size and groups (CON-1396) #1134

Open
jonsmirl opened this issue Oct 31, 2024 · 1 comment
Open

Binding table size and groups (CON-1396) #1134

jonsmirl opened this issue Oct 31, 2024 · 1 comment

Comments

@jonsmirl
Copy link
Contributor

What is the difference between these two config variables?
CONFIG_MAX_BINDINGS=8
CONFIG_ESP_MATTER_BINDING_TABLE_SIZE=100

I am running into a constraint somewhere trying to increase the number of bindings we support.

Are groups constrained somehow?
We have two GroupKeys
But then up to 25 GroupIds using those two keys.

@github-actions github-actions bot changed the title Binding table size and groups Binding table size and groups (CON-1396) Oct 31, 2024
@shubhamdp
Copy link
Contributor

CONFIG_MAX_BINDINGS is unused, I think this can be removed.

CONFIG_ESP_MATTER_BINDING_TABLE_SIZE maps to MATTER_BINDING_TABLE_SIZE and used for bindings size. (https://github.com/project-chip/connectedhomeip/blob/master/src/app/util/binding-table.h#L113)

For increasing number of bindings you should increase CONFIG_ESP_MATTER_BINDING_TABLE_SIZE.

Are groups constrained somehow?

Yes, they are. Minimum supported number of groups per fabric should be (4 * no-of-endpoints-implementing-groups-cluster). I have attached the screenshot as well. This gets automatically picked up after 974997d

Screenshot 2024-11-13 at 15 39 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

No branches or pull requests

2 participants