We previously verified that "action" is either BUS_NOTIFY_ADD_DEVICE or BUS_NOTIFY_DEL_DEVICE, so there's no need for the default "?" case.
Found by Coverity (CID 146520). Signed-off-by: Bjorn Helgaas <[email protected]> --- drivers/xen/pci.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/xen/pci.c b/drivers/xen/pci.c index dd9c249ea311..59a25f937bd7 100644 --- a/drivers/xen/pci.c +++ b/drivers/xen/pci.c @@ -177,8 +177,7 @@ static int xen_pci_notifier(struct notifier_block *nb, } if (r) dev_err(dev, "Failed to %s - passthrough or MSI/MSI-X might fail!\n", - action == BUS_NOTIFY_ADD_DEVICE ? "add" : - (action == BUS_NOTIFY_DEL_DEVICE ? "delete" : "?")); + action == BUS_NOTIFY_ADD_DEVICE ? "add" : "del"); return NOTIFY_OK; } -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
