linux-kernel - Re: [PATCH v5] pwm: add sysfs interface

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: <20130621093207.GC31464@manwe>
Date:	Fri, 21 Jun 2013 11:32:08 +0200
From:	Thierry Reding <thierry.reding@...il.com>
To:	H Hartley Sweeten <hartleys@...ionengravers.com>
Cc:	Linux Kernel <linux-kernel@...r.kernel.org>,
	linux-pwm@...r.kernel.org, linux-doc@...r.kernel.org,
	poeschel@...onage.de, Ryan Mallon <rmallon@...il.com>,
	rob@...dley.net
Subject: Re: [PATCH v5] pwm: add sysfs interface

On Tue, Jun 11, 2013 at 10:38:59AM -0700, H Hartley Sweeten wrote:
> Add a simple sysfs interface to the generic PWM framework.
> 
>   /sys/class/pwm/
>   `-- pwmchipN/           for each PWM chip
>       |-- export          (w/o) ask the kernel to export a PWM channel
>       |-- npwm            (r/o) number of PWM channels in this PWM chip
>       |-- pwmX/           for each exported PWM channel
>       |   |-- duty_cycle  (r/w) duty cycle (in nanoseconds)
>       |   |-- enable      (r/w) enable/disable PWM
>       |   |-- period      (r/w) period (in nanoseconds)
>       |   `-- polarity    (r/w) polarity of PWM (normal/inversed)
>       `-- unexport        (w/o) return a PWM channel to the kernel
> 
> Signed-off-by: H Hartley Sweeten <hsweeten@...ionengravers.com>
> Cc: Thierry Reding <thierry.reding@...il.com>
> Cc: Lars Poeschel <poeschel@...onage.de>
> Cc: Ryan Mallon <rmallon@...il.com>
> Cc: Rob Landley <rob@...dley.net>
> ---
> v5: * rename the 'duty' attribute to 'duty_cycle'
>     * make the Kconfig option hidden and enabled when CONFIG_SYSFS
>       is enabled
>     * use sysfs_streq() in pwm_polarity_store()
> v4: * address a number of issues pointed out by Thierry Reding
>       - fix some typos and wording issues in the Documentation
>       - rename the new source file to sysfs.c
>       - rename some of the variables in sysfs.c to clarify them
>       - fix the period store so it does not change the duty cycle
>       - change the polarity attribute to use a string representation
>       - add a warning message is the pwmchip_sysfs_export() fails
> v3: * fix an issue with the export/unexport of the PWM chip
> v2: * add API documentation and update Documentation/pwm.txt
>     * fix some issues pointed out by Ryan Mallon
>     * add the pwm attributes to dev.groups so they are created
>       when the device is registered for the exported PWM
> v1: * Based on previous work by Lars Poecshel

Applied, thanks.

Thierry

Content of type "application/pgp-signature" skipped