linux-serial.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Heiko Stübner" <heiko@sntech.de>
To: Detlev Casanova <detlev.casanova@collabora.com>,
	Guenter Roeck <linux@roeck-us.net>
Cc: linux-kernel@vger.kernel.org, Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
	Maxime Ripard <mripard@kernel.org>,
	Thomas Zimmermann <tzimmermann@suse.de>,
	David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>,
	Andi Shyti <andi.shyti@kernel.org>,
	Jonathan Cameron <jic23@kernel.org>,
	Lars-Peter Clausen <lars@metafoo.de>, Lee Jones <lee@kernel.org>,
	Ulf Hansson <ulf.hansson@linaro.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Jiri Slaby <jirislaby@kernel.org>,
	Mark Brown <broonie@kernel.org>,
	Wim Van Sebroeck <wim@linux-watchdog.org>,
	Chris Morgan <macromorgan@hotmail.com>,
	Jonas Karlman <jonas@kwiboo.se>, Tim Lunn <tim@feathertop.org>,
	Chukun Pan <amadeus@jmu.edu.cn>, Andy Yan <andyshrk@163.com>,
	Muhammed Efe Cetin <efectn@protonmail.com>,
	Jagan Teki <jagan@edgeble.ai>, Dragan Simic <dsimic@manjaro.org>,
	Ondrej Jirman <megi@xff.cz>,
	Michael Riesch <michael.riesch@wolfvision.net>,
	Jimmy Hon <honyuenkwun@gmail.com>,
	Alexey Charkov <alchark@gmail.com>,
	Elon Zhang <zhangzj@rock-chips.com>,
	Elaine Zhang <zhangqing@rock-chips.com>,
	Yifeng Zhao <yifeng.zhao@rock-chips.com>,
	Finley Xiao <finley.xiao@rock-chips.com>,
	Liang Chen <cl@rock-chips.com>,
	Jisheng Zhang <jszhang@kernel.org>,
	Jamie Iles <jamie@jamieiles.com>,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-rockchip@lists.infradead.org,
	dri-devel@lists.freedesktop.org, linux-i2c@vger.kernel.org,
	linux-iio@vger.kernel.org, linux-mmc@vger.kernel.org,
	linux-serial@vger.kernel.org, linux-spi@vger.kernel.org,
	linux-watchdog@vger.kernel.org, kernel@collabora.com
Subject: Re: [PATCH v2 09/12] dt-bindings: watchdog: Add rockchip,rk3576-wdt compatible
Date: Thu, 29 Aug 2024 13:52:37 +0200	[thread overview]
Message-ID: <61607306.matp6XCIr4@diego> (raw)
In-Reply-To: <5254cf14-65d1-4ffa-a1fb-265a51dda37d@roeck-us.net>

Hi Guenter,

Am Dienstag, 27. August 2024, 21:38:35 CEST schrieb Guenter Roeck:
> On 8/27/24 00:20, Heiko Stübner wrote:
> > Am Samstag, 24. August 2024, 18:44:29 CEST schrieb Guenter Roeck:
> >> On Fri, Aug 23, 2024 at 10:52:36AM -0400, Detlev Casanova wrote:
> >>> It is compatible with the other rockchip SoCs.
> >>>
> >>> Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com>
> >>
> >> Acked-by: Guenter Roeck <linux@roeck-us.net>
> > 
> > For my understanding, does this Ack mean you expect the patch to go in
> > with the other patches?
> > 
> Yes
> 
> > Because in theory this patch could also be picked and go through the
> > watchdog tree, similar to how the saradc binding went into the
> > iio tree already.
> > 
> 
> I thought it was all supposed to be pushed together.

I think at this point the rk3576 has still quite a number of separate
pieces. A series for clocks, one for mmc and I think there are more.

So for stuff that is obvious, like the saradc compatible Jonathan already,
picked or this watchdog compatible, a strategy of "someone picks the
individual" patch also is helpful, because it reduces the number of pieces
on the "chess board" ;-) .


So I guess both ways (wdt binding getting applied, or me just applying it
with the rest) are doable and we'll just follow what you feel comfortable
with doing.

