Re: [U-Boot] [PATCH] mtd: nand: mxs: reset BCH earlier, too, to avoid NAND startup problems

2012-12-11 Thread Scott Wood
On 12/05/2012 02:48:47 PM, Wolfram Sang wrote: It could happen (1 out of 100 times) that NAND did not start up correctly after warm rebooting, so we end up with various failures or DMA timed out due to a stalled BCH. When resetting BCH together with GPMI, the issue could not be observed anym

Re: [U-Boot] [PATCH] mtd: nand: mxs: reset BCH earlier, too, to avoid NAND startup problems

2012-12-10 Thread Marek Vasut
Dear Wolfram Sang, > It could happen (1 out of 100 times) that NAND did not start up correctly > after warm rebooting, so we end up with various failures or DMA timed out > due to a stalled BCH. When resetting BCH together with GPMI, the issue > could not be observed anymore (after 1+ reboots)

Re: [U-Boot] [PATCH] mtd: nand: mxs: reset BCH earlier, too, to avoid NAND startup problems

2012-12-06 Thread Fabio Estevam
On Wed, Dec 5, 2012 at 9:39 PM, Wolfram Sang wrote: > Both patches are correct. Check the kernel code again, please, > especially the function arguments. Ok, all is clear after reading the comments of the kernel gpmi_reset_block function. Regards, Fabio Estevam

Re: [U-Boot] [PATCH] mtd: nand: mxs: reset BCH earlier, too, to avoid NAND startup problems

2012-12-05 Thread Wolfram Sang
On Wed, Dec 05, 2012 at 09:35:26PM -0200, Fabio Estevam wrote: > Hi Wolfram, > > On Wed, Dec 5, 2012 at 6:48 PM, Wolfram Sang wrote: > > > diff --git a/drivers/mtd/nand/mxs_nand.c b/drivers/mtd/nand/mxs_nand.c > > index 4701be8..e38e151 100644 > > --- a/drivers/mtd/nand/mxs_nand.c > > +++ b/driv

Re: [U-Boot] [PATCH] mtd: nand: mxs: reset BCH earlier, too, to avoid NAND startup problems

2012-12-05 Thread Fabio Estevam
Hi Wolfram, On Wed, Dec 5, 2012 at 6:48 PM, Wolfram Sang wrote: > diff --git a/drivers/mtd/nand/mxs_nand.c b/drivers/mtd/nand/mxs_nand.c > index 4701be8..e38e151 100644 > --- a/drivers/mtd/nand/mxs_nand.c > +++ b/drivers/mtd/nand/mxs_nand.c > @@ -1058,6 +1058,8 @@ int mxs_nand_init(struct mxs_na

Re: [U-Boot] [PATCH] mtd: nand: mxs: reset BCH earlier, too, to avoid NAND startup problems

2012-12-05 Thread Otavio Salvador
On Wed, Dec 5, 2012 at 6:48 PM, Wolfram Sang wrote: > It could happen (1 out of 100 times) that NAND did not start up correctly > after > warm rebooting, so we end up with various failures or DMA timed out due to > a > stalled BCH. When resetting BCH together with GPMI, the issue could not be > o

[U-Boot] [PATCH] mtd: nand: mxs: reset BCH earlier, too, to avoid NAND startup problems

2012-12-05 Thread Wolfram Sang
It could happen (1 out of 100 times) that NAND did not start up correctly after warm rebooting, so we end up with various failures or DMA timed out due to a stalled BCH. When resetting BCH together with GPMI, the issue could not be observed anymore (after 1+ reboots). We probably need the consi