Re: [PATCH v2] gpio: menz127: add support for 16Z034 and 16Z037 GPIO controllers

[Krzysztof Kozlowski <krzk@kernel.org>]

On 17/11/2025 14:38, Jose Javier Rodriguez Barbarin wrote:
>>>  MODULE_DEVICE_TABLE(mcb, men_z127_ids);
>>> @@ -204,4 +224,6 @@ MODULE_AUTHOR("Andreas Werner <andreas.werner@men.de>");
>>>  MODULE_DESCRIPTION("MEN 16z127 GPIO Controller");
>>>  MODULE_LICENSE("GPL v2");
>>>  MODULE_ALIAS("mcb:16z127");
>>> +MODULE_ALIAS("mcb:16z034");
>>> +MODULE_ALIAS("mcb:16z037");
>>
>> Why do you need these? You have MODULE_DEVICE_TABLE() just few lines above.
>>
> 
> I added this new MODULE_ALIAS() because it is the mechanism that previous contributors
> used to enable autoloading of mcb device drivers. After reading your comment,
> I decided to investigate further how the MODULE_DEVICE_TABLE() macro is used and processed.
> I discovered that MODULE_DEVICE_TABLE(mcb, ...) was being ignored by the kernel build
> system. For this reason, I'm preparing a new patch to add support for MCB devices 
> in file2alias.
> 
> Therefore, I will send a V3 patch that simply removes these new MODULE_ALIAS() entries,
> and another patch to add support for automatically generating module aliases based on 
> MODULE_DEVICE_TABLE() instead of MODULE_ALIAS().

Thanks, looks like good approach.

Best regards,
Krzysztof