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/a5b99d1c406fdf92cd1d7d50d566a4e1e66c447b
@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...
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]
