Problem description at: http://discuss.px4.io/t/multicopter-lock-if-primary-gyro-fails/9685

- Accel: use cutoff of 62.5 Hz instead of 500 Hz
- Gyro: the cutoff frequency is coupled with the ODR and is fixed to 116 Hz
  at 1 kHz readout rate. So this patch does not change anything for the
  gyro.

Move variable initializations to header file that were no longer in the correct order in the constructor list.

Deprecate #defines that are not needed, format whitespace and minor (incomplete) alphebetization of a few lists.

There is only one occurence of the namespace specified,
so it's more readable to simply specify the full namespace in this case.

Because BATT_SMBUS is using a different structure (the ModuleBase class), and `BATT_SMBUS::task_spawn` is only called for the `start` command. This is the reason why you don't need to check for it in here.

Co-Authored-By: BazookaJoe1900 <BazookaJoe1900@gmail.com>

Co-Authored-By: BazookaJoe1900 <BazookaJoe1900@gmail.com>

Co-Authored-By: BazookaJoe1900 <BazookaJoe1900@gmail.com>

Co-Authored-By: BazookaJoe1900 <BazookaJoe1900@gmail.com>

Co-Authored-By: BazookaJoe1900 <BazookaJoe1900@gmail.com>

using only <px4_getopt.h> as include, cleanup related includes
added check that all the option are valid (myoptind >= argc). if there are invalid options on some script that might now lead not to run commands

the mode can be used for VIO algorithms and obstacle avoidance to close the loop

Add perf_free() calls orb_unsubscribe and remove delete _instance in the Simulator class destructor.

Move remaining variable initialization from constructor list and alphabetize/organize methods and vars ordering.

 - closes #10133

Rename tune_control_s strength to volume to match Nuttx and Linux standard nomenclature for audio (loudness) control.

e.g. 
Preflight Fail: Height estimate error
Preflight Fail: Velocity estimate error
Preflight Fail: Position estimate error
Preflight Fail: Yaw estimate error

My recent PR exceeded the 50 char limit (apparently the last character must be null?).

This new attempt changes: "Preflight Fail: Horizontal position estimate Error"  to: "Preflight Fail: Position estimate Error"

In fact, this newest wording is more easily understood by the average user and is actually more consistent with the other COM_ARM_EKF_xxx related errors:

COM_ARM_EKF_HGT --> "Preflight Fail: Height estimate Error"
COM_ARM_EKF_VEL --> "Preflight Fail: Velocity estimate Error"
COM_ARM_EKF_POS --> "Preflight Fail: Position estimate Error"

Includes https://github.com/PX4/sitl_gazebo/pull/283

This updates the jMAVSim submodule which includes a fix for HITL.
In order to fix HITL, a CLI argument `-lockstep` was required to enable
lockstep. This has now been added to the command in jmavsim_run.sh.