Hi, Tested with the non-stacked default single configuration on ZynqMP zcu102 board and didn’t see any issue.
ZynqMP> sf probe 0 0 0 SF: Detected mt25qu512a with page size 256 Bytes, erase size 64 KiB, total 64 MiB ZynqMP> sf erase 0x0 0x4000000;mw.b 0x8000 aabbccdd 0x4000000;sf write 0x8000 0x0 0x4000000;mw.b 0x8008000 0x0 0x4000000;sf read 0x8008000 0x0 0x4000000;cmp.b 0x8000 0x8008000 0x4000000 SF: 67108864 bytes @ 0x0 Erased: OK device 0 whole chip SF: 67108864 bytes @ 0x0 Written: OK device 0 whole chip SF: 67108864 bytes @ 0x0 Read: OK Total of 67108864 byte(s) were the same Thanks Venkatesh > -----Original Message----- > From: Marek Vasut <[email protected]> > Sent: Wednesday, October 23, 2024 2:12 PM > To: Abbarapu, Venkatesh <[email protected]>; Marek Vasut > <[email protected]>; [email protected] > Cc: Andre Przywara <[email protected]>; Ashok Reddy Soma > <[email protected]>; Jagan Teki <[email protected]>; > Michael Walle <[email protected]>; Simek, Michal <[email protected]>; > Patrice Chotard <[email protected]>; Patrick Delaunay > <[email protected]>; Pratyush Yadav <[email protected]>; Quentin > Schulz <[email protected]>; Sean Anderson <[email protected]>; > Simon Glass <[email protected]>; Takahiro Kuwano > <[email protected]>; Tom Rini <[email protected]>; Tudor > Ambarus <[email protected]>; uboot-stm32@st-md- > mailman.stormreply.com > Subject: Re: [PATCH 1/6] Revert "spi: zynq_qspi: Add parallel memories > support in > QSPI driver" > > On 10/23/24 5:18 AM, Abbarapu, Venkatesh wrote: > > Hi Marek, > > There was some issue and fix is sent > > https://lore.kernel.org/u-boot/20241018082644.22495-1-venkatesh.abbara > > [email protected]/T/#u > > Is this one fix or three fixes for three different issues ? > > This seems to fix READ errors, which is apparently another error introduced > by this > stuff. In my case, plain and simply 'sf probe ; sf update' combination with > single non- > stacked SPI NOR does not work. Was such a simple configuration ever tested ? > > > Not sure we need to revert whole parallel/stacked support? > Please stop top-posting.
