Re: [PATCH v4 1/3] mtd: nand: increase ready wait timeout and report timeouts

2015-09-07 Thread Alex Smith
Hi Ezequiel, Thanks for reviewing the series. On 06/09/2015 21:37, Ezequiel Garcia wrote: > On 27 Jul 02:50 PM, Alex Smith wrote: >> If nand_wait_ready() times out, this is silently ignored, and its >> caller will then proceed to read from/write to the chip before it is >> ready. This can

Re: [PATCH v4 1/3] mtd: nand: increase ready wait timeout and report timeouts

2015-09-07 Thread Alex Smith
Hi Ezequiel, Thanks for reviewing the series. On 06/09/2015 21:37, Ezequiel Garcia wrote: > On 27 Jul 02:50 PM, Alex Smith wrote: >> If nand_wait_ready() times out, this is silently ignored, and its >> caller will then proceed to read from/write to the chip before it is >> ready. This can

Re: [PATCH v4 1/3] mtd: nand: increase ready wait timeout and report timeouts

2015-09-06 Thread Ezequiel Garcia
On 27 Jul 02:50 PM, Alex Smith wrote: > If nand_wait_ready() times out, this is silently ignored, and its > caller will then proceed to read from/write to the chip before it is > ready. This can potentially result in corruption with no indication as > to why. > > While a 20ms timeout seems like

Re: [PATCH v4 1/3] mtd: nand: increase ready wait timeout and report timeouts

2015-09-06 Thread Ezequiel Garcia
On 27 Jul 02:50 PM, Alex Smith wrote: > If nand_wait_ready() times out, this is silently ignored, and its > caller will then proceed to read from/write to the chip before it is > ready. This can potentially result in corruption with no indication as > to why. > > While a 20ms timeout seems like

[PATCH v4 1/3] mtd: nand: increase ready wait timeout and report timeouts

2015-07-27 Thread Alex Smith
If nand_wait_ready() times out, this is silently ignored, and its caller will then proceed to read from/write to the chip before it is ready. This can potentially result in corruption with no indication as to why. While a 20ms timeout seems like it should be plenty enough, certain behaviour can

[PATCH v4 1/3] mtd: nand: increase ready wait timeout and report timeouts

2015-07-27 Thread Alex Smith
If nand_wait_ready() times out, this is silently ignored, and its caller will then proceed to read from/write to the chip before it is ready. This can potentially result in corruption with no indication as to why. While a 20ms timeout seems like it should be plenty enough, certain behaviour can