Tidy compass probing defines

[peterbarker]

Stops using "ifdef" defines.

Removes the somewhat odd #ifdef HAL_COMPASS_DISABLE_IST8310_INTERNAL_PROBE.

Removes double-negatives.

Output changes:

pbarker@threads:~/rc/ardupilot$ ./Tools/scripts/filter_size_compare_branches_csv.py --hide /tmp/some.csv 
------------------------  -----  ------  ----  -----  -----  ---
Board                     blimp  copter  heli  plane  rover  sub
SITL_arm_linux_gnueabihf  0      0       80    48     16     -64
SITL_x86_64_linux_gnu     0      0       0     0      0      0
aero                      -48    0       -32   48     48     0
bebop                            96
dark                      32     -144    0     0      -48    -32
disco                                          0
linux                     0      0       0     0      0      0
vnav                      0      0       0     -48    0      48
------------------------  -----  ------  ----  -----  -----  ---

The Linux boards aren't losing any symbols, the changes seem to centre around paths getting embedded in compiler products.

[tpwrules]

This inexplicably has #31424 mixed in but otherwise looks okay with a brief check (this statement does not constitute an approving review).

Poor Bob :(

[peterbarker]

This inexplicably has #31424 mixed in but otherwise looks okay with a brief check (this statement does not constitute an approving review).

Those patches are gone; my build-everything routine showed up the pocket2 problem, and I finally broke and fixed the problem :-)

Poor Bob :(

He had it coming.

[tpwrules]

LGTM, confirmed some no compiler output changes.