From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9940D2EBB96 for ; Fri, 12 Dec 2025 08:34:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1765528488; cv=none; b=K9hCUJKiNfjMZWaS8p5cF6t09Rub42MldbHGwiUXWOE01rc7yTl9SaG3Gut13u/rSpOUM/NpdN7Fhp2/bsg/b6q+05PLR9XuGM8hKk2mIY5WFeoFGTvNZE8aDce/u838h0YaoZkzSJqnRhc+NaPSVvCuoKWwaEy3X/liDPvjplQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1765528488; c=relaxed/simple; bh=1PoCoI/iDulrDrCBG+dRMcgiy7qUslVONnQjTC1K6B8=; h=Mime-Version:Content-Type:Date:Message-Id:Subject:Cc:From:To: References:In-Reply-To; b=u/C4OOZrd+patbVGm/mUSq5lpK2kDyDn4J/xPVdkIgSK+/tNMnOOQ3+qPX+vpvacJe3amH8CHmbSWLmcAJBBjw2jcp9ZOzeyTCs7jopJg7XIWQo6/P1v+MAveW01ImZcS7G1K5avIHFv+dZfnzUBEc99DZ92vsshzB8pnl0rnJk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=PtYSTgqQ; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="PtYSTgqQ" Received: by smtp.kernel.org (Postfix) with ESMTPSA id A3BC0C4CEF1; Fri, 12 Dec 2025 08:34:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1765528486; bh=1PoCoI/iDulrDrCBG+dRMcgiy7qUslVONnQjTC1K6B8=; h=Date:Subject:Cc:From:To:References:In-Reply-To:From; b=PtYSTgqQXlaD4uASHYQ+jFFe7vFU6jbOxC+csC2IniERZeEH1qMeOYw4/eAJhW1o2 cq/4dDOWI5QmqgBT1Oy6rKzSiTjAY1g5h14aGAkOZznSo4NNiUi+iKgHCYZWm9haeZ +8Pmoo/7pO8DFAVKEbGUh/VHpfQBxDofha99unrKpu2o9JX3eZgS2G31Oe8idElCrf tAnTc4/2DTAf0mbDE5CDfFI/uX9kF3VClY0NVleNVynNbMulGNG3iJvg02SPpwY22e SeWhfQjBzsutsUbV32LykwU+8W1cWluzQQsQW8XzdSN+01rC8GFfb3cdtrkaiKx0II SkqnkeZaUg3Aw== Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Type: multipart/signed; boundary=3a447c79091588e2a94044d6d01cf0e8baa40503869af63b848f49e4df49; micalg=pgp-sha384; protocol="application/pgp-signature" Date: Fri, 12 Dec 2025 09:34:41 +0100 Message-Id: Subject: Re: [PATCH] Revert "mtd: spi-nor: micron-st: use SFDP of mt35xu512aba" Cc: , , From: "Michael Walle" To: "Haibo Chen" , "Tudor Ambarus" , "Pratyush Yadav" , "Miquel Raynal" , "Richard Weinberger" , "Vignesh Raghavendra" X-Mailer: aerc 0.20.0 References: <20251212-nor-v1-1-20a5a381979c@nxp.com> In-Reply-To: <20251212-nor-v1-1-20a5a381979c@nxp.com> --3a447c79091588e2a94044d6d01cf0e8baa40503869af63b848f49e4df49 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Hi, On Fri Dec 12, 2025 at 9:30 AM CET, Haibo Chen wrote: > This reverts commit 44dd635cd632824f412ff2a6b320d9302a277ad0. > > Find two batches mt35xu512aba has different SFDP but with same > jedec ID. To make all batches of mt35xu512aba work well and > support OCT DTR mode, back to hardcode the flags. What are "batches" in this case? A new revision of the same flash chip? Why would they use a different ID then. What's wrong with the parsed flags? Both support SFDP, so if you really need to fixup any flags, please use a fixup callback to actually fix them. But first, what will go wrong? -michael > Signed-off-by: Haibo Chen > --- > mt35xu512aba chip with label 4KA17 RW303: > root@imx943evk:/sys/bus/spi/devices/spi8.0/spi-nor# cat jedec_id > 2c5b1a > root@imx943evk:/sys/bus/spi/devices/spi8.0/spi-nor# cat manufacturer > micron > root@imx943evk:/sys/bus/spi/devices/spi8.0/spi-nor# hexdump -Cv sfdp > 00000000 53 46 44 50 0a 01 03 ff 00 08 01 17 30 00 00 ff |SFDP........= 0...| > 00000010 84 00 01 02 90 00 00 ff 05 01 01 06 a0 00 00 ff |............= ....| > 00000020 0a 00 01 08 b0 00 00 ff ff ff ff ff ff ff ff ff |............= ....| > 00000030 e5 20 8a ff ff ff ff 1f 00 00 00 00 00 00 00 00 |. ..........= ....| > 00000040 ee ff ff ff ff ff 00 00 ff ff 00 00 0c 20 11 d8 |............= . ..| > 00000050 0f 52 00 00 39 61 99 00 87 8e 03 d3 ac a1 27 3d |.R..9a......= ..'=3D| > 00000060 7a 75 7a 75 fb bd d5 5c 00 00 70 ff 81 50 f8 a1 |zuzu...\..p.= .P..| > 00000070 2f cb 27 8b 00 00 04 01 00 06 01 00 ff ff ff 8e |/.'.........= ....| > 00000080 00 00 00 00 00 00 00 00 00 00 00 00 ff ff ff ff |............= ....| > 00000090 43 0e ff ff 21 dc 5c ff ff ff ff ff ff ff ff ff |C...!.\.....= ....| > 000000a0 00 0b 80 9e b1 81 b5 85 00 f0 ff 9f 00 0a 00 00 |............= ....| > 000000b0 00 0a 1a 88 10 00 00 00 ff ff ff ff ff ff ff ff |............= ....| > 000000c0 00 00 06 01 00 00 00 00 14 01 81 03 00 00 00 00 |............= ....| > 000000d0 > > mt35xu512aba chip with label 0DA15 RW303: > root@imx8qmmek:/sys/bus/spi/devices/spi4.0/spi-nor# ls > jedec_id manufacturer sfdp > root@imx8qmmek:/sys/bus/spi/devices/spi4.0/spi-nor# cat jedec_id > 2c5b1a > root@imx8qmmek:/sys/bus/spi/devices/spi4.0/spi-nor# cat manufacturer > micron > root@imx8qmmek:/sys/bus/spi/devices/spi4.0/spi-nor# hexdump -Cv sfdp > 00000000 53 46 44 50 06 01 01 ff 00 06 01 10 30 00 00 ff |SFDP........= 0...| > 00000010 84 00 01 02 80 00 00 ff ff ff ff ff ff ff ff ff |............= ....| > 00000020 ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff |............= ....| > 00000030 e5 20 8a ff ff ff ff 1f 00 00 00 00 00 00 00 00 |. ..........= ....| > 00000040 ee ff ff ff ff ff 00 00 ff ff 00 00 0c 20 11 d8 |............= . ..| > 00000050 0f 52 00 00 24 5a 99 00 8b 8e 03 e1 ac 01 27 38 |.R..$Z......= ..'8| > 00000060 7a 75 7a 75 fb bd d5 5c 00 00 70 ff 81 b0 38 36 |zuzu...\..p.= ..86| > 00000070 ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff |............= ....| > 00000080 43 0e ff ff 21 dc 5c ff |C...!.\.| > 00000088 Thanks! -michael --3a447c79091588e2a94044d6d01cf0e8baa40503869af63b848f49e4df49 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iKgEABMJADAWIQTIVZIcOo5wfU/AngkSJzzuPgIf+AUCaTvToRIcbXdhbGxlQGtl cm5lbC5vcmcACgkQEic87j4CH/hZmgGAhPjGan8Un+Viv7qOmKS+fdhnb/3Hrob0 vdlrD71Loer2dAjqQJIqW78LDjaApxKfAYCZAMOLSAt6K1oxrqEQE6bC74C5SEIc YqRLmzwFwKRkZPsUknGWy6sDPpuOaPjDgrQ= =JjRL -----END PGP SIGNATURE----- --3a447c79091588e2a94044d6d01cf0e8baa40503869af63b848f49e4df49--