| Message ID | 20180421135537.24716-6-miquel.raynal@bootlin.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hello, On Sat, 21 Apr 2018 15:55:25 +0200, Miquel Raynal wrote: > Rewrite a small section to clarify the reset operation of interrupts > already configured by ATF that we want to handle in the driver. This > will simplify the introduction of System Error Interrupts support. > > Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com> Looks good to me. Reviewed-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com> Thomas
diff --git a/drivers/irqchip/irq-mvebu-icu.c b/drivers/irqchip/irq-mvebu-icu.c index 6a77b9ea8e41..5af2520445c4 100644 --- a/drivers/irqchip/irq-mvebu-icu.c +++ b/drivers/irqchip/irq-mvebu-icu.c @@ -198,8 +198,8 @@ static int mvebu_icu_probe(struct platform_device *pdev) struct device_node *gicp_dn; struct resource *res; phys_addr_t setspi, clrspi; - u32 i, icu_int; int ret; + u32 i; icu = devm_kzalloc(&pdev->dev, sizeof(struct mvebu_icu), GFP_KERNEL); @@ -257,8 +257,12 @@ static int mvebu_icu_probe(struct platform_device *pdev) * avoid unpredictable SPI assignments done by firmware. */ for (i = 0 ; i < ICU_MAX_IRQS ; i++) { + u32 icu_int, icu_grp; + icu_int = readl(icu->base + ICU_INT_CFG(i)); - if ((icu_int >> ICU_GROUP_SHIFT) == ICU_GRP_NSR) + icu_grp = icu_int >> ICU_GROUP_SHIFT; + + if (icu_grp == ICU_GRP_NSR) writel_relaxed(0x0, icu->base + ICU_INT_CFG(i)); }
Rewrite a small section to clarify the reset operation of interrupts already configured by ATF that we want to handle in the driver. This will simplify the introduction of System Error Interrupts support. Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com> --- drivers/irqchip/irq-mvebu-icu.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-)