Re: [PATCH v2 05/10] pwm: ntxec: Add driver for PWM function in Netronix EC

From: Andy Shevchenko <hidden>
Date: 2020-09-08 08:48:23
Also in: linux-arm-kernel, linux-pwm, linux-rtc, lkml

On Tue, Sep 8, 2020 at 11:14 AM Lee Jones [off-list ref] wrote:

On Sat, 05 Sep 2020, Andy Shevchenko wrote:

quoted

On Saturday, September 5, 2020, Jonathan Neuschäfer [off-list ref]
wrote:

...

quoted

+config PWM_NTXEC
+       tristate "Netronix embedded controller PWM support"

quoted

+       depends on MFD_NTXEC && OF


I don’t see need to reduce test coverage and use of the driver by sticking
with OF. Actually it’s not used.

If the device is only known to boot with OF, then it's pointless
building it when !OF.

No, it's not. As I pointed out the (compilation) test coverage is better.

If you want to increase test coverage enable
COMPILE_TEST instead.

It is one way to achieve that, yes;

       depends on OF || COMPILE_TEST
       depends on MFD_NTXEC

-- 
With Best Regards,
Andy Shevchenko