Re: [PATCH v2 4/9] board: phytec: phycore_imx8mp: Change debug UART

[Teresa Remmet <T.Remmet@phytec.de>]

Hello Heiko,

Am Montag, den 05.07.2021, 06:58 +0200 schrieb Heiko Schocher:
> Hello Teresa,
> 
> On 02.07.21 13:19, Teresa Remmet wrote:
> > With the first redesign the debug UART had changed from
> > UART2 to UART1.
> > As the first hardware revision is considered as alpha and
> > will not be supported in future. The old setup will not
> > be preserved.
> > 
> > Signed-off-by: Teresa Remmet <t.remmet@phytec.de>
> > ---
> >  arch/arm/dts/imx8mp-phyboard-pollux-rdk-u-boot.dtsi | 4 ++--
> >  board/phytec/phycore_imx8mp/spl.c                   | 6 +++---
> >  include/configs/phycore_imx8mp.h                    | 4 ++--
> >  3 files changed, 7 insertions(+), 7 deletions(-)
> 
> Your patch
> "[PATCH v2 3/9] arm: dts: imx8mp-phyboard-pollux: Sync dts files with
> kernel"
> 
> changed uart2 to uart1 in arch/arm/dts/imx8mp-phyboard-pollux-rdk.dts
> 
> May this part should be done also in this patch?

yes, this would make the uart change more atomic and not separated over
2 patches. I will update this in v3.

Thanks,
Teresa

> 
> Beside of this:
> 
> Reviewed-by: Heiko Schocher <hs@denx.de>
> 
> bye,
> Heiko