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

[Heiko Schocher]

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/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: 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