[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <cfd19433-c8de-4157-bdde-dcb6aa490792@app.fastmail.com>
Date: Tue, 07 May 2024 09:43:33 +0200
From: "Arnd Bergmann" <arnd@...db.de>
To: "Esben Haabendal" <esben@...nix.com>
Cc: "Russell King" <linux@...linux.org.uk>, "Shawn Guo" <shawnguo@...nel.org>,
"Sascha Hauer" <s.hauer@...gutronix.de>,
"Pengutronix Kernel Team" <kernel@...gutronix.de>,
"Fabio Estevam" <festevam@...il.com>, "Dong Aisheng" <aisheng.dong@....com>,
"Jacky Bai" <ping.bai@....com>, "Linus Walleij" <linus.walleij@...aro.org>,
"Rasmus Villemoes" <rasmus.villemoes@...vas.dk>,
linux-arm-kernel@...ts.infradead.org, imx@...ts.linux.dev,
linux-kernel@...r.kernel.org,
"open list:GPIO SUBSYSTEM" <linux-gpio@...r.kernel.org>
Subject: Re: [PATCH v2 2/3] pinctrl: freescale: Use CONFIG_SOC_IMXRT to guard i.MX
RT1xxx drivers
On Tue, May 7, 2024, at 09:37, Esben Haabendal wrote:
> "Arnd Bergmann" <arnd@...db.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.
Ok, great! Sorry I missed that bit. Whole series
Acked-by: Arnd Bergmann <arnd@...db.de>
Powered by blists - more mailing lists