On Fri, May 10, 2024, at 09:55, Greg Kroah-Hartman wrote:
> On Fri, May 10, 2024 at 09:10:41AM +0200, Luca Ceresoli wrote:
>>
>> +config GE_SUNH_CONNECTOR
>> + tristate "GE SUNH hotplug add-on connector"
>> + depends on OF
>> + select OF_OVERLAY
>> + select FW_LOADER
>> + select NVMEM
>> + select DRM_HOTPLUG_BRIDGE
>
> Can these be depends instead of select? 'select' causes dependencies
> that are hard, if not almost impossible, to detect at times why
> something is being enabled.
I think FW_LOADER needs to be 'select' since it is normally
a hidden symbol and gets selected by its users, all the other
ones should be 'depends on'.
Arnd
