Re: [PATCH 2/2] pwm: Kconfig: Enable ehrpwm driver to be compiled for ARCH_K3

[Vignesh Raghavendra]

Hi,

On 12/03/19 3:00 PM, Uwe Kleine-KÃnig wrote:
> On Tue, Mar 12, 2019 at 02:46:29PM +0530, Vignesh Raghavendra wrote:
>> K3 devices have the same EHRPWM IP as OMAP SoCs. Enable driver to be built
>> for K3 devices. Also, drop reference to AM33xx in help text, as IP is
>> found on multiple TI SoCs.
>>
>> Signed-off-by: Vignesh Raghavendra <vigneshr@xxxxxx>
>> ---
>>  drivers/pwm/Kconfig | 5 ++---
>>  1 file changed, 2 insertions(+), 3 deletions(-)
>>
>> diff --git a/drivers/pwm/Kconfig b/drivers/pwm/Kconfig
>> index 54f8238aac0d..c054bd1dba36 100644
>> --- a/drivers/pwm/Kconfig
>> +++ b/drivers/pwm/Kconfig
>> @@ -467,10 +467,9 @@ config  PWM_TIECAP
>>  
>>  config  PWM_TIEHRPWM
>>  	tristate "EHRPWM PWM support"
>> -	depends on ARCH_OMAP2PLUS || ARCH_DAVINCI_DA8XX
>> +	depends on ARCH_OMAP2PLUS || ARCH_DAVINCI_DA8XX || ARCH_K3
>>  	help
>> -	  PWM driver support for the EHRPWM controller found on AM33XX
>> -	  TI SOC
>> +	  PWM driver support for the EHRPWM controller found on TI SOCs
> 
> It would be great to be able to compile this driver with COMPILE_TEST.
> From a quick look the following would be appropriate:
> 
> 	depends on ARCH_OMAP2PLUS || ARCH_DAVINCI_DA8XX || ARCH_K3 || COMPILE_TEST
> 	depends on HAVE_CLK && HAS_IOMEM
>  

Thanks for the suggestion! I will send a separate patch on top of this
series to enable COMPILE TEST for this driver and other TI PWM drivers.

> Best regards
> Uwe
> 

-- 
Regards
Vignesh