Re: [PATCH v2 06/15] xen: arm64: Map xgene PCI memory regions and interrupts to dom0.

[Julien Grall <julien.grall@linaro.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 <julien.grall@linaro.org>

> 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