Hi,

In my testing of this branch, I've been having trouble clearing the following build error:

In file included from C:\Users\geoma\AppData\Local\Arduino15\packages\STM32\hardware\stm32\1.7.0\cores\arduino/stm32/low_power.h:44,
                 from C:\Users\geoma\AppData\Local\Arduino15\packages\STM32\hardware\stm32\1.7.0\cores\arduino/board.h:15,
                 from C:\Users\geoma\AppData\Local\Arduino15\packages\STM32\hardware\stm32\1.7.0\cores\arduino/wiring.h:41,
                 from C:\Users\geoma\AppData\Local\Arduino15\packages\STM32\hardware\stm32\1.7.0\cores\arduino/Arduino.h:32,
                 from C:\Users\geoma\AppData\Local\Temp\arduino_build_17788\sketch\main.c:41:
C:\Users\geoma\AppData\Local\Arduino15\packages\STM32\hardware\stm32\1.7.0\cores\arduino/stm32/uart.h:69:3: error: unknown type name 'UART_HandleTypeDef'
   UART_HandleTypeDef handle;
   ^~~~~~~~~~~~~~~~~~

I've tried enabling UART HAL module in my build_opt.h file:

-DHAL_COMP_MODULE_ENABLED
-DHAL_IWDG_MODULE_ENABLED
-DHAL_UART_MODULE_ENABLED
-DHAL_I2C_MODULE_DISABLED
-DHAL_SPI_MODULE_DISABLED
-DHAL_RTC_MODULE_DISABLED
-DUSE_FULL_LL_DRIVER

You have to use the definitions I've described in the first post.
HAL_UART_MODULE_ONLY

Is it ok @geosmall ?

My test application appears to compile but something seems strange. I have a TIM interrupt in a C file as part of my test project. It compiles fine even with no defines in build_opt.h. It appears to me that both are getting compiled and linked but with no errors. I need to boil it down to a small test application to confirm.

I'm not sure I am understanding the default, HAL_NNN_MODULE_ONLY and HAL_NNN_MODULE_ENABLED define behaviors. Could you provide a bit of guidance on how users are supposed to use these options in build_opt.h?

Sorry @geosmall didn't see you wait for a guidance.
The goal of HAL_NNN_MODULE_ONLY is to remove all Arduino stuff using the HAL NNN.
So for example if HAL_ADC_MODULE_ENABLED is defined and you define also HAL_ADC_MODULE_ONLY then all Ardunio API related to analog will not be available.
Same for TIM, ...
To sum up this allow to get access to all HAL NNN without any weak redefinition of a HAL API.

@geosmall
is it OK for you?
I would like merge as it is a pain to rebase it. 😭

It appears to be working as intended. I would go ahead and merge. I will continue testing. Thanks!