PCI: pciehp: make check_link_active more helpful
authorYinghai Lu <yinghai@kernel.org>
Fri, 27 Jan 2012 18:55:12 +0000 (10:55 -0800)
committerJesse Barnes <jbarnes@virtuousgeek.org>
Tue, 14 Feb 2012 16:45:00 +0000 (08:45 -0800)
A few changes:
  - remove the 'inline' and let the complier decide
  - return a bool to indicate whether the link was active
  - add a debug message to indicate link state when it beocmes active

Signed-off-by: Yinghai Lu <yinghai@kernel.org>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
drivers/pci/hotplug/pciehp_hpc.c

index 7dc9e33746a65ca8bacf82e0ec98d74f454290fa..a8c1f74a1e367d7ec2d3aa3afcdcb551369f76a2 100644 (file)
@@ -241,13 +241,20 @@ static int pcie_write_cmd(struct controller *ctrl, u16 cmd, u16 mask)
        return retval;
 }
 
-static inline int check_link_active(struct controller *ctrl)
+static bool check_link_active(struct controller *ctrl)
 {
-       u16 link_status;
+       bool ret = false;
+       u16 lnk_status;
 
-       if (pciehp_readw(ctrl, PCI_EXP_LNKSTA, &link_status))
-               return 0;
-       return !!(link_status & PCI_EXP_LNKSTA_DLLLA);
+       if (pciehp_readw(ctrl, PCI_EXP_LNKSTA, &lnk_status))
+               return ret;
+
+       ret = !!(lnk_status & PCI_EXP_LNKSTA_DLLLA);
+
+       if (ret)
+               ctrl_dbg(ctrl, "%s: lnk_status = %x\n", __func__, lnk_status);
+
+       return ret;
 }
 
 static void pcie_wait_link_active(struct controller *ctrl)