Skip to content

Commit 6aecc0a

Browse files
bjorn-helgaasmpe
authored andcommitted
cxl: Remove unnecessary cxl_pci_window_alignment()
cxl_pci_window_alignment() is referenced only via the struct pci_controller_ops.window_alignment function pointer, and only in the powerpc implementation of pcibios_window_alignment(). pcibios_window_alignment() defaults to returning 1 if the function pointer is NULL, which is the same was what cxl_pci_window_alignment() does. cxl_pci_window_alignment() is unnecessary, so remove it. No functional change intended. Signed-off-by: Bjorn Helgaas <[email protected]> Acked-by: Frederic Barrat <[email protected]> Signed-off-by: Michael Ellerman <[email protected]> Link: https://lore.kernel.org/r/[email protected]
1 parent 8f4ab7d commit 6aecc0a

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

drivers/misc/cxl/vphb.c

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -67,12 +67,6 @@ static void cxl_pci_disable_device(struct pci_dev *dev)
6767
}
6868
}
6969

70-
static resource_size_t cxl_pci_window_alignment(struct pci_bus *bus,
71-
unsigned long type)
72-
{
73-
return 1;
74-
}
75-
7670
static void cxl_pci_reset_secondary_bus(struct pci_dev *dev)
7771
{
7872
/* Should we do an AFU reset here ? */
@@ -200,7 +194,6 @@ static struct pci_controller_ops cxl_pci_controller_ops =
200194
.enable_device_hook = cxl_pci_enable_device_hook,
201195
.disable_device = cxl_pci_disable_device,
202196
.release_device = cxl_pci_disable_device,
203-
.window_alignment = cxl_pci_window_alignment,
204197
.reset_secondary_bus = cxl_pci_reset_secondary_bus,
205198
.setup_msi_irqs = cxl_setup_msi_irqs,
206199
.teardown_msi_irqs = cxl_teardown_msi_irqs,

0 commit comments

Comments
 (0)