Re: [PATCH v4 3/3] arm64: dts: imx93: Add phyBOARD-Segin-i.MX93 support

From: Mathieu Othacehe <othacehe@gnu.org>
To: Stefan Wahren <wahrenst@gmx.net>
Cc: Wadim Egorov <w.egorov@phytec.de>,
	 Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	 Conor Dooley <conor+dt@kernel.org>,
	 Shawn Guo <shawnguo@kernel.org>,
	 Sascha Hauer <s.hauer@pengutronix.de>,
	 Pengutronix Kernel Team <kernel@pengutronix.de>,
	 Fabio Estevam <festevam@gmail.com>,
	 NXP Linux Team <linux-imx@nxp.com>,
	 Li Yang <leoyang.li@nxp.com>,
	 Primoz Fiser <primoz.fiser@norik.com>,
	 Christoph Stoidner <c.stoidner@phytec.de>,
	devicetree@vger.kernel.org,  linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,  upstream@lists.phytec.de
Subject: Re: [PATCH v4 3/3] arm64: dts: imx93: Add phyBOARD-Segin-i.MX93 support
Date: Wed, 24 Jan 2024 14:48:14 +0100	[thread overview]
Message-ID: <87o7da4zc1.fsf@gnu.org> (raw)
In-Reply-To: <47c79a0a-5be0-4ee8-87d4-fd03809a9664@gmx.net> (Stefan Wahren's message of "Wed, 24 Jan 2024 12:39:26 +0100")

Hello Stefan,

>> Defining line names should be fine. But I would still prefer to have
>> the muxing in an overlay bound to a specific use case.
> I'm fine with this. Unfortunately Mathieu dropped the line names in V5
> today :-(
>
> AFAIR reviewers should have 2 weeks time maximum. This was just 2 days.

I am sorry but it is not easy for me to deal with contradictory input. I
chose to remove the gpio-line-names even though it also seemed like a
nice addition to me. The idea was to not interfere with Phytec plans in
the future.

There is no hurry and I can always restore them in a v6.

Let me know what you think,

Thanks,

Mathieu