From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH v2 06/15] xen: arm64: Map xgene PCI memory regions and interrupts to dom0. Date: Fri, 22 Nov 2013 17:17:00 +0000 Message-ID: <528F918C.60703@linaro.org> References: <1385137474-31245-6-git-send-email-ian.campbell@citrix.com> <528F8D8B.2070808@linaro.org> <1385140429.25845.131.camel@kazak.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1385140429.25845.131.camel@kazak.uk.xensource.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: Ian Campbell Cc: pranavkumar@linaro.org, stefano.stabellini@eu.citrix.com, tim@xen.org, Anup Patel , xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On 11/22/2013 05:13 PM, Ian Campbell wrote: > On Fri, 2013-11-22 at 16:59 +0000, Julien Grall wrote: >> Can you add a comment to specify that for the future we plan to support >> PCI for all platforms ? (ie this patch a kind of "WORKAROUND" for now) > > I had intended to and plain forgot. > > Here is the incremental patch I just merged into this one: With this incremental patch: Acked-by: Julien Grall > diff --git a/xen/arch/arm/platforms/xgene-storm.c b/xen/arch/arm/platforms/xgene-storm.c > index 3846f86..1f506fe 100644 > --- a/xen/arch/arm/platforms/xgene-storm.c > +++ b/xen/arch/arm/platforms/xgene-storm.c > @@ -60,6 +60,12 @@ static int map_one_spi(struct domain *d, const char *what, > return ret; > } > > +/* > + * Xen does not currently support mapping MMIO regions and interrupt > + * for bus child devices (referenced via the "ranges" and > + * "interrupt-map" properties to domain 0). Instead for now map the > + * necessary resources manually. > + */ > static int xgene_storm_specific_mapping(struct domain *d) > { > int ret; > > That OK? > -- Julien Grall