Re: [PATCH v3 2/2] arm64: dts: Replace deprecated extcon-usb-gpio id-gpio/vbus-gpio properties

From: Alexander Stein <alexander.stein@ew.tq-group.com>
To: "Rob Herring" <robh+dt@kernel.org>,
	"Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Benoît Cousson" <bcousson@baylibre.com>,
	"Tony Lindgren" <tony@atomide.com>,
	"Shawn Guo" <shawnguo@kernel.org>,
	"Sascha Hauer" <s.hauer@pengutronix.de>,
	"Fabio Estevam" <festevam@gmail.com>,
	"Matthias Brugger" <matthias.bgg@gmail.com>,
	"AngeloGioacchino Del Regno"
	<angelogioacchino.delregno@collabora.com>,
	"Andy Gross" <agross@kernel.org>,
	"Bjorn Andersson" <andersson@kernel.org>,
	"Konrad Dybcio" <konrad.dybcio@linaro.org>,
	"Heiko Stuebner" <heiko@sntech.de>,
	"Krzysztof Kozlowski" <krzysztof.kozlowski@linaro.org>
Cc: Pengutronix Kernel Team <kernel@pengutronix.de>,
	NXP Linux Team <linux-imx@nxp.com>,
	devicetree@vger.kernel.org, linux-omap@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-mediatek@lists.infradead.org,
	linux-arm-msm@vger.kernel.org,
	linux-rockchip@lists.infradead.org
Subject: Re: [PATCH v3 2/2] arm64: dts: Replace deprecated extcon-usb-gpio id-gpio/vbus-gpio properties
Date: Mon, 24 Jul 2023 10:49:16 +0200	[thread overview]
Message-ID: <3178020.5fSG56mABF@steina-w> (raw)
In-Reply-To: <e0897453-54b0-2a2c-2bf0-1f771c0c6565@linaro.org>

Hi Krzysztof,

Am Montag, 24. Juli 2023, 09:10:43 CEST schrieb Krzysztof Kozlowski:
> On 24/07/2023 08:05, Alexander Stein wrote:
> > Hi,
> > 
> > Am Freitag, 21. Juli 2023, 14:22:06 CEST schrieb Krzysztof Kozlowski:
> >> On 21/07/2023 10:19, Alexander Stein wrote:
> >>> Use id-gpios and vbus-gpios instead.
> >>> 
> >>> Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
> >>> Acked-by: Heiko Stuebner <heiko@sntech.de> #rockchip
> >>> Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com> #mediatek
> >>> Reviewed-by: AngeloGioacchino Del Regno
> >>> <angelogioacchino.delregno@collabora.com> Acked-by: Shawn Guo
> >>> <shawnguo@kernel.org>
> >>> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> >>> ---
> >>> Changes in v3:
> >>> * Rebased to next-20230721
> >>> * Split from bindings patch
> >> 
> >> I think you wanted to split it per subsystem, right? That's why you
> >> resent/v3? But the split did not happen.
> > 
> > Yes, I split it into dt bindings and DT changes patches. Is this not
> > correct?
> We talked about DTS patch - this one. It was already split between
> bindings and DTS, so this would not have been a topic at all.

Ah, sorry. I misunderstood and thought you were referring to bindings and DT 
changes in one series. My bad.

> >> If you decide not to split,
> >> then try to figure out: who should pick up this patchset?
> > 
> > Well, intention was one patch for DT bindings maintainers and these two
> > patches for imx maintainer (Shawn AFAIK).
> 
> You touch there much more than IMX, so if you intend that you need to be
> pretty clear. I see there around 5% of changes from IMX, so targeting
> IMX is a weird choice.

That's true. There are several SoC families.

> > I've send patches separated by arch/
> > arm and arch/arm64 in one series, so I'm slightly confused now.
> 
> So telling you second time - don't. Split per subsystem.

Okay, I think I got it. I'll respin a new series, including patches for 
bindings and for each SoC family separately.

Best regards.
Alexander
-- 
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/