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

[Leto, Enrico]

Hi Heiko

> -----Original Message-----
> From: Heiko Schocher <hs at denx.de>
> Sent: Monday, November 11, 2024 9:48 AM
> To: Leto, Enrico (SI B PRO TI EAC CCP) <enrico.leto at siemens.com>; Fabio
> Estevam <festevam at gmail.com>
> Cc: U-Boot Mailing List <u-boot at lists.denx.de>; 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 Enrico,
> 
> On 11.11.24 09:25, Leto, Enrico wrote:
> > 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.
> 
> Okay, i remove it and test, thanks!
> 
> >>>> - 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/a5b99d1c406fdf92cd1d7d50d
> > 566a4e1e66c447b
> >
> > @Heiko: Check if we have a chance to put the whole commit upstream.
> 
> Aehm... this is the "reworked for mainline patch" we currently talk about!
> 
> :-P
> 
> The question is, if we can get the siemens linux port also to mainline linux!
> But we should discuss first internally...
> 

Overseen. The patch mentioned above is now shared in 01/22 and here.
I would update the commit text like "siemens: imx8-capricorn.dtsi: enable scu-wdt". 

> bye,
> Heiko
> >
> >>
> >> 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
> >
> 
> --
> 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