]> git.ipfire.org Git - thirdparty/u-boot.git/commit
Revert "zynq: cfi: Fix nor issue for Zynq"
authorMichal Simek <michal.simek@xilinx.com>
Tue, 17 Aug 2021 14:11:50 +0000 (16:11 +0200)
committerMichal Simek <michal.simek@xilinx.com>
Wed, 6 Oct 2021 11:12:31 +0000 (13:12 +0200)
commit4bfb090ec368693d934ffca34a3e8866e89227c6
treeff837891f8506f8357439608f9d89701f7a2a7a3
parentd1b891873257d630649f6dd4155f0ac4813af78c
Revert "zynq: cfi: Fix nor issue for Zynq"

This reverts commit 4bdea3b8b7ec13f6fdb8030b0d069ab7dd147b2b.

The only board which we are testing is zc770-xm012 where this patch is
used. Based on schematics it is M29EW device which is manufactured by
Numonyx 0x89 (Linux kernel is saying Intel).

Device on this board is detected as:
manufacturer is 2
manufacturer id is 0xff
device id is 0xff
device id2 is 0x0
cfi version is 0x3133
size_ratio 1 port 16 bits chip 8 bits

As is visible maufacturer is 2 which is any known one that's why this patch
was forcing max_buf_write_size to 8 likely because of reasons described by
commit c502321c4a1b ("mtd: cfi_flash: Write buffer size adjustment for
M29EW Numonyx devices").

When query data is dump it is visible at offset 0x1a there is already 0x08
which means that max_buf_write_size is correctly setup and this workaround
is not needed anymore.

max_buf_write_size offset 1a /* for offset information */
00 : 51 52 59 02 00 40 00 00 00 00 00 27 36 85 95 05  QRY..@.....'6...
10 : 09 08 0f 03 02 03 03 18 02 00 08 00 01 7f 00 00  ................
20 : 02 00 00 00 00 00 00 00 00 00 00 00 00 b4 f3 3f  ...............?

Just for summary parallel flash on zc770-xm012 is likely incorrectly
detected but max_buf_write_size is setup already to 8 which was the reason
for this patch and there is no need to have this workaround in our tree
because value is setup correctly already.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
arch/arm/mach-zynq/Kconfig
configs/xilinx_zynq_virt_defconfig
configs/zynq_cse_nor_defconfig
drivers/mtd/cfi_flash.c