zigpy-espzb icon indicating copy to clipboard operation
zigpy-espzb copied to clipboard

ZGP (Zigbee Green Power) Green Power Sink (GPS) support in zigpy-espzb radio library for ESP32-H2 and ESP32-C6?

Open Hedda opened this issue 1 year ago • 3 comments

Not tested myself but wondering if this zigpy-espzb radio library supports Green Power Sink (GPS) and/or forwarding to ZGP controller support with the draft ZGP (Zigbee Green Power) patches for zigpy that are still a work in progress?

https://github.com/zigpy/zigpy/pull/1282

https://github.com/konistehrad/zigpy/tree/zgp

https://github.com/zigpy/zigpy/issues/341

FYI, some of the other reference radio libraries for zigpy specifically had to be patched for proper GPDF (Green Power Data Frames) handling and/or fixed for ZGP profile and endpoint handling, see example:

https://github.com/zigpy/zigpy-deconz/pull/235

https://github.com/zigpy/zigpy-znp/pull/235

https://github.com/zigpy/bellows/pull/594

PS: At least esp-zigbee-sdk contains Zigbee GreenPower examples so it looks like ZGP should be fully supported upstream there?

https://github.com/espressif/esp-zigbee-sdk/tree/main/examples/esp_zigbee_greenpower/esp_zigbee_gpd

https://github.com/espressif/esp-zigbee-sdk/tree/main/examples/esp_zigbee_greenpower/esp_zigbee_gpc

Hedda avatar Mar 19 '24 10:03 Hedda

@Hedda It doesn't support Green Power Now.

lhespress avatar Mar 20 '24 01:03 lhespress

@Hedda It doesn't support Green Power Now.

zigpy as downstream does not yet have initial ZGP device support until this PR is merged -> https://github.com/zigpy/zigpy/pull/1282

But Espressif's esp-zigbee-sdk does support ZGP upstream or? https://github.com/espressif/esp-zigbee-sdk

Again, there are at least examples that seem to indicate that is the case? See:

https://github.com/espressif/esp-zigbee-sdk/tree/main/examples/esp_zigbee_greenpower/esp_zigbee_gpd

and

https://github.com/espressif/esp-zigbee-sdk/tree/main/examples/esp_zigbee_greenpower/esp_zigbee_gpc

Hedda avatar Mar 22 '24 07:03 Hedda

@Hedda Yes, the SDK has supported it, i mean the NCP firmware doesn't support.

lhespress avatar Mar 22 '24 07:03 lhespress