[PATCH v1 15/22] siemens: imx8-capricorn.dtsi: small adaptions

[Leto, Enrico]

Hello Heiko, Fabio

> -----Original Message-----
> From: Heiko Schocher <hs at denx.de>
> Sent: Monday, November 11, 2024 6:53 AM
> To: Fabio Estevam <festevam at gmail.com>
> Cc: U-Boot Mailing List <u-boot at lists.denx.de>; Leto, Enrico (SI B PRO TI EAC
> CCP) <enrico.leto at siemens.com>; Schweizer, Walter (SI B PRO TI EAC CCP)
> <walter.schweizer at siemens.com>; Sverdlin, Alexander (SI B PRO TI EAC CCP)
> <alexander.sverdlin at siemens.com>; Anatolij Gustschin <agust at denx.de>;
> NXP i.MX U-Boot Team <uboot-imx at nxp.com>; Babic, Stefano (EXT) (DENX
> Software Engineering GmbH) <sbabic at denx.de>; Tom Rini
> <trini at konsulko.com>
> Subject: Re: [PATCH v1 15/22] siemens: imx8-capricorn.dtsi: small adaptions
> 
> Hello Fabio,
> 
> On 09.11.24 13:03, Fabio Estevam wrote:
> > Hi Heiko,
> >
> > On Fri, Nov 8, 2024 at 2:22 AM Heiko Schocher <hs at denx.de> wrote:
> >>
> >> - remove DMA for console
> >
> > Why? Does it cause problems in U-Boot?
> 
> Forward this question to Enrico.

The reference implementation of the imx8qxp/imx8dx (mek board) uses lpuart0 for the console. We have the console on lpuart2.

Getting a look on NXP u-boot that is today the base of our code, DMA is defined for lpuart1-2-3, not for lpuart0. NXP doesn't use DMA for the console anywhere for imx8. It seems it doesn't work properly with their driver. For this reason, we disabled the DMA here.

Getting now a look to u-boot upstream I see that the DMA is not configured anywhere.
Moving to upstream we don't have to disable anymore.

@Heiko: I propose to remove.

> 
> >> - add wdt device
> >>
> >> Signed-off-by: Heiko Schocher <hs at denx.de>
> >> ---
> >>
> >>   arch/arm/dts/imx8-capricorn.dtsi | 16 ++++++++++++++++
> >
> > Are there any plans to upstream this dtsi and switch to OF_UPSTREAM?
> 
> Hmm... good question! Currently we work on U-Boot to get mainline ready
> for use in production, so we get rid from NXP code.
> 
> I do not know, if we can reach this with mainline linux too...
> 
> May the siemens guys can answer this...

Here the commit "feat(scu_wdt): add option to control external wdt via IMX8 SCU" https://code.siemens.com/ccp/u-boot/-/commit/a5b99d1c406fdf92cd1d7d50d566a4e1e66c447b

@Heiko: Check if we have a chance to put the whole commit upstream.

> 
> bye,
> Heiko
> --
> DENX Software Engineering GmbH,      Managing Director: Erika Unter
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs at denx.de

Thanks,
Enrico