Browse Source

powerpc: Set I/O port resource types correctly

Set I/O port resource structs to have IORESOURCE_IO in their type field.

Previously we marked these as merely IORESOURCE_BUSY without indicating the
type.  Setting the type doesn't fix any functional problem but makes %pR
on the resource work better.

Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Bjorn Helgaas 7 years ago
parent
commit
961292e664
2 changed files with 4 additions and 4 deletions
  1. 1 1
      arch/powerpc/platforms/maple/time.c
  2. 3 3
      arch/powerpc/sysdev/i8259.c

+ 1 - 1
arch/powerpc/platforms/maple/time.c

@@ -134,7 +134,7 @@ int maple_set_rtc_time(struct rtc_time *tm)
 
 
 static struct resource rtc_iores = {
 static struct resource rtc_iores = {
 	.name = "rtc",
 	.name = "rtc",
-	.flags = IORESOURCE_BUSY,
+	.flags = IORESOURCE_IO | IORESOURCE_BUSY,
 };
 };
 
 
 unsigned long __init maple_get_boot_time(void)
 unsigned long __init maple_get_boot_time(void)

+ 3 - 3
arch/powerpc/sysdev/i8259.c

@@ -145,21 +145,21 @@ static struct resource pic1_iores = {
 	.name = "8259 (master)",
 	.name = "8259 (master)",
 	.start = 0x20,
 	.start = 0x20,
 	.end = 0x21,
 	.end = 0x21,
-	.flags = IORESOURCE_BUSY,
+	.flags = IORESOURCE_IO | IORESOURCE_BUSY,
 };
 };
 
 
 static struct resource pic2_iores = {
 static struct resource pic2_iores = {
 	.name = "8259 (slave)",
 	.name = "8259 (slave)",
 	.start = 0xa0,
 	.start = 0xa0,
 	.end = 0xa1,
 	.end = 0xa1,
-	.flags = IORESOURCE_BUSY,
+	.flags = IORESOURCE_IO | IORESOURCE_BUSY,
 };
 };
 
 
 static struct resource pic_edgectrl_iores = {
 static struct resource pic_edgectrl_iores = {
 	.name = "8259 edge control",
 	.name = "8259 edge control",
 	.start = 0x4d0,
 	.start = 0x4d0,
 	.end = 0x4d1,
 	.end = 0x4d1,
-	.flags = IORESOURCE_BUSY,
+	.flags = IORESOURCE_IO | IORESOURCE_BUSY,
 };
 };
 
 
 static int i8259_host_match(struct irq_domain *h, struct device_node *node,
 static int i8259_host_match(struct irq_domain *h, struct device_node *node,