labapart / polymcu

An open framework for micro-controller software
http://labapart.com/products/polymcu
202 stars 41 forks source link

Support for STM32 F767ZI Nucleo #3

Closed sergiuszm closed 7 years ago

sergiuszm commented 7 years ago

I have tried to add support for STM32 F767ZI Nucleo but I can't get it working.

Examples/Baremetal compiles without any error but I can't get serial output. I have checked with L467RG and it works perfectly but not for F767ZI.

@oliviermartin Could you help me figure out if I am not missing something?

oliviermartin commented 7 years ago

Hi @sergiuszm,

have you tried to investigate the issue with the embedded HW debugger? Are you sure the code is running well? Can you see the LED turns on with the application Examples/Barematal? If you cannot then it means the issue is before. On board without HW debugger, I actually make sure the LED works fine and I try to investigate how far the code is running.

Some comments about the patches as I would happy to merge them in polymcu at some point soon :-) If you can squash your both commit together or even maybe remove your changes to /CMakeLists.txt to avoid to mix it with your STM32L767ZI-Nucleo port.

Could you avoid to duplicate Device/ST/Driver/VCom/Driver_USART.c and use for instance in Driver_USART.c:

#ifdef STM32L476xx
  #include "stm32l4xx_hal.h"
  #include "stm32l4xx_hal_conf.h"
#elif STM32F767xx
  #include "stm32f7xx_hal_conf.h"
  #include "stm32f7xx_hal.h"
#else
  #error Unsupport device
#endif

Thanks, Olivier