Linux-Clk Archive mirror
 help / color / mirror / Atom feed
From: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
To: Alexandre Mergnat <amergnat@baylibre.com>
Cc: Chun-Kuang Hu <chunkuang.hu@kernel.org>,
	 Philipp Zabel <p.zabel@pengutronix.de>,
	David Airlie <airlied@gmail.com>,
	 Daniel Vetter <daniel@ffwll.ch>,
	Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
	 Maxime Ripard <mripard@kernel.org>,
	Thomas Zimmermann <tzimmermann@suse.de>,
	 Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	 Conor Dooley <conor+dt@kernel.org>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	 AngeloGioacchino Del Regno
	<angelogioacchino.delregno@collabora.com>,
	Jitao Shi <jitao.shi@mediatek.com>, CK Hu <ck.hu@mediatek.com>,
	 Michael Turquette <mturquette@baylibre.com>,
	Stephen Boyd <sboyd@kernel.org>,
	 Catalin Marinas <catalin.marinas@arm.com>,
	Will Deacon <will@kernel.org>,
	dri-devel@lists.freedesktop.org,
	 linux-mediatek@lists.infradead.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	 linux-arm-kernel@lists.infradead.org, linux-pwm@vger.kernel.org,
	linux-clk@vger.kernel.org
Subject: Re: [PATCH v2 12/18] dt-bindings: pwm: mediatek,pwm-disp: add compatible for mt8365 SoC
Date: Thu, 18 Apr 2024 18:05:41 +0200	[thread overview]
Message-ID: <jl35q23p7axlxaxxwo7652qke5luft4x6zsz3ldntegdpvmdom@ge6taosuwtnj> (raw)
In-Reply-To: <286945bc-f207-4373-9589-0a9b62df1b36@baylibre.com>

