From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexandre Belloni Subject: Re: [PATCH v2 03/18] AT91: G45: DT: Declare a second ram controller Date: Mon, 7 Jul 2014 16:57:21 +0200 Message-ID: <20140707145721.GC3113@piout.net> References: <1404743131-24569-1-git-send-email-maxime.ripard@free-electrons.com> <1404743131-24569-4-git-send-email-maxime.ripard@free-electrons.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1404743131-24569-4-git-send-email-maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Maxime Ripard Cc: linux-PelNFVqkFnVyf+4FbqDuWQ@public.gmane.org, Nicolas Ferre , Jean-Christophe Plagniol-Villard , dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org, dbaryshkov-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, Boris Brezillon , Thomas Petazzoni , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org Hi, On 07/07/2014 at 16:25:16 +0200, Maxime Ripard wrote : > The G45 has two identical ram controller, that are defined as a single one, > with two reg cells. > > The proper way to support such a case is to have two separate DT nodes. > This is also the case for the sam9263, please fix it there too. > Signed-off-by: Maxime Ripard > --- > arch/arm/boot/dts/at91sam9g45.dtsi | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/boot/dts/at91sam9g45.dtsi b/arch/arm/boot/dts/at91sam9g45.dtsi > index ace6bf197b70..6f648b85b725 100644 > --- a/arch/arm/boot/dts/at91sam9g45.dtsi > +++ b/arch/arm/boot/dts/at91sam9g45.dtsi > @@ -75,8 +75,12 @@ > > ramc0: ramc@ffffe400 { > compatible = "atmel,at91sam9g45-ddramc"; > - reg = <0xffffe400 0x200 > - 0xffffe600 0x200>; > + reg = <0xffffe400 0x200>; > + }; > + > + ramc1: ramc@ffffe600 { > + compatible = "atmel,at91sam9g45-ddramc"; > + reg = <0xffffe600 0x200>; > }; > > pmc: pmc@fffffc00 { > -- > 2.0.1 > -- Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html