linux-kernel - Re: [PATCH v2 4/4] arm64: dts: ti: Add support for J784S4 EVM board

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: <20221115045226.6l4j4qbvedv2whjq@effects>
Date:   Mon, 14 Nov 2022 22:52:26 -0600
From:   Nishanth Menon <nm@...com>
To:     Apurva Nandan <a-nandan@...com>
CC:     Andrew Davis <afd@...com>, Vignesh Raghavendra <vigneshr@...com>,
        Tero Kristo <kristo@...nel.org>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Linus Walleij <linus.walleij@...aro.org>,
        <linux-arm-kernel@...ts.infradead.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <linux-gpio@...r.kernel.org>, Hari Nagalla <hnagalla@...com>
Subject: Re: [PATCH v2 4/4] arm64: dts: ti: Add support for J784S4 EVM board

On 17:48-20221111, Apurva Nandan wrote:
> 
[...]

> > > Schematics: https://www.ti.com/lit/zip/sprr458


> > > 
> > > Signed-off-by: Hari Nagalla <hnagalla@...com>
> > > Signed-off-by: Apurva Nandan <a-nandan@...com>
> > > Signed-off-by: Nishanth Menon <nm@...com>
> > > Signed-off-by: Matt Ranostay <mranostay@...com>
> > > Signed-off-by: Rahul T R <r-ravikumar@...com>
> > > Signed-off-by: Suman Anna <s-anna@...com>
> > > Signed-off-by: Pratyush Yadav <p.yadav@...com>
> > > Signed-off-by: Siddharth Vadapalli <s-vadapalli@...com>

Uggh - too many Signed-off-by -> Can we either trim this down or use
Co-developed-by ?

The last of the SoB should be yours as you are handing the patch over.
> > > ---
> > >    arch/arm64/boot/dts/ti/Makefile          |   2 +
> > >    arch/arm64/boot/dts/ti/k3-j784s4-evm.dts | 199 +++++++++++++++++++++++
> > >    2 files changed, 201 insertions(+)
> > >    create mode 100644 arch/arm64/boot/dts/ti/k3-j784s4-evm.dts
> > > 
> > > diff --git a/arch/arm64/boot/dts/ti/Makefile b/arch/arm64/boot/dts/ti/Makefile
> > > index 02e5d80344d0..6381c458738a 100644
> > > --- a/arch/arm64/boot/dts/ti/Makefile
> > > +++ b/arch/arm64/boot/dts/ti/Makefile
> > > @@ -19,6 +19,8 @@ dtb-$(CONFIG_ARCH_K3) += k3-j7200-common-proc-board.dtb
> > >    dtb-$(CONFIG_ARCH_K3) += k3-j721s2-common-proc-board.dtb

One EoL here please? helps keeps the dtbs grouped.

> > > +dtb-$(CONFIG_ARCH_K3) += k3-j784s4-evm.dtb
> > > +
> > >    dtb-$(CONFIG_ARCH_K3) += k3-am642-evm.dtb
> > >    dtb-$(CONFIG_ARCH_K3) += k3-am642-sk.dtb
> > > diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-evm.dts b/arch/arm64/boot/dts/ti/k3-j784s4-evm.dts
> > > new file mode 100644
> > > index 000000000000..bf2f2dfb7658
> > > --- /dev/null
> > > +++ b/arch/arm64/boot/dts/ti/k3-j784s4-evm.dts
> > > @@ -0,0 +1,199 @@
> > > +// SPDX-License-Identifier: GPL-2.0
> > > +/*
> > > + * Copyright (C) 2022 Texas Instruments Incorporated - https://www.ti.com/
> > > + *
> > > + * Common Processor Board: https://www.ti.com/tool/J721EXCPXEVM
> > This doesn't seem to be the right EVM, I'd just drop this link.

Please use the proper link -> I think it is in commit message.

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D