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? > > 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 -- 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
