From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Mon, 28 Mar 2011 19:32:31 +0200 Subject: [U-Boot] [PATCH] cfi_flash: fix bug with flash banks with different sector numbers In-Reply-To: <1300727276-10204-1-git-send-email-martin.krause@tqs.de> References: <> <1300727276-10204-1-git-send-email-martin.krause@tqs.de> Message-ID: <201103281932.31952.sr@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Monday 21 March 2011 18:07:56 Martin Krause wrote: > The function find_sector() does not take into account if the flash bank > has changed since the last call. This could lead to illegal accesses inside > and beyond the flash_info_t info strcture. For example if the current > flash bank has less sectors than the last used flash bank. > > This patch adds two cheks. One that insures, that the current sector does > not exceed the allowed maximum (which is always a good idea). And one that > checks if the current access is to the same flash bank as the last access. > If not, the search loop will start with sector 0. Applied to u-boot-cfi-flash/master. Thanks. Cheers, Stefan -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de