Skip to content

Disable the blue LED on RAK19003 and 19007#1242

Open
ivmden wants to merge 21 commits intomeshcore-dev:devfrom
ivmden:wisblock-disable-blue-led
Open

Disable the blue LED on RAK19003 and 19007#1242
ivmden wants to merge 21 commits intomeshcore-dev:devfrom
ivmden:wisblock-disable-blue-led

Conversation

@ivmden
Copy link

@ivmden ivmden commented Dec 18, 2025

Adding a flag to the build to disable the Blue LED on the RAK19007 and 19003 boards, and a call in SerialBLEInterface.cpp to turn the LED off.

Tested on a 19007 and 19003.

silverphish-io and others added 20 commits September 25, 2025 14:59
added pyMC_core to meshcore projects
mentioned Cisien's meshcoretomqtt fork from Andrew-a-g
updated Coding Rate explanation and recommendation
updated radio presets and added how to update presets listed in the app
Fix debug log: use c->extra.room.push_failures instead of c->push_failures
Change source of truth to this repo, remove whitespace
# Conflicts:
#	docs/payloads.md
add get and set prv.key
add web site to generate new private key and specific its public key's first byte value
add link to repeater observer instruction
add links to The Comms Channel's meshcore video, MCarper's Meshcore Advantages, and Austin Mesh's MeshCore vs Meshtastic comparison
add deafness instruction for agc reset interval
add reference to Liam's Windows and Intel Mac client apps
add reference to Tree's Meshcore packet decoder
add OTA BLE update addendum for Seeed Wio Tracker L1 Pro
add instruction to use T-deck's software keyboard to enter `=` at the end of the base64 public key
fix typo bugs found by @4np
Fix typo
@liamcottle
Copy link
Member

Howdy! Thanks for the contribution :)

I would probably opt to keep the LED enabled in default builds for now, as it's a very good indicator of when we have connection issues or the board locks up etc. We've had many support cases with connection issues and users have been able to tell us the LED is stuck on even when disconnected. It would be great to keep this for now.

Would you be willing to remove the -D DISABLE_BLUE_LED=1 from platformio.ini, or at least comment it out, and when you build your own firmware you can set this flag in the build script?

@liquidraver
Copy link
Contributor

also remove this to keep it clean :)
// If we want to control BLE LED ourselves, uncomment this:
// Bluefruit.autoConnLed(false);

@ivmden
Copy link
Author

ivmden commented Dec 18, 2025

I would probably opt to keep the LED enabled in default builds for now, as it's a very good indicator of when we have connection issues or the board locks up etc. We've had many support cases with connection issues and users have been able to tell us the LED is stuck on even when disconnected. It would be great to keep this for now.

(Please read this as conversational, not argumentative). You've also had multiple people asking about how to turn the LED off via discord. How can we strike a balance of keeping good debug tools and keeping energy usage reasonable/users happy? I tried getting it to turn off after the connection was well established but struggled with it.

Could a configuration: Off when connected, Flashing when trying to pair be a reasonable compromise?

also remove this to keep it clean :) // If we want to control BLE LED ourselves, uncomment this: // Bluefruit.autoConnLed(false);

Done.

@4np
Copy link

4np commented Dec 30, 2025

Can't it be a console command (and app setting) to disable the led? My solar repeater doesn't need the LED and it probably saves additional power usage if it is disabled, but by default it's probably best to have it enabled. To make it platform agnostic maybe introduce a generic property disable all unnecessary bells & whistles, then platform specific implementations can disable -say- the led or -say- the buzzer.

@ivmden
Copy link
Author

ivmden commented Dec 30, 2025

@4np These changes do not apply to anything but Bluetooth companion nodes. Your solar repeater should not have this blue LED enabled.

@liamcottle I'd love some feedback from you from my above questions. Many thanks!

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.

10 participants