diff options
author | Rob Herring <robh@kernel.org> | 2018-11-16 17:11:02 -0500 |
---|---|---|
committer | Michael Ellerman <mpe@ellerman.id.au> | 2018-11-26 06:33:37 -0500 |
commit | d2db097905d6ec439b7e863dc9a7e790889a7607 (patch) | |
tree | 1e3870835271d72dddb35bdad1b9cb0a8578303b | |
parent | bf82d3758d4a075d17f4930c9872b05bd277ab0c (diff) |
cxl: Use device_type helpers to access the node type
Remove directly accessing device_node.type pointer and use the
accessors instead. This will eventually allow removing the type
pointer.
Signed-off-by: Rob Herring <robh@kernel.org>
Acked-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
-rw-r--r-- | drivers/misc/cxl/pci.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/misc/cxl/pci.c b/drivers/misc/cxl/pci.c index b66d832d3233..c79ba1c699ad 100644 --- a/drivers/misc/cxl/pci.c +++ b/drivers/misc/cxl/pci.c | |||
@@ -1718,7 +1718,6 @@ int cxl_slot_is_switched(struct pci_dev *dev) | |||
1718 | { | 1718 | { |
1719 | struct device_node *np; | 1719 | struct device_node *np; |
1720 | int depth = 0; | 1720 | int depth = 0; |
1721 | const __be32 *prop; | ||
1722 | 1721 | ||
1723 | if (!(np = pci_device_to_OF_node(dev))) { | 1722 | if (!(np = pci_device_to_OF_node(dev))) { |
1724 | pr_err("cxl: np = NULL\n"); | 1723 | pr_err("cxl: np = NULL\n"); |
@@ -1727,8 +1726,7 @@ int cxl_slot_is_switched(struct pci_dev *dev) | |||
1727 | of_node_get(np); | 1726 | of_node_get(np); |
1728 | while (np) { | 1727 | while (np) { |
1729 | np = of_get_next_parent(np); | 1728 | np = of_get_next_parent(np); |
1730 | prop = of_get_property(np, "device_type", NULL); | 1729 | if (!of_node_is_type(np, "pciex")) |
1731 | if (!prop || strcmp((char *)prop, "pciex")) | ||
1732 | break; | 1730 | break; |
1733 | depth++; | 1731 | depth++; |
1734 | } | 1732 | } |