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

LEDs for 52840 dongle not working #1

Closed
mame82 opened this issue Jul 15, 2019 · 16 comments
Closed

LEDs for 52840 dongle not working #1

mame82 opened this issue Jul 15, 2019 · 16 comments

Comments

@mame82
Copy link

mame82 commented Jul 15, 2019

The LED assignment for 52840 dongle isn't working at all.

Here's an excerpt of my board definitions, with respective comments on LEDs.

RED isn't working at all. Blue and green show weird behavior.

#ifndef APRBROTHER_DONGLE_H
#define APRBROTHER_DONGLE_H

#ifdef __cplusplus
extern "C" {
#endif

#include "nrf_gpio.h"

// LED definitions for APRBROTHER dongle
// Each LED color is considered a separate LED
#define LEDS_NUMBER    4

#define LED1_G         NRF_GPIO_PIN_MAP(0,6)
#define LED2_R         NRF_GPIO_PIN_MAP(0,8)
#define LED2_G         NRF_GPIO_PIN_MAP(1,9)
#define LED2_B         NRF_GPIO_PIN_MAP(0,12)

#define LED_1          LED1_G
#define LED_2          LED2_R
#define LED_3          LED2_G
#define LED_4          LED2_B

#define LEDS_ACTIVE_STATE 0

#define LEDS_LIST { LED_1, LED_2, LED_3, LED_4 }

#define LEDS_INV_MASK  LEDS_MASK

// Totally messed, no RED at all
#define BSP_LED_0      LED_1 // blue, bright
#define BSP_LED_1      LED_2 // blue, barely noticeable
#define BSP_LED_2      LED_3 // green, only if LED_2 is on, too
#define BSP_LED_3      LED_4 // seems to toggle LED_2 somehow, without changing LED_4

// There is only one button for the application
#define BUTTONS_NUMBER 1

#define BUTTON_1       NRF_GPIO_PIN_MAP(1,6)
#define BUTTON_PULL    NRF_GPIO_PIN_PULLUP

#define BUTTONS_ACTIVE_STATE 0

#define BUTTONS_LIST { BUTTON_1 }

#define BSP_BUTTON_0   BUTTON_1

#define BSP_SELF_PINRESET_PIN NRF_GPIO_PIN_MAP(0,19)

#define HWFC           true

#ifdef __cplusplus
}
#endif

#endif // APRBROTHER_DONGLE_H
@mame82
Copy link
Author

mame82 commented Jul 15, 2019

@nieldk
Copy link

nieldk commented Jul 15, 2019

There is an interresting observation here. adafruit/circuitpython#1174 (comment)

@mame82
Copy link
Author

mame82 commented Jul 15, 2019

Behavior differs ... especially on BLUE LED ... not sure on undervoltage

#define BSP_LED_0      LED_1 // blue, bright
#define BSP_LED_1      LED_2 // blue, barely noticeable
#define BSP_LED_2      LED_3 // green, only if LED_2 is on, too
#define BSP_LED_3      LED_4 // seems to toggle LED_2 somehow, without changing LED_4

@volca
Copy link
Contributor

volca commented Jul 16, 2019

@mame82 When did you buy the USB dongle? We'll try to re-produce the issue with your code first

@volca
Copy link
Contributor

volca commented Jul 16, 2019

I just tested one dongle with a test firmware. Please check the video.

Here's the firmware file
nrf52840.uf2.zip

Some early dongles are soldered with wrong RGB led so it has the issue.

@nieldk
Copy link

nieldk commented Jul 16, 2019

I just tested one dongle with a test firmware. Please check the video.

Here's the firmware file
nrf52840.uf2.zip

Some early dongles are soldered with wrong RGB led so it has the issue.

Thanks @volca , but your video only show green blinking LED, and the blue LED. @mame82 mentioned that the issue is getting Red LED, and Green LED is only working together with Blue.

@volca
Copy link
Contributor

volca commented Jul 16, 2019

The video shows blinking green --> blue -> red. Please check it again?

@mame82
Copy link
Author

mame82 commented Jul 16, 2019

I just tested one dongle with a test firmware. Please check the video.
Here's the firmware file
nrf52840.uf2.zip
Some early dongles are soldered with wrong RGB led so it has the issue.

Thanks @volca , but your video only show green blinking LED, and the blue LED. @mame82 mentioned that the issue is getting Red LED, and Green LED is only working together with Blue.

No, all 4 LEDs are shown (single color blue and RGB).

Unfortunately it still doesn't work out with my dongle, here's a video from the LED test firmware provided above video.

The dongles was have been ordered on May 16th (order #2169). Only 1 out of 3 arrived, still waiting for the others ... maybe it is a malfunction of the one which arrived.

Also, is there a way to disassemble the dongle, without destroying it. I noticed that you removed the upper side from the housing. On my version, the upper part of the housing is holding the USB-A plug together. Removing it would render the plug unusable.

@volca
Copy link
Contributor

volca commented Jul 16, 2019

The dongles was have been ordered on May 16th (order #2169). Only 1 out of 3 arrived, still waiting for the others

I can confirm the dongle was soldered wrong RGB LED. We've replaced all RGB LED for other dongles. Two dongles with new RGB LEDs were shipped out yesterday.

Also, is there a way to disassemble the dongle, without destroying it.

Unfortunately you must destroy the casing to disassemble the dongle (I also destroyed one casing today). But we can send some more new casings to you if you place another order for the dongle.

@nieldk
Copy link

nieldk commented Jul 16, 2019

OK, my bad, sorry guys. I also ordered new dongles (the ones I had are bricked), will be looking forward to test once they arrive. @volca it seems my order 2310 is just about to ship, perhaps you can manage to add some housing to that ?

@mame82
Copy link
Author

mame82 commented Jul 16, 2019

I can confirm the dongle was soldered wrong RGB LED. We've replaced all RGB LED for other dongles. Two dongles with new RGB LEDs were shipped out yesterday.

So this means this issue does not exist, anymore! Nice.

Btw. the external antenna performs really well, compared to the Nordic version with PCB antenna.

Looking forward to test the new dongles ... thanks for the support!

@volca
Copy link
Contributor

volca commented Jul 16, 2019

@nieldk My colleague are making package for order #2310. She will add extra 3 empty housings and one more usb dongle to the package.

@nieldk
Copy link

nieldk commented Jul 16, 2019

@nieldk My colleague are making package for order #2310. She will add extra 3 empty housings and one more usb dongle to the package.

Marvelous, thanks guys!
I will forward some to you @mame82 if you would like ?

@volca
Copy link
Contributor

volca commented Jul 16, 2019

Thanks @mame82 and @nieldk for get the dongle compatible with the great software LOGITacker!

@mame82
Copy link
Author

mame82 commented Jul 16, 2019

👍🍻

@volca
Copy link
Contributor

volca commented Jul 26, 2019

closed. Reopen if need

@volca volca closed this as completed Jul 26, 2019
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

3 participants