linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] irqchip: Simplify the TI_PRUSS_INTC Kconfig
@ 2021-01-08 16:29 Suman Anna
  2021-01-08 23:34 ` David Lechner
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Suman Anna @ 2021-01-08 16:29 UTC (permalink / raw)
  To: Marc Zyngier, Thomas Gleixner
  Cc: Santosh Shilimkar, Tony Lindgren, Lokesh Vutla, David Lechner,
	linux-kernel, linux-omap, linux-arm-kernel, Grzegorz Jaszczyk,
	Suman Anna

The TI PRUSS INTC irqchip driver handles the local interrupt controller
which is a child device of it's parent PRUSS/ICSSG device. The driver
was upstreamed in parallel with the PRUSS platform driver, and was
configurable independently previously. The PRUSS interrupt controller
is an integral part of the overall PRUSS software architecture, and is
not useful at all by itself.

Simplify the TI_PRUSS_INTC Kconfig dependencies by making it silent and
selected automatically when the TI_PRUSS platform driver is enabled.

Signed-off-by: Suman Anna <s-anna@ti.com>
---
 drivers/irqchip/Kconfig | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/irqchip/Kconfig b/drivers/irqchip/Kconfig
index 94920a51c628..b147f22a78f4 100644
--- a/drivers/irqchip/Kconfig
+++ b/drivers/irqchip/Kconfig
@@ -493,8 +493,9 @@ config TI_SCI_INTA_IRQCHIP
 	  TI System Controller, say Y here. Otherwise, say N.
 
 config TI_PRUSS_INTC
-	tristate "TI PRU-ICSS Interrupt Controller"
-	depends on ARCH_DAVINCI || SOC_AM33XX || SOC_AM43XX || SOC_DRA7XX || ARCH_KEYSTONE || ARCH_K3
+	tristate
+	depends on TI_PRUSS
+	default TI_PRUSS
 	select IRQ_DOMAIN
 	help
 	  This enables support for the PRU-ICSS Local Interrupt Controller
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 4+ messages in thread
* Re: [PATCH] irqchip: Simplify the TI_PRUSS_INTC Kconfig
  2021-01-08 16:29 [PATCH] irqchip: Simplify the TI_PRUSS_INTC Kconfig Suman Anna
@ 2021-01-08 23:34 ` David Lechner
  2021-01-10 10:24 ` Marc Zyngier
  2021-01-10 10:29 ` [irqchip: irq/irqchip-next] irqchip/pruss: " irqchip-bot for Suman Anna
  2 siblings, 0 replies; 4+ messages in thread
From: David Lechner @ 2021-01-08 23:34 UTC (permalink / raw)
  To: Suman Anna, Marc Zyngier, Thomas Gleixner
  Cc: Santosh Shilimkar, Tony Lindgren, Lokesh Vutla, linux-kernel,
	linux-omap, linux-arm-kernel, Grzegorz Jaszczyk

On 1/8/21 10:29 AM, Suman Anna wrote:
> The TI PRUSS INTC irqchip driver handles the local interrupt controller
> which is a child device of it's parent PRUSS/ICSSG device. The driver
> was upstreamed in parallel with the PRUSS platform driver, and was
> configurable independently previously. The PRUSS interrupt controller
> is an integral part of the overall PRUSS software architecture, and is
> not useful at all by itself.
> 
> Simplify the TI_PRUSS_INTC Kconfig dependencies by making it silent and
> selected automatically when the TI_PRUSS platform driver is enabled.
> 
> Signed-off-by: Suman Anna <s-anna@ti.com>
> ---

Reviewed-by: David Lechner <david@lechnology.com>


^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [PATCH] irqchip: Simplify the TI_PRUSS_INTC Kconfig
  2021-01-08 16:29 [PATCH] irqchip: Simplify the TI_PRUSS_INTC Kconfig Suman Anna
  2021-01-08 23:34 ` David Lechner
@ 2021-01-10 10:24 ` Marc Zyngier
  2021-01-10 10:29 ` [irqchip: irq/irqchip-next] irqchip/pruss: " irqchip-bot for Suman Anna
  2 siblings, 0 replies; 4+ messages in thread
From: Marc Zyngier @ 2021-01-10 10:24 UTC (permalink / raw)
  To: Mathias Kresin, linux-mips, linux-kernel, Thomas Gleixner,
	Thomas Bogendoerfer, Serge Semin, Suman Anna
  Cc: Hauke Mehrtens, Martin Blumenstingl, stable, linux-arm-kernel,
	Grzegorz Jaszczyk, Lokesh Vutla, linux-omap, Tony Lindgren,
	Santosh Shilimkar, David Lechner

On Fri, 8 Jan 2021 10:29:01 -0600, Suman Anna wrote:
> The TI PRUSS INTC irqchip driver handles the local interrupt controller
> which is a child device of it's parent PRUSS/ICSSG device. The driver
> was upstreamed in parallel with the PRUSS platform driver, and was
> configurable independently previously. The PRUSS interrupt controller
> is an integral part of the overall PRUSS software architecture, and is
> not useful at all by itself.
> 
> [...]

Applied to irq/irqchip-next, thanks!

[1/1] irqchip: Simplify the TI_PRUSS_INTC Kconfig
      commit: b8e594fa20d2e33d40c7a8c7c106549a35c38972

Cheers,

	M.
-- 
Without deviation from the norm, progress is not possible.



^ permalink raw reply	[flat|nested] 4+ messages in thread
* [irqchip: irq/irqchip-next] irqchip/pruss: Simplify the TI_PRUSS_INTC Kconfig
  2021-01-08 16:29 [PATCH] irqchip: Simplify the TI_PRUSS_INTC Kconfig Suman Anna
  2021-01-08 23:34 ` David Lechner
  2021-01-10 10:24 ` Marc Zyngier
@ 2021-01-10 10:29 ` irqchip-bot for Suman Anna
  2 siblings, 0 replies; 4+ messages in thread
From: irqchip-bot for Suman Anna @ 2021-01-10 10:29 UTC (permalink / raw)
  To: linux-kernel; +Cc: Suman Anna, David Lechner, Marc Zyngier, tglx

The following commit has been merged into the irq/irqchip-next branch of irqchip:

Commit-ID:     b8e594fa20d2e33d40c7a8c7c106549a35c38972
Gitweb:        https://git.kernel.org/pub/scm/linux/kernel/git/maz/arm-platforms/b8e594fa20d2e33d40c7a8c7c106549a35c38972
Author:        Suman Anna <s-anna@ti.com>
AuthorDate:    Fri, 08 Jan 2021 10:29:01 -06:00
Committer:     Marc Zyngier <maz@kernel.org>
CommitterDate: Sun, 10 Jan 2021 10:19:14 

irqchip/pruss: Simplify the TI_PRUSS_INTC Kconfig

The TI PRUSS INTC irqchip driver handles the local interrupt controller
which is a child device of it's parent PRUSS/ICSSG device. The driver
was upstreamed in parallel with the PRUSS platform driver, and was
configurable independently previously. The PRUSS interrupt controller
is an integral part of the overall PRUSS software architecture, and is
not useful at all by itself.

Simplify the TI_PRUSS_INTC Kconfig dependencies by making it silent and
selected automatically when the TI_PRUSS platform driver is enabled.

Signed-off-by: Suman Anna <s-anna@ti.com>
Reviewed-by: David Lechner <david@lechnology.com>
Signed-off-by: Marc Zyngier <maz@kernel.org>
Link: https://lore.kernel.org/r/20210108162901.6003-1-s-anna@ti.com
---
 drivers/irqchip/Kconfig | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/irqchip/Kconfig b/drivers/irqchip/Kconfig
index 94920a5..b147f22 100644
--- a/drivers/irqchip/Kconfig
+++ b/drivers/irqchip/Kconfig
@@ -493,8 +493,9 @@ config TI_SCI_INTA_IRQCHIP
 	  TI System Controller, say Y here. Otherwise, say N.
 
 config TI_PRUSS_INTC
-	tristate "TI PRU-ICSS Interrupt Controller"
-	depends on ARCH_DAVINCI || SOC_AM33XX || SOC_AM43XX || SOC_DRA7XX || ARCH_KEYSTONE || ARCH_K3
+	tristate
+	depends on TI_PRUSS
+	default TI_PRUSS
 	select IRQ_DOMAIN
 	help
 	  This enables support for the PRU-ICSS Local Interrupt Controller

^ permalink raw reply related	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-01-10 10:30 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-08 16:29 [PATCH] irqchip: Simplify the TI_PRUSS_INTC Kconfig Suman Anna
2021-01-08 23:34 ` David Lechner
2021-01-10 10:24 ` Marc Zyngier
2021-01-10 10:29 ` [irqchip: irq/irqchip-next] irqchip/pruss: " irqchip-bot for Suman Anna

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).