RE: [PATCH 1/4] ARM: Xilinx: Adding Xilinx platform infrastructuresupport

[John Linn]

> -----Original Message-----
> From: Russell King - ARM Linux [mailto:linux@xxxxxxxxxxxxxxxx]
> Sent: Saturday, February 05, 2011 9:16 AM
> To: John Linn
> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; kernel@xxxxxxxxxxxxxxx;
catalin.marinas@xxxxxxx;
> glikely@xxxxxxxxxxxx
> Subject: Re: [PATCH 1/4] ARM: Xilinx: Adding Xilinx platform
infrastructuresupport
> 
> On Sat, Feb 05, 2011 at 09:08:41AM -0700, John Linn wrote:
> > +config ARCH_XILINX
> > +	bool "Xilinx ARM Cortex A9 Based FPGA"
> > +	select CPU_V7
> 
> Ok, so your platform only supports ARMv7, so you select this symbol.
> 

Yes.  I added linux-kernel list also as I had hosed it up.

> > diff --git a/arch/arm/mm/Kconfig b/arch/arm/mm/Kconfig
> > index 9d30c6f..cf0c6f6 100644
> > --- a/arch/arm/mm/Kconfig
> > +++ b/arch/arm/mm/Kconfig
> > @@ -415,7 +415,7 @@ config CPU_32v6K
> >
> >  # ARMv7
> >  config CPU_V7
> > -	bool "Support ARM V7 processor" if ARCH_INTEGRATOR ||
MACH_REALVIEW_EB || MACH_REALVIEW_PBX
> > +	bool "Support ARM V7 processor" if ARCH_INTEGRATOR ||
MACH_REALVIEW_EB || MACH_REALVIEW_PBX ||
> ARCH_XILINX
> 
> However, this makes the prompt visible to the user, and with the
'select'
> above, the only possible answer is 'y'.  You don't need to do this -
just
> remove this change.  That means "Support ARM V7 processor" won't be
offered
> to the user, but it will be set in the configuration.

Thanks Russell. Will do.

-- John



This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/