linux-kernel - Re: [PATCH v2] pwm: Add support for configuring the PWM polarity

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] [day] [month] [year] [list]

Message-ID: <20120809054708.GD29806@avionic-0098.mockup.avionic-design.de>
Date:	Thu, 9 Aug 2012 07:47:08 +0200
From:	Thierry Reding <thierry.reding@...onic-design.de>
To:	"Philip, Avinash" <avinashphilip@...com>
Cc:	linux-kernel@...r.kernel.org, nsekhar@...com
Subject: Re: [PATCH v2] pwm: Add support for configuring the PWM polarity

On Tue, Jul 24, 2012 at 07:35:32PM +0530, Philip, Avinash wrote:
> Some hardware supports inverting the polarity of the PWM signal. This
> commit adds support to the PWM framework to allow users of the PWM API
> to configure the polarity. Note that in order to reduce complexity,
> changing the polarity of a PWM signal is only allowed while the PWM is
> disabled.
> 
> A practical example where this can prove useful is to simulate inversion
> of the duty cycle. While inversion of polarity and duty cycle are not
> exactly the same, the differences for most use-cases are negligible.
> 
> Signed-off-by: Philip, Avinash <avinashphilip@...com>
> ---
> changes in  V2:
> 	- Updates commit message
> 	- Provides proper comments
> 	- Removes usage of polarity variable.
> 	- Removes encoding of polarity in flags
> 
> Configuring polarity to be done with PWM device disabled, if not failure
> reported.
> 
> If PWM device is enabled while configuring polarity, disabling and
> re_enabling make it complex. Whoever uses this API has to taken care of
> the basic rules.
> 
> Discussions related to this can found at
> http://www.spinics.net/lists/kernel/msg1372110.html
> 
> :100644 100644 ecb7690... f0a1db3... M	drivers/pwm/core.c
> :100644 100644 21d076c... 354764c... M	include/linux/pwm.h
>  drivers/pwm/core.c  |   22 ++++++++++++++++++++++
>  include/linux/pwm.h |   23 +++++++++++++++++++++++
>  2 files changed, 45 insertions(+), 0 deletions(-)

Applied, thanks.

Thierry

Content of type "application/pgp-signature" skipped