From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH 1/5] xen: arm: propagate gic's #address-cells property to dom0. Date: Wed, 5 Nov 2014 10:47:32 +0000 Message-ID: <1415184452.11486.80.camel@citrix.com> References: <1414144694.15687.31.camel@citrix.com> <1414144717-32328-1-git-send-email-ian.campbell@citrix.com> <54513A08.3000908@linaro.org> <1415096635.11486.15.camel@citrix.com> <20141104171144.GJ4510@laptop.dumpdata.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20141104171144.GJ4510@laptop.dumpdata.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: Konrad Rzeszutek Wilk Cc: stefano.stabellini@eu.citrix.com, Julien Grall , tim@xen.org, xen-devel@lists.xen.org, Clark Laughlin , Pranavkumar Sawargaonkar List-Id: xen-devel@lists.xenproject.org On Tue, 2014-11-04 at 12:11 -0500, Konrad Rzeszutek Wilk wrote: > On Tue, Nov 04, 2014 at 10:23:55AM +0000, Ian Campbell wrote: > > On Wed, 2014-10-29 at 19:03 +0000, Julien Grall wrote: > > > Hi Ian, > > > > > > On 24/10/2014 10:58, Ian Campbell wrote: > > > > The interrupt-map property requires that the interrupt-parent node > > > > must have both #address-cells and #interrupt-cells properties (see > > > > ePAPR 2.4.3.1). Therefore propagate the property if it is present. > > > > > > > > We must propagate (rather than invent our own value) since this value > > > > is used to size fields within other properties within the tree. > > > > > > > > ePAPR strictly speaking requires that the interrupt-parent node > > > > always has these properties. However reality has diverged from this > > > > and implementations will recursively search parents for #*-cells > > > > properties. Hence we only copy if it is present. > > > > > > > > Signed-off-by: Ian Campbell > > > > > > Reviewed-by: Julien Grall > > > > > > Without this patch I can't boot Xen on the Foundation model with GIC-v3. > > > Is it possible to push this patch for Xen 4.5 rc1? > > > > Konrad, I think this is needed for 4.5 since without it dom0 can fail to > > parse certain other constructs within the DT (in bits which we don't > > generate and can't easily/don't want to rewrite as we pass them > > through). > > Release-Acked-by: Konrad Rzeszutek Wilk Applied, thanks. Ian.