linux-kernel - Re: [PATCH v14 3/4] arm64: dts: Add DTS for Marvell PXA1908 and samsung,coreprimevelte

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: <5849160.DvuYhMxLoT@radijator>
Date: Thu, 16 Jan 2025 16:26:59 +0100
From: Duje Mihanović <duje.mihanovic@...le.hr>
To: Krzysztof Kozlowski <krzk@...nel.org>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>, Lubomir Rintel <lkundrak@...sk>,
 Catalin Marinas <catalin.marinas@....com>, Will Deacon <will@...nel.org>,
 Andrew Lunn <andrew@...n.ch>, Gregory Clement <gregory.clement@...tlin.com>,
 Sebastian Hesselbarth <sebastian.hesselbarth@...il.com>,
 Kees Cook <kees@...nel.org>, Tony Luck <tony.luck@...el.com>,
 "Guilherme G. Piccoli" <gpiccoli@...lia.com>,
 David Wronek <david@...nlining.org>, Karel Balej <balejk@...fyz.cz>,
 devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
 linux-arm-kernel@...ts.infradead.org, linux-hardening@...r.kernel.org,
 phone-devel@...r.kernel.org, ~postmarketos/upstreaming@...ts.sr.ht
Subject:
 Re: [PATCH v14 3/4] arm64: dts: Add DTS for Marvell PXA1908 and
 samsung,coreprimevelte

On Thursday 16 January 2025 09:00:40 Central European Standard Time Krzysztof 
Kozlowski wrote:
> On Wed, Jan 15, 2025 at 09:35:56PM +0100, Duje Mihanović wrote:
> > Add DTS for Marvell PXA1908 SoC and Samsung Galaxy Core Prime Value
> > Edition LTE, a smartphone based on said SoC.
> > 
> > Signed-off-by: Duje Mihanović <duje.mihanovic@...le.hr>
> > ---
> > 
> >  arch/arm64/boot/dts/marvell/Makefile               |   3 +
> >  .../dts/marvell/pxa1908-samsung-coreprimevelte.dts | 336
> >  +++++++++++++++++++++ arch/arm64/boot/dts/marvell/pxa1908.dtsi           
|
> >  300 ++++++++++++++++++ 3 files changed, 639 insertions(+)
> > 
> > diff --git a/arch/arm64/boot/dts/marvell/Makefile
> > b/arch/arm64/boot/dts/marvell/Makefile index
> > ce751b5028e2628834340b5c50f8992092226eba..
39c5749e631db33aa8fb0386a951c0a70
> > 215bc02 100644 --- a/arch/arm64/boot/dts/marvell/Makefile
> > +++ b/arch/arm64/boot/dts/marvell/Makefile
> > @@ -32,3 +32,6 @@ dtb-$(CONFIG_ARCH_MVEBU) += cn9130-cf-base.dtb
> > 
> >  dtb-$(CONFIG_ARCH_MVEBU) += cn9130-cf-pro.dtb
> >  dtb-$(CONFIG_ARCH_MVEBU) += cn9131-cf-solidwan.dtb
> >  dtb-$(CONFIG_ARCH_MVEBU) += cn9132-clearfog.dtb
> > 
> > +
> > +# MMP SoC Family
> > +dtb-$(CONFIG_ARCH_MMP) += pxa1908-samsung-coreprimevelte.dtb
> 
> Hm, why separate ARCH if this is part of Marvell Makefile? One ARCH per
> vendor, so if you think this is different than MVEBU, then should be in
> a separate subdirectory of marvell.

I see, for v15 I'll put these in a mmp/ subdir.

> > +
> > +		ramoops@...0000 {
> > +			compatible = "ramoops";
> > +			reg = <0 0x8100000 0 0x40000>;
> > +			record-size = <0x8000>;
> > +			console-size = <0x20000>;
> > +			max-reason = <5>;
> > +		};
> > +	};
> > +
> > +
> 
> If there is going to be resend/new version:
> Just one blank line.

Will fix.

Regards,
-- 
Duje