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

Add support for incomingNetworkStatusHandler #640

Closed
wants to merge 3 commits into from

Conversation

fuzzy01
Copy link

@fuzzy01 fuzzy01 commented Jan 4, 2023

Fix for Koenkk/zigbee2mqtt#15724
Based on latest UG100 reference guide 0x00C4 has been redefined.

@Koenkk
Copy link
Owner

Koenkk commented Jan 4, 2023

@kirovilya can you review this?

@kirovilya
Copy link
Contributor

kirovilya commented Jan 6, 2023

image
image

I see that the changes appeared in the new version of the stack 7.2.0
so it's strange to see that these changes affect the bug Koenkk/zigbee2mqtt#15724 where stack 6.10.3 was used

And another question will be how to combine work with old stacks, where there was another frame under the code 0xC4? It is necessary to make some bypass to support both stacks.

image
image

@MattWestb
Copy link

In GSDK 3.2 for EZSP 6.10.X is not allocated 0x00C4 --unassigned-- so very likely its taken away between EZSP 6.0.X and 6.9.x and very likely in 6.7.X that was getting the V8 framing.
Perhaps it was implanted but not documented. Interesting if 6.7.X (= stable version of EZSP) is using the new or old command for the frame or not using it at all.
`

@github-actions
Copy link
Contributor

This pull request is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants