| Message ID | 20220121205150.1151607-1-helgaas@kernel.org |
|---|---|
| State | New |
| Headers | show |
| Series | [v1] microblaze/PCI: Remove pci_phys_mem_access_prot() dead code | expand |
On 1/21/22 21:51, Bjorn Helgaas wrote: > From: Bjorn Helgaas <bhelgaas@google.com> > > pci_phys_mem_access_prot() is defined but never used. Remove it. > > Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> > --- > arch/microblaze/include/asm/pci.h | 4 --- > arch/microblaze/pci/pci-common.c | 49 ------------------------------- > 2 files changed, 53 deletions(-) > > diff --git a/arch/microblaze/include/asm/pci.h b/arch/microblaze/include/asm/pci.h > index 7c4dc5d85f53..d90528064604 100644 > --- a/arch/microblaze/include/asm/pci.h > +++ b/arch/microblaze/include/asm/pci.h > @@ -61,10 +61,6 @@ extern int pci_mmap_legacy_page_range(struct pci_bus *bus, > extern void pcibios_resource_survey(void); > > struct file; > -extern pgprot_t pci_phys_mem_access_prot(struct file *file, > - unsigned long pfn, > - unsigned long size, > - pgprot_t prot); > > /* This part of code was originally in xilinx-pci.h */ > #ifdef CONFIG_PCI_XILINX > diff --git a/arch/microblaze/pci/pci-common.c b/arch/microblaze/pci/pci-common.c > index 622a4867f9e9..33bab7eec731 100644 > --- a/arch/microblaze/pci/pci-common.c > +++ b/arch/microblaze/pci/pci-common.c > @@ -165,55 +165,6 @@ int pci_iobar_pfn(struct pci_dev *pdev, int bar, struct vm_area_struct *vma) > return 0; > } > > -/* > - * This one is used by /dev/mem and fbdev who have no clue about the > - * PCI device, it tries to find the PCI device first and calls the > - * above routine > - */ > -pgprot_t pci_phys_mem_access_prot(struct file *file, > - unsigned long pfn, > - unsigned long size, > - pgprot_t prot) > -{ > - struct pci_dev *pdev = NULL; > - struct resource *found = NULL; > - resource_size_t offset = ((resource_size_t)pfn) << PAGE_SHIFT; > - int i; > - > - if (page_is_ram(pfn)) > - return prot; > - > - prot = pgprot_noncached(prot); > - for_each_pci_dev(pdev) { > - for (i = 0; i <= PCI_ROM_RESOURCE; i++) { > - struct resource *rp = &pdev->resource[i]; > - int flags = rp->flags; > - > - /* Active and same type? */ > - if ((flags & IORESOURCE_MEM) == 0) > - continue; > - /* In the range of this resource? */ > - if (offset < (rp->start & PAGE_MASK) || > - offset > rp->end) > - continue; > - found = rp; > - break; > - } > - if (found) > - break; > - } > - if (found) { > - if (found->flags & IORESOURCE_PREFETCH) > - prot = pgprot_noncached_wc(prot); > - pci_dev_put(pdev); > - } > - > - pr_debug("PCI: Non-PCI map for %llx, prot: %lx\n", > - (unsigned long long)offset, pgprot_val(prot)); > - > - return prot; > -} > - > /* This provides legacy IO read access on a bus */ > int pci_legacy_read(struct pci_bus *bus, loff_t port, u32 *val, size_t size) > { Applied. M
diff --git a/arch/microblaze/include/asm/pci.h b/arch/microblaze/include/asm/pci.h index 7c4dc5d85f53..d90528064604 100644 --- a/arch/microblaze/include/asm/pci.h +++ b/arch/microblaze/include/asm/pci.h @@ -61,10 +61,6 @@ extern int pci_mmap_legacy_page_range(struct pci_bus *bus, extern void pcibios_resource_survey(void); struct file; -extern pgprot_t pci_phys_mem_access_prot(struct file *file, - unsigned long pfn, - unsigned long size, - pgprot_t prot); /* This part of code was originally in xilinx-pci.h */ #ifdef CONFIG_PCI_XILINX diff --git a/arch/microblaze/pci/pci-common.c b/arch/microblaze/pci/pci-common.c index 622a4867f9e9..33bab7eec731 100644 --- a/arch/microblaze/pci/pci-common.c +++ b/arch/microblaze/pci/pci-common.c @@ -165,55 +165,6 @@ int pci_iobar_pfn(struct pci_dev *pdev, int bar, struct vm_area_struct *vma) return 0; } -/* - * This one is used by /dev/mem and fbdev who have no clue about the - * PCI device, it tries to find the PCI device first and calls the - * above routine - */ -pgprot_t pci_phys_mem_access_prot(struct file *file, - unsigned long pfn, - unsigned long size, - pgprot_t prot) -{ - struct pci_dev *pdev = NULL; - struct resource *found = NULL; - resource_size_t offset = ((resource_size_t)pfn) << PAGE_SHIFT; - int i; - - if (page_is_ram(pfn)) - return prot; - - prot = pgprot_noncached(prot); - for_each_pci_dev(pdev) { - for (i = 0; i <= PCI_ROM_RESOURCE; i++) { - struct resource *rp = &pdev->resource[i]; - int flags = rp->flags; - - /* Active and same type? */ - if ((flags & IORESOURCE_MEM) == 0) - continue; - /* In the range of this resource? */ - if (offset < (rp->start & PAGE_MASK) || - offset > rp->end) - continue; - found = rp; - break; - } - if (found) - break; - } - if (found) { - if (found->flags & IORESOURCE_PREFETCH) - prot = pgprot_noncached_wc(prot); - pci_dev_put(pdev); - } - - pr_debug("PCI: Non-PCI map for %llx, prot: %lx\n", - (unsigned long long)offset, pgprot_val(prot)); - - return prot; -} - /* This provides legacy IO read access on a bus */ int pci_legacy_read(struct pci_bus *bus, loff_t port, u32 *val, size_t size) {