Re: [PATCH] Revert "ARM: dts: imx: move nand related property under nand@0"

Krzysztof Kozlowski
2026-03-05T20:37:31Z

On 05/03/2026 21:05, Frank Li wrote:
> On Mon, Mar 02, 2026 at 10:49:28AM -0500, Frank Li wrote:
>> On Mon, Mar 02, 2026 at 11:53:00AM +0100, Max Krummenacher wrote:
>>> for our NAND-based modules, and our CI no longer produces any useful
>>> results as a consequence.
>>>
>>> I strongly believe that, as an immediate first step, the commit
>>> that moved the DT NAND chip configuration into its subnode should
>>> be reverted.
>>
>> As my previous said, can do partial revert? Just add back property under
>> gpmi and keep nand@0 node.
>>
>> For example
>>
>> --- a/arch/arm/boot/dts/nxp/imx/imx6qdl-skov-cpu.dtsi
>> +++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-skov-cpu.dtsi
>> @@ -260,14 +260,10 @@ fixed-link {
>> &gpmi {
>> pinctrl-names = "default";
>> pinctrl-0 = <&pinctrl_gpmi_nand>;
>> + nand-on-flash-bbt;
>> #address-cells = <1>;
>> #size-cells = <0>;
>> status = "okay";
>>
>> Remove "- nand@0" part. You can use "git add -p" to do that.
>
> Do you plan rework this patch as what I said?

Issue and revert were posted two weeks ago. That's maximum time to
handle regression.

Please apply the patch to fix the regression.

#regzbot introduced: 8124b4a4a96b

>
> Frank
>
>>
>> Frank
>>
>>

Please kindly trim the replies from unnecessary context. It makes it
much easier to find new content.

There is huge chunk of quotes below which no one should scroll through
to look for your replies.

Best regards,
Krzysztof