On Fri, Feb 23, 2024 at 03:39:10PM +0000, Richard Fitzgerald wrote: > +config SND_SOC_CS_AMP_LIB_TEST > + tristate "KUnit test for Cirrus Logic cs-amp-lib" > + depends on ACPI || COMPILE_TEST || KUNIT Should this not depend unconditionally on KUNIT rather than KUNIT or some other stuff? ie depends on ACPI || COMPILE_TEST depends on KUNIT or equivalent. > +#define TYPESAFE_ACTIVATE_STATIC_STUB_PTR(test, fn_ptr, replacement_fn) \ > + do { \ > + typecheck_fn(typeof(fn_ptr), replacement_fn); \ > + __kunit_activate_static_stub(test, fn_ptr, replacement_fn); \ > + } while (0) Should this be somewhere more generic in the kunit headers?
Attachment:
signature.asc
Description: PGP signature