On Monday 25 February 2008, Zhang, Rui wrote: > > + if (!ACPI_FAILURE(status)) { > > + int ret; > > + > > + ret = sysfs_create_link(&dev->kobj, &acpi_dev->dev.kobj, > > + "firmware_node"); > > + ret = sysfs_create_link(&acpi_dev->dev.kobj, &dev->kobj, > > + "physical_node"); > > + } > > ret is not needed here... But what compiler warnings do you see when you omit it? :) - 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