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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Date:	Sat, 5 Feb 2011 09:20:12 -0700
From:	John Linn <John.Linn@...inx.com>
To:	Russell King - ARM Linux <linux@....linux.org.uk>
CC:	<linux-arm-kernel@...ts.infradead.org>, <kernel@...r.kernel.org>,
	<catalin.marinas@....com>, <glikely@...retlab.ca>,
	<linux-kernel@...r.kernel.org>
Subject: RE: [PATCH 1/4] ARM: Xilinx: Adding Xilinx platform infrastructuresupport

> -----Original Message-----
> From: Russell King - ARM Linux [mailto:linux@....linux.org.uk]
> Sent: Saturday, February 05, 2011 9:16 AM
> To: John Linn
> Cc: linux-arm-kernel@...ts.infradead.org; kernel@...r.kernel.org;
catalin.marinas@....com;
> glikely@...retlab.ca
> 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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/