From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chen Baozi Subject: Re: [PATCH V2] xen/arm64: Use virtual address when setting up early_printk fixmap Date: Fri, 17 Apr 2015 19:39:49 +0800 Message-ID: <20150417113949.GA23710@cbz-thinkpad> References: <1428405884-19431-1-git-send-email-baozich@gmail.com> <552511FA.5080800@citrix.com> <1429108904.15516.307.camel@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1429108904.15516.307.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: Julien Grall , julien.grall@linaro.org, xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On Wed, Apr 15, 2015 at 03:41:44PM +0100, Ian Campbell wrote: > On Wed, 2015-04-08 at 12:33 +0100, Julien Grall wrote: > > Hi Chen, > > > > On 07/04/15 12:24, Chen Baozi wrote: > > > We have already had the boot pagetable when reaching the point > > > > s/had/added/ ? > > I think "switched too" is what was meant and more correct. > So do I ;-) > > > of early_printk fixmap setup. Thus there is no longer necessary > > > > s/there is/it is/ ? > > Yes. > > > > to calculate physical address of xen_fixmap. > > > > > > Signed-off-by: Chen Baozi > > > > Other than the possible typoes above: > > > > Reviewed-by: Julien Grall > > Acked-by: Ian Campbell > > If it is confirmed that I got the correct meaning in the first one I can > fix both on commit. Yes, please. Thanks. Baozi