Re: [U-Boot] [PATCH] cfi_flash: fix bug with flash banks with different sector numbers

2011-03-28 Thread Stefan Roese
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

Re: [U-Boot] [PATCH] cfi_flash: fix bug with flash banks with different sector numbers

2011-03-28 Thread Stefan Roese
Hi Detlev, On Monday 28 March 2011 18:11:37 Detlev Zundel wrote: > >> Can you please comment on Martins fix? Thanks! > > > > I already did and asked for a non line-wrapped patch version: > > > > http://lists.denx.de/pipermail/u-boot/2011-March/088950.html > > > > Still no answer though. Martin

Re: [U-Boot] [PATCH] cfi_flash: fix bug with flash banks with different sector numbers

2011-03-28 Thread Detlev Zundel
Hi Stefan, > Hi Detlev, > > On Monday 28 March 2011 17:55:03 Detlev Zundel 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 exampl

Re: [U-Boot] [PATCH] cfi_flash: fix bug with flash banks with different sector numbers

2011-03-28 Thread Stefan Roese
Hi Detlev, On Monday 28 March 2011 17:55:03 Detlev Zundel 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

Re: [U-Boot] [PATCH] cfi_flash: fix bug with flash banks with different sector numbers

2011-03-28 Thread Detlev Zundel
Hello Stefan, > 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.

Re: [U-Boot] [PATCH] cfi_flash: fix bug with flash banks with different sector numbers

2011-03-28 Thread Detlev Zundel
Martin Krause writes: > 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 fl

[U-Boot] [PATCH] cfi_flash: fix bug with flash banks with different sector numbers

2011-03-21 Thread Martin Krause
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 c