[-- Attachment #1: Type: text/plain, Size: 2121 bytes --]

Hello Alexandre,

On Thu, Apr 18, 2024 at 11:46:11AM +0200, Alexandre Mergnat wrote:
> On 17/04/2024 10:06, Uwe Kleine-König wrote:
> > On Tue, Apr 16, 2024 at 05:53:13PM +0200, Alexandre Mergnat wrote:
> > > Add a compatible string for MediaTek Genio 350 MT8365's display PWM
> > > block: this is the same as MT8183.
> > > 
> > > Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
> > > Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> > > Signed-off-by: Alexandre Mergnat <amergnat@baylibre.com>
> > 
> > I already asked in reply to v1 what the merge plan is here. There are
> > changes in my pwm tree to the mediatek,pwm-disp binding already. I don't
> > think they conflict with this patch, but maybe it's still easier to take
> > this via pwm?!
> 
> Sorry, I though the merge strategy wasn't addressed to me because I'm not a maintainer.

Well, you're the canonical person who has an interest to get this series
in, so you can at least have and express an idea. In my experience it's
good to be proactive and suggest a merge plan. Maintainers are triggered
more to reply by a presented plan they don't agree with than with
suggesting a plan themselves. And this way at least there is less
surprise for you as submitter.

> It's fine for me to merge it in the PWM tree.
> IMO, this change [1] shouldn't conflict with this patch.

FTR: [1] is in Linus's tree already now.

> Can you add the "Acked-by: Rob Herring (Arm) <robh@kernel.org>" please if you merge this version ?

I picked the one from your v3 series now where you included Rob's Ack. I
was a bit grumpy because I only spotted your reply here afterwards.
(Everything fine now on my side now.)

Best regards
Uwe

> [1]: https://git.kernel.org/pub/scm/linux/kernel/git/ukleinek/linux.git/commit/Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml?h=pwm/for-next&id=fb7c3d8ba039df877886fd457538d8b24ca9c84b

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

  reply	other threads:[~2024-04-18 16:06 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-04-16 15:53 [PATCH v2 00/18] Add display support for the MT8365-EVK board Alexandre Mergnat
2024-04-16 15:53 ` [PATCH v2 01/18] dt-bindings: display: mediatek: aal: add compatible for MT8365 SoC Alexandre Mergnat
2024-04-17 10:19   ` AngeloGioacchino Del Regno
2024-04-17 18:43   ` Rob Herring
2024-04-16 15:53 ` [PATCH v2 02/18] dt-bindings: display: mediatek: ccorr: " Alexandre Mergnat
2024-04-17 10:19   ` AngeloGioacchino Del Regno
2024-04-17 18:43   ` Rob Herring
2024-04-16 15:53 ` [PATCH v2 03/18] dt-bindings: display: mediatek: color: " Alexandre Mergnat
2024-04-17 10:19   ` AngeloGioacchino Del Regno
2024-04-17 18:43   ` Rob Herring
2024-04-16 15:53 ` [PATCH v2 04/18] dt-bindings: display: mediatek: dither: " Alexandre Mergnat
2024-04-17 10:19   ` AngeloGioacchino Del Regno
2024-04-17 18:43   ` Rob Herring
2024-04-16 15:53 ` [PATCH v2 05/18] dt-bindings: display: mediatek: dsi: " Alexandre Mergnat
2024-04-17 10:21   ` AngeloGioacchino Del Regno
2024-04-17 18:44   ` Rob Herring
2024-04-16 15:53 ` [PATCH v2 06/18] dt-bindings: display: mediatek: dpi: add power-domains property amergnat
2024-04-17 10:15   ` AngeloGioacchino Del Regno
2024-04-17 18:44   ` Rob Herring
2024-04-16 15:53 ` [PATCH v2 07/18] dt-bindings: display: mediatek: dpi: add compatible for MT8365 Alexandre Mergnat
2024-04-17 10:14   ` AngeloGioacchino Del Regno
2024-04-16 15:53 ` [PATCH v2 08/18] dt-bindings: display: mediatek: gamma: add compatible for MT8365 SoC Alexandre Mergnat
2024-04-17 10:19   ` AngeloGioacchino Del Regno
2024-04-17 18:47   ` Rob Herring
2024-04-16 15:53 ` [PATCH v2 09/18] dt-bindings: display: mediatek: ovl: " Alexandre Mergnat
2024-04-17 10:19   ` AngeloGioacchino Del Regno
2024-04-17 18:47   ` Rob Herring
2024-04-16 15:53 ` [PATCH v2 10/18] dt-bindings: display: mediatek: rdma: " Alexandre Mergnat
2024-04-17 10:19   ` AngeloGioacchino Del Regno
2024-04-17 18:48   ` Rob Herring
2024-04-16 15:53 ` [PATCH v2 11/18] dt-bindings: pwm: mediatek,pwm-disp: add power-domains property Alexandre Mergnat
2024-04-17  8:01   ` Uwe Kleine-König
2024-04-17  8:06     ` Alexandre Mergnat
2024-04-17 10:19   ` AngeloGioacchino Del Regno
2024-04-17 13:25     ` Uwe Kleine-König
2024-04-17 13:50       ` AngeloGioacchino Del Regno
2024-04-16 15:53 ` [PATCH v2 12/18] dt-bindings: pwm: mediatek,pwm-disp: add compatible for mt8365 SoC Alexandre Mergnat
2024-04-17  8:06   ` Uwe Kleine-König
2024-04-18  9:46     ` Alexandre Mergnat
2024-04-18 16:05       ` Uwe Kleine-König [this message]
2024-04-17 18:48   ` Rob Herring
2024-04-16 15:53 ` [PATCH v2 13/18] drm/mediatek: dsi: Improves the DSI lane setup robustness Alexandre Mergnat
2024-04-16 15:53 ` [PATCH v2 14/18] drm/mediatek: add mt8365 dpi support Alexandre Mergnat
2024-04-17 10:14   ` AngeloGioacchino Del Regno
2024-04-16 15:53 ` [PATCH v2 15/18] drm/mediatek: add MT8365 SoC support amergnat
2024-04-16 15:53 ` [PATCH v2 16/18] arm64: defconfig: enable display connector support Alexandre Mergnat
2024-04-16 15:53 ` [PATCH v2 17/18] arm64: dts: mediatek: add display blocks support for the MT8365 SoC Alexandre Mergnat
2024-04-17 10:27   ` AngeloGioacchino Del Regno
2024-04-16 15:53 ` [PATCH v2 18/18] arm64: dts: mediatek: add display support for mt8365-evk Alexandre Mergnat
2024-04-17 10:27   ` AngeloGioacchino Del Regno
2024-04-18 13:02     ` Alexandre Mergnat
2024-04-17  7:58 ` [PATCH v2 00/18] Add display support for the MT8365-EVK board Uwe Kleine-König
2024-04-17  8:12   ` Alexandre Mergnat

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=jl35q23p7axlxaxxwo7652qke5luft4x6zsz3ldntegdpvmdom@ge6taosuwtnj \
    --to=u.kleine-koenig@pengutronix.de \
    --cc=airlied@gmail.com \
    --cc=amergnat@baylibre.com \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=catalin.marinas@arm.com \
    --cc=chunkuang.hu@kernel.org \
    --cc=ck.hu@mediatek.com \
    --cc=conor+dt@kernel.org \
    --cc=daniel@ffwll.ch \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=jitao.shi@mediatek.com \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=linux-pwm@vger.kernel.org \
    --cc=maarten.lankhorst@linux.intel.com \
    --cc=matthias.bgg@gmail.com \
    --cc=mripard@kernel.org \
    --cc=mturquette@baylibre.com \
    --cc=p.zabel@pengutronix.de \
    --cc=robh@kernel.org \
    --cc=sboyd@kernel.org \
    --cc=tzimmermann@suse.de \
    --cc=will@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 read-only IMAP folder(s) and NNTP newsgroup(s).