On Mon, Feb 25, 2019 at 04:57:15PM +0200, Andy Shevchenko wrote: > > > + if (!dev->driver->acpi_match_table) > > > > If a driver does not have acpi_match_table is it expected to be > > compatible with PRP0001 or should you check for adev->data.of_compatible > > here as well? > > It's checked anyway in acpi_of_match_device(). I see no point for the check > duplication. Indeed, then it does not make sense. Thanks for the clarification.