linux-kernel - Re: [PATCH 2/3] arm64: dts: ti: k3-j784s4-j742s2-evm: Add overlay for PCIe NTB

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20250224121110.w3mu55otlgplcw6j@uda0492258>
Date: Mon, 24 Feb 2025 17:41:10 +0530
From: Siddharth Vadapalli <s-vadapalli@...com>
To: Vignesh Raghavendra <vigneshr@...com>
CC: Siddharth Vadapalli <s-vadapalli@...com>, <nm@...com>, <kristo@...nel.org>,
        <robh@...nel.org>, <krzk+dt@...nel.org>, <conor+dt@...nel.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <linux-arm-kernel@...ts.infradead.org>, <srk@...com>
Subject: Re: [PATCH 2/3] arm64: dts: ti: k3-j784s4-j742s2-evm: Add overlay
 for PCIe NTB

On Mon, Feb 24, 2025 at 05:35:47PM +0530, Vignesh Raghavendra wrote:

Hello Vignesh,

> 
> 
> On 02/02/25 15:06, Siddharth Vadapalli wrote:
> > diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-evm-pcie-ntb.dtso b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-evm-pcie-ntb.dtso
> > new file mode 100644
> > index 000000000000..e44173f670fa
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-evm-pcie-ntb.dtso
> > @@ -0,0 +1,92 @@
> > +// SPDX-License-Identifier: GPL-2.0-only OR MIT
> > +/**
> > + * DT Overlay for enabling NTB functionality using PCIE0 and PCIE1 instances of
> > + * PCIe on the J784S4 EVM and the J742S2 EVM.
> > + *
> > + * J784S4 EVM Product Link: https://www.ti.com/tool/J784S4XEVM
> > + * J742S2 EVM Product Link: https://www.ti.com/tool/J742S2XH01EVM
> > + *
> > + * Copyright (C) 2025 Texas Instruments Incorporated - https://www.ti.com/
> > + */
> > +
> > +/dts-v1/;
> > +/plugin/;
> > +
> > +#include <dt-bindings/interrupt-controller/arm-gic.h>
> > +#include <dt-bindings/soc/ti,sci_pm_domain.h>
> > +
> > +#include "k3-pinctrl.h"
> > +
> > +&{/} {
> > +	epf_bus {
> 
> 
> no underscore in node name please

Thank you for pointing this out. I will fix it in this as well as the
previous patch in this series and post the v2 series.

Regards,
Siddharth.