Non-default motor pin configuration fails

[slavicd]

I'm trying to set up madflight with motors mapped like this:

//--- OUT Pins ---
pin_out0      2
pin_out1      3
pin_out2      5
pin_out3      4

Building succeeds but Madflight spins motors in the wrong order (last two are swapped). Printing the config reveals that my madflight_config.h is ignored:

dump out

[...]
pin_out0         2
pin_out1         3
pin_out10        -1
pin_out11        -1
pin_out12        -1
pin_out13        -1
pin_out14        -1
pin_out15        -1
pin_out2         4
pin_out3         5
pin_out4         -1
[...]

Swapping the order using motor_outputs variable in main.cpp leads to the same behaviour.

int motor_outputs[] = {0, 1, 3, 2}; //right-rear, right-front, left-rear, left-front motor

[slavicd]

I suspect this boot message has something to do with the problem:

CFG: Loading config - Skipping madflight_board and madflight_config (EEPROM is newer)

Circumvented the problem by manually setting the configs in the cli, then performing a save.

[qqqlab]

The idea is that if you make changes with CLI 'save' that these changes will survive uploading modified code. Unless the uploaded modified code also modified the madflight_board or madflight_config strings.

Thus allowing you to modify the program without loosing calibration settings.

The confing logic was extensively tested, but might be broken by recent updates. I'll leave the issue open until I or somebody else can test this.

[slavicd]

Unless the uploaded modified code also modified the madflight_board or madflight_config strings.

That is precisely what I did. There is a bug where it disregards the modifications, perhaps having to do with GPS data getting saved right on boot.