From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH 1/5] xen: arm: propagate gic's #address-cells property to dom0. Date: Thu, 30 Oct 2014 10:31:24 +0000 Message-ID: <5452137C.5070100@linaro.org> References: <1414144694.15687.31.camel@citrix.com> <1414144717-32328-1-git-send-email-ian.campbell@citrix.com> <54513A08.3000908@linaro.org> <1414663586.2064.20.camel@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1414663586.2064.20.camel@citrix.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 Sawargaonkar , Clark Laughlin , stefano.stabellini@eu.citrix.com, tim@xen.org, xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On 30/10/2014 10:06, 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? > > rc1 is long gone... I'll push for rc2 though. Hmm right, I though the first version was rc0 not rc1. -- Julien Grall