Hi Heiko > -----Original Message----- > From: Heiko Schocher <[email protected]> > Sent: Monday, November 11, 2024 9:48 AM > To: Leto, Enrico (SI B PRO TI EAC CCP) <[email protected]>; Fabio > Estevam <[email protected]> > Cc: U-Boot Mailing List <[email protected]>; Schweizer, Walter (SI B PRO TI > EAC CCP) <[email protected]>; Sverdlin, Alexander (SI B PRO TI > EAC CCP) <[email protected]>; Anatolij Gustschin > <[email protected]>; NXP i.MX U-Boot Team <[email protected]>; Babic, > Stefano (EXT) (DENX Software Engineering GmbH) <[email protected]>; Tom > Rini <[email protected]> > 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 <[email protected]> > >> Sent: Monday, November 11, 2024 6:53 AM > >> To: Fabio Estevam <[email protected]> > >> Cc: U-Boot Mailing List <[email protected]>; Leto, Enrico (SI B > >> PRO TI EAC > >> CCP) <[email protected]>; Schweizer, Walter (SI B PRO TI EAC > >> CCP) <[email protected]>; Sverdlin, Alexander (SI B PRO TI > >> EAC CCP) <[email protected]>; Anatolij Gustschin > >> <[email protected]>; NXP i.MX U-Boot Team <[email protected]>; Babic, > >> Stefano (EXT) (DENX Software Engineering GmbH) <[email protected]>; Tom > >> Rini <[email protected]> > >> 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 <[email protected]> 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 <[email protected]> > >>>> --- > >>>> > >>>> 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: [email protected] > > > > 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: [email protected]
