On Tue, 2012-11-27 at 11:24 +0200, Westerberg, Mika wrote: > On Tue, Nov 27, 2012 at 11:03:45AM +0200, Andy Shevchenko wrote: > > There is no need to initialize the node before appending it to the list. > > I might be missing something but why do we then have macros like: > list_del_init() etc? list_del_init(&list->node) allows to reuse &list->node as a new list. I guess the often use case of that when you extract the leaf node from the list of lists and create a new list based on it. In our case we just add a node to the existed list. > > > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > > --- > > drivers/acpi/resource.c | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/drivers/acpi/resource.c b/drivers/acpi/resource.c > > index 4107c00..a3868f6 100644 > > --- a/drivers/acpi/resource.c > > +++ b/drivers/acpi/resource.c > > @@ -426,7 +426,6 @@ static acpi_status acpi_dev_new_resource_entry(struct resource *r, > > c->error = -ENOMEM; > > return AE_NO_MEMORY; > > } > > - INIT_LIST_HEAD(&rentry->node); > > rentry->res = *r; > > list_add_tail(&rentry->node, c->list); > > c->count++; > > -- > > 1.7.10.4 -- Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Intel Finland Oy -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