Heiko





  reply	other threads:[~2024-08-29 11:52 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-08-23 14:52 [PATCH v2 00/12] Add device tree for ArmSoM Sige 5 board Detlev Casanova
2024-08-23 14:52 ` [PATCH v2 01/12] dt-bindings: arm: rockchip: Add ArmSoM Sige 5 Detlev Casanova
2024-08-23 14:52 ` [PATCH v2 02/12] dt-bindings: arm: rockchip: Add rk3576 compatible string to pmu.yaml Detlev Casanova
2024-08-23 14:52 ` [PATCH v2 03/12] dt-bindings: i2c: i2c-rk3x: Add rk3576 compatible Detlev Casanova
2024-08-23 14:52 ` [PATCH v2 04/12] dt-bindings: iio: adc: Add rockchip,rk3576-saradc string Detlev Casanova
2024-08-24  6:31   ` Krzysztof Kozlowski
2024-08-23 14:52 ` [PATCH v2 05/12] dt-bindings: mfd: syscon: Add rk3576 QoS register compatible Detlev Casanova
2024-08-23 14:52 ` [PATCH v2 06/12] dt-bindings: serial: snps-dw-apb-uart: Add Rockchip RK3576 Detlev Casanova
2024-08-23 14:52 ` [PATCH v2 07/12] dt-bindings: mmc: Add support for rk3576 eMMC Detlev Casanova
2024-08-24  7:49   ` Krzysztof Kozlowski
2024-08-23 14:52 ` [PATCH v2 08/12] dt-bindings: gpu: Add rockchip,rk3576-mali compatible Detlev Casanova
2024-08-24  7:51   ` Krzysztof Kozlowski
2024-08-23 14:52 ` [PATCH v2 09/12] dt-bindings: watchdog: Add rockchip,rk3576-wdt compatible Detlev Casanova
2024-08-24  7:52   ` Krzysztof Kozlowski
2024-08-24 16:44   ` Guenter Roeck
2024-08-27  7:20     ` Heiko Stübner
2024-08-27 19:38       ` Guenter Roeck
2024-08-29 11:52         ` Heiko Stübner [this message]
2024-08-23 16:07 ` [PATCH v2 10/12] dt-bindings: spi: Add rockchip,rk3576-spi compatible Detlev Casanova
2024-08-24  7:53   ` Krzysztof Kozlowski
2024-08-23 16:11 ` [PATCH v2 11/12] arm64: dts: rockchip: Add rk3576 SoC base DT Detlev Casanova
2024-08-25 14:08   ` Chukun Pan
2024-08-26 18:28     ` Detlev Casanova
2024-08-26 19:56       ` Diederik de Haas
     [not found]   ` <4849dae7-24c8-4581-9ad1-1d731be2fd51@rock-chips.com>
2024-08-26 17:54     ` Detlev Casanova
2024-08-27  1:02       ` zhangqing
2024-08-23 16:12 ` [PATCH v2 12/12] arm64: dts: rockchip: Add rk3576-armsom-sige5 board Detlev Casanova
2024-08-25 14:25   ` Chukun Pan
2024-08-26 19:38     ` Detlev Casanova

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=61607306.matp6XCIr4@diego \
    --to=heiko@sntech.de \
    --cc=airlied@gmail.com \
    --cc=alchark@gmail.com \
    --cc=amadeus@jmu.edu.cn \
    --cc=andi.shyti@kernel.org \
    --cc=andyshrk@163.com \
    --cc=broonie@kernel.org \
    --cc=cl@rock-chips.com \
    --cc=conor+dt@kernel.org \
    --cc=daniel@ffwll.ch \
    --cc=detlev.casanova@collabora.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=dsimic@manjaro.org \
    --cc=efectn@protonmail.com \
    --cc=finley.xiao@rock-chips.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=honyuenkwun@gmail.com \
    --cc=jagan@edgeble.ai \
    --cc=jamie@jamieiles.com \
    --cc=jic23@kernel.org \
    --cc=jirislaby@kernel.org \
    --cc=jonas@kwiboo.se \
    --cc=jszhang@kernel.org \
    --cc=kernel@collabora.com \
    --cc=krzk+dt@kernel.org \
    --cc=lars@metafoo.de \
    --cc=lee@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-i2c@vger.kernel.org \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=linux-rockchip@lists.infradead.org \
    --cc=linux-serial@vger.kernel.org \
    --cc=linux-spi@vger.kernel.org \
    --cc=linux-watchdog@vger.kernel.org \
    --cc=linux@roeck-us.net \
    --cc=maarten.lankhorst@linux.intel.com \
    --cc=macromorgan@hotmail.com \
    --cc=megi@xff.cz \
    --cc=michael.riesch@wolfvision.net \
    --cc=mripard@kernel.org \
    --cc=robh@kernel.org \
    --cc=tim@feathertop.org \
    --cc=tzimmermann@suse.de \
    --cc=ulf.hansson@linaro.org \
    --cc=wim@linux-watchdog.org \
    --cc=yifeng.zhao@rock-chips.com \
    --cc=zhangqing@rock-chips.com \
    --cc=zhangzj@rock-chips.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;
as well as URLs for NNTP newsgroup(s).