Received-SPF: pass (google.com: domain of linux-kernel+bounces-34843-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
From: Mathieu Othacehe <othacehe@gnu.org>
To: Wadim Egorov <w.egorov@phytec.de>
Cc: 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>,  Stefan
 Wahren <wahrenst@gmx.net>,  Christoph Stoidner <c.stoidner@phytec.de>,
  <devicetree@vger.kernel.org>,  <linux-kernel@vger.kernel.org>,
  <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v4 3/3] arm64: dts: imx93: Add phyBOARD-Segin-i.MX93
 support
In-Reply-To: <537266fe-0bf7-4208-a9f3-ae27f462c6ed@phytec.de> (Wadim Egorov's
	message of "Tue, 23 Jan 2024 07:11:27 +0100")
References: <20240122095306.14084-1-othacehe@gnu.org>
	<20240122095306.14084-4-othacehe@gnu.org>
	<537266fe-0bf7-4208-a9f3-ae27f462c6ed@phytec.de>
Date: Tue, 23 Jan 2024 08:44:55 +0100
Message-ID: <87ttn45w94.fsf@gnu.org>
User-Agent: Gnus/5.13 (Gnus v5.13)
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain


Hey Wadim,

> You are doing more than you describing in your changes log.
> Here you are forcing a gpio-only functionality for the X16 header. But the
> pins we route down to the X16 expansion connector can be also used
> differently.
> Typically we provide device tree overlays for different use cases on this
> expansion connectors.

Adding those GPIOs on X16 was a suggestion from Stefan back in v3. I
must say that it is quite convenient as a user to be able to interact
with gpiolib on some GPIOs on the eval board.

Would it be a possibility to change the muxing of those two pins in the
overlays if the need arises?

> Maybe this comment was lost, but I asked to move the dts-v1 tag into the
> carrier board device tree.

Missed that, sorry.

Thanks,

Mathieu