From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2a00:1450:4864:20::343; helo=mail-wm1-x343.google.com; envelope-from=leif.lindholm@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-wm1-x343.google.com (mail-wm1-x343.google.com [IPv6:2a00:1450:4864:20::343]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 9E341211BA47C for ; Thu, 31 Jan 2019 04:12:06 -0800 (PST) Received: by mail-wm1-x343.google.com with SMTP id b11so2261194wmj.1 for ; Thu, 31 Jan 2019 04:12:06 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to :user-agent; bh=Ej/Lj+VksKUIskf85Y/N//g8OAebgRUr8XB9RNaXyI4=; b=UD6K1WnHEEt4eTmWQyk0Xs9qaew36k385zAzY/i/C3nR9cWZk3pNCn9kxGcFR/J5M4 hevcv4mQd9bixJUjiVUGCG1iv15s/JWrGNyJYkPFJ4dHkk+89OT1pgpic6m64R0XDo7B 69lD4kUeD2OTavWRfAABNs+jvSNaqYCPNaJ30= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to:user-agent; bh=Ej/Lj+VksKUIskf85Y/N//g8OAebgRUr8XB9RNaXyI4=; b=IoKO/0IqnP3g52zQBOxX2y4oc7sRDGzI0C95ezpJFrtpACvx1EYX3uWH9HNkoKgUSN AB/WyeATkSCUETpdFnmh4FppAq5ek9HCqL4skt30THrIdBI/REfI+nDM659wbViV/C1U 7kulH2N3+mQTp8KcNRZO2eDD6nBXw8BcYr6cgD7+tSJBHwArH/THp+05atlkS3aPaoG8 XA5HAZ2V1RX7+7kvNSP+m9C5FCUoctrHAZt9kkI+L65q0wY8c7wlUcpdkQeeXh+dsI59 SBt9gOZqCoRUVj55C5sKM9XCa8V43ysq8iGJkuBc5phSUPHpuWeFMtnZ3eH2Wm1vp+7C th1Q== X-Gm-Message-State: AJcUukcKYak5nVdmpj0J+dAHgf1ZorzECmBPH3Lui19jiGYCICx04mjT MRoKxghU+uYHtOfqgPmx7IrxVA== X-Google-Smtp-Source: ALg8bN5HNSD4NkAjktGApBh1Nphz0DI/78N53oMbgtkO1LbxPwerfOiFZYr7v1I6A6wS10PhXDnbhg== X-Received: by 2002:a1c:760c:: with SMTP id r12mr27447705wmc.127.1548936724379; Thu, 31 Jan 2019 04:12:04 -0800 (PST) Received: from bivouac.eciton.net (bivouac.eciton.net. [2a00:1098:0:86:1000:23:0:2]) by smtp.gmail.com with ESMTPSA id p10sm2648489wrt.25.2019.01.31.04.12.03 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 31 Jan 2019 04:12:03 -0800 (PST) Date: Thu, 31 Jan 2019 12:12:02 +0000 From: Leif Lindholm To: Marcin Wojtas Cc: edk2-devel-01 , Ard Biesheuvel , nadavh@marvell.com, "jsd@semihalf.com" , Grzegorz Jaszczyk , Kostya Porotchkin Message-ID: <20190131121202.ots5zr6x6kqbtmge@bivouac.eciton.net> References: <1548668715-15042-1-git-send-email-mw@semihalf.com> <20190130164737.glko5nprvhfcy7ap@bivouac.eciton.net> <20190131102759.fsww3vevkktmxse7@bivouac.eciton.net> MIME-Version: 1.0 In-Reply-To: User-Agent: NeoMutt/20170113 (1.7.2) Subject: Re: [platforms: PATCH v3 0/5] Armada7k8k memory handling update X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 31 Jan 2019 12:12:07 -0000 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit On Thu, Jan 31, 2019 at 01:06:15PM +0100, Marcin Wojtas wrote: > czw., 31 sty 2019 o 11:28 Leif Lindholm napisał(a): > > > > On Thu, Jan 31, 2019 at 08:01:08AM +0100, Marcin Wojtas wrote: > > > Hi Leif, > > > > > > Thanks a lot. While at it - do you think ArmPkg/Include/Library/ArmLib.h > > > / ArmPkg/Library/ArmLib/ArmLib.c would be a proper place for it? > > > > As good a place as any. While not ARM-architecture specific, I feel > > it's probably ARM-platform specific. > > > > I mean, hopefully we'll some day we'll get a sane reporting mechanism > > of Secure-reserved regions by ARM-TF and we can drop this juggling in > > Non-secure firmware. > > Agree on ARM-TF dependency, but the code is IMO pretty generic itself. > In the merged version I also used this routine to cut out a hole in > the HoB for the reason not related to NS region - therefore I think > even MdePkg/Library would be good to go :) I don't really disagree with you, but that would be something to take up with MdeModulePkg maintainers. Also, I'm not seeing an obvious existing library to put it into. Would it really be worth creating a new one for just this function? / Leif > Marcin > > > > Best regards, > > > Marcin > > > > > > śr., 30 sty 2019 o 17:47 Leif Lindholm > > > napisał(a): > > > > > > > Thanks for the rework. > > > > > > > > (We should probably move that broken-out function to ArmPkg at some point.) > > > > > > > > For the series: > > > > Reviewed-by: Leif Lindholm > > > > > > > > Pushed as b0bb325f20..0a7d8e7d93. > > > > > > > > On Mon, Jan 28, 2019 at 10:45:10AM +0100, Marcin Wojtas wrote: > > > > > Hi, > > > > > > > > > > The third version of the patchset moves the new common > > > > > header for Marvell SMC ID's to the IndustryStandard directory. > > > > > What is more important, now 3 regions (described by new PCDs) > > > > > are reserved separately. For that purpose a preparation > > > > > patch was added, which extract existing reservation code > > > > > into a new subroutine. More details can be found in > > > > > the changelog below and the commit messages. > > > > > > > > > > Patches are available in the github: > > > > > > > > > https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commits/dram-upstream-r20190128 > > > > > > > > > > I'm looking forward to the comments and remarks. > > > > > > > > > > Best regards, > > > > > Marcin > > > > > > > > > > Changelog: > > > > > v2 -> v3 > > > > > * 1/5 > > > > > - New patch - extract memory reservation to a separate routine > > > > > > > > > > * 2/2 > > > > > - Add new PCDs and reserve 3 regions (ARM-TF, PEI stack, OP-TEE) > > > > > separately > > > > > - Update commit message accordingly > > > > > > > > > > * 3/5 > > > > > - Move MvSmc.h to Include/IndustryStandard > > > > > > > > > > * 4,5/5 > > > > > - Add Leif's RB > > > > > > > > > > v1 -> v2: > > > > > * 1/4 > > > > > - Improve commit log - mention single area size and new PEI stack base > > > > > > > > > > * 2/4 (new patch) > > > > > - Add common header for Marvell SMC ID's > > > > > > > > > > * 3/4 > > > > > - Add function description comment > > > > > - Define and use ARMADA7K8K_AP806_INDEX > > > > > - Change function argument to EFI_PHYSICAL_ADDRESS > > > > > > > > > > * 4/4 > > > > > - Move new SMC ID to MvSmc.h > > > > > - Include ArmadaSoCDescLib.h directly (instead indirectly via > > > > BoardDesc.h) > > > > > - Remove ARMADA7K8K_AP806_INDEX macro > > > > > > > > > > Grzegorz Jaszczyk (2): > > > > > Marvell/Library: ArmadaSoCDescLib: Add North Bridge description > > > > > Marvell/Armada7k8k: Read DRAM settings from ARM-TF > > > > > > > > > > Marcin Wojtas (3): > > > > > Marvell/Armada7k8k: Refactor reserving memory regions > > > > > Marvell/Armada7k8k: Shift PEI stack base and extend memory reservation > > > > > Marvell/Library: Introduce common header for the SMC ID's > > > > > > > > > > Silicon/Marvell/Marvell.dec > > > > | 8 +- > > > > > Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc > > > > | 16 ++- > > > > > Silicon/Marvell/Armada7k8k/Library/Armada7k8kLib/Armada7k8kLib.inf > > > > | 3 + > > > > > > > > > Silicon/Marvell/Armada7k8k/Library/Armada7k8kMemoryInitPeiLib/Armada7k8kMemoryInitPeiLib.inf > > > > | 8 +- > > > > > Silicon/Marvell/Armada7k8k/Library/Armada7k8kLib/Armada7k8kLibMem.h > > > > | 25 ----- > > > > > > > > > Silicon/Marvell/Armada7k8k/Library/Armada7k8kSoCDescLib/Armada7k8kSoCDescLib.h > > > > | 6 ++ > > > > > Silicon/Marvell/Include/IndustryStandard/MvSmc.h > > > > | 24 +++++ > > > > > Silicon/Marvell/Include/Library/ArmadaSoCDescLib.h > > > > | 28 +++++ > > > > > Silicon/Marvell/Library/ComPhyLib/ComPhySipSvc.h > > > > | 8 +- > > > > > Silicon/Marvell/Armada7k8k/Library/Armada7k8kLib/Armada7k8kLibMem.c > > > > | 60 ++++------- > > > > > > > > > Silicon/Marvell/Armada7k8k/Library/Armada7k8kMemoryInitPeiLib/Armada7k8kMemoryInitPeiLib.c > > > > | 107 +++++++++++++------- > > > > > > > > > Silicon/Marvell/Armada7k8k/Library/Armada7k8kSoCDescLib/Armada7k8kSoCDescLib.c > > > > | 34 +++++++ > > > > > Silicon/Marvell/Library/ComPhyLib/ComPhyCp110.c > > > > | 14 +-- > > > > > 13 files changed, 220 insertions(+), 121 deletions(-) > > > > > create mode 100644 Silicon/Marvell/Include/IndustryStandard/MvSmc.h > > > > > > > > > > -- > > > > > 2.7.4 > > > > > > > > >