Skip to content

Commit 3228950

Browse files
apmswJason Cooper
authored andcommitted
irqchip: gic: Preserve gic V2 bypass bits in cpu ctrl register
This change is made to preserve the GIC v2 bypass bits in the GIC_CPU_CTRL register (also known as the GICC_CTLR register in spec). This code will preserve all bits configured by the bootloader regarding v2 bypass group bits. In the X-Gene platform, the bypass functionality is not used and bypass bits should not be changed by the kernel gic code as it could lead to incorrect behavior. Signed-off-by: Feng Kan <[email protected]> Reviewed-by: Vinayak Kale <[email protected]> Reviewed-by: Anup Patel <[email protected]> Link: https://lkml.kernel.org/r/[email protected] Signed-off-by: Jason Cooper <[email protected]>
1 parent e5f8153 commit 3228950

File tree

2 files changed

+23
-3
lines changed

2 files changed

+23
-3
lines changed

drivers/irqchip/irq-gic.c

Lines changed: 22 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -353,6 +353,21 @@ static u8 gic_get_cpumask(struct gic_chip_data *gic)
353353
return mask;
354354
}
355355

356+
static void gic_cpu_if_up(void)
357+
{
358+
void __iomem *cpu_base = gic_data_cpu_base(&gic_data[0]);
359+
u32 bypass = 0;
360+
361+
/*
362+
* Preserve bypass disable bits to be written back later
363+
*/
364+
bypass = readl(cpu_base + GIC_CPU_CTRL);
365+
bypass &= GICC_DIS_BYPASS_MASK;
366+
367+
writel_relaxed(bypass | GICC_ENABLE, cpu_base + GIC_CPU_CTRL);
368+
}
369+
370+
356371
static void __init gic_dist_init(struct gic_chip_data *gic)
357372
{
358373
unsigned int i;
@@ -401,13 +416,17 @@ static void gic_cpu_init(struct gic_chip_data *gic)
401416
gic_cpu_config(dist_base, NULL);
402417

403418
writel_relaxed(GICC_INT_PRI_THRESHOLD, base + GIC_CPU_PRIMASK);
404-
writel_relaxed(GICC_ENABLE, base + GIC_CPU_CTRL);
419+
gic_cpu_if_up();
405420
}
406421

407422
void gic_cpu_if_down(void)
408423
{
409424
void __iomem *cpu_base = gic_data_cpu_base(&gic_data[0]);
410-
writel_relaxed(0, cpu_base + GIC_CPU_CTRL);
425+
u32 val = 0;
426+
427+
val = readl(cpu_base + GIC_CPU_CTRL);
428+
val &= ~GICC_ENABLE;
429+
writel_relaxed(val, cpu_base + GIC_CPU_CTRL);
411430
}
412431

413432
#ifdef CONFIG_CPU_PM
@@ -543,7 +562,7 @@ static void gic_cpu_restore(unsigned int gic_nr)
543562
dist_base + GIC_DIST_PRI + i * 4);
544563

545564
writel_relaxed(GICC_INT_PRI_THRESHOLD, cpu_base + GIC_CPU_PRIMASK);
546-
writel_relaxed(GICC_ENABLE, cpu_base + GIC_CPU_CTRL);
565+
gic_cpu_if_up();
547566
}
548567

549568
static int gic_notifier(struct notifier_block *self, unsigned long cmd, void *v)

include/linux/irqchip/arm-gic.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
#define GICC_INT_PRI_THRESHOLD 0xf0
2626
#define GICC_IAR_INT_ID_MASK 0x3ff
2727
#define GICC_INT_SPURIOUS 1023
28+
#define GICC_DIS_BYPASS_MASK 0x1e0
2829

2930
#define GIC_DIST_CTRL 0x000
3031
#define GIC_DIST_CTR 0x004

0 commit comments

Comments
 (0)