MTB_ODIN_v2_fixes

[ashok-rao]

Description

Fixes to MTB_ODIN target based on v2.0.0 of the MCB hardware:

1. Removed redundant code comments. Added relevant ones for MTB pins.
2. More SPI_CS pins added for peripherals on the MTB.
3. Disabled LSE_Clock as it is not present on the MTB in targets.json

@0xc0170 , @cmonr : Could you please let me know what test results I need to attach for this PR? I've tested a basic blinky that tests the SPI (LCD), I2C (temp sensor) and PWMs (LEDs) on the MTB & it is working fine. Thanks.

Pull request type

[ ] Fix
[ ] Refactor
[x] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

@cmonr @0xc0170 : Could you please review? Thanks.

[0xc0170]

Can you add the list from description to the commit msg ?

v2.0.0 - what for users with older revision, or only 2.0.0 out there?

[ashok-rao]

Thanks @0xc0170 .. only v2.0.0 going out.. earlier version was for internal development only..

[0xc0170]

IF you can extend the commit message to contain details that are provided here, we will start CI

[ciarmcom]

@ashok-rao, thank you for your changes.
@cmonr @0xc0170@ARMmbed/mbed-os-maintainers please review.

[0xc0170]

Improve the commit message as requested above

[cmonr]

@ashok-rao If there are any tests that run LSE_Clock, would be good to check that those are no logner run.

Otherwise, I'm not sure which tests would test the SPI pins.

[ashok-rao]

@0xc0170 : Force pushed with changes to commit message :-)
@cmonr : Some of the RTC tests used the LSE Clock. There is an internal JIRA : https://jira.arm.com/browse/IOTPART-7257 ongoing for this too.

[0xc0170]

CI started

[mbed-ci]

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 1
Build artifacts