Skip to content

Commit

Permalink
[POWERPC] Remove set_cfg_type for PCI indirect users that don't need it
Browse files Browse the repository at this point in the history
The Freescale and Marvell PCI controllers dont require explicit setting for
type 1 config cycles.  They handle producing them by implicitly looking at the
bus, devfn.

The TSI108 and 52xx don't use the generic PCI indirect code and thus don't
bother with set_cfg_type.

Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
  • Loading branch information
Kumar Gala committed Jun 29, 2007
1 parent d3c7ffa commit 5ab9c45
Show file tree
Hide file tree
Showing 7 changed files with 0 additions and 9 deletions.
1 change: 0 additions & 1 deletion arch/powerpc/platforms/52xx/mpc52xx_pci.c
Original file line number Diff line number Diff line change
Expand Up @@ -390,7 +390,6 @@ mpc52xx_add_bridge(struct device_node *node)
return -ENOMEM;

hose->arch_data = node;
hose->set_cfg_type = 1;

hose->first_busno = bus_range ? bus_range[0] : 0;
hose->last_busno = bus_range ? bus_range[1] : 0xff;
Expand Down
3 changes: 0 additions & 3 deletions arch/powerpc/platforms/82xx/mpc82xx_ads.c
Original file line number Diff line number Diff line change
Expand Up @@ -548,14 +548,11 @@ static void __init mpc82xx_add_bridge(struct device_node *np)
return;

hose->arch_data = np;
hose->set_cfg_type = 1;

hose->first_busno = bus_range ? bus_range[0] : 0;
hose->last_busno = bus_range ? bus_range[1] : 0xff;
hose->bus_offset = 0;

hose->set_cfg_type = 1;

setup_indirect_pci(hose,
r.start + offsetof(pci_cpm2_t, pci_cfg_addr),
r.start + offsetof(pci_cpm2_t, pci_cfg_data));
Expand Down
1 change: 0 additions & 1 deletion arch/powerpc/platforms/83xx/pci.c
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,6 @@ int __init mpc83xx_add_bridge(struct device_node *dev)
if (!hose)
return -ENOMEM;
hose->arch_data = dev;
hose->set_cfg_type = 1;

hose->first_busno = bus_range ? bus_range[0] : 0;
hose->last_busno = bus_range ? bus_range[1] : 0xff;
Expand Down
1 change: 0 additions & 1 deletion arch/powerpc/platforms/85xx/pci.c
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,6 @@ int __init mpc85xx_add_bridge(struct device_node *dev)
if (!hose)
return -ENOMEM;
hose->arch_data = dev;
hose->set_cfg_type = 1;

hose->first_busno = bus_range ? bus_range[0] : 0;
hose->last_busno = bus_range ? bus_range[1] : 0xff;
Expand Down
1 change: 0 additions & 1 deletion arch/powerpc/platforms/86xx/pci.c
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,6 @@ int __init mpc86xx_add_bridge(struct device_node *dev)
if (!hose)
return -ENOMEM;
hose->arch_data = dev;
hose->set_cfg_type = 1;

/* last_busno = 0xfe cause by MPC8641 PCIE bug */
hose->first_busno = bus_range ? bus_range[0] : 0x0;
Expand Down
1 change: 0 additions & 1 deletion arch/powerpc/sysdev/mv64x60_pci.c
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,6 @@ static int __init mv64x60_add_bridge(struct device_node *dev)
return -ENOMEM;

hose->arch_data = dev;
hose->set_cfg_type = 1;

hose->first_busno = bus_range ? bus_range[0] : 0;
hose->last_busno = bus_range ? bus_range[1] : 0xff;
Expand Down
1 change: 0 additions & 1 deletion arch/powerpc/sysdev/tsi108_pci.c
Original file line number Diff line number Diff line change
Expand Up @@ -226,7 +226,6 @@ int __init tsi108_setup_pci(struct device_node *dev, u32 cfg_phys, int primary)
return -ENOMEM;
}
hose->arch_data = dev;
hose->set_cfg_type = 1;

hose->first_busno = bus_range ? bus_range[0] : 0;
hose->last_busno = bus_range ? bus_range[1] : 0xff;
Expand Down

0 comments on commit 5ab9c45

Please sign in to comment.