Devicetree
 help / color / mirror / Atom feed
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


      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