From: Marc Zyngier <maz@kernel.org>
To: Lubomir Rintel <lkundrak@v3.sk>, Olof Johansson <olof@lixom.net>
Cc: Rob Herring <robh+dt@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
Thomas Gleixner <tglx@linutronix.de>,
Jason Cooper <jason@lakedaemon.net>,
Kishon Vijay Abraham I <kishon@ti.com>,
Russell King <linux@armlinux.org.uk>,
Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@kernel.org>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, linux-clk@vger.kernel.org,
Andres Salomon <dilinger@queued.net>
Subject: Re: [PATCH 07/19] irqchip/mmp: mask off interrupts from other cores
Date: Fri, 9 Aug 2019 13:18:24 +0100 [thread overview]
Message-ID: <19a21c54-93ac-19dc-d679-8d376d44e68c@kernel.org> (raw)
In-Reply-To: <20190809093158.7969-8-lkundrak@v3.sk>
On 09/08/2019 10:31, Lubomir Rintel wrote:
> From: Andres Salomon <dilinger@queued.net>
>
> On mmp3, there's an extra set of ICU registers (ICU2) that handle
> interrupts on the extra cores. When masking off interrupts on MP1,
> these should be masked as well.
>
> We add a new interrupt controller via device tree to identify when we're
> looking at an mmp3 machine via compatible field of "marvell,mmp3-intc".
>
> [lkundrak@v3.sk: Changed "mrvl,mmp3-intc" compatible strings to
> "marvell,mmp3-intc". Tidied up the subject line a bit.]
>
> Signed-off-by: Andres Salomon <dilinger@queued.net>
> Signed-off-by: Lubomir Rintel <lkundrak@v3.sk>
>
> ---
> arch/arm/mach-mmp/regs-icu.h | 3 +++
> drivers/irqchip/irq-mmp.c | 51 ++++++++++++++++++++++++++++++++++++
> 2 files changed, 54 insertions(+)
>
> diff --git a/arch/arm/mach-mmp/regs-icu.h b/arch/arm/mach-mmp/regs-icu.h
> index 0375d5a7fcb2b..410743d2b4020 100644
> --- a/arch/arm/mach-mmp/regs-icu.h
> +++ b/arch/arm/mach-mmp/regs-icu.h
> @@ -11,6 +11,9 @@
> #define ICU_VIRT_BASE (AXI_VIRT_BASE + 0x82000)
> #define ICU_REG(x) (ICU_VIRT_BASE + (x))
>
> +#define ICU2_VIRT_BASE (AXI_VIRT_BASE + 0x84000)
> +#define ICU2_REG(x) (ICU2_VIRT_BASE + (x))
> +
> #define ICU_INT_CONF(n) ICU_REG((n) << 2)
> #define ICU_INT_CONF_MASK (0xf)
>
> diff --git a/drivers/irqchip/irq-mmp.c b/drivers/irqchip/irq-mmp.c
> index cd8d2253f56d1..25497c75cc861 100644
> --- a/drivers/irqchip/irq-mmp.c
> +++ b/drivers/irqchip/irq-mmp.c
> @@ -44,6 +44,7 @@ struct icu_chip_data {
> unsigned int conf_enable;
> unsigned int conf_disable;
> unsigned int conf_mask;
> + unsigned int conf2_mask;
> unsigned int clr_mfp_irq_base;
> unsigned int clr_mfp_hwirq;
> struct irq_domain *domain;
> @@ -53,9 +54,11 @@ struct mmp_intc_conf {
> unsigned int conf_enable;
> unsigned int conf_disable;
> unsigned int conf_mask;
> + unsigned int conf2_mask;
> };
>
> static void __iomem *mmp_icu_base;
> +static void __iomem *mmp_icu2_base;
> static struct icu_chip_data icu_data[MAX_ICU_NR];
> static int max_icu_nr;
>
> @@ -98,6 +101,16 @@ static void icu_mask_irq(struct irq_data *d)
> r &= ~data->conf_mask;
> r |= data->conf_disable;
> writel_relaxed(r, mmp_icu_base + (hwirq << 2));
> +
> + if (data->conf2_mask) {
> + /*
> + * ICU1 (above) only controls PJ4 MP1; if using SMP,
> + * we need to also mask the MP2 and MM cores via ICU2.
> + */
> + r = readl_relaxed(mmp_icu2_base + (hwirq << 2));
> + r &= ~data->conf2_mask;
> + writel_relaxed(r, mmp_icu2_base + (hwirq << 2));
> + }
> } else {
> r = readl_relaxed(data->reg_mask) | (1 << hwirq);
> writel_relaxed(r, data->reg_mask);
> @@ -201,6 +214,14 @@ static const struct mmp_intc_conf mmp2_conf = {
> MMP2_ICU_INT_ROUTE_PJ4_FIQ,
> };
>
> +static struct mmp_intc_conf mmp3_conf = {
> + .conf_enable = 0x20,
> + .conf_disable = 0x0,
> + .conf_mask = MMP2_ICU_INT_ROUTE_PJ4_IRQ |
> + MMP2_ICU_INT_ROUTE_PJ4_FIQ,
> + .conf2_mask = 0xf0,
> +};
> +
> static void __exception_irq_entry mmp_handle_irq(struct pt_regs *regs)
> {
> int hwirq;
> @@ -364,6 +385,14 @@ static int __init mmp_init_bases(struct device_node *node)
> pr_err("Failed to get interrupt controller register\n");
> return -ENOMEM;
> }
> + if (of_device_is_compatible(node, "marvell,mmp3-intc")) {
Instead of harcoding the compatible property once more, why don't you
simply pass a flag from mmpx_of_init()?
> + mmp_icu2_base = of_iomap(node, 1);
> + if (!mmp_icu2_base) {
> + pr_err("Failed to get interrupt controller register #2\n");
> + iounmap(mmp_icu_base);
> + return -ENOMEM;
> + }
> + }
>
> icu_data[0].virq_base = 0;
> icu_data[0].domain = irq_domain_add_linear(node, nr_irqs,
> @@ -386,6 +415,8 @@ static int __init mmp_init_bases(struct device_node *node)
> irq_dispose_mapping(icu_data[0].virq_base + i);
> }
> irq_domain_remove(icu_data[0].domain);
> + if (of_device_is_compatible(node, "marvell,mmp3-intc"))
> + iounmap(mmp_icu2_base);
> iounmap(mmp_icu_base);
> return -EINVAL;
> }
> @@ -428,6 +459,26 @@ static int __init mmp2_of_init(struct device_node *node,
> }
> IRQCHIP_DECLARE(mmp2_intc, "mrvl,mmp2-intc", mmp2_of_init);
>
> +static int __init mmp3_of_init(struct device_node *node,
> + struct device_node *parent)
> +{
> + int ret;
> +
> + ret = mmp_init_bases(node);
> + if (ret < 0)
> + return ret;
> +
> + icu_data[0].conf_enable = mmp3_conf.conf_enable;
> + icu_data[0].conf_disable = mmp3_conf.conf_disable;
> + icu_data[0].conf_mask = mmp3_conf.conf_mask;
> + icu_data[0].conf2_mask = mmp3_conf.conf2_mask;
> + irq_set_default_host(icu_data[0].domain);
Why do you need this? On a fully DT-ified platform, there should be no
notion of a default domain.
> + set_handle_irq(mmp2_handle_irq);
> + max_icu_nr = 1;
> + return 0;
> +}
> +IRQCHIP_DECLARE(mmp3_intc, "marvell,mmp3-intc", mmp3_of_init);
> +
> static int __init mmp2_mux_of_init(struct device_node *node,
> struct device_node *parent)
> {
>
Thanks,
M.
--
Jazz is not dead, it just smells funny...
next prev parent reply other threads:[~2019-08-09 12:18 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-09 9:31 [PATCH 00/19] Initial support for Marvell MMP3 SoC Lubomir Rintel
2019-08-09 9:31 ` [PATCH 01/19] dt-bindings: arm: cpu: Add Marvell MMP3 SMP enable method Lubomir Rintel
2019-08-12 18:57 ` Rob Herring
2019-08-09 9:31 ` [PATCH 02/19] dt-bindings: arm: mrvl: Document MMP3 compatible string Lubomir Rintel
2019-08-21 21:03 ` Rob Herring
2019-08-22 8:12 ` Lubomir Rintel
2019-08-22 12:42 ` Rob Herring
2019-08-09 9:31 ` [PATCH 03/19] dt-bindings: mrvl,intc: Add a MMP3 interrupt controller Lubomir Rintel
2019-08-21 21:11 ` Rob Herring
2019-08-09 9:31 ` [PATCH 04/19] dt-bindings: phy-mmp3-usb: Add bindings Lubomir Rintel
2019-08-21 21:13 ` Rob Herring
2019-08-09 9:31 ` [PATCH 05/19] irqchip/mmp: do not use of_address_to_resource() to get mux regs Lubomir Rintel
2019-08-09 12:12 ` Marc Zyngier
2019-08-16 18:41 ` Lubomir Rintel
2019-08-17 18:17 ` Marc Zyngier
2019-08-09 9:31 ` [PATCH 06/19] irqchip/mmp: add missing chained_irq_{enter,exit}() Lubomir Rintel
2019-08-09 10:56 ` Marc Zyngier
2019-08-09 9:31 ` [PATCH 07/19] irqchip/mmp: mask off interrupts from other cores Lubomir Rintel
2019-08-09 12:18 ` Marc Zyngier [this message]
2019-08-16 18:15 ` Lubomir Rintel
2019-08-09 9:31 ` [PATCH 08/19] irqchip/mmp: coexist with GIC root IRQ controller Lubomir Rintel
2019-08-09 9:31 ` [PATCH 09/19] ARM: l2c: add definition for FWA in PL310 aux register Lubomir Rintel
2019-08-09 9:31 ` [PATCH 10/19] ARM: mmp: don't select CACHE_TAUROS2 on all ARCH_MMP Lubomir Rintel
2019-08-09 9:31 ` [PATCH 11/19] ARM: mmp: map the PGU as well Lubomir Rintel
2019-08-09 9:31 ` [PATCH 12/19] ARM: mmp: DT: convert timer driver to use TIMER_OF_DECLARE Lubomir Rintel
2019-08-09 9:31 ` [PATCH 13/19] ARM: mmp: define MMP_CHIPID by the means of CIU_REG() Lubomir Rintel
2019-08-09 9:31 ` [PATCH 14/19] ARM: mmp: add support for MMP3 SoC Lubomir Rintel
2019-08-16 18:06 ` Stephen Boyd
2019-08-09 9:31 ` [PATCH 15/19] ARM: mmp: add SMP support Lubomir Rintel
2019-08-16 18:11 ` Florian Fainelli
2019-08-09 9:31 ` [PATCH 16/19] ARM: mmp: move cputype.h to include/linux/soc/ Lubomir Rintel
2019-08-09 9:31 ` [PATCH 17/19] ARM: mmp: remove MMP3 USB PHY registers from regs-usb.h Lubomir Rintel
2019-08-09 9:31 ` [PATCH 18/19] phy: phy-mmp3-usb: add a new driver Lubomir Rintel
2019-08-09 9:31 ` [PATCH 19/19] ARM: dts: mmp3: Add MMP3 SoC dts file Lubomir Rintel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=19a21c54-93ac-19dc-d679-8d376d44e68c@kernel.org \
--to=maz@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dilinger@queued.net \
--cc=jason@lakedaemon.net \
--cc=kishon@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=lkundrak@v3.sk \
--cc=mark.rutland@arm.com \
--cc=mturquette@baylibre.com \
--cc=olof@lixom.net \
--cc=robh+dt@kernel.org \
--cc=sboyd@kernel.org \
--cc=tglx@linutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).