Re: [Xen-devel] [PATCH] ARM: add Calxeda Midway platform

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

On Wed, 2013-08-14 at 10:33 +0100, Julien Grall wrote:
> On 13 August 2013 23:02, Ian Campbell <ian.campbell@xxxxxxxxxx> wrote:
> > On Tue, 2013-08-13 at 17:13 +0200, Andre Przywara wrote:
> >> avoid a warning on boot and provide the proper reset method.
> >
> > I've been wondering about taming that warning down to something less
> > alarming like "Using generic platform fallback for platform
> > $some_dt_name", or something like that. There are likely (going to be)
> > many platforms where no specific code is really needed and the big
> > WARNING is needlessly scary IMHO.
> 
> All the platforms need at least the reset callback.

Doesn't PSCI eventually cover this?

> So, I think this WARNING should stay to let the developper know that
> something is missing.

A warning if the reset hook was NULL when Xen tries to call it would
serve much the same purpose without worrying people needlessly during
boot.

Ian



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel