From: Julien Grall <julien.grall@citrix.com>
To: Ian Campbell <ian.campbell@citrix.com>, xen-devel@lists.xen.org
Cc: julien.grall@linaro.org, tim@xen.org, stefano.stabellini@eu.citrix.com
Subject: Re: [PATCH v4 6/6] xen: arm: Import of_bus PCI entry from Linux (as a dt_bus entry)
Date: Fri, 26 Jun 2015 20:08:10 +0200	[thread overview]
Message-ID: <558D950A.7060101@citrix.com> (raw)
In-Reply-To: <1431084420-14372-6-git-send-email-ian.campbell@citrix.com>
Hi Ian,
On 08/05/2015 13:27, Ian Campbell wrote:
> This provides specific handlers for the PCI bus relating to matching
> and translating. It's mostly similar to the defaults but includes some
> additional error checks and other PCI specific bits.
I though the previous patch (#5) was enough to handle PCI. May I ask why 
we need it?
> Functions/types renamed and reindented (because apparently we do
> that for these).
My mistake, when I first port the functions from Linux I changed the 
name and the coding style to match the existent functions in 
xen/common/device_tree.c
All the functions to work directly on the DT blob have been moved in a 
separate file (arch/arm/bootfdt.c), so this file is mostly Linux code. 
I'm thinking to resync it correctly for Xen 4.7.
> Needs a selection of IORESOURCE_* defines, which I've taken from Linux
> and have included locally for now until we figure out where else they
> might be needed.
Regards,
-- 
Julien Grall
next prev parent reply	other threads:[~2015-06-26 18:08 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-08 11:26 [PATCH v3 0/6] xen: arm: Parse PCI DT nodes' ranges and interrupt-map Ian Campbell
2015-05-08 11:26 ` [PATCH v4 1/6] xen: dt: add dt_for_each_irq_map helper Ian Campbell
2015-06-26 17:47   ` Julien Grall
2015-07-03 14:16     ` Ian Campbell
2015-07-03 15:15       ` Julien Grall
2015-07-03 15:28         ` Ian Campbell
2015-05-08 11:26 ` [PATCH v4 2/6] xen: dt: add dt_for_each_range helper Ian Campbell
2015-05-08 11:26 ` [PATCH v4 3/6] xen: arm: slightly refactor gic DT node creation for domain 0 Ian Campbell
2015-05-08 16:21   ` Ian Campbell
2015-05-08 11:26 ` [PATCH v4 4/6] xen: arm: drop redundant extra call to vgic_reserve_virq Ian Campbell
2015-06-26 17:49   ` Julien Grall
2015-05-08 11:26 ` [PATCH v4 5/6] xen: arm: map child MMIO and IRQs to dom0 for PCI bus DT nodes Ian Campbell
2015-06-26 17:56   ` Julien Grall
2015-07-03 10:59     ` Ian Campbell
2015-07-03 11:26       ` Julien Grall
2015-05-08 11:27 ` [PATCH v4 6/6] xen: arm: Import of_bus PCI entry from Linux (as a dt_bus entry) Ian Campbell
2015-06-26 18:08   ` Julien Grall [this message]
2015-07-03 10:47     ` Ian Campbell
2015-07-03 10:56       ` Ian Campbell
2015-07-03 11:24         ` Julien Grall
2015-07-03 11:30           ` Ian Campbell
2015-05-08 11:41 ` [PATCH v3 0/6] xen: arm: Parse PCI DT nodes' ranges and interrupt-map Ian Campbell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox
  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):
  git send-email \
    --in-reply-to=558D950A.7060101@citrix.com \
    --to=julien.grall@citrix.com \
    --cc=ian.campbell@citrix.com \
    --cc=julien.grall@linaro.org \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=tim@xen.org \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY
  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
  Be sure your reply has a Subject: header at the top and a blank line
  before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).