Received-SPF: pass (google.com: domain of linux-kernel+bounces-170820-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: Esben Haabendal <esben@geanix.com>
To: "Arnd Bergmann" <arnd@arndb.de>
Cc: "Russell King" <linux@armlinux.org.uk>,  "Shawn Guo"
 <shawnguo@kernel.org>,  "Sascha Hauer" <s.hauer@pengutronix.de>,
  "Pengutronix Kernel Team" <kernel@pengutronix.de>,  "Fabio Estevam"
 <festevam@gmail.com>,  "Dong Aisheng" <aisheng.dong@nxp.com>,  "Jacky Bai"
 <ping.bai@nxp.com>,  "Linus Walleij" <linus.walleij@linaro.org>,  "Rasmus
 Villemoes" <rasmus.villemoes@prevas.dk>,
  linux-arm-kernel@lists.infradead.org,  imx@lists.linux.dev,
  linux-kernel@vger.kernel.org,  "open list:GPIO SUBSYSTEM"
 <linux-gpio@vger.kernel.org>
Subject: Re: [PATCH v2 2/3] pinctrl: freescale: Use CONFIG_SOC_IMXRT to
 guard i.MX RT1xxx drivers
In-Reply-To: <8accb26e-c7a8-43aa-90d5-d83d5a1575de@app.fastmail.com> (Arnd
	Bergmann's message of "Mon, 06 May 2024 13:29:43 +0200")
References: <20240506-imx-pinctrl-optional-v2-0-bdff75085156@geanix.com>
	<20240506-imx-pinctrl-optional-v2-2-bdff75085156@geanix.com>
	<8accb26e-c7a8-43aa-90d5-d83d5a1575de@app.fastmail.com>
Date: Tue, 07 May 2024 09:37:28 +0200
Message-ID: <87r0eef4xz.fsf@geanix.com>
User-Agent: Gnus/5.13 (Gnus v5.13)
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain

"Arnd Bergmann" <arnd@arndb.de> writes:

> On Mon, May 6, 2024, at 12:23, Esben Haabendal wrote:
>>
>>  config PINCTRL_IMXRT1050
>>  	bool "IMXRT1050 pinctrl driver"
>> -	depends on ARCH_MXC
>> +	depends on SOC_IMXRT
>> +	default SOC_IMXRT
>>  	select PINCTRL_IMX
>>  	help
>>  	  Say Y here to enable the imxrt1050 pinctrl driver
>
> Maybe make this
>
>        depends on SOC_IMXRT || COMPILE_TEST

That is done in patch 3/3.

> I see that all the i.MX pinctrl drivers are currently missing
> this, but a lot of other platforms have the ||COMPILE_TEST
> bit so it gets included in x86 allmodconfig tests that
> often gets run before sending or merging changes.

Take a look at patch 3/3 in this series. It does a wholesale addition of
||COMPILE_TEST to these drivers.

/Esben