Add LED color remapping definitions to Odin target

[kegilbert]

Odin target was missing LED_RED/GREEN/BLUE definitions that other client targets used. Adding these definitions removes the need to #ifdef around Odin targets in client source when referencing LEDs.

Majority of whitespace changes were to line up the assignments. Lines 157-159 are the import changes with the addition of:

LED_RED     = LED1,
LED_GREEN   = LED2,
LED_BLUE    = LED3,

CC @mray19027

[JanneKiiskila]

Is there some weird pin overflow here, since LED2= PB_6 and also SW1 = PB_6. I.e. does it mean SW1 button aka BUTTON2 will not work if the LED works?

[kegilbert]

That is a bit strange...it seems to match the Ublox documentation though.

This test program also built and ran as expected:

#include "mbed.h"

DigitalOut led2(LED2);
DigitalIn  sw1(SW1, PullUp);

int main() {
    while (true) {
        led2 = !sw1;
        printf("%d\r\n", sw1.read());
    }
}

[JanneKiiskila]

Yep, I think the LEDs overrun the button.

[0xc0170]

Majority of whitespace changes were to line up the assignments. Lines 157-159 are the import changes with the addition of:

Always split whitespace changes to a new commit, it produces lot of noise , distraction from the real changes. Can you please?

[0xc0170]

cc @andreaslarssonublox @andreaspeterssonublox

[andreaslarssonublox]

LGTM

[0xc0170]

@kegilbert Let us know once you rebase, thanks

[kegilbert]

@0xc0170 Sorry for the wait, rebased it with just the remapping.

[0xc0170]

/morph test

[mbed-bot]

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1205

All builds and test passed!