[U-Boot] [PATCH 03/19] mtd: nand: pxa3xx_nand: sync pxa3xx_nand_set_sdr_timing()

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


On 29.08.2018 10:56, kostap at marvell.com wrote:
> From: Ofer Heifetz <oferh at marvell.com>
> 
> Since the pxa3xx_nand driver was added there has been a discrepancy in
> pxa3xx_nand_set_sdr_timing() around the setting of tWP_min and tRP_min.
> This brings us into line with the current Linux code.
> 
> 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 | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
> index 6295886..8e450fb 100644
> --- a/drivers/mtd/nand/pxa3xx_nand.c
> +++ b/drivers/mtd/nand/pxa3xx_nand.c
> @@ -349,9 +349,9 @@ static void pxa3xx_nand_set_sdr_timing(struct pxa3xx_nand_host *host,
>   	u32 tCH_min = DIV_ROUND_UP(t->tCH_min, 1000);
>   	u32 tCS_min = DIV_ROUND_UP(t->tCS_min, 1000);
>   	u32 tWH_min = DIV_ROUND_UP(t->tWH_min, 1000);
> -	u32 tWP_min = DIV_ROUND_UP(t->tWC_min - tWH_min, 1000);
> +	u32 tWP_min = DIV_ROUND_UP(t->tWC_min - t->tWH_min, 1000);
>   	u32 tREH_min = DIV_ROUND_UP(t->tREH_min, 1000);
> -	u32 tRP_min = DIV_ROUND_UP(t->tRC_min - tREH_min, 1000);
> +	u32 tRP_min = DIV_ROUND_UP(t->tRC_min - t->tREH_min, 1000);
>   	u32 tR = chip->chip_delay * 1000;
>   	u32 tWHR_min = DIV_ROUND_UP(t->tWHR_min, 1000);
>   	u32 tAR_min = DIV_ROUND_UP(t->tAR_min, 1000);
> 

Applied to u-boot-marvell/master

Thanks,
Stefan


More information about the U-Boot mailing list