On 25/03/2023 06:59, Herbert Xu wrote: > On Fri, Mar 24, 2023 at 11:51:43AM +0100, Krzysztof Kozlowski wrote: >> >>> You should either add a dependency on OF or __maybe_unused. >>> Adding both makes no sense. >> >> Of course it makes, otherwise you have warnings which is also fixed here. > > It won't create a warning if you depend on OF without COMPILE_TEST. In that case yes, but we want COMPILE_TEST for build coverage. > > Anyway, I think adding __maybe_unused by itself is sufficient. Best regards, Krzysztof