From: Stefan Wahren <wahrenst@gmx.net>
To: "Uwe Kleine-König" <u.kleine-koenig@baylibre.com>,
"Linus Walleij" <linus.walleij@linaro.org>
Cc: Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
andy.shevchenko@gmail.com, Philip Howard <phil@gadgetoid.com>,
Sean Young <sean@mess.org>,
Chris Morgan <macromorgan@hotmail.com>,
linux-gpio@vger.kernel.org, linux-pwm@vger.kernel.org,
devicetree@vger.kernel.org,
Nicola Di Lieto <nicola.dilieto@gmail.com>,
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
Dhruva Gole <d-gole@ti.com>,
Vincent Whitchurch <vincent.whitchurch@axis.com>,
Andy Shevchenko <andy@kernel.org>
Subject: Re: [PATCH v7 0/2] pwm: Add GPIO PWM driver
Date: Thu, 27 Jun 2024 11:54:05 +0200 [thread overview]
Message-ID: <964072fc-1aa9-4d4e-955f-8777e2692346@gmx.net> (raw)
In-Reply-To: <oytpr6q3rmyj4n6ufktl5okg7ugusks5csgkg3zvs32b4jqh24@lmfovdemzi4y>
Am 27.06.24 um 08:52 schrieb Uwe Kleine-König:
> Hello,
>
> On Tue, Jun 04, 2024 at 11:00:39PM +0200, Linus Walleij wrote:
>> Add a software PWM which toggles a GPIO from a high-resolution timer.
>>
>> Recent discussions in the Raspberry Pi community revealt that a lot
>> of users still use MMIO userspace tools for GPIO access. One argument
>> for this approach is the lack of a GPIO PWM kernel driver. So this
>> series tries to fill this gap.
>>
>> This continues the work of Vincent Whitchurch [1], which is easier
>> to read and more consequent by rejecting sleeping GPIOs than Nicola's
>> approach [2]. It further takes over where Stefan Wahren left off.
>>
>> I have not looked into the interrupt storm problem mentioned in [3]
>> but instead focused on some real-life tests:
>>
>> The IXP4xx NSLU2 has a speaker connected directly to a GPIO, and I
>> wanted to use this patch to provide a proper beeper for the machine
>> and not have to rely on custom hacks.
>>
>> I added a DTS patch like this:
>>
>> gpio_pwm: pwm {
>> #pwm-cells = <3>;
>> compatible = "pwm-gpio";
>> gpios = <&gpio0 4 GPIO_ACTIVE_HIGH>;
>> };
>>
>> beeper {
>> compatible = "pwm-beeper";
>> pwms = <&gpio_pwm 0 1 0>;
>> beeper-hz = <1000>;
>> };
> Applied both patches to
> https://git.kernel.org/pub/scm/linux/kernel/git/ukleinek/linux.git pwm/for-next
\o/
Thanks
>
> Thanks
> Uwe
prev parent reply other threads:[~2024-06-27 9:54 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-04 21:00 [PATCH v7 0/2] pwm: Add GPIO PWM driver Linus Walleij
2024-06-04 21:00 ` [PATCH v7 1/2] dt-bindings: pwm: Add pwm-gpio Linus Walleij
2024-06-04 21:00 ` [PATCH v7 2/2] pwm: Add GPIO PWM driver Linus Walleij
2024-06-05 5:52 ` Dhruva Gole
2024-06-18 11:51 ` [PATCH v7 0/2] " Phil Howard
2024-06-27 6:52 ` Uwe Kleine-König
2024-06-27 9:54 ` Stefan Wahren [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=964072fc-1aa9-4d4e-955f-8777e2692346@gmx.net \
--to=wahrenst@gmx.net \
--cc=andy.shevchenko@gmail.com \
--cc=andy@kernel.org \
--cc=conor+dt@kernel.org \
--cc=d-gole@ti.com \
--cc=devicetree@vger.kernel.org \
--cc=krzk+dt@kernel.org \
--cc=krzysztof.kozlowski@linaro.org \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-pwm@vger.kernel.org \
--cc=macromorgan@hotmail.com \
--cc=nicola.dilieto@gmail.com \
--cc=phil@gadgetoid.com \
--cc=robh@kernel.org \
--cc=sean@mess.org \
--cc=u.kleine-koenig@baylibre.com \
--cc=vincent.whitchurch@axis.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox