fmu_v5 NSH some commands force an instant reboot

[ryanjAA]

Certain NSH commands via QGC on Master and Beta send it to an instant reboot.

For instance ver all works fine, tests all, etc but any of the airspeed commands send it to a reboot. Plugged in directly via usb only and only one airspeed (sdp3x) at a time plugged in (to i2c a). Even with no airspeed plugged in it does the same thing, reboots.

[dagar]

@DanielePettenuzzo any ideas? All currently supported differential pressure sensors (airspeed) are i2c.

[DanielePettenuzzo]

@ryanjAA how do you start the driver?
I start it like this:
sdp3x_airspeed start -b [bus-number]
for i2c-a the bus number should be 4.

@dagar some drivers don't have the argc check so if you just write the driver's name with no arguments there is a reboot.

[DanielePettenuzzo]

I got ms4525 working on i2c-a on friday.

[ryanjAA]

@DanielePettenuzzo That does it. Its the argc check. Using sdp3x_airspeed start -b works. Using sdp3x_airspeed start -b works, sdp3x_airspeed help works but just sdp3x_airspeed reboots it.

[ryanjAA]

I guess that means a separate issue. That command doesnt reboot it but it also wont let the sdp3x start.

[DanielePettenuzzo]

@ryanjAA if you connect the sensor to i2c-a and start the driver what message do you get on the console?
sdp3x_airspeed start -b 4

[ryanjAA]

Two firmware flashs and all set. Maybe something lingering from whatever firmware was sent over on it? I was playing with some can units but it’s working now. If it happens again I won’t reflash. I’ll keeep it and figure out why if it sticks but so far have several airspeed units working.

[ryanjAA]

So I guess to close this then it’s the argc checks. Maybe we should get that in the docs or add them to all commands.

[DanielePettenuzzo]

@ryanjAA for the px4 firmware are you using latest master?

[ryanjAA]

@DanielePettenuzzo Yes. Latest master.

[LorenzMeier]

@bkueng Sounds like argc handling needs an audit.

[dagar]

@bkueng Sounds like argc handling needs an audit.

The driver boilerplate is all copypasta and would greatly benefit from ModuleBase or similar.

EDIT: one reason we haven't already started using ModuleBase for drivers is that we need to be able to start multiple instances. #9143

[bkueng]

@ryanjAA can you write down the affected commands so that we can fix them up quickly?
In the long run we'll need a ModuleBase for multi instances, but I have not come up with a good solution yet.

[ryanjAA]

@bkueng I am out of the office for at least another week (no Pix 4 with me either) but so far the airspeed tests with no arguments were doing it. The ver command did not but it probably just needs someone to go through each one with out an argument to see what happens if no other way to check. I’m happy to do so but it won’t be possible for a week or so.

[bkueng]

@ryanjAA I went through all commands and fixed the ones affected in #9589.

[garfieldG]

Hi,

seems like there are some untouched places where the issue isn't handled, such as https://github.com/PX4/Firmware/blob/27e75920bd560b7d8de295db45aba8cf036cd98d/src/drivers/imu/l3gd20/l3gd20.cpp#L1358

[bkueng]

@garfieldG right, this driver is not enabled on Pixhawk 4. Fixed in #9647