Re: [Xen-devel] [PATCH 06/45] xen: arm64: initial build + config changes, start of day code

[Tim Deegan <tim@xxxxxxx>]

At 13:20 +0000 on 07 Feb (1360243206), Ian Campbell wrote:
> On Thu, 2013-02-07 at 13:12 +0000, Tim Deegan wrote:
> > At 15:56 +0000 on 23 Jan (1358956572), Ian Campbell wrote:
> > > diff --git a/config/arm64.mk b/config/arm64.mk
> > > new file mode 100644
> > > index 0000000..b2457eb
> > > --- /dev/null
> > > +++ b/config/arm64.mk
> > > @@ -0,0 +1,12 @@
> > > +CONFIG_ARM := y
> > > +CONFIG_ARM_64 := y
> > > +CONFIG_ARM_$(XEN_OS) := y
> > > +
> > > +CFLAGS += #-marm -march= -mcpu= etc
> > > +
> > > +HAS_PL011 := y
> > > +
> > > +# Use only if calling $(LD) directly.
> > > +LDFLAGS_DIRECT += -maarch64elf
> > > +
> > > +CONFIG_LOAD_ADDRESS ?= 0x80000000
> > > diff --git a/xen/arch/arm/Rules.mk b/xen/arch/arm/Rules.mk
> > > index f83bfee..2250366 100644
> > > --- a/xen/arch/arm/Rules.mk
> > > +++ b/xen/arch/arm/Rules.mk
> > > @@ -25,6 +25,12 @@ arm32 := y
> > >  arm64 := n
> > >  endif
> > >  
> > > +ifeq ($(TARGET_SUBARCH),arm64)
> > > +CFLAGS += -mcpu=generic
> > 
> > Should this go in config/arm64.mk?  I realise the equivalent flags for
> > arm32 are in this file, just wondering if we ought to tidy them all into
> > config/.
> 
> The x86 ones are in xen/arch/x86/Rules.mk too, I'm not averse to fixing
> that though (perhaps as a subsequent cleanup though?)

OK; no need to disturb this patch for it. 

Cheers,

Tim.

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