linux-kernel - Re: [PATCH 1/3] riscv: dts: thead: add BeagleV Ahead board device tree

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <ZL1jlacHj7sgNg8M@x1>
Date:   Sun, 23 Jul 2023 10:29:57 -0700
From:   Drew Fustini <dfustini@...libre.com>
To:     Conor Dooley <conor@...nel.org>, Jisheng Zhang <jszhang@...nel.org>
Cc:     Guo Ren <guoren@...nel.org>, Fu Wei <wefu@...hat.com>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Conor Dooley <conor+dt@...nel.org>,
        Paul Walmsley <paul.walmsley@...ive.com>,
        Palmer Dabbelt <palmer@...belt.com>,
        Albert Ou <aou@...s.berkeley.edu>,
        linux-riscv@...ts.infradead.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org,
        Robert Nelson <robertcnelson@...gleboard.org>,
        Jason Kridner <jkridner@...gleboard.org>
Subject: Re: [PATCH 1/3] riscv: dts: thead: add BeagleV Ahead board device
 tree

On Sun, Jul 23, 2023 at 11:32:17AM +0100, Conor Dooley wrote:
> On Sat, Jul 22, 2023 at 02:55:39PM -0700, Drew Fustini wrote:
> 
> > +++ b/arch/riscv/boot/dts/thead/th1520-beaglev-ahead.dts
> > @@ -0,0 +1,61 @@
> > +// SPDX-License-Identifier: GPL-2.0
> 
> Hmm, should this not be dual licensed?
> I notice the other th1520 stuff isn't either..

Good point, I'll resubmit with a dual license.

Jisheng: are you okay with the other arch/riscv/boot/dts/thead files
changing to a dual license?

> 
> 
> > +	memory@0 {
> > +		device_type = "memory";
> > +		reg = <0x0  0x00000000  0x1 0x00000000>;
> 
> There's some extra spaces in here for some reason.

Thanks for spotting that. I'll fix when I resubmit with dual licnese.

Drew