On Thu, Jul 13, 2017 at 5:38 PM, Sakari Ailus <sakari.ailus@xxxxxx> wrote:
[snip]
>>
>> -static const struct platform_device_id vimc_sen_driver_ids[] = {
>> +static const __maybe_unused
>> +struct platform_device_id vimc_sen_driver_ids[] = {
>> {
>> .name = VIMC_SEN_DRV_NAME,
>> },
>
> Shouldn't these be set to the corresponding driver structs' id_table
> fields? Or do I miss something...?
>
Agreed, the real problem is that the .id_table is not set for these
drivers. The match only works because the platform subsystem fallbacks
to the driver's name if an .id_table isn't defined:
http://elixir.free-electrons.com/linux/latest/source/drivers/base/platform.c#L964
Best regards,
Javier
