ArduPilot / ardupilot

ArduPlane, ArduCopter, ArduRover, ArduSub source
http://ardupilot.org/
GNU General Public License v3.0
10.73k stars 17.18k forks source link

AP_Periph: No Neopixel Output From AP_Periph f405-MatekGPS #27597

Open davidbitton opened 1 month ago

davidbitton commented 1 month ago

Bug report

Issue details

A Matek M9N-F4-5883 was not driving an 8-neopixel strip attached to the LED port; as an AP_Periph. If I flash Rover for MATEK-CAN, the LED strip works as expected. I can also get the neopixels to work if I use the WITHOUT_NOTIFY defines. The resolution that I found is to add define AP_SERIALLED_ENABLED 1 to all AP_Periph devices that have HAL_PERIPH_ENABLE_NOTIFY defined. If serial LED is not explicitly defined, AP_Periph chooses RCOutputRGBLed over SerialLed.

Version This is in Release as well as master

Platform [ ] All [ ] AntennaTracker [ ] Copter [ ] Plane [ ] Rover [ ] Submarine [X] AP_Periph

Airframe type Rover

Hardware type M9N-F4-5883

Logs

peterbarker commented 1 month ago

@davidbitton isn't the default behaviour more likely to be correct - i.e. most periph boards won't be having serial LEDs attached to them.

Periph boards in particular don't have vast amounts of resources to throw features in.... arguments could definitely be made that the board you nominate can both afford the resources and be more suited than other boards to use the LEDs.

... so I'm not sure what you want to have happen here.