diff options
| author | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2016-07-07 21:17:16 +1000 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2016-07-12 20:15:26 +1000 |
| commit | 052d7b967ed451d232c02e1cabce8d89ae3266fe (patch) | |
| tree | 1ffe79ef4b7583367f540352fd16d513b900d56c | |
| parent | b48141527446857e26f13c6283185434f0a276d6 (diff) | |
| download | skiboot-052d7b967ed451d232c02e1cabce8d89ae3266fe.zip skiboot-052d7b967ed451d232c02e1cabce8d89ae3266fe.tar.gz skiboot-052d7b967ed451d232c02e1cabce8d89ae3266fe.tar.bz2 | |
interrupts: LPC is always an interrupt controller
Whether we have Naples or not, the "interrupts" properties
of the child of the LPC node are always in the LPC interrupt
number space, and so should always have the LPC controller as
their interrupt controller.
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Acked-by: Michael Neuling <mikey@neuling.org>
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
| -rw-r--r-- | hw/lpc.c | 5 |
1 files changed, 3 insertions, 2 deletions
@@ -816,8 +816,9 @@ static void lpc_init_chip_p8(struct dt_node *xn) chip->id, chip->lpc_xbase); lpc_init_interrupts(chip); - if (chip->type == PROC_CHIP_P8_NAPLES) - dt_add_property(xn, "interrupt-controller", NULL, 0); + dt_add_property(xn, "interrupt-controller", NULL, 0); + dt_add_property_cells(xn, "#interrupt-cells", 1); + assert(dt_prop_get_u32(xn, "#address-cells") == 2); } static void lpc_init_chip_p9(struct dt_node *opb_node) |