devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Heiko Stübner" <heiko@sntech.de>
To: Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Cristian Ciocaltea <cristian.ciocaltea@collabora.com>,
	Jean Delvare <jdelvare@suse.com>,
	Guenter Roeck <linux@roeck-us.net>
Cc: linux-hwmon@vger.kernel.org, devicetree@vger.kernel.org,
	kernel@collabora.com, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-rockchip@lists.infradead.org
Subject: Re: (subset) [PATCH v2 0/2] Add PWM fan support to Rock 5B board
Date: Wed, 05 Apr 2023 23:35:04 +0200	[thread overview]
Message-ID: <840877554.0ifERbkFSE@diego> (raw)
In-Reply-To: <1a914522-8780-febd-7224-121b1f1fc2ba@roeck-us.net>

Am Mittwoch, 5. April 2023, 21:04:04 CEST schrieb Guenter Roeck:
> On 4/5/23 10:45, Heiko Stuebner wrote:
> > On Tue, 4 Apr 2023 20:38:05 +0300, Cristian Ciocaltea wrote:
> >> This patch series enables support for the PWM controlled heat sink fan
> >> on the Rock 5B SBC and, additionally, converts the hwmon PWM fan bindings
> >> to DT schema format.
> >>
> >> Changes in v2:
> >>   - Updated PATCH 1/2 according to Rob's review
> >>   - Added Reviewed-by from Christopher to PATCH 2/2
> >>   - v1: https://lore.kernel.org/lkml/20230403105052.426135-1-cristian.ciocaltea@collabora.com/
> >>
> >> [...]
> > 
> > Applied, thanks!
> > 
> 
> Both patches or only the second one ?

only the second one of course. According to earlier talks the dts patch
does already follow the text binding, so the yaml conversion is actually
not a requirement for it.

So I picked the dts patch let the binding sort itself when people have
time for it :-) .


Looking at the mail b4 created, I guessed this would also be visible
with the subject saying (subset) and the listed commits only showing

[2/2] arm64: dts: rockchip: rk3588-rock-5b: Add pwm-fan
      commit: f36bb17653e4b9e26bbdb1224027d20614e77636


Heiko



  reply	other threads:[~2023-04-05 21:35 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-04 17:38 [PATCH v2 0/2] Add PWM fan support to Rock 5B board Cristian Ciocaltea
2023-04-04 17:38 ` [PATCH v2 1/2] dt-bindings: hwmon: pwm-fan: Convert to DT schema Cristian Ciocaltea
2023-04-06 16:45   ` Rob Herring
2023-04-06 16:55     ` Cristian Ciocaltea
2023-04-04 17:38 ` [PATCH v2 2/2] arm64: dts: rockchip: rk3588-rock-5b: Add pwm-fan Cristian Ciocaltea
2023-04-05 17:45 ` (subset) [PATCH v2 0/2] Add PWM fan support to Rock 5B board Heiko Stuebner
2023-04-05 19:04   ` Guenter Roeck
2023-04-05 21:35     ` Heiko Stübner [this message]
2023-04-06 18:27       ` Guenter Roeck
2023-04-06 18:40         ` Heiko Stübner

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=840877554.0ifERbkFSE@diego \
    --to=heiko@sntech.de \
    --cc=cristian.ciocaltea@collabora.com \
    --cc=devicetree@vger.kernel.org \
    --cc=jdelvare@suse.com \
    --cc=kernel@collabora.com \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-hwmon@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-rockchip@lists.infradead.org \
    --cc=linux@roeck-us.net \
    --cc=robh+dt@kernel.org \
    /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;
as well as URLs for NNTP newsgroup(s).