From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH] xen: arm: enable sync console in machine_reboot. Date: Thu, 15 Jan 2015 13:00:49 +0000 Message-ID: <1421326849.5437.3.camel@citrix.com> References: <1421320947-17049-1-git-send-email-ian.campbell@citrix.com> <54B7B940.7000909@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <54B7B940.7000909@linaro.org> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Julien Grall Cc: stefano.stabellini@eu.citrix.com, tim@xen.org, xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On Thu, 2015-01-15 at 12:57 +0000, Julien Grall wrote: > Hi Ian, > > On 15/01/15 11:22, Ian Campbell wrote: > > Otherwise the last thing printed is "(XE" or something. > > > > In line with x86 also disable the watchdog and spin debugging. > > > > Signed-off-by: Ian Campbell > > Reviewed-by: Julien Grall > > > --- > > xen/arch/arm/shutdown.c | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/xen/arch/arm/shutdown.c b/xen/arch/arm/shutdown.c > > index 4988b03..e53ec97 100644 > > --- a/xen/arch/arm/shutdown.c > > +++ b/xen/arch/arm/shutdown.c > > @@ -38,6 +38,10 @@ void machine_restart(unsigned int delay_millisecs) > > { > > int timeout = 10; > > > > + watchdog_disable(); > > OOI, it looks like watchdog_disable is a no-op on ARM. Is there any plan > to implement it? Not by me, would be nice to have though. > > Regards, >