8000 Allow option to disable and enable LED/LEDs on adapter or board · Issue #20 · zigpy/zigpy-znp · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Allow option to disable and enable LED/LEDs on adapter or board #20

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.

Sign up for GitHub

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

Jump to bottom
Closed
Hedda opened this issue May 13, 2020 · 5 comments
Closed

Allow option to disable and enable LED/LEDs on adapter or board #20

Hedda opened this issue May 13, 2020 · 5 comments
Labels
enhancement New feature or request

Comments

@Hedda
Copy link
Contributor
Hedda commented May 13, 2020 •
edited
Loading

Copy request for LED control config from zigpy-cc zigpy/zigpy-cc#42 and zigpy/zigpy#384

Feature suggestion for optional settings to disable LED on CC2531 USB adapter/dongle/stick.

Option to configure setting for disabling and enabling LED/LEDs on the Zigbee radio adapter/dongle.

Requested for CC2531 dongle by Yevgenium in zigpy/zigpy-cc#42

I understand this is dependent on zigpy/zigpy#381 as another example of an optional extension?
@Hedda Hedda mentioned this issue May 13, 2020
Closed
@puddly
Copy link
Collaborator
puddly commented May 13, 2020

I don't think this is possible without customizing Z-Stack.

@Adminiuga
Copy link
Contributor

And only would be applicable to one board 🤷 I'd rather have that dev time spent for anything with a better ROI

@puddly puddly added the wontfix This will not be worked on label May 13, 2020
@puddly
Copy link
Collaborator
puddly commented May 13, 2020

Agreed. Just open the clamshell (if your adapter has one) and stick on a little piece of electrical tape.

Also, zigpy-znp doesn't support older radios at the moment because I don't have any older radios to test with (yet).

@puddly puddly closed this as completed May 13, 2020
@Hedda
Copy link
Contributor Author
Hedda commented May 14, 2020

FYI, was possible in zigpy-cc so @sanyatuning set LED to always disabled except for when pairing, see:

zigpy/zigpy-cc@3855695

zigpy/zigpy-cc#42

But you are of course both right, this is not critical, just a nice to have and not a need to have ;)

@puddly
Copy link
Collaborator
puddly commented May 14, 2020

Hmm, I did see this command earlier but I just thought it let you toggle the LEDs momentarily. If it indeed works on the dev kits, you should be able to send a c.Util.LEDControl(Laded=0xFF, On=False) command (though fixing the typos and omitted information pulled from the docs would make the command make mor 8000 e sense).

I'll re-open this issue, since it indeed seems to be possible.

@puddly puddly reopened this May 14, 2020
@puddly puddly added enhancement New feature or request and removed wontfix This will not be worked on labels May 14, 2020
@puddly puddly changed the title [REUQEST] Allow option to disable and enable LED/LEDs on adapter or board Allow option to disable and enable LED/LEDs on adapter or board May 16, 2020
@puddly puddly closed this as completed in f840914 Jun 29, 2020
This was referenced Sep 3, 2020
Closed
Closed
@Hedda Hedda mentioned this issue Oct 14, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Adminiuga @Hedda @puddly
0