Re: [Xen-devel] [RFC 23/29] xen/arm: Add versatile express platform

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Mon, 2013-04-29 at 18:52 +0100, Julien Grall wrote:
> On 04/29/2013 05:27 PM, Ian Campbell wrote:
> 
> > On Mon, 2013-04-29 at 00:02 +0100, Julien Grall wrote:
> >> This platform contains nearly nothing specific except the reset function.
> >>
> >> Signed-off-by: Julien Grall <julien.grall@xxxxxxxxxx>
> > 
> > Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
> > 
> >> +/* Board-specific: base address of system controller */
> >> +#define SP810_ADDRESS 0x1C020000
> > 
> > Even better would be if this is in DT.
> 
> It seems there is an *arm,vexpress-reset* node in the device tree.
> 
> It needs some work, so I will send a separate patch to add support for
> the vexpress-reset function.

That's fine. If you are respinning this patch you could add an "XXX get
from DT" marker.


Ian.


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