An opencapi slot doesn't have an associated bridge device. It's not
needed for operation, but any warning is displayed through pci_warn()
which uses the pci_dev struct of the assocated bridge device. So wrap
those warning so that a different trace mechanism can be used if it's
an opencapi slot.

Signed-off-by: Frederic Barrat <fbar...@linux.ibm.com>
---
 drivers/pci/hotplug/pnv_php.c | 33 ++++++++++++++++++---------------
 1 file changed, 18 insertions(+), 15 deletions(-)

diff --git a/drivers/pci/hotplug/pnv_php.c b/drivers/pci/hotplug/pnv_php.c
index 74b62a8e11e7..08ac8f0df06c 100644
--- a/drivers/pci/hotplug/pnv_php.c
+++ b/drivers/pci/hotplug/pnv_php.c
@@ -18,6 +18,9 @@
 #define DRIVER_AUTHOR  "Gavin Shan, IBM Corporation"
 #define DRIVER_DESC    "PowerPC PowerNV PCI Hotplug Driver"
 
+#define SLOT_WARN(slot, x...) \
+       (slot->pdev ? pci_warn(slot->pdev, x) : dev_warn(&slot->bus->dev, x))
+
 struct pnv_php_event {
        bool                    added;
        struct pnv_php_slot     *php_slot;
@@ -265,7 +268,7 @@ static int pnv_php_add_devtree(struct pnv_php_slot 
*php_slot)
 
        ret = pnv_pci_get_device_tree(php_slot->dn->phandle, fdt1, 0x10000);
        if (ret) {
-               pci_warn(php_slot->pdev, "Error %d getting FDT blob\n", ret);
+               SLOT_WARN(php_slot, "Error %d getting FDT blob\n", ret);
                goto free_fdt1;
        }
 
@@ -279,7 +282,7 @@ static int pnv_php_add_devtree(struct pnv_php_slot 
*php_slot)
        dt = of_fdt_unflatten_tree(fdt, php_slot->dn, NULL);
        if (!dt) {
                ret = -EINVAL;
-               pci_warn(php_slot->pdev, "Cannot unflatten FDT\n");
+               SLOT_WARN(php_slot, "Cannot unflatten FDT\n");
                goto free_fdt;
        }
 
@@ -289,7 +292,7 @@ static int pnv_php_add_devtree(struct pnv_php_slot 
*php_slot)
        ret = pnv_php_populate_changeset(&php_slot->ocs, php_slot->dn);
        if (ret) {
                pnv_php_reverse_nodes(php_slot->dn);
-               pci_warn(php_slot->pdev, "Error %d populating changeset\n",
+               SLOT_WARN(php_slot, "Error %d populating changeset\n",
                         ret);
                goto free_dt;
        }
@@ -297,7 +300,7 @@ static int pnv_php_add_devtree(struct pnv_php_slot 
*php_slot)
        php_slot->dn->child = NULL;
        ret = of_changeset_apply(&php_slot->ocs);
        if (ret) {
-               pci_warn(php_slot->pdev, "Error %d applying changeset\n", ret);
+               SLOT_WARN(php_slot, "Error %d applying changeset\n", ret);
                goto destroy_changeset;
        }
 
@@ -337,7 +340,7 @@ int pnv_php_set_slot_power_state(struct hotplug_slot *slot,
        if (ret > 0) {
                if (be64_to_cpu(msg.params[1]) != php_slot->dn->phandle ||
                    be64_to_cpu(msg.params[2]) != state) {
-                       pci_warn(php_slot->pdev, "Wrong msg (%lld, %lld, 
%lld)\n",
+                       SLOT_WARN(php_slot, "Wrong msg (%lld, %lld, %lld)\n",
                                 be64_to_cpu(msg.params[1]),
                                 be64_to_cpu(msg.params[2]),
                                 be64_to_cpu(msg.params[3]));
@@ -359,7 +362,7 @@ int pnv_php_set_slot_power_state(struct hotplug_slot *slot,
        return ret;
 
 error:
-       pci_warn(php_slot->pdev, "Error %d powering %s\n",
+       SLOT_WARN(php_slot, "Error %d powering %s\n",
                ret, (state == OPAL_PCI_SLOT_POWER_ON) ? "on" : "off");
        return ret;
 }
@@ -378,7 +381,7 @@ static int pnv_php_get_power_state(struct hotplug_slot 
*slot, u8 *state)
         */
        ret = pnv_pci_get_power_state(php_slot->id, &power_state);
        if (ret) {
-               pci_warn(php_slot->pdev, "Error %d getting power status\n",
+               SLOT_WARN(php_slot, "Error %d getting power status\n",
                         ret);
        } else {
                *state = power_state;
@@ -402,7 +405,7 @@ static int pnv_php_get_adapter_state(struct hotplug_slot 
*slot, u8 *state)
                *state = presence;
                ret = 0;
        } else {
-               pci_warn(php_slot->pdev, "Error %d getting presence\n", ret);
+               SLOT_WARN(php_slot, "Error %d getting presence\n", ret);
        }
 
        return ret;
@@ -637,7 +640,7 @@ static int pnv_php_register_slot(struct pnv_php_slot 
*php_slot)
        ret = pci_hp_register(&php_slot->slot, php_slot->bus,
                              php_slot->slot_no, php_slot->name);
        if (ret) {
-               pci_warn(php_slot->pdev, "Error %d registering slot\n", ret);
+               SLOT_WARN(php_slot, "Error %d registering slot\n", ret);
                return ret;
        }
 
@@ -690,7 +693,7 @@ static int pnv_php_enable_msix(struct pnv_php_slot 
*php_slot)
        /* Enable MSIx */
        ret = pci_enable_msix_exact(pdev, &entry, 1);
        if (ret) {
-               pci_warn(pdev, "Error %d enabling MSIx\n", ret);
+               SLOT_WARN(php_slot, "Error %d enabling MSIx\n", ret);
                return ret;
        }
 
@@ -734,7 +737,7 @@ static irqreturn_t pnv_php_interrupt(int irq, void *data)
                   (sts & PCI_EXP_SLTSTA_PDC)) {
                ret = pnv_pci_get_presence_state(php_slot->id, &presence);
                if (ret) {
-                       pci_warn(pdev, "PCI slot [%s] error %d getting presence 
(0x%04x), to retry the operation.\n",
+                       SLOT_WARN(php_slot, "PCI slot [%s] error %d getting 
presence (0x%04x), to retry the operation.\n",
                                 php_slot->name, ret, sts);
                        return IRQ_HANDLED;
                }
@@ -764,7 +767,7 @@ static irqreturn_t pnv_php_interrupt(int irq, void *data)
         */
        event = kzalloc(sizeof(*event), GFP_ATOMIC);
        if (!event) {
-               pci_warn(pdev, "PCI slot [%s] missed hotplug event 0x%04x\n",
+               SLOT_WARN(php_slot, "PCI slot [%s] missed hotplug event 
0x%04x\n",
                         php_slot->name, sts);
                return IRQ_HANDLED;
        }
@@ -789,7 +792,7 @@ static void pnv_php_init_irq(struct pnv_php_slot *php_slot, 
int irq)
        /* Allocate workqueue */
        php_slot->wq = alloc_workqueue("pciehp-%s", 0, 0, php_slot->name);
        if (!php_slot->wq) {
-               pci_warn(pdev, "Cannot alloc workqueue\n");
+               SLOT_WARN(php_slot, "Cannot alloc workqueue\n");
                pnv_php_disable_irq(php_slot, true);
                return;
        }
@@ -813,7 +816,7 @@ static void pnv_php_init_irq(struct pnv_php_slot *php_slot, 
int irq)
                          php_slot->name, php_slot);
        if (ret) {
                pnv_php_disable_irq(php_slot, true);
-               pci_warn(pdev, "Error %d enabling IRQ %d\n", ret, irq);
+               SLOT_WARN(php_slot, "Error %d enabling IRQ %d\n", ret, irq);
                return;
        }
 
@@ -849,7 +852,7 @@ static void pnv_php_enable_irq(struct pnv_php_slot 
*php_slot)
 
        ret = pci_enable_device(pdev);
        if (ret) {
-               pci_warn(pdev, "Error %d enabling device\n", ret);
+               SLOT_WARN(php_slot, "Error %d enabling device\n", ret);
                return;
        }
 
-- 
2.21.0

Reply via email to