diff mbox

lspci: Decode DevCap SlotPowerLimit for all components with Upstream Ports

Message ID 20151210195001.8111.55134.stgit@bhelgaas-glaptop2.roam.corp.google.com
State Not Applicable
Headers show

Commit Message

Bjorn Helgaas Dec. 10, 2015, 7:50 p.m. UTC
The SlotPowerLimit in the Slot Capability indicates how much power the slot
can supply to a downstream device.  A Root Port or Switch Downstream Port
communicates the limit via a Set_Slot_Power_Limit Message on the link.  The
component on the other end of the link copies the limit from the message to
the Captured Slot Power Limit in its Device Capability [see PCIe r3.0, sec
2.2.8.5].

The Captured SlotPowerLimit is relevant for all devices on the downstream
end of a Link.  This includes Endpoints and Bridges as well as
Switch Upstream Ports.

Decode the DevCap Captured SlotPowerLimit for Endpoints and Bridges as well
as Switch Upstream Ports.

Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
---
 ls-caps.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)


--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Martin Mareš Dec. 22, 2015, 3:36 p.m. UTC | #1
Hi!

> The SlotPowerLimit in the Slot Capability indicates how much power the slot
> can supply to a downstream device.  A Root Port or Switch Downstream Port
> communicates the limit via a Set_Slot_Power_Limit Message on the link.  The
> component on the other end of the link copies the limit from the message to
> the Captured Slot Power Limit in its Device Capability [see PCIe r3.0, sec
> 2.2.8.5].
> 
> The Captured SlotPowerLimit is relevant for all devices on the downstream
> end of a Link.  This includes Endpoints and Bridges as well as
> Switch Upstream Ports.
> 
> Decode the DevCap Captured SlotPowerLimit for Endpoints and Bridges as well
> as Switch Upstream Ports.

Thanks, applied.

				Have a nice fortnight
diff mbox

Patch

diff --git a/ls-caps.c b/ls-caps.c
index c145ed6..639f931 100644
--- a/ls-caps.c
+++ b/ls-caps.c
@@ -678,7 +678,8 @@  static void cap_express_dev(struct device *d, int where, int type)
   if ((type == PCI_EXP_TYPE_ENDPOINT) || (type == PCI_EXP_TYPE_LEG_END))
     printf(" FLReset%c",
 	FLAG(t, PCI_EXP_DEVCAP_FLRESET));
-  if (type == PCI_EXP_TYPE_UPSTREAM)
+  if ((type == PCI_EXP_TYPE_ENDPOINT) || (type == PCI_EXP_TYPE_UPSTREAM) ||
+      (type == PCI_EXP_TYPE_PCI_BRIDGE))
     printf(" SlotPowerLimit %.3fW",
 	power_limit((t & PCI_EXP_DEVCAP_PWR_VAL) >> 18,
 		    (t & PCI_EXP_DEVCAP_PWR_SCL) >> 26));