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 Espressif System Zigbee radio library support #42

Closed
wants to merge 1 commit into from

Conversation

lhespress
Copy link

Add Espressif System Zigbee radio library support which use Espressif Zigbee Network Core-Processor Protocol (EZNP).

@lhespress
Copy link
Author

@puddly PTAL

@puddly
Copy link
Contributor

puddly commented Dec 29, 2023

Could you point me to where the zigpy-eznp package resides? It is not available on PyPI (https://pypi.org/project/zigpy-eznp/) and I cannot find it on GitHub.

The name of this package is very close to zigpy-znp (in fact, Google autocorrects it). Perhaps zigpy-esp would be a more distinct name?

@Hedda
Copy link

Hedda commented Dec 31, 2023

@chshu I guess this is about the zigpy request in zigpy/zigpy#1052 as well related to your posts in espressif/esp-zigbee-sdk#124 (comment) and -> espressif/esp-zigbee-sdk#124 (comment) about ESP Zigbee NCP host support espressif/esp-zigbee-sdk#124 ?

We are working on a ESP Zigbee NCP solution, which will be based on the ESP Zigbee APIs, and provide the serial interface to communicate with host.

We will also prepare a corresponding zigpy python library to work with Home Assistant. The draft version will be ready in 2024 January.

The ESP Zigbee NCP will not be compatible with ZBOSS NCP. With ZBOSS NCP, users need to run ZCL and part of ZDO on host. We want to integrate ZCL and ZDO in the ESP Zigbee NCP as well, so host can handle the application logic only. If some hosts already have ZCL implementation, it's also flexible to access the NCP network layer directly.

@Hedda

This comment was marked as off-topic.

@lhespress
Copy link
Author

Could you point me to where the zigpy-eznp package resides? It is not available on PyPI (https://pypi.org/project/zigpy-eznp/) and I cannot find it on GitHub.

The name of this package is very close to zigpy-znp (in fact, Google autocorrects it). Perhaps zigpy-esp would be a more distinct name?

We also have some changes in review so haven't been released. BTW, We hope release it with zigpy on github, Could you help create a project for it?

The PR just want to add it to the zigpy-cli for test.

@lhespress
Copy link
Author

@lhespress
Copy link
Author

@chshu I guess this is about the zigpy request in zigpy/zigpy#1052 as well related to your posts in espressif/esp-zigbee-sdk#124 (comment) and -> espressif/esp-zigbee-sdk#124 (comment) about ESP Zigbee NCP host support espressif/esp-zigbee-sdk#124 ?

We are working on a ESP Zigbee NCP solution, which will be based on the ESP Zigbee APIs, and provide the serial interface to communicate with host.

We will also prepare a corresponding zigpy python library to work with Home Assistant. The draft version will be ready in 2024 January.

The ESP Zigbee NCP will not be compatible with ZBOSS NCP. With ZBOSS NCP, users need to run ZCL and part of ZDO on host. We want to integrate ZCL and ZDO in the ESP Zigbee NCP as well, so host can handle the application logic only. If some hosts already have ZCL implementation, it's also flexible to access the NCP network layer directly.

@Hedda Right

@puddly
Copy link
Contributor

puddly commented Jan 3, 2024

We can talk about naming later, it isn't important.

BTW, We hope release it with zigpy on github, Could you help create a project for it?

Do you mean under the zigpy organization? I can do that. You can also create the project within the Espressif organization or a personal account and then transfer it.

The PR just want to add it to the zigpy-cli for test.

zigpy-cli requires all of its dependencies be installable. Otherwise, zigpy-cli can't be installed itself. Let me know once you've published the radio library to PyPI and I can test it out or help get it running.

@Hedda

This comment was marked as off-topic.

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.

3 participants