Commit 5a1ff480 authored by Suravee Suthikulpanit's avatar Suravee Suthikulpanit Committed by Thomas Gleixner

irqchip/gicv2m: Miscellaneous fixes for v2m resources and SPI ranges

This patch contain fixes for v2m resources and SPI ranges:

  * Fix off-by-one error when set up v2m resource end range in
    gicv2m_acpi_init().

  * Fix the off-by-one print error for SPI range.

  * Use %pR to properly print resource range information.

Both ACPI and DT should now print:

  GICv2m: range[mem 0xe1180000-0xe1180fff], SPI[64:319]
Suggested-by: default avatarBjorn Helgaas <bhelgaas@google.com>
Signed-off-by: default avatarSuravee Suthikulpanit <Suravee.Suthikulpanit@amd.com>
Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Cc: Will Deacon <will.deacon@arm.com>
Cc: Catalin Marinas <Catalin.Marinas@arm.com>
Cc: <hanjun.guo@linaro.org>
Cc: <tomasz.nowicki@linaro.org>
Cc: <graeme.gregory@linaro.org>
Cc: <dhdang@apm.com>
Cc: <linux-arm-kernel@lists.infradead.org>
Cc: <marc.zyngier@arm.com>
Cc: <jason@lakedaemon.net>
Cc: <rjw@rjwysocki.net>
Link: http://lkml.kernel.org/r/1450830263-28914-1-git-send-email-Suravee.Suthikulpanit@amd.comSigned-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
parent 64103f06
...@@ -365,9 +365,8 @@ static int __init gicv2m_init_one(struct fwnode_handle *fwnode, ...@@ -365,9 +365,8 @@ static int __init gicv2m_init_one(struct fwnode_handle *fwnode,
list_add_tail(&v2m->entry, &v2m_nodes); list_add_tail(&v2m->entry, &v2m_nodes);
pr_info("range[%#lx:%#lx], SPI[%d:%d]\n", pr_info("range%pR, SPI[%d:%d]\n", res,
(unsigned long)res->start, (unsigned long)res->end, v2m->spi_start, (v2m->spi_start + v2m->nr_spis - 1));
v2m->spi_start, (v2m->spi_start + v2m->nr_spis));
return 0; return 0;
err_iounmap: err_iounmap:
...@@ -456,7 +455,8 @@ acpi_parse_madt_msi(struct acpi_subtable_header *header, ...@@ -456,7 +455,8 @@ acpi_parse_madt_msi(struct acpi_subtable_header *header,
return -EINVAL; return -EINVAL;
res.start = m->base_address; res.start = m->base_address;
res.end = m->base_address + SZ_4K; res.end = m->base_address + SZ_4K - 1;
res.flags = IORESOURCE_MEM;
if (m->flags & ACPI_MADT_OVERRIDE_SPI_VALUES) { if (m->flags & ACPI_MADT_OVERRIDE_SPI_VALUES) {
spi_start = m->spi_base; spi_start = m->spi_base;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment