On Mon, Jan 11, 2016 at 08:31:57AM -0600, Bjorn Helgaas wrote: > From: Julia Lawall <julia.lawall@xxxxxxx> > > Remove unneeded NULL test. The index variable of list_for_each_entry is > never NULL, as it is the structure that contains the list pointer. > > Generated by: scripts/coccinelle/iterators/itnull.cocci > > Signed-off-by: Fengguang Wu <fengguang.wu@xxxxxxxxx> > Signed-off-by: Julia Lawall <julia.lawall@xxxxxxx> > Signed-off-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> > CC: Geliang Tang <geliangtang@xxxxxxx> I applied this to pci/hotplug and plan to include it for v4.5. I'm just resending it so it's on the linux-pci list. > --- > drivers/pci/hotplug/ibmphp_core.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/pci/hotplug/ibmphp_core.c b/drivers/pci/hotplug/ibmphp_core.c > index 010e46a..49adf22 100644 > --- a/drivers/pci/hotplug/ibmphp_core.c > +++ b/drivers/pci/hotplug/ibmphp_core.c > @@ -503,9 +503,6 @@ static int __init init_ops(void) > int rc; > > list_for_each_entry(slot_cur, &ibmphp_slot_head, ibm_slot_list) { > - if (!slot_cur) > - return -ENODEV; > - > debug("BEFORE GETTING SLOT STATUS, slot # %x\n", > slot_cur->number); > if (slot_cur->ctrl->revision == 0xFF) > > -- > To unsubscribe from this list: send the line "unsubscribe linux-pci" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html