PCI ASPM: remove redundant list check

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Linux Kernel Mailing List
Date: Wednesday, September 16, 2009 - 7:59 am

Gitweb:     http://git.kernel.org/linus/8a339e7321f10dc2f28928ffadb69b6c7c2d5c3b
Commit:     8a339e7321f10dc2f28928ffadb69b6c7c2d5c3b
Parent:     b127bd55d9cd9d5b40278b30645669d6d46933bc
Author:     Kenji Kaneshige <kaneshige.kenji@jp.fujitsu.com>
AuthorDate: Wed Aug 19 10:58:09 2009 +0900
Committer:  Jesse Barnes <jbarnes@virtuousgeek.org>
CommitDate: Wed Sep 9 13:29:45 2009 -0700

    PCI ASPM: remove redundant list check
    
    Remove the following check in __pcie_aspm_config_link() because it
    nerver be true.
    
    Acked-by: Shaohua Li <shaohua.li@intel.com>
    Signed-off-by: Kenji Kaneshige <kaneshige.kenji@jp.fujitsu.com>
    Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
---
 drivers/pci/pcie/aspm.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/drivers/pci/pcie/aspm.c b/drivers/pci/pcie/aspm.c
index 9759714..638881b 100644
--- a/drivers/pci/pcie/aspm.c
+++ b/drivers/pci/pcie/aspm.c
@@ -458,9 +458,6 @@ static void __pcie_aspm_config_link(struct pcie_link_state *link, u32 state)
 	struct pci_dev *child, *parent = link->pdev;
 	struct pci_bus *linkbus = parent->subordinate;
 
-	/* If no child, disable the link */
-	if (list_empty(&linkbus->devices))
-		state = 0;
 	/*
 	 * If the downstream component has pci bridge function, don't
 	 * do ASPM now.
--
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
PCI ASPM: remove redundant list check, Linux Kernel Mailing ..., (Wed Sep 16, 7:59 am)