Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932882AbeAKJOZ (ORCPT + 1 other); Thu, 11 Jan 2018 04:14:25 -0500 Received: from mail.free-electrons.com ([62.4.15.54]:46733 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932566AbeAKJOX (ORCPT ); Thu, 11 Jan 2018 04:14:23 -0500 From: Gregory CLEMENT To: Chris Packham Cc: robh+dt@kernel.org, linux-arm-kernel@lists.infradead.org, andrew@lunn.ch, jason@lakedaemon.net, sebastian.hesselbarth@gmail.com, linux@armlinux.org.uk, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 0/3] ARM: mvebu: dts: updates to enable EDAC References: <20180111015903.11322-1-chris.packham@alliedtelesis.co.nz> Date: Thu, 11 Jan 2018 10:14:22 +0100 In-Reply-To: <20180111015903.11322-1-chris.packham@alliedtelesis.co.nz> (Chris Packham's message of "Thu, 11 Jan 2018 14:59:00 +1300") Message-ID: <87a7xk7o5t.fsf@free-electrons.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: Hi Chris, On jeu., janv. 11 2018, Chris Packham wrote: > I've split this off from my earlier series[1] this is just the dts changes that > will enable support for the EDAC series when it lands. > > The Armada 38x as well as the 98dx3236 and similar switch chips with integrated > CPUs use the same SDRAM controller block as the Armada XP. The key difference > is the width of the DDR interface. > > [1] - https://marc.info/?l=linux-kernel&m=151545124505964&w=2 The series is looks good now. For patch 1 I still wait for that the "marvell,,ecc-enable" property was accepted before merging it. So I can either wait for that it was accepted before applying the series, or just applying patch 2 and 3 for now, as you want. Thanks, Gregory > > Changes in v2: > - update commit message > - add labels to dts > > Chris Packham (3): > ARM: dts: armada-xp: enable L2 cache parity and ecc on db-xc3-24g4xg > ARM: dts: armada-xp: add label to sdram-controller node > ARM: dts: mvebu: add sdram controller node to Armada-38x > > arch/arm/boot/dts/armada-38x.dtsi | 5 +++++ > arch/arm/boot/dts/armada-xp-98dx3236.dtsi | 2 +- > arch/arm/boot/dts/armada-xp-db-xc3-24g4xg.dts | 5 +++++ > arch/arm/boot/dts/armada-xp.dtsi | 2 +- > 4 files changed, 12 insertions(+), 2 deletions(-) > > -- > 2.15.1 > -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com