linux-kernel - Re: [PATCH 2/2] pwm: Kconfig: Enable TI ECAP driver for ARCH

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <ef269f60-b92d-c8af-77b2-d3d299814b5c@ti.com>
Date:   Fri, 5 Oct 2018 10:51:13 -0500
From:   "Andrew F. Davis" <afd@...com>
To:     Vignesh R <vigneshr@...com>,
        Thierry Reding <thierry.reding@...il.com>,
        Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>
CC:     <linux-pwm@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>,
        Linux ARM Mailing List <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH 2/2] pwm: Kconfig: Enable TI ECAP driver for ARCH_K3

On 10/03/2018 12:06 PM, Vignesh R wrote:
> K3 devices have the same ECAP IP as OMAP SoC. Enable driver to be built
> for K3 devices.
> 
> Signed-off-by: Vignesh R <vigneshr@...com>
> Signed-off-by: Sekhar Nori <nsekhar@...com>
> ---
>  drivers/pwm/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/pwm/Kconfig b/drivers/pwm/Kconfig
> index 504d252716f2..0c7bc6f554cf 100644
> --- a/drivers/pwm/Kconfig
> +++ b/drivers/pwm/Kconfig
> @@ -447,7 +447,7 @@ config PWM_TEGRA
>  
>  config  PWM_TIECAP
>  	tristate "ECAP PWM support"
> -	depends on ARCH_OMAP2PLUS || ARCH_DAVINCI_DA8XX || ARCH_KEYSTONE
> +	depends on ARCH_OMAP2PLUS || ARCH_DAVINCI_DA8XX || ARCH_KEYSTONE || ARCH_K3
>  	help
>  	  PWM driver support for the ECAP APWM controller found on AM33XX
>  	  TI SOC
> 

Minor, perhaps calling out AM33XX specifically in the help text is no
longer needed and a bit confusing.

Andrew