linux-kernel - Re: [PATCH v4 7/7] x86: add CONFIG_ARM

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>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <50C925F5.7030703@zytor.com>
Date:	Wed, 12 Dec 2012 16:48:53 -0800
From:	"H. Peter Anvin" <hpa@...or.com>
To:	Davide Ciminaghi <ciminaghi@...dd.com>
CC:	jaswinder.singh@...aro.org, will.deacon@....com,
	linux@....linux.org.uk, tglx@...utronix.de, mingo@...hat.com,
	djbw@...com, vinod.koul@...el.com, grant.likely@...retlab.ca,
	linus.walleij@...aro.org, rubini@...dd.com, wim@...ana.be,
	cjb@...top.org, davidb@...eaurora.org, nico@...xnic.net,
	ben-linux@...ff.org, viresh.linux@...il.com,
	rajeev-dlh.kumar@...com, linux-kernel@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org, giancarlo.asnaghi@...com
Subject: Re: [PATCH v4 7/7] x86: add CONFIG_ARM_AMBA, selected by STA2X11

On 12/10/2012 05:42 AM, Davide Ciminaghi wrote:
> From: Alessandro Rubini <rubini@...dd.com>
>
> The sta2x11 I/O Hub is a bridge from PCIe to AMBA. It reuses a number
> of amba drivers and needs to activate core bus support.
>
> Signed-off-by: Alessandro Rubini <rubini@...dd.com>
> Acked-by: Giancarlo Asnaghi <giancarlo.asnaghi@...com>
> ---
>   arch/x86/Kconfig |    4 ++++
>   1 file changed, 4 insertions(+)
>
> diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
> index 5f67642..0659e7b 100644
> --- a/arch/x86/Kconfig
> +++ b/arch/x86/Kconfig
> @@ -521,6 +521,7 @@ config STA2X11
>   	select SWIOTLB
>   	select MFD_STA2X11
>   	select ARCH_REQUIRE_GPIOLIB
> +	select ARM_AMBA
>   	default n
>   	---help---
>   	  This adds support for boards based on the STA2X11 IO-Hub,
> @@ -2237,6 +2238,9 @@ config GEOS
>
>   endif # X86_32
>
> +config ARM_AMBA
> +        bool
> +
>   config AMD_NB
>   	def_bool y
>   	depends on CPU_SUP_AMD && PCI
>

I would be happier if instead we made this match PCI and made this a 
depends on, but I'm not sure if it is worth holding up the patchset over.

	-hpa

-- 
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel.  I don't speak on their behalf.

--
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/