From mboxrd@z Thu Jan  1 00:00:00 1970
From: Pavel Machek <pavel@denx.de>
Date: Mon, 30 May 2016 09:04:17 +0200
Subject: [U-Boot] Between 2015.10 and 2016.01,
 SPI flash changed erase size, and saveenv stopped working
In-Reply-To: <574BB1B8.8010400@denx.de>
References: <20160529204041.GA8588@amd>
 <574BB1B8.8010400@denx.de>
Message-ID: <20160530070417.GB9828@amd>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Mon 2016-05-30 05:21:28, Stefan Roese wrote:
> Hi Pavel,
> 
> On 29.05.2016 22:40, Pavel Machek wrote:
> >Hi!
> >
> >On board with Altera Cyclone 5:
> >
> >=> version
> >
> >U-Boot 2016.01-05658-g81689ec (May 29 2016 - 22:25:45 +0200)
> >arm-linux-gnueabi-gcc (GCC) 4.8.2
> >GNU ld (GNU Binutils) 2.24
> >Saving Environment to SPI Flash...
> >spi_flash_std_probe: slave=0bf367d0, cs=0
> >SF: Detected N25Q512 with page size 256 Bytes, erase size 64 KiB,
> >total 64 MiB
> >offset 0x40000 is protected and cannot be erased
> >
> >...
> >
> >=> version
> >
> >U-Boot 2015.10-04078-g3b6d086 (May 29 2016 - 22:33:36 +0200)
> >arm-linux-gnueabi-gcc (GCC) 4.8.2
> >GNU ld (GNU Binutils) 2.24
> >SF: Detected N25Q512 with page size 256 Bytes, erase size 4 KiB, total
> >64 MiB
> >
> >
> >Does anyone else see the problem? Any ideas?
> 
> I think you have run into the S-bit / cache issue with version
> the 2016 U-Boot version. This was fixed with these patches:
> 
> mtd: cqspi: Simplify indirect write code
> mtd: cqspi: Simplify indirect read code
> 
> Please use the v2016.05 release.

Sorry if I was unclear, but it is also broken in v2016.03, v2016.05
and v2016.07. I hoped narowing down range where it broke would help.

Best regards,
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html