Re: [PATCH] This patch adds support for a STM32H7 variation

2021-01-13 Thread Chris Johns
On 9/1/21 7:49 am, Sebastian Huber wrote: > On 08/01/2021 15:30, Joel Sherrill wrote: >> On Fri, Jan 8, 2021 at 6:58 AM Robin Müller > > wrote: >> >> Are they really 12 BSPs or just build variants of this one BSP family? >> >> If just variants, I have come full cir

Re: [PATCH] This patch adds support for a STM32H7 variation

2021-01-13 Thread Robin Müller
Hi, I can test the new BSP variant on my Nucleo board. Let me know when you have pushed the changes. Kind Regards Robin On Sun, 10 Jan 2021 at 01:02, Chris Johns wrote: > On 9/1/21 7:49 am, Sebastian Huber wrote: > > On 08/01/2021 15:30, Joel Sherrill wrote: > >> On Fri, Jan 8, 2021 at 6:58 AM

Re: [PATCH] This patch adds support for a STM32H7 variation

2021-01-08 Thread Sebastian Huber
On 08/01/2021 15:30, Joel Sherrill wrote: On Fri, Jan 8, 2021 at 6:58 AM Robin Müller > wrote: Hi Sebastian, That would also be a way. The question is whether all other board variation should be included as well. There are 12 boards listed in

Re: [PATCH] This patch adds support for a STM32H7 variation

2021-01-08 Thread Joel Sherrill
On Fri, Jan 8, 2021 at 6:58 AM Robin Müller wrote: > Hi Sebastian, > > That would also be a way. The question is whether all other board > variation should be included as well. > There are 12 boards listed in the STM32CubeH7 repository example > applications and I have not checked which pin and H

Re: [PATCH] This patch adds support for a STM32H7 variation

2021-01-08 Thread Robin Müller
Hi Sebastian, That would also be a way. The question is whether all other board variation should be included as well. There are 12 boards listed in the STM32CubeH7 repository example applications and I have not checked which pin and HSE values these boards use or if there are any other important d

Re: [PATCH] This patch adds support for a STM32H7 variation

2021-01-07 Thread Sebastian Huber
Hello Robin, thanks for the updated patch. I am not sure if we should address the problem with a STM32H743ZI_NUCLEO option. An alternative would be to add a new BSP variant, e.g. stm32h743zi_nucleo, and two new options STM32H7_HSE_FREQUENCY and STM32H7_USART3_USE_GPIOD_PIN_8_PIN_9 with a defa

[PATCH] This patch adds support for a STM32H7 variation

2021-01-06 Thread Robin.Mueller
It was made so that the default settings work out of the box for this board This includes a new BSP configuration option and using different HSE value and console GPIO pins if that option is set. --- bsps/arm/stm32h7/console/console-usart3-cfg.c | 21 +++ bsps/arm/stm32h7/include