From: robherring2@gmail.com (Rob Herring)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 07/10] irqchip: sunxi: Rename sunxi to sun4i
Date: Mon, 25 Mar 2013 09:00:09 -0500 [thread overview]
Message-ID: <51505869.8010203@gmail.com> (raw)
In-Reply-To: <1364218233-29636-8-git-send-email-maxime.ripard@free-electrons.com>
On 03/25/2013 08:30 AM, Maxime Ripard wrote:
> During the introduction of the Allwinner SoC platforms, sunxi was
> initially meant as a generic name for all the variants of the Allwinner
> SoC.
>
> It was ok at the time of the support of only the A10 and A13 that
> looks pretty much the same, but it's beginning to be troublesome with
> the future addition of the Allwinner A31 (sun6i) that is quite
> different, and would introduce some weird logic, where sunxi would
> actually mean in some case sun4i and sun5i but without sun6i...
>
> Moreover, it makes the compatible strings naming scheme not consistent
> with other architectures, where usually for this kind of compability, we
> just use the oldest SoC name that has this IP, so let's do just this.
>
> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
> ---
> drivers/irqchip/Makefile | 2 +-
> drivers/irqchip/irq-sun4i.c | 149 +++++++++++++++++++++++++++++++++++++++++++
> drivers/irqchip/irq-sunxi.c | 149 -------------------------------------------
> 3 files changed, 150 insertions(+), 150 deletions(-)
You need to use the -M option for git-format-patch when sending this patch.
Rob
> create mode 100644 drivers/irqchip/irq-sun4i.c
> delete mode 100644 drivers/irqchip/irq-sunxi.c
>
> diff --git a/drivers/irqchip/Makefile b/drivers/irqchip/Makefile
> index 98e3b87..5416965 100644
> --- a/drivers/irqchip/Makefile
> +++ b/drivers/irqchip/Makefile
> @@ -4,7 +4,7 @@ obj-$(CONFIG_ARCH_BCM2835) += irq-bcm2835.o
> obj-$(CONFIG_ARCH_EXYNOS) += exynos-combiner.o
> obj-$(CONFIG_METAG) += irq-metag-ext.o
> obj-$(CONFIG_METAG_PERFCOUNTER_IRQS) += irq-metag.o
> -obj-$(CONFIG_ARCH_SUNXI) += irq-sunxi.o
> +obj-$(CONFIG_ARCH_SUNXI) += irq-sun4i.o
> obj-$(CONFIG_ARCH_SPEAR3XX) += spear-shirq.o
> obj-$(CONFIG_ARM_GIC) += irq-gic.o
> obj-$(CONFIG_ARM_VIC) += irq-vic.o
> diff --git a/drivers/irqchip/irq-sun4i.c b/drivers/irqchip/irq-sun4i.c
> new file mode 100644
> index 0000000..b66d4ae
> --- /dev/null
> +++ b/drivers/irqchip/irq-sun4i.c
> @@ -0,0 +1,149 @@
> +/*
> + * Allwinner A1X SoCs IRQ chip driver.
> + *
> + * Copyright (C) 2012 Maxime Ripard
> + *
> + * Maxime Ripard <maxime.ripard@free-electrons.com>
> + *
> + * Based on code from
> + * Allwinner Technology Co., Ltd. <www.allwinnertech.com>
> + * Benn Huang <benn@allwinnertech.com>
> + *
> + * This file is licensed under the terms of the GNU General Public
> + * License version 2. This program is licensed "as is" without any
> + * warranty of any kind, whether express or implied.
> + */
> +
> +#include <linux/io.h>
> +#include <linux/irq.h>
> +#include <linux/of.h>
> +#include <linux/of_address.h>
> +#include <linux/of_irq.h>
> +
> +#include <asm/exception.h>
> +#include <asm/mach/irq.h>
> +
> +#include "irqchip.h"
> +
> +#define SUN4I_IRQ_VECTOR_REG 0x00
> +#define SUN4I_IRQ_PROTECTION_REG 0x08
> +#define SUN4I_IRQ_NMI_CTRL_REG 0x0c
> +#define SUN4I_IRQ_PENDING_REG(x) (0x10 + 0x4 * x)
> +#define SUN4I_IRQ_FIQ_PENDING_REG(x) (0x20 + 0x4 * x)
> +#define SUN4I_IRQ_ENABLE_REG(x) (0x40 + 0x4 * x)
> +#define SUN4I_IRQ_MASK_REG(x) (0x50 + 0x4 * x)
> +
> +static void __iomem *sun4i_irq_base;
> +static struct irq_domain *sun4i_irq_domain;
> +
> +static asmlinkage void __exception_irq_entry sun4i_handle_irq(struct pt_regs *regs);
> +
> +void sun4i_irq_ack(struct irq_data *irqd)
> +{
> + unsigned int irq = irqd_to_hwirq(irqd);
> + unsigned int irq_off = irq % 32;
> + int reg = irq / 32;
> + u32 val;
> +
> + val = readl(sun4i_irq_base + SUN4I_IRQ_PENDING_REG(reg));
> + writel(val | (1 << irq_off),
> + sun4i_irq_base + SUN4I_IRQ_PENDING_REG(reg));
> +}
> +
> +static void sun4i_irq_mask(struct irq_data *irqd)
> +{
> + unsigned int irq = irqd_to_hwirq(irqd);
> + unsigned int irq_off = irq % 32;
> + int reg = irq / 32;
> + u32 val;
> +
> + val = readl(sun4i_irq_base + SUN4I_IRQ_ENABLE_REG(reg));
> + writel(val & ~(1 << irq_off),
> + sun4i_irq_base + SUN4I_IRQ_ENABLE_REG(reg));
> +}
> +
> +static void sun4i_irq_unmask(struct irq_data *irqd)
> +{
> + unsigned int irq = irqd_to_hwirq(irqd);
> + unsigned int irq_off = irq % 32;
> + int reg = irq / 32;
> + u32 val;
> +
> + val = readl(sun4i_irq_base + SUN4I_IRQ_ENABLE_REG(reg));
> + writel(val | (1 << irq_off),
> + sun4i_irq_base + SUN4I_IRQ_ENABLE_REG(reg));
> +}
> +
> +static struct irq_chip sun4i_irq_chip = {
> + .name = "sun4i_irq",
> + .irq_ack = sun4i_irq_ack,
> + .irq_mask = sun4i_irq_mask,
> + .irq_unmask = sun4i_irq_unmask,
> +};
> +
> +static int sun4i_irq_map(struct irq_domain *d, unsigned int virq,
> + irq_hw_number_t hw)
> +{
> + irq_set_chip_and_handler(virq, &sun4i_irq_chip,
> + handle_level_irq);
> + set_irq_flags(virq, IRQF_VALID | IRQF_PROBE);
> +
> + return 0;
> +}
> +
> +static struct irq_domain_ops sun4i_irq_ops = {
> + .map = sun4i_irq_map,
> + .xlate = irq_domain_xlate_onecell,
> +};
> +
> +static int __init sun4i_of_init(struct device_node *node,
> + struct device_node *parent)
> +{
> + sun4i_irq_base = of_iomap(node, 0);
> + if (!sun4i_irq_base)
> + panic("%s: unable to map IC registers\n",
> + node->full_name);
> +
> + /* Disable all interrupts */
> + writel(0, sun4i_irq_base + SUN4I_IRQ_ENABLE_REG(0));
> + writel(0, sun4i_irq_base + SUN4I_IRQ_ENABLE_REG(1));
> + writel(0, sun4i_irq_base + SUN4I_IRQ_ENABLE_REG(2));
> +
> + /* Mask all the interrupts */
> + writel(0, sun4i_irq_base + SUN4I_IRQ_MASK_REG(0));
> + writel(0, sun4i_irq_base + SUN4I_IRQ_MASK_REG(1));
> + writel(0, sun4i_irq_base + SUN4I_IRQ_MASK_REG(2));
> +
> + /* Clear all the pending interrupts */
> + writel(0xffffffff, sun4i_irq_base + SUN4I_IRQ_PENDING_REG(0));
> + writel(0xffffffff, sun4i_irq_base + SUN4I_IRQ_PENDING_REG(1));
> + writel(0xffffffff, sun4i_irq_base + SUN4I_IRQ_PENDING_REG(2));
> +
> + /* Enable protection mode */
> + writel(0x01, sun4i_irq_base + SUN4I_IRQ_PROTECTION_REG);
> +
> + /* Configure the external interrupt source type */
> + writel(0x00, sun4i_irq_base + SUN4I_IRQ_NMI_CTRL_REG);
> +
> + sun4i_irq_domain = irq_domain_add_linear(node, 3 * 32,
> + &sun4i_irq_ops, NULL);
> + if (!sun4i_irq_domain)
> + panic("%s: unable to create IRQ domain\n", node->full_name);
> +
> + set_handle_irq(sun4i_handle_irq);
> +
> + return 0;
> +}
> +IRQCHIP_DECLARE(allwinner_sun4i_ic, "allwinner,sun4i-ic", sun4i_of_init);
> +
> +static asmlinkage void __exception_irq_entry sun4i_handle_irq(struct pt_regs *regs)
> +{
> + u32 irq, hwirq;
> +
> + hwirq = readl(sun4i_irq_base + SUN4I_IRQ_VECTOR_REG) >> 2;
> + while (hwirq != 0) {
> + irq = irq_find_mapping(sun4i_irq_domain, hwirq);
> + handle_IRQ(irq, regs);
> + hwirq = readl(sun4i_irq_base + SUN4I_IRQ_VECTOR_REG) >> 2;
> + }
> +}
> diff --git a/drivers/irqchip/irq-sunxi.c b/drivers/irqchip/irq-sunxi.c
> deleted file mode 100644
> index 0fc49c5..0000000
> --- a/drivers/irqchip/irq-sunxi.c
> +++ /dev/null
> @@ -1,149 +0,0 @@
> -/*
> - * Allwinner A1X SoCs IRQ chip driver.
> - *
> - * Copyright (C) 2012 Maxime Ripard
> - *
> - * Maxime Ripard <maxime.ripard@free-electrons.com>
> - *
> - * Based on code from
> - * Allwinner Technology Co., Ltd. <www.allwinnertech.com>
> - * Benn Huang <benn@allwinnertech.com>
> - *
> - * This file is licensed under the terms of the GNU General Public
> - * License version 2. This program is licensed "as is" without any
> - * warranty of any kind, whether express or implied.
> - */
> -
> -#include <linux/io.h>
> -#include <linux/irq.h>
> -#include <linux/of.h>
> -#include <linux/of_address.h>
> -#include <linux/of_irq.h>
> -
> -#include <asm/exception.h>
> -#include <asm/mach/irq.h>
> -
> -#include "irqchip.h"
> -
> -#define SUNXI_IRQ_VECTOR_REG 0x00
> -#define SUNXI_IRQ_PROTECTION_REG 0x08
> -#define SUNXI_IRQ_NMI_CTRL_REG 0x0c
> -#define SUNXI_IRQ_PENDING_REG(x) (0x10 + 0x4 * x)
> -#define SUNXI_IRQ_FIQ_PENDING_REG(x) (0x20 + 0x4 * x)
> -#define SUNXI_IRQ_ENABLE_REG(x) (0x40 + 0x4 * x)
> -#define SUNXI_IRQ_MASK_REG(x) (0x50 + 0x4 * x)
> -
> -static void __iomem *sunxi_irq_base;
> -static struct irq_domain *sunxi_irq_domain;
> -
> -static asmlinkage void __exception_irq_entry sunxi_handle_irq(struct pt_regs *regs);
> -
> -void sunxi_irq_ack(struct irq_data *irqd)
> -{
> - unsigned int irq = irqd_to_hwirq(irqd);
> - unsigned int irq_off = irq % 32;
> - int reg = irq / 32;
> - u32 val;
> -
> - val = readl(sunxi_irq_base + SUNXI_IRQ_PENDING_REG(reg));
> - writel(val | (1 << irq_off),
> - sunxi_irq_base + SUNXI_IRQ_PENDING_REG(reg));
> -}
> -
> -static void sunxi_irq_mask(struct irq_data *irqd)
> -{
> - unsigned int irq = irqd_to_hwirq(irqd);
> - unsigned int irq_off = irq % 32;
> - int reg = irq / 32;
> - u32 val;
> -
> - val = readl(sunxi_irq_base + SUNXI_IRQ_ENABLE_REG(reg));
> - writel(val & ~(1 << irq_off),
> - sunxi_irq_base + SUNXI_IRQ_ENABLE_REG(reg));
> -}
> -
> -static void sunxi_irq_unmask(struct irq_data *irqd)
> -{
> - unsigned int irq = irqd_to_hwirq(irqd);
> - unsigned int irq_off = irq % 32;
> - int reg = irq / 32;
> - u32 val;
> -
> - val = readl(sunxi_irq_base + SUNXI_IRQ_ENABLE_REG(reg));
> - writel(val | (1 << irq_off),
> - sunxi_irq_base + SUNXI_IRQ_ENABLE_REG(reg));
> -}
> -
> -static struct irq_chip sunxi_irq_chip = {
> - .name = "sunxi_irq",
> - .irq_ack = sunxi_irq_ack,
> - .irq_mask = sunxi_irq_mask,
> - .irq_unmask = sunxi_irq_unmask,
> -};
> -
> -static int sunxi_irq_map(struct irq_domain *d, unsigned int virq,
> - irq_hw_number_t hw)
> -{
> - irq_set_chip_and_handler(virq, &sunxi_irq_chip,
> - handle_level_irq);
> - set_irq_flags(virq, IRQF_VALID | IRQF_PROBE);
> -
> - return 0;
> -}
> -
> -static struct irq_domain_ops sunxi_irq_ops = {
> - .map = sunxi_irq_map,
> - .xlate = irq_domain_xlate_onecell,
> -};
> -
> -static int __init sunxi_of_init(struct device_node *node,
> - struct device_node *parent)
> -{
> - sunxi_irq_base = of_iomap(node, 0);
> - if (!sunxi_irq_base)
> - panic("%s: unable to map IC registers\n",
> - node->full_name);
> -
> - /* Disable all interrupts */
> - writel(0, sunxi_irq_base + SUNXI_IRQ_ENABLE_REG(0));
> - writel(0, sunxi_irq_base + SUNXI_IRQ_ENABLE_REG(1));
> - writel(0, sunxi_irq_base + SUNXI_IRQ_ENABLE_REG(2));
> -
> - /* Mask all the interrupts */
> - writel(0, sunxi_irq_base + SUNXI_IRQ_MASK_REG(0));
> - writel(0, sunxi_irq_base + SUNXI_IRQ_MASK_REG(1));
> - writel(0, sunxi_irq_base + SUNXI_IRQ_MASK_REG(2));
> -
> - /* Clear all the pending interrupts */
> - writel(0xffffffff, sunxi_irq_base + SUNXI_IRQ_PENDING_REG(0));
> - writel(0xffffffff, sunxi_irq_base + SUNXI_IRQ_PENDING_REG(1));
> - writel(0xffffffff, sunxi_irq_base + SUNXI_IRQ_PENDING_REG(2));
> -
> - /* Enable protection mode */
> - writel(0x01, sunxi_irq_base + SUNXI_IRQ_PROTECTION_REG);
> -
> - /* Configure the external interrupt source type */
> - writel(0x00, sunxi_irq_base + SUNXI_IRQ_NMI_CTRL_REG);
> -
> - sunxi_irq_domain = irq_domain_add_linear(node, 3 * 32,
> - &sunxi_irq_ops, NULL);
> - if (!sunxi_irq_domain)
> - panic("%s: unable to create IRQ domain\n", node->full_name);
> -
> - set_handle_irq(sunxi_handle_irq);
> -
> - return 0;
> -}
> -IRQCHIP_DECLARE(allwinner_sunxi_ic, "allwinner,sunxi-ic", sunxi_of_init);
> -
> -static asmlinkage void __exception_irq_entry sunxi_handle_irq(struct pt_regs *regs)
> -{
> - u32 irq, hwirq;
> -
> - hwirq = readl(sunxi_irq_base + SUNXI_IRQ_VECTOR_REG) >> 2;
> - while (hwirq != 0) {
> - irq = irq_find_mapping(sunxi_irq_domain, hwirq);
> - handle_IRQ(irq, regs);
> - hwirq = readl(sunxi_irq_base + SUNXI_IRQ_VECTOR_REG) >> 2;
> - }
> -}
>
next prev parent reply other threads:[~2013-03-25 14:00 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-25 13:30 [PATCH 00/10] ARM: sunxi: Architecture cleanups and rework Maxime Ripard
2013-03-25 13:30 ` [PATCH 01/10] ARM: sunxi: dt: Reorganize the dtsi Maxime Ripard
2013-03-25 13:30 ` [PATCH 02/10] clocksource: sunxi: Cleanup the timer code Maxime Ripard
2013-03-25 13:30 ` [PATCH 03/10] clocksource: sunxi: make use of CLKSRC_OF Maxime Ripard
2013-03-25 13:50 ` Rob Herring
2013-03-25 16:23 ` Maxime Ripard
2013-03-25 13:30 ` [PATCH 04/10] clocksource: sunxi: Rename sunxi to sun4i Maxime Ripard
2013-03-25 13:30 ` [PATCH 05/10] ARM: sunxi: dt: Update timer compatible string Maxime Ripard
2013-03-25 13:30 ` [PATCH 06/10] irqchip: sunxi: Make use of the IRQCHIP_DECLARE macro Maxime Ripard
2013-03-25 13:30 ` [PATCH 07/10] irqchip: sunxi: Rename sunxi to sun4i Maxime Ripard
2013-03-25 14:00 ` Rob Herring [this message]
2013-03-25 13:30 ` [PATCH 08/10] ARM: sunxi: dt: Update interrupt controller compatible string Maxime Ripard
2013-03-25 13:59 ` Rob Herring
2013-03-25 16:27 ` Maxime Ripard
2013-03-25 13:30 ` [PATCH 09/10] ARM: sunxi: Rework the restart code Maxime Ripard
2013-03-25 13:30 ` [PATCH 10/10] ARM: sunxi: dt: Update watchdog compatible string Maxime Ripard
-- strict thread matches above, loose matches on Subject: below --
2013-03-26 9:13 [PATCHv2 00/10] ARM: sunxi: Architecture cleanups and rework Maxime Ripard
2013-03-26 9:13 ` [PATCH 07/10] irqchip: sunxi: Rename sunxi to sun4i Maxime Ripard
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=51505869.8010203@gmail.com \
--to=robherring2@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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).