On Tue, Feb 5, 2019 at 4:21 PM Life is hard, and then you die
[off-list ref] wrote:

quoted

quoted

+config KEYBOARD_APPLESPI
+   tristate "Apple SPI keyboard and trackpad"

quoted

+   depends on (X86 && ACPI && SPI) || COMPILE_TEST

COMPILE_TEST more or less makes sense in conjunction with architecture selection.
It means, your code always dependant to ACPI and SPI frameworks.
That's why 0day complained.

Thanks. Yes, looking at this again I realized I somewhat misunderstood
the uses of COMPILE_TEST. I've changed this now to

        depends on ACPI && SPI && (X86 || COMPILE_TEST)

Better to split like

depends on SPI && ACPI
depends on X86 || COMPILE_TEST

-- 
With Best Regards,
Andy Shevchenko