Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755846AbbHYOFF (ORCPT ); Tue, 25 Aug 2015 10:05:05 -0400 Received: from mail-pa0-f53.google.com ([209.85.220.53]:36374 "EHLO mail-pa0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751691AbbHYOFC (ORCPT ); Tue, 25 Aug 2015 10:05:02 -0400 Date: Tue, 25 Aug 2015 22:04:50 +0800 From: Leo Yan To: Sudeep Holla Cc: "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "guodong.xu@linaro.org" , Jian Zhang , Zhenwei Wang , Haoju Mo , Dan Zhao , "kongfei@hisilicon.com" , Guangyue Zeng , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Catalin Marinas , Will Deacon , Jassi Brar , Bintian Wang , Haojian Zhuang , Yiping Xu , Wei Xu Subject: Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Message-ID: <20150825140450.GB28262@leoy-linaro> References: <1439977055-1747-1-git-send-email-leo.yan@linaro.org> <1439977055-1747-4-git-send-email-leo.yan@linaro.org> <55DC532C.3020005@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <55DC532C.3020005@arm.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3060 Lines: 92 Hi Sudeep, On Tue, Aug 25, 2015 at 12:36:12PM +0100, Sudeep Holla wrote: > > > On 19/08/15 10:37, Leo Yan wrote: > >On Hi6220, below memory regions in DDR have specific purpose: > > > > 0x05e0,0000 - 0x05ef,ffff: For MCU firmware using at runtime; > > 0x0740,f000 - 0x0740,ffff: For MCU firmware's section; > > 0x06df,f000 - 0x06df,ffff: For mailbox message data. > > > > Unless I am reading the DTS file completely wrong, I don't think the > above memory regions are in DDR as per the memory node. i'm not sure if understand correctly for your question; Hikey board has DDR 1GB@0x0, but there have some memory regions are used for MCU. So this patch is to reserve these memory regions so that make sure kernel will not map and allocate them. Will remove these memory regions from memory node in next version. > >This patch reserves these memory regions and add device node for > >mailbox in dts. > > > >Signed-off-by: Leo Yan > >--- > > arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts | 20 +++++++++++++++++--- > > arch/arm64/boot/dts/hisilicon/hi6220.dtsi | 8 ++++++++ > > 2 files changed, 25 insertions(+), 3 deletions(-) > > > >diff --git a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts > >index e36a539..d5470d3 100644 > >--- a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts > >+++ b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts > >@@ -7,9 +7,6 @@ > > > > /dts-v1/; > > > >-/*Reserved 1MB memory for MCU*/ > >-/memreserve/ 0x05e00000 0x00100000; > >- > > #include "hi6220.dtsi" > > > > / { > >@@ -28,4 +25,21 @@ > > device_type = "memory"; > > reg = <0x0 0x0 0x0 0x40000000>; > > }; > > I have no access to the spec, but I read this as 1GB RAM @0x0 > Unless this entry is completely wrong, what your commit log claims is > incorrect. If this entry is wrong I wonder how is it booting with this > DT then. Do you mean should remove all reserved memory regions from memory node? Will submit next version's patch for this. Kernel can boot successfully on Hikey with this patch [1]. [1] https://github.com/96boards/linux > >+ > >+ reserved-memory { > >+ #address-cells = <2>; > >+ #size-cells = <2>; > >+ ranges; > >+ > >+ mcu-buf@05e00000 { > >+ no-map; > >+ reg = <0x0 0x05e00000 0x0 0x00100000>, /* MCU firmware buffer */ > >+ <0x0 0x0740f000 0x0 0x00001000>; /* MCU firmware section */ > > So I don't see how can this be part of DDR ? Or at-least part of DDR > that's mapped by kernel ? Here use reserved-memory node to remove these regions from memblock during kernel's boot up; kernel also will not map for them with property "no-map". I think this is the same question which have been brought up by Mark in his early mail and suggested to use UEFI to do this. Thanks, Leo Yan -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/