of_address: Guard of_bus_pci_get_flags with CONFIG_PCI

FlyGoat · tsbogend · commit 0fc0ead34888 · 2020-07-31T17:30:52.000+02:00
After 2f96593 ("of_address: Add bus type match for pci ranges parser"), the last user of of_bus_pci_get_flags when CONFIG_PCI is disabled had gone. This caused unused function warning when compiling without CONFIG_PCI. Fix by guarding it with CONFIG_PCI. Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au> Fixes: 2f96593 ("of_address: Add bus type match for pci ranges parser") Acked-by: Rob Herring <robh@kernel.org> Signed-off-by: Thomas Bogendoerfer <tsbogend@alpha.franken.de>
diff --git a/drivers/of/address.c b/drivers/of/address.c
@@ -101,6 +101,7 @@ static unsigned int of_bus_default_get_flags(const __be32 *addr)
 	return IORESOURCE_MEM;
 }
 
+#ifdef CONFIG_PCI
 static unsigned int of_bus_pci_get_flags(const __be32 *addr)
 {
 	unsigned int flags = 0;
@@ -123,7 +124,6 @@ static unsigned int of_bus_pci_get_flags(const __be32 *addr)
 	return flags;
 }
 
-#ifdef CONFIG_PCI
 /*
  * PCI bus specific translator
  */

Original file line number	Diff line number	Diff line change
`@@ -101,6 +101,7 @@ static unsigned int of_bus_default_get_flags(const __be32 *addr)`
`101`	`101`	`return IORESOURCE_MEM;`
`102`	`102`	`}`
`103`	`103`
	`104`	`+#ifdef CONFIG_PCI`
`104`	`105`	`static unsigned int of_bus_pci_get_flags(const __be32 *addr)`
`105`	`106`	`{`
`106`	`107`	`unsigned int flags = 0;`
`@@ -123,7 +124,6 @@ static unsigned int of_bus_pci_get_flags(const __be32 *addr)`
`123`	`124`	`return flags;`
`124`	`125`	`}`
`125`	`126`
`126`		`-#ifdef CONFIG_PCI`
`127`	`127`	`/*`
`128`	`128`	`* PCI bus specific translator`
`129`	`129`	`*/`