Commit 6428bc7b authored by Helge Deller's avatar Helge Deller

parisc: sba_iommu: Fix build warning if procfs if disabled

Clean up the code, e.g. make proc_mckinley_root static, drop the now
empty mckinley header file and remove some unneeded ifdefs around procfs
functions.
Signed-off-by: default avatarHelge Deller <deller@gmx.de>
Reported-by: default avatarkernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202308300800.Jod4sHzM-lkp@intel.com/
Fixes: 77e0ddf0 ("parisc: ccio-dma: Create private runway procfs root entry")
parent 6c1b980a
/* SPDX-License-Identifier: GPL-2.0 */
#ifndef ASM_PARISC_MCKINLEY_H
#define ASM_PARISC_MCKINLEY_H
/* declared in arch/parisc/kernel/setup.c */
extern struct proc_dir_entry * proc_mckinley_root;
#endif /*ASM_PARISC_MCKINLEY_H*/
...@@ -46,8 +46,6 @@ ...@@ -46,8 +46,6 @@
#include <linux/module.h> #include <linux/module.h>
#include <asm/ropes.h> #include <asm/ropes.h>
#include <asm/mckinley.h> /* for proc_mckinley_root */
#include <asm/runway.h> /* for proc_runway_root */
#include <asm/page.h> /* for PAGE0 */ #include <asm/page.h> /* for PAGE0 */
#include <asm/pdc.h> /* for PDC_MODEL_* */ #include <asm/pdc.h> /* for PDC_MODEL_* */
#include <asm/pdcpat.h> /* for is_pdc_pat() */ #include <asm/pdcpat.h> /* for is_pdc_pat() */
...@@ -122,7 +120,7 @@ MODULE_PARM_DESC(sba_reserve_agpgart, "Reserve half of IO pdir as AGPGART"); ...@@ -122,7 +120,7 @@ MODULE_PARM_DESC(sba_reserve_agpgart, "Reserve half of IO pdir as AGPGART");
#endif #endif
static struct proc_dir_entry *proc_runway_root __ro_after_init; static struct proc_dir_entry *proc_runway_root __ro_after_init;
struct proc_dir_entry *proc_mckinley_root __ro_after_init; static struct proc_dir_entry *proc_mckinley_root __ro_after_init;
/************************************ /************************************
** SBA register read and write support ** SBA register read and write support
...@@ -1899,9 +1897,7 @@ static int __init sba_driver_callback(struct parisc_device *dev) ...@@ -1899,9 +1897,7 @@ static int __init sba_driver_callback(struct parisc_device *dev)
int i; int i;
char *version; char *version;
void __iomem *sba_addr = ioremap(dev->hpa.start, SBA_FUNC_SIZE); void __iomem *sba_addr = ioremap(dev->hpa.start, SBA_FUNC_SIZE);
#ifdef CONFIG_PROC_FS struct proc_dir_entry *root __maybe_unused;
struct proc_dir_entry *root;
#endif
sba_dump_ranges(sba_addr); sba_dump_ranges(sba_addr);
...@@ -1967,7 +1963,6 @@ static int __init sba_driver_callback(struct parisc_device *dev) ...@@ -1967,7 +1963,6 @@ static int __init sba_driver_callback(struct parisc_device *dev)
hppa_dma_ops = &sba_ops; hppa_dma_ops = &sba_ops;
#ifdef CONFIG_PROC_FS
switch (dev->id.hversion) { switch (dev->id.hversion) {
case PLUTO_MCKINLEY_PORT: case PLUTO_MCKINLEY_PORT:
if (!proc_mckinley_root) if (!proc_mckinley_root)
...@@ -1985,7 +1980,6 @@ static int __init sba_driver_callback(struct parisc_device *dev) ...@@ -1985,7 +1980,6 @@ static int __init sba_driver_callback(struct parisc_device *dev)
proc_create_single("sba_iommu", 0, root, sba_proc_info); proc_create_single("sba_iommu", 0, root, sba_proc_info);
proc_create_single("sba_iommu-bitmap", 0, root, sba_proc_bitmap_info); proc_create_single("sba_iommu-bitmap", 0, root, sba_proc_bitmap_info);
#endif
return 0; return 0;
} }
......
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