linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] arm64: Add Broadcom Set Top Box Kconfig entry point
@ 2016-06-29 19:49 Markus Mayer
  2016-07-11 23:50 ` Florian Fainelli
  2016-07-12  9:06 ` Sudeep Holla
  0 siblings, 2 replies; 4+ messages in thread
From: Markus Mayer @ 2016-06-29 19:49 UTC (permalink / raw)
  To: linux-arm-kernel

From: Florian Fainelli <f.fainelli@gmail.com>

Add an ARCH_BRCMSTB Kconfig symbol which allows us not to update the
dependencies for all STB-related drivers. Select BRCMSTB_L2_IRQ and
GENERIC_IRQ_CHIP which are required for proper functioning.

Signed-off-by: Doug Berger <opendmb@gmail.com>
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: Markus Mayer <mmayer@broadcom.com>
---

 arch/arm64/Kconfig.platforms | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
index 7ef1d05..22b5c7f 100644
--- a/arch/arm64/Kconfig.platforms
+++ b/arch/arm64/Kconfig.platforms
@@ -29,6 +29,13 @@ config ARCH_BERLIN
 	help
 	  This enables support for Marvell Berlin SoC Family
 
+config ARCH_BRCMSTB
+	bool "Broadcom Set-Top-Box SoCs"
+	select BRCMSTB_L2_IRQ
+	select GENERIC_IRQ_CHIP
+	help
+	  This enables support for Broadcom's ARMv8 Set Top Box SoCs
+
 config ARCH_EXYNOS
 	bool "ARMv8 based Samsung Exynos SoC family"
 	select COMMON_CLK_SAMSUNG
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH] arm64: Add Broadcom Set Top Box Kconfig entry point
  2016-06-29 19:49 [PATCH] arm64: Add Broadcom Set Top Box Kconfig entry point Markus Mayer
@ 2016-07-11 23:50 ` Florian Fainelli
  2016-07-12  9:06 ` Sudeep Holla
  1 sibling, 0 replies; 4+ messages in thread
From: Florian Fainelli @ 2016-07-11 23:50 UTC (permalink / raw)
  To: linux-arm-kernel

On 06/29/2016 12:49 PM, Markus Mayer wrote:
> From: Florian Fainelli <f.fainelli@gmail.com>
> 
> Add an ARCH_BRCMSTB Kconfig symbol which allows us not to update the
> dependencies for all STB-related drivers. Select BRCMSTB_L2_IRQ and
> GENERIC_IRQ_CHIP which are required for proper functioning.
> 
> Signed-off-by: Doug Berger <opendmb@gmail.com>
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
> Signed-off-by: Markus Mayer <mmayer@broadcom.com>

Applied to soc-arm64/next thanks Markus!
-- 
Florian

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] arm64: Add Broadcom Set Top Box Kconfig entry point
  2016-06-29 19:49 [PATCH] arm64: Add Broadcom Set Top Box Kconfig entry point Markus Mayer
  2016-07-11 23:50 ` Florian Fainelli
@ 2016-07-12  9:06 ` Sudeep Holla
  2016-07-13 21:24   ` Florian Fainelli
  1 sibling, 1 reply; 4+ messages in thread
From: Sudeep Holla @ 2016-07-12  9:06 UTC (permalink / raw)
  To: linux-arm-kernel



On 29/06/16 20:49, Markus Mayer wrote:
> From: Florian Fainelli <f.fainelli@gmail.com>
>
> Add an ARCH_BRCMSTB Kconfig symbol which allows us not to update the
> dependencies for all STB-related drivers. Select BRCMSTB_L2_IRQ and
> GENERIC_IRQ_CHIP which are required for proper functioning.
>
> Signed-off-by: Doug Berger <opendmb@gmail.com>
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
> Signed-off-by: Markus Mayer <mmayer@broadcom.com>
> ---
>
>   arch/arm64/Kconfig.platforms | 7 +++++++
>   1 file changed, 7 insertions(+)
>
> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
> index 7ef1d05..22b5c7f 100644
> --- a/arch/arm64/Kconfig.platforms
> +++ b/arch/arm64/Kconfig.platforms
> @@ -29,6 +29,13 @@ config ARCH_BERLIN
>   	help
>   	  This enables support for Marvell Berlin SoC Family
>
> +config ARCH_BRCMSTB
> +	bool "Broadcom Set-Top-Box SoCs"
> +	select BRCMSTB_L2_IRQ

I see that BRCMSTB_L2_IRQ selects GENERIC_IRQ_CHIP already so the below
one is redundant, generally irqchip code should select GENERIC_IRQ_CHIP

> +	select GENERIC_IRQ_CHIP
> +	help
> +	  This enables support for Broadcom's ARMv8 Set Top Box SoCs
> +
>   config ARCH_EXYNOS
>   	bool "ARMv8 based Samsung Exynos SoC family"
>   	select COMMON_CLK_SAMSUNG
>

-- 
Regards,
Sudeep

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] arm64: Add Broadcom Set Top Box Kconfig entry point
  2016-07-12  9:06 ` Sudeep Holla
@ 2016-07-13 21:24   ` Florian Fainelli
  0 siblings, 0 replies; 4+ messages in thread
From: Florian Fainelli @ 2016-07-13 21:24 UTC (permalink / raw)
  To: linux-arm-kernel

On 07/12/2016 02:06 AM, Sudeep Holla wrote:
> 
> 
> On 29/06/16 20:49, Markus Mayer wrote:
>> From: Florian Fainelli <f.fainelli@gmail.com>
>>
>> Add an ARCH_BRCMSTB Kconfig symbol which allows us not to update the
>> dependencies for all STB-related drivers. Select BRCMSTB_L2_IRQ and
>> GENERIC_IRQ_CHIP which are required for proper functioning.
>>
>> Signed-off-by: Doug Berger <opendmb@gmail.com>
>> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
>> Signed-off-by: Markus Mayer <mmayer@broadcom.com>
>> ---
>>
>>   arch/arm64/Kconfig.platforms | 7 +++++++
>>   1 file changed, 7 insertions(+)
>>
>> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
>> index 7ef1d05..22b5c7f 100644
>> --- a/arch/arm64/Kconfig.platforms
>> +++ b/arch/arm64/Kconfig.platforms
>> @@ -29,6 +29,13 @@ config ARCH_BERLIN
>>       help
>>         This enables support for Marvell Berlin SoC Family
>>
>> +config ARCH_BRCMSTB
>> +    bool "Broadcom Set-Top-Box SoCs"
>> +    select BRCMSTB_L2_IRQ
> 
> I see that BRCMSTB_L2_IRQ selects GENERIC_IRQ_CHIP already so the below
> one is redundant, generally irqchip code should select GENERIC_IRQ_CHIP

Thanks, I applied a fix in place which removes the select
GENERIC_IRQ_CHIP since it is redundant.
-- 
Florian

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-07-13 21:24 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-06-29 19:49 [PATCH] arm64: Add Broadcom Set Top Box Kconfig entry point Markus Mayer
2016-07-11 23:50 ` Florian Fainelli
2016-07-12  9:06 ` Sudeep Holla
2016-07-13 21:24   ` Florian Fainelli

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).