Re: [PATCH v7 3/3] virt: vmgenid: add support for devicetree bindings

[Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>]

On 19/04/2024 16:02, Jason A. Donenfeld wrote:
> Hi Rob,
> 
> On Fri, Apr 19, 2024 at 3:30 PM Landge, Sudan <sudanl@amazon.co.uk> wrote:
>>>   config VMGENID
>>>          tristate "Virtual Machine Generation ID driver"
>>>          default y
>>> -       depends on ACPI
>>> +       depends on ACPI || OF
>>
>> Version 2 of the patches had these flag but were removed in version3
>> following the below review comment from Rob:
>> "One of those is pretty much always enabled, so it can probably be dropped."
>> reference :
>> https://lore.kernel.org/lkml/CAL_JsqJoB5CYajWuntMdQrJZir+ZA-69Q0cwvxcVZAqs-mXC+Q@mail.gmail.com/
> 
> Are you sure about that? I see other drivers that depend on `ACPI ||
> OF`, and this driver is totally useless without either disabled
> (right?), so the dependency makes sense. I think? Not an expert in
> this area, so I'm happy to defer to what you know is best, but I did
> want to lightly pushback a bit.
> 

There are just 18 cases, so I would argue that it's not a real pattern
but could be just growing organically, without really thinking.

The driver could still be probed on some !ACPI and !OF system via driver
name matching.

Keeping it as is seems harmless, but OTOH, if this stays, you miss ||
COMPILE_TEST

Best regards,
Krzysztof