[PATCH v1 1/2] bsps/stm32h7: Make UART7 pins configurable

[Kinsey Moore]

Hi Karel,
I happen to have a board in hand that doesn't nominally use UART7, but has
been reconfigured to use it since the typical USART1 connection was ripped
off of the board. It won't ever have a public BSP, but I'm using the base
stm32h7 BSP with the right options layered on top to get the necessary
functionality. I have a few more pieces that I'm condensing into proper
options that will let me make use of the rest of the board's functionality.

Kinsey

On Mon, Jul 24, 2023 at 3:06 PM Karel Gardas <karel at functional.vision>
wrote:

>
> Hello Kinsey,
>
> I think the patch looks good, although I've not verified precise
> PIN/REGs assignment value. I trust you test this somehow otherwise you
> would not submit it. And this steers my curiosity. The only board (from
> ST Micro) I know which provides connection to UART7 is stm32h735g-dk
> which is not supported by RTEMS due to missing MCU support in old HAL
> code. I submitted new HAL patches last week but they are not in yet. So
> if this is not company secrete let me ask, what board you are using with
> UART7 connection?
>
> Thanks,
> Karel
>
> On 7/24/23 20:28, Kinsey Moore wrote:
> > This change allows for the pins assigned to UART7 to be reconfigured via
> > config.ini.
> > ---
> >   bsps/arm/stm32h7/console/console-uart7-cfg.c    |  4 ++--
> >   spec/build/bsps/arm/stm32h7/grp.yml             |  4 ++++
> >   .../build/bsps/arm/stm32h7/optuart7gpiopins.yml | 17 +++++++++++++++++
> >   .../build/bsps/arm/stm32h7/optuart7gpioregs.yml | 17 +++++++++++++++++
> >   4 files changed, 40 insertions(+), 2 deletions(-)
> >   create mode 100644 spec/build/bsps/arm/stm32h7/optuart7gpiopins.yml
> >   create mode 100644 spec/build/bsps/arm/stm32h7/optuart7gpioregs.yml
> >
> > diff --git a/bsps/arm/stm32h7/console/console-uart7-cfg.c
> b/bsps/arm/stm32h7/console/console-uart7-cfg.c
> > index dca19e652d..2ced4cf9c2 100644
> > --- a/bsps/arm/stm32h7/console/console-uart7-cfg.c
> > +++ b/bsps/arm/stm32h7/console/console-uart7-cfg.c
> > @@ -33,9 +33,9 @@
> >
> >   const stm32h7_uart_config stm32h7_uart7_config = {
> >     .gpio = {
> > -    .regs = GPIOB,
> > +    .regs = STM32H7_UART7_GPIO_REGS,
> >       .config = {
> > -      .Pin = GPIO_PIN_3 | GPIO_PIN_4,
> > +      .Pin = STM32H7_UART7_GPIO_PINS,
> >         .Mode = GPIO_MODE_AF_PP,
> >         .Pull = GPIO_NOPULL,
> >         .Speed = GPIO_SPEED_FREQ_LOW,
> > diff --git a/spec/build/bsps/arm/stm32h7/grp.yml
> b/spec/build/bsps/arm/stm32h7/grp.yml
> > index 595762c665..239df9b898 100644
> > --- a/spec/build/bsps/arm/stm32h7/grp.yml
> > +++ b/spec/build/bsps/arm/stm32h7/grp.yml
> > @@ -106,6 +106,10 @@ links:
> >     uid: optusart3gpiopins
> >   - role: build-dependency
> >     uid: optusart3gpioregs
> > +- role: build-dependency
> > +  uid: optuart7gpiopins
> > +- role: build-dependency
> > +  uid: optuart7gpioregs
> >   - role: build-dependency
> >     uid: optvariant
> >   - role: build-dependency
> > diff --git a/spec/build/bsps/arm/stm32h7/optuart7gpiopins.yml
> b/spec/build/bsps/arm/stm32h7/optuart7gpiopins.yml
> > new file mode 100644
> > index 0000000000..0252bdab7c
> > --- /dev/null
> > +++ b/spec/build/bsps/arm/stm32h7/optuart7gpiopins.yml
> > @@ -0,0 +1,17 @@
> > +SPDX-License-Identifier: CC-BY-SA-4.0 OR BSD-2-Clause
> > +actions:
> > +- get-string: null
> > +- define-unquoted: null
> > +build-type: option
> > +copyrights:
> > +- Copyright (C) 2023 On-Line Applications Research (OAR)
> > +default:
> > +- enabled-by: true
> > +  value: ( GPIO_PIN_3 | GPIO_PIN_4 )
> > +description: |
> > +  GPIO pins used for the UART7 pin configuration.
> > +enabled-by: true
> > +format: '{}'
> > +links: []
> > +name: STM32H7_UART7_GPIO_PINS
> > +type: build
> > diff --git a/spec/build/bsps/arm/stm32h7/optuart7gpioregs.yml
> b/spec/build/bsps/arm/stm32h7/optuart7gpioregs.yml
> > new file mode 100644
> > index 0000000000..16ce3bf526
> > --- /dev/null
> > +++ b/spec/build/bsps/arm/stm32h7/optuart7gpioregs.yml
> > @@ -0,0 +1,17 @@
> > +SPDX-License-Identifier: CC-BY-SA-4.0 OR BSD-2-Clause
> > +actions:
> > +- get-string: null
> > +- define-unquoted: null
> > +build-type: option
> > +copyrights:
> > +- Copyright (C) 2023 On-Line Applications Research (OAR)
> > +default:
> > +- enabled-by: true
> > +  value: GPIOB
> > +description: |
> > +  GPIO registers used for the UART7 pin configuration.
> > +enabled-by: true
> > +format: '{}'
> > +links: []
> > +name: STM32H7_UART7_GPIO_REGS
> > +type: build
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20230724/797ce79b/attachment-0001.htm>