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

Pizigate on odroid #86

Open
CinPoU opened this issue Apr 30, 2021 · 10 comments
Open

Pizigate on odroid #86

CinPoU opened this issue Apr 30, 2021 · 10 comments

Comments

@CinPoU
Copy link

CinPoU commented Apr 30, 2021

Could you please add the odroid as a compatible hardware for the pizigate?

@Hedda
Copy link
Contributor

Hedda commented Jun 10, 2021

FYI, ZiGate compatible with different computers is not decided at the zigpy-zigate level at all. ZiGate models compatible with different computers depends on ZiGate hardware and not the zigpy-zigate project. To make this fact a little more clear suggest change readme as per #94

@CinPoU As such you might want to try contacting ZiGate and get them to also clarify compatibility on their webpage for PiZiGate:

https://zigate.fr/produit/pizigatev2/

@MattWestb
Copy link

It can being problem then the Pi is using 5 V and normal TTL is using 3.3 V but very likely the HAT cant handle both OK.

@Hedda
Copy link
Contributor

Hedda commented Jun 10, 2021

Obviously comment "Raspberry Pi compatible GPIO header" also means that it must also offer the same voltage as Raspberry Pi.

@CinPoU
Copy link
Author

CinPoU commented Jun 10, 2021

Hi!

The pizigate is compatible with odroid, i used it with raspbian and jeedom.

But the zigpy-zigate's code check the model of the motherboard, and if it's not a raspberry, there's an error.
So, could you add the odroid motherboard as a compatible device please?

Best regards

@CinPoU
Copy link
Author

CinPoU commented Jun 10, 2021

in setup.py

try: with open('/proc/cpuinfo', 'r') as cpuinfo: found = False for line in cpuinfo: if line.startswith('Hardware'): found = True label, value = line.strip().split(':', 1) value = value.strip() if value not in ( 'BCM2708', 'BCM2709', 'BCM2835', 'BCM2836' ): if raise_on_errors: raise ValueError( 'This system does not appear to be a Raspberry Pi.' ) else: return False

@Hedda
Copy link
Contributor

Hedda commented Jun 10, 2021

@doudz did you intend that zigpy-zigate should only work with Raspberry Pi?

Does zigpy-zigate really need to check CPU and hardware at all?

If so then should not either work with OpenLumi ("Open Lumi Gateway" DIY ZiGate WiFi bridge hacked from Xiaomi Lumi Gateway ZHWG11LM or DGNWG02LM with modded OpenWRT firmware for the OS and ZiGate firmware on its NXP module for Zigbee), or?

https://openlumi.github.io

https://github.com/openlumi

#59

@doudz doudz closed this as completed in #94 Jun 10, 2021
@Hedda
Copy link
Contributor

Hedda commented Jun 11, 2021

in setup.py

try: with open('/proc/cpuinfo', 'r') as cpuinfo: found = False for line in cpuinfo: if line.startswith('Hardware'): found = True label, value = line.strip().split(':', 1) value = value.strip() if value not in ( 'BCM2708', 'BCM2709', 'BCM2835', 'BCM2836' ): if raise_on_errors: raise ValueError( 'This system does not appear to be a Raspberry Pi.' ) else: return False

@doudz This should maybe not be closed as CinPoU points out that zigpy-zigate setup is currently requiring Raspberry Pi to run?

@doudz
Copy link
Collaborator

doudz commented Jun 11, 2021

The Rpi detection is used to add rpi.gpio requirement dynamically during setup.
Pizigate need it , not other zigate boards.

@doudz doudz reopened this Jun 11, 2021
@doudz
Copy link
Collaborator

doudz commented Jun 11, 2021

Is there a way to check if we are on odroid board ?

@doudz
Copy link
Collaborator

doudz commented Jun 11, 2021

Maybe on odroid we need to install Odroid.GPIO instead of Rpi.GPIO ?

puddly added a commit to puddly/zigpy-zigate that referenced this issue Mar 17, 2022
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

4 participants