* [PATCH] acpiphp: only size new bus
@ 2005-12-14 1:49 Kristen Accardi
2005-12-14 4:21 ` [Pcihpd-discuss] " Kenji Kaneshige
0 siblings, 1 reply; 3+ messages in thread
From: Kristen Accardi @ 2005-12-14 1:49 UTC (permalink / raw)
To: pcihpd-discuss; +Cc: linux-kernel, rajesh.shah, greg
Only size the bus that has been added.
Signed-off-by: Kristen Carlson Accardi <kristen.c.accardi@intel.com>
drivers/pci/hotplug/acpiphp_glue.c | 4 +++-
drivers/pci/hotplug/acpiphp_glue.c | 7 +++++--
1 files changed, 5 insertions(+), 2 deletions(-)
--- linux-2.6.15-rc5.orig/drivers/pci/hotplug/acpiphp_glue.c
+++ linux-2.6.15-rc5/drivers/pci/hotplug/acpiphp_glue.c
@@ -794,12 +794,15 @@ static int enable_device(struct acpiphp_
if (PCI_SLOT(dev->devfn) != slot->device)
continue;
if (dev->hdr_type == PCI_HEADER_TYPE_BRIDGE ||
- dev->hdr_type == PCI_HEADER_TYPE_CARDBUS)
+ dev->hdr_type == PCI_HEADER_TYPE_CARDBUS) {
max = pci_scan_bridge(bus, dev, max, pass);
+ if (pass)
+ if (dev->subordinate)
+ pci_bus_size_bridges(dev->subordinate);
+ }
}
}
- pci_bus_size_bridges(bus);
pci_bus_assign_resources(bus);
acpiphp_sanitize_bus(bus);
pci_enable_bridges(bus);
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Pcihpd-discuss] [PATCH] acpiphp: only size new bus
2005-12-14 1:49 [PATCH] acpiphp: only size new bus Kristen Accardi
@ 2005-12-14 4:21 ` Kenji Kaneshige
2005-12-14 17:37 ` Kristen Accardi
0 siblings, 1 reply; 3+ messages in thread
From: Kenji Kaneshige @ 2005-12-14 4:21 UTC (permalink / raw)
To: Kristen Accardi; +Cc: pcihpd-discuss, linux-kernel, rajesh.shah, greg
Hi Kristen,
> + if (pass)
> + if (dev->subordinate)
> + pci_bus_size_bridges(dev->subordinate);
How about doing as follows. This also satisfies 80 columns rule.
if (pass && dev->subordinate)
pci_bus_size_bridges(dev->subordinate);
Thanks,
Kenji Kaneshige
Kristen Accardi wrote:
> Only size the bus that has been added.
>
> Signed-off-by: Kristen Carlson Accardi <kristen.c.accardi@intel.com>
>
> drivers/pci/hotplug/acpiphp_glue.c | 4 +++-
> drivers/pci/hotplug/acpiphp_glue.c | 7 +++++--
> 1 files changed, 5 insertions(+), 2 deletions(-)
>
> --- linux-2.6.15-rc5.orig/drivers/pci/hotplug/acpiphp_glue.c
> +++ linux-2.6.15-rc5/drivers/pci/hotplug/acpiphp_glue.c
> @@ -794,12 +794,15 @@ static int enable_device(struct acpiphp_
> if (PCI_SLOT(dev->devfn) != slot->device)
> continue;
> if (dev->hdr_type == PCI_HEADER_TYPE_BRIDGE ||
> - dev->hdr_type == PCI_HEADER_TYPE_CARDBUS)
> + dev->hdr_type == PCI_HEADER_TYPE_CARDBUS) {
> max = pci_scan_bridge(bus, dev, max, pass);
> + if (pass)
> + if (dev->subordinate)
> + pci_bus_size_bridges(dev->subordinate);
> + }
> }
> }
>
> - pci_bus_size_bridges(bus);
> pci_bus_assign_resources(bus);
> acpiphp_sanitize_bus(bus);
> pci_enable_bridges(bus);
>
>
>
> -------------------------------------------------------
> This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
> for problems? Stop! Download the new AJAX search engine that makes
> searching your log files as easy as surfing the web. DOWNLOAD SPLUNK!
> http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click
> _______________________________________________
> Pcihpd-discuss mailing list
> Pcihpd-discuss@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/pcihpd-discuss
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Pcihpd-discuss] [PATCH] acpiphp: only size new bus
2005-12-14 4:21 ` [Pcihpd-discuss] " Kenji Kaneshige
@ 2005-12-14 17:37 ` Kristen Accardi
0 siblings, 0 replies; 3+ messages in thread
From: Kristen Accardi @ 2005-12-14 17:37 UTC (permalink / raw)
To: Kenji Kaneshige; +Cc: pcihpd-discuss, linux-kernel, rajesh.shah, greg
Only size the bus that has been added.
Signed-off-by: Kristen Carlson Accardi <kristen.c.accardi@intel.com>
drivers/pci/hotplug/acpiphp_glue.c | 4 +++-
drivers/pci/hotplug/acpiphp_glue.c | 6 ++++--
1 files changed, 4 insertions(+), 2 deletions(-)
--- linux-2.6.15-rc5.orig/drivers/pci/hotplug/acpiphp_glue.c
+++ linux-2.6.15-rc5/drivers/pci/hotplug/acpiphp_glue.c
@@ -794,12 +794,14 @@ static int enable_device(struct acpiphp_
if (PCI_SLOT(dev->devfn) != slot->device)
continue;
if (dev->hdr_type == PCI_HEADER_TYPE_BRIDGE ||
- dev->hdr_type == PCI_HEADER_TYPE_CARDBUS)
+ dev->hdr_type == PCI_HEADER_TYPE_CARDBUS) {
max = pci_scan_bridge(bus, dev, max, pass);
+ if (pass && dev->subordinate)
+ pci_bus_size_bridges(dev->subordinate);
+ }
}
}
- pci_bus_size_bridges(bus);
pci_bus_assign_resources(bus);
acpiphp_sanitize_bus(bus);
pci_enable_bridges(bus);
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2005-12-14 17:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-12-14 1:49 [PATCH] acpiphp: only size new bus Kristen Accardi
2005-12-14 4:21 ` [Pcihpd-discuss] " Kenji Kaneshige
2005-12-14 17:37 ` Kristen Accardi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).