Improve handling of drivers that use mutexes

[josesimoes]

Description

• BusRelease are now called after the event code block and not on the event handler.
• The drivers HAL init are called at nanoHAL_Initialize() to prevent drivers being locked on a nanoCLR restart.

Motivation and Context

• Attempt to fix device locking when drivers that use mutexes are involved. There is the potential to lock the device on nanoCLR restart.

How Has This Been Tested?

Screenshots

Types of changes

• Improvement (non-breaking change that improves a feature, code or algorithm)
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

Signed-off-by: José Simões [email protected]

[nfbot]

Hi @josesimoes,

I'm nanoFramework bot.
Thank you for your contribution!

A human will be reviewing it shortly. 😉

[gligorov]

I see a small typo/bug on the line 95, targets/CMSIS-OS/ChibiOS/nanoCLR/targetHAL.cpp

#if STM32_I2C_USE_I2C2

i2cReleaseBus(&I2CD1);

Should be, i2cReleaseBus(&I2CD2);

[josesimoes]

@gligorov 😯 wow! Thanks for the code review and catching that. 👍