From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH v8 04/14] arch/arm: let map_mmio_regions() use start and count Date: Sun, 25 May 2014 16:56:34 +0100 Message-ID: <538212B2.5040004@linaro.org> References: <1401015115-7610-1-git-send-email-avanzini.arianna@gmail.com> <1401015115-7610-5-git-send-email-avanzini.arianna@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1401015115-7610-5-git-send-email-avanzini.arianna@gmail.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Arianna Avanzini , xen-devel@lists.xen.org Cc: julien.grall@citrix.com, paolo.valente@unimore.it, keir@xen.org, stefano.stabellini@eu.citrix.com, tim@xen.org, dario.faggioli@citrix.com, Ian.Jackson@eu.citrix.com, Ian.Campbell@eu.citrix.com, etrudeau@broadcom.com, JBeulich@suse.com, andrew.cooper3@citrix.com, viktor.kleinik@globallogic.com List-Id: xen-devel@lists.xenproject.org Hi Arianna, On 25/05/14 11:51, Arianna Avanzini wrote: > Currently, the arguments given to the function map_mmio_regions() to > describe the memory range to be mapped are the start and end page frame > numbers of the range to be mapped. However, this could give rise to > issues due to the range being inclusive or exclusive of the end gfn > given as parameter. This commit changes the interface of the function > to accept the start gfn and the number of gfns to be mapped. > This commit also changes the interface of the function map_one_mmio(), > helper for the xgene-storm platform, which is a wrapper for the function > map_mmio_regions() and does not need its arguments to be paddr_t. > > NOTE: platform-specific code has not been tested. FYI, I'm able to run Xen with your serie on the Arndale (an exynos5 platform). So I believe there is not issue on this platform :). Acked-by: Julien Grall Regards, -- Julien Grall