Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934166AbeALP0W (ORCPT + 1 other); Fri, 12 Jan 2018 10:26:22 -0500 Received: from mail.free-electrons.com ([62.4.15.54]:58178 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933997AbeALP0R (ORCPT ); Fri, 12 Jan 2018 10:26:17 -0500 From: Gregory CLEMENT To: Chris Packham Cc: "andrew\@lunn.ch" , "jason\@lakedaemon.net" , "devicetree\@vger.kernel.org" , "linux\@armlinux.org.uk" , "linux-kernel\@vger.kernel.org" , "robh+dt\@kernel.org" , "linux-arm-kernel\@lists.infradead.org" , "sebastian.hesselbarth\@gmail.com" Subject: Re: [PATCH v2 0/3] ARM: mvebu: dts: updates to enable EDAC References: <20180111015903.11322-1-chris.packham@alliedtelesis.co.nz> <87a7xk7o5t.fsf@free-electrons.com> Date: Fri, 12 Jan 2018 16:25:51 +0100 In-Reply-To: (Chris Packham's message of "Thu, 11 Jan 2018 20:06:01 +0000") Message-ID: <87shbb5cao.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: > On 11/01/18 22:14, Gregory CLEMENT wrote: >> 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. > > I'm happy either way. If it's easier for you to keep the 3 patches > together that's fine by me. So I will apply all of them in the same time. Thanks, Gregory > >> >> 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 >>> >> > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com