Message ID | 1436837955-26279-4-git-send-email-rjui@broadcom.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Mon, Jul 13, 2015 at 06:39:14PM -0700, Ray Jui wrote: > This patch adds support to Broadcom's North Star 2 SoC in the arm64 > Kconfig and defconfig files > > Signed-off-by: Ray Jui <rjui@broadcom.com> > Reviewed-by: Scott Branden <sbranden@broadcom.com> > --- > arch/arm64/Kconfig | 12 ++++++++++++ > arch/arm64/configs/defconfig | 2 ++ > 2 files changed, 14 insertions(+) > > diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig > index 318175f..6c5c279 100644 > --- a/arch/arm64/Kconfig > +++ b/arch/arm64/Kconfig > @@ -162,6 +162,18 @@ source "kernel/Kconfig.freezer" > > menu "Platform selection" > > +config ARCH_BCM_IPROC > + bool > + help > + This enables support for Broadcom iProc based SoCs > + > +config ARCH_BCM_NS2 > + bool "Broadcom North Start 2 (BCM5871X)" > + select ARCH_BCM_IPROC > + help > + This enables support for Broadcom NS2 (BCM5871X). NS2 is in > + the iProc family of SoCs based on the ARMv8 architecture Do you really need the second config option? Just leave the one covering the family as we do for most of the other family SoCs on arm64 (apart from ARCH_TEGRA_123_SOC which I plan to remove and use ARCH_TEGRA instead).
On 7/14/2015 2:14 AM, Catalin Marinas wrote: > On Mon, Jul 13, 2015 at 06:39:14PM -0700, Ray Jui wrote: >> This patch adds support to Broadcom's North Star 2 SoC in the arm64 >> Kconfig and defconfig files >> >> Signed-off-by: Ray Jui <rjui@broadcom.com> >> Reviewed-by: Scott Branden <sbranden@broadcom.com> >> --- >> arch/arm64/Kconfig | 12 ++++++++++++ >> arch/arm64/configs/defconfig | 2 ++ >> 2 files changed, 14 insertions(+) >> >> diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig >> index 318175f..6c5c279 100644 >> --- a/arch/arm64/Kconfig >> +++ b/arch/arm64/Kconfig >> @@ -162,6 +162,18 @@ source "kernel/Kconfig.freezer" >> >> menu "Platform selection" >> >> +config ARCH_BCM_IPROC >> + bool >> + help >> + This enables support for Broadcom iProc based SoCs >> + >> +config ARCH_BCM_NS2 >> + bool "Broadcom North Start 2 (BCM5871X)" >> + select ARCH_BCM_IPROC >> + help >> + This enables support for Broadcom NS2 (BCM5871X). NS2 is in >> + the iProc family of SoCs based on the ARMv8 architecture > > Do you really need the second config option? Just leave the one covering > the family as we do for most of the other family SoCs on arm64 (apart > from ARCH_TEGRA_123_SOC which I plan to remove and use ARCH_TEGRA > instead). > ARCH_BCM_NS2 helps in cases where a NS2 specific driver may just default to ARCH_BCM_NS2 in its Kconfig, so those NS2 drivers got turned on whenever ARCH_BCM_NS2 is turned on. If the plan is to allow only one ARCH flag per family of SoCs for ARMv8, I can go ahead and remove ARCH_BCM_NS2. Thanks, Ray
diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig index 318175f..6c5c279 100644 --- a/arch/arm64/Kconfig +++ b/arch/arm64/Kconfig @@ -162,6 +162,18 @@ source "kernel/Kconfig.freezer" menu "Platform selection" +config ARCH_BCM_IPROC + bool + help + This enables support for Broadcom iProc based SoCs + +config ARCH_BCM_NS2 + bool "Broadcom North Start 2 (BCM5871X)" + select ARCH_BCM_IPROC + help + This enables support for Broadcom NS2 (BCM5871X). NS2 is in + the iProc family of SoCs based on the ARMv8 architecture + config ARCH_EXYNOS bool help diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig index 4e17e7e..3138700 100644 --- a/arch/arm64/configs/defconfig +++ b/arch/arm64/configs/defconfig @@ -31,6 +31,7 @@ CONFIG_MODULES=y CONFIG_MODULE_UNLOAD=y # CONFIG_BLK_DEV_BSG is not set # CONFIG_IOSCHED_DEADLINE is not set +CONFIG_ARCH_BCM_NS2=y CONFIG_ARCH_EXYNOS7=y CONFIG_ARCH_FSL_LS2085A=y CONFIG_ARCH_HISI=y @@ -102,6 +103,7 @@ CONFIG_SERIO_AMBAKMI=y CONFIG_LEGACY_PTY_COUNT=16 CONFIG_SERIAL_8250=y CONFIG_SERIAL_8250_CONSOLE=y +CONFIG_SERIAL_8250_DW=y CONFIG_SERIAL_8250_MT6577=y CONFIG_SERIAL_AMBA_PL011=y CONFIG_SERIAL_AMBA_PL011_CONSOLE=y