mbox series

[v3,0/4] STM32 early console

Message ID 20200203140425.26579-1-erwan.leray@st.com (mailing list archive)
Headers show
Series STM32 early console | expand

Message

Erwan LE RAY Feb. 3, 2020, 2:04 p.m. UTC
Add UART instance configuration to STM32 F4 and F7 early console.
Add STM32 H7 and MP1 early console support.

Changes in v3:
- fix a missing condition for STM32MP1
  
Changes in v2:
- split "[PATCH] ARM: debug: stm32: add UART early console configuration"
  into separate patches as suggested by Clement into [1]

[1] https://lkml.org/lkml/2019/4/10/199

Erwan Le Ray (4):
  ARM: debug: stm32: add UART early console configuration for STM32F4
  ARM: debug: stm32: add UART early console configuration for STM32F7
  ARM: debug: stm32: add UART early console support for STM32H7
  ARM: debug: stm32: add UART early console support for STM32MP1

 arch/arm/Kconfig.debug         | 42 +++++++++++++++++++++++++++++-----
 arch/arm/include/debug/stm32.S |  9 ++++----
 2 files changed, 40 insertions(+), 11 deletions(-)

Comments

Clément Péron Feb. 3, 2020, 2:35 p.m. UTC | #1
Hi Erwan,

On Mon, 3 Feb 2020 at 15:04, Erwan Le Ray <erwan.leray@st.com> wrote:
>
> Add UART instance configuration to STM32 F4 and F7 early console.
> Add STM32 H7 and MP1 early console support.
>
> Changes in v3:
> - fix a missing condition for STM32MP1
>
> Changes in v2:
> - split "[PATCH] ARM: debug: stm32: add UART early console configuration"
>   into separate patches as suggested by Clement into [1]

Thanks for splitting the patch, the whole series looks fine to me.

Acked-by: Clément Péron <peron.clem@gmail.com>

Clément



>
> [1] https://lkml.org/lkml/2019/4/10/199
>
> Erwan Le Ray (4):
>   ARM: debug: stm32: add UART early console configuration for STM32F4
>   ARM: debug: stm32: add UART early console configuration for STM32F7
>   ARM: debug: stm32: add UART early console support for STM32H7
>   ARM: debug: stm32: add UART early console support for STM32MP1
>
>  arch/arm/Kconfig.debug         | 42 +++++++++++++++++++++++++++++-----
>  arch/arm/include/debug/stm32.S |  9 ++++----
>  2 files changed, 40 insertions(+), 11 deletions(-)
>
> --
> 2.17.1
>
Erwan LE RAY Feb. 21, 2020, 3:54 p.m. UTC | #2
Hi Russel, Max, Alex, Arnd, Linus and Olof,

Gentle reminder.

Can you please provide your feedback on this series ?

Best Regards, Erwan.


On 2/3/20 3:35 PM, Clément Péron wrote:
> Hi Erwan,
>
> On Mon, 3 Feb 2020 at 15:04, Erwan Le Ray <erwan.leray@st.com> wrote:
>> Add UART instance configuration to STM32 F4 and F7 early console.
>> Add STM32 H7 and MP1 early console support.
>>
>> Changes in v3:
>> - fix a missing condition for STM32MP1
>>
>> Changes in v2:
>> - split "[PATCH] ARM: debug: stm32: add UART early console configuration"
>>    into separate patches as suggested by Clement into [1]
> Thanks for splitting the patch, the whole series looks fine to me.
>
> Acked-by: Clément Péron <peron.clem@gmail.com>
>
> Clément
>
>
>
>> [1] https://lkml.org/lkml/2019/4/10/199
>>
>> Erwan Le Ray (4):
>>    ARM: debug: stm32: add UART early console configuration for STM32F4
>>    ARM: debug: stm32: add UART early console configuration for STM32F7
>>    ARM: debug: stm32: add UART early console support for STM32H7
>>    ARM: debug: stm32: add UART early console support for STM32MP1
>>
>>   arch/arm/Kconfig.debug         | 42 +++++++++++++++++++++++++++++-----
>>   arch/arm/include/debug/stm32.S |  9 ++++----
>>   2 files changed, 40 insertions(+), 11 deletions(-)
>>
>> --
>> 2.17.1
>>
Alexandre TORGUE Feb. 21, 2020, 3:58 p.m. UTC | #3
Hi

On 2/3/20 3:04 PM, Erwan Le Ray wrote:
> Add UART instance configuration to STM32 F4 and F7 early console.
> Add STM32 H7 and MP1 early console support.
> 
> Changes in v3:
> - fix a missing condition for STM32MP1
>    
> Changes in v2:
> - split "[PATCH] ARM: debug: stm32: add UART early console configuration"
>    into separate patches as suggested by Clement into [1]
> 
> [1] https://lkml.org/lkml/2019/4/10/199
> 
> Erwan Le Ray (4):
>    ARM: debug: stm32: add UART early console configuration for STM32F4
>    ARM: debug: stm32: add UART early console configuration for STM32F7
>    ARM: debug: stm32: add UART early console support for STM32H7
>    ARM: debug: stm32: add UART early console support for STM32MP1
> 
>   arch/arm/Kconfig.debug         | 42 +++++++++++++++++++++++++++++-----
>   arch/arm/include/debug/stm32.S |  9 ++++----
>   2 files changed, 40 insertions(+), 11 deletions(-)
> 

Acked-by: Alexandre TORGUE <alexandre.torgue@st.com>

Russel, Arnd, Olof, Linus
Do I have to take this series in my next PR ? or you'll ?

Thanks
alex
Linus Walleij Feb. 22, 2020, 11:22 a.m. UTC | #4
On Fri, Feb 21, 2020 at 4:58 PM Alexandre Torgue
<alexandre.torgue@st.com> wrote:

> > Erwan Le Ray (4):
> >    ARM: debug: stm32: add UART early console configuration for STM32F4
> >    ARM: debug: stm32: add UART early console configuration for STM32F7
> >    ARM: debug: stm32: add UART early console support for STM32H7
> >    ARM: debug: stm32: add UART early console support for STM32MP1
(...)
> Do I have to take this series in my next PR ? or you'll ?

Sign it off and send a PR to ARM SoC.

Yours,
Linus Walleij
Alexandre TORGUE March 13, 2020, 12:56 p.m. UTC | #5
Hi Erwan

On 2/3/20 3:04 PM, Erwan Le Ray wrote:
> Add UART instance configuration to STM32 F4 and F7 early console.
> Add STM32 H7 and MP1 early console support.
> 
> Changes in v3:
> - fix a missing condition for STM32MP1
>    
> Changes in v2:
> - split "[PATCH] ARM: debug: stm32: add UART early console configuration"
>    into separate patches as suggested by Clement into [1]
> 
> [1] https://lkml.org/lkml/2019/4/10/199
> 
> Erwan Le Ray (4):
>    ARM: debug: stm32: add UART early console configuration for STM32F4
>    ARM: debug: stm32: add UART early console configuration for STM32F7
>    ARM: debug: stm32: add UART early console support for STM32H7
>    ARM: debug: stm32: add UART early console support for STM32MP1
> 
>   arch/arm/Kconfig.debug         | 42 +++++++++++++++++++++++++++++-----
>   arch/arm/include/debug/stm32.S |  9 ++++----
>   2 files changed, 40 insertions(+), 11 deletions(-)
> 

Series applied on stm32-next.

Regards
Alex