[U-Boot] [PATCH 04/19] mtd: nand: pxa3xx_nand: fix early spurious interrupt

Stefan Roese sr at denx.de
Wed Sep 19 12:29:54 UTC 2018


On 29.08.2018 10:56, kostap at marvell.com wrote:
> From: Ofer Heifetz <oferh at marvell.com>
> 
> When the nand is first probe, and upon the first command start, the
> status bits should be cleared before the interrupts are unmasked.
> 
> This commit is taken from Linux:
> 'commit 0b14392db2e'
> ("mtd: nand: pxa3xx_nand: fix early spurious interrupt")
> 
> Signed-off-by: Chris Packham <judge.packham at gmail.com>
> Signed-off-by: Ofer Heifetz <oferh at marvell.com>
> Reviewed-by: Igal Liberman <igall at marvell.com>
> Cc: Stefan Roese <sr at denx.de>
> Cc: Simon Glass <sjg at chromium.org>
> ---
>   drivers/mtd/nand/pxa3xx_nand.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
> index 8e450fb..7b1dcb2 100644
> --- a/drivers/mtd/nand/pxa3xx_nand.c
> +++ b/drivers/mtd/nand/pxa3xx_nand.c
> @@ -481,8 +481,8 @@ static void pxa3xx_nand_start(struct pxa3xx_nand_info *info)
>   	ndcr |= NDCR_ND_RUN;
>   
>   	/* clear status bits and run */
> -	nand_writel(info, NDCR, 0);
>   	nand_writel(info, NDSR, NDSR_MASK);
> +	nand_writel(info, NDCR, 0);
>   	nand_writel(info, NDCR, ndcr);
>   }
>   
> 

Applied to u-boot-marvell/master

Thanks,
Stefan


More information about the U-Boot mailing list