linux-kernel - RE: [PWM PATCH 0/5] Implements a common PWM API

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: <BD79186B4FD85F4B8E60E381CAEE190902152C2F@mi8nycmail19.Mi8.com>
Date:	Thu, 4 Feb 2010 18:20:41 -0500
From:	"H Hartley Sweeten" <hartleys@...ionengravers.com>
To:	"H Hartley Sweeten" <hartleys@...ionengravers.com>,
	"Bill Gatliff" <bgat@...lgatliff.com>,
	<linux-embedded@...r.kernel.org>
Cc:	<linux-kernel@...r.kernel.org>
Subject: RE: [PWM PATCH 0/5] Implements a common PWM API

On Tuesday, February 02, 2010 10:44 AM, H Hartley Sweeten wrote:
> On Tuesday, February 02, 2010 12:15 AM, Bill Gatliff wrote:
>> This patch series implements a common PWM API.  This series incorporates
>> the feedback from the linux-embedded mailing list and elsewhere; the author
>> greatly appreciates the efforts of everyone who reviewed the previous version
>> of this code.
>> 
>> Bill Gatliff (5):
>>   API to consolidate PWM devices behind a common user and kernel
>>     interface
>>   Emulates PWM hardware using a high-resolution timer and a GPIO pin
>>   Expunge old Atmel PWMC driver, replacing it with one that conforms to
>>     the PWM API
>>   PWM-based LED control
>>   LED "dimmer" trigger
>> 
>>  Documentation/pwm.txt       |  260 ++++++++++++++++++
>>  drivers/leds/leds-pwm.c     |  224 +++++++++-------
>>  drivers/leds/ledtrig-dim.c  |   95 +++++++
>>  drivers/misc/Makefile       |    6 +-
>>  drivers/misc/atmel_pwm.c    |  409 ----------------------------
>>  drivers/pwm/atmel-pwm.c     |  589 ++++++++++++++++++++++++++++++++++++++++
>>  drivers/pwm/gpio.c          |  307 +++++++++++++++++++++
>>  drivers/pwm/pwm.c           |  633 +++++++++++++++++++++++++++++++++++++++++++
>>  include/linux/pwm.h         |   31 --
>>  include/linux/pwm/pwm-led.h |   34 +++
>>  include/linux/pwm/pwm.h     |  170 ++++++++++++
>>  11 files changed, 2217 insertions(+), 541 deletions(-)
>>  create mode 100644 Documentation/pwm.txt
>>  create mode 100644 drivers/leds/ledtrig-dim.c
>>  delete mode 100644 drivers/misc/atmel_pwm.c
>>  create mode 100644 drivers/pwm/atmel-pwm.c
>>  create mode 100644 drivers/pwm/gpio.c
>>  create mode 100644 drivers/pwm/pwm.c
>>  delete mode 100644 include/linux/pwm.h
>>  create mode 100644 include/linux/pwm/pwm-led.h
>>  create mode 100644 include/linux/pwm/pwm.h
>
> I think the following files/patches are missing from this series:
>
> drivers/Kconfig
> drivers/Makefile
> drivers/leds/Kconfig
> drivers/leds/Makefile
> drivers/pwm/Kconfig
> drivers/pwm/Makefile

Bill,

When you do repost this series please make sure that pwm support can only be
selected if CONFIG_SYSFS is enabled.  Your original patches did not have this.

I look forward to seeing the update.

BTW, setting pwm.dev = &pdev->dev in my driver did fix the Ooops.

Regards,
Hartley
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/