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

On 05/03/2026 21:05, Frank Li wrote:

On Mon, Mar 02, 2026 at 10:49:28AM -0500, Frank Li wrote:

quoted

On Mon, Mar 02, 2026 at 11:53:00AM +0100, Max Krummenacher wrote:

quoted

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

quoted

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