On Tue, Jun 28, 2016 at 10:44:25AM +0100, Tony Makkiel wrote: > +#ifdef CONFIG_ACPI > +static const struct acpi_device_id lp3952_acpi_match[] = { > + {LP3952_ACPI_NAME, 0}, > + {} > +}; > + > +MODULE_DEVICE_TABLE(acpi, lp3952_acpi_match); > +#endif Do you actually have ACPI system with this device included or are you just making this up? If there is no such system please consider dropping ACPI probing support from the driver. It can be added later if actually found on some real system. -- To unsubscribe from this list: send the line "unsubscribe linux-leds" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html