mtd: rawnand: timings: Fix default tR_max and tCCS_min timings

commit 4d8ec041d9c454029f6cd90622f6d81eb61e781c upstream.

tR and tCCS are currently wrongly expressed in femtoseconds, while we
expect these values to be expressed in picoseconds. Set right
hardcoded values.

Fixes: 6a943386ee36 mtd: rawnand: add default values for dynamic timings
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: Boris Brezillon <boris.brezillon@collabora.com>
Link: https://lore.kernel.org/linux-mtd/20200428094302.14624-3-miquel.raynal@bootlin.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/mtd/nand/raw/nand_timings.c b/drivers/mtd/nand/raw/nand_timings.c
index f64b06a71dfa1e2f1713b5cfb3dd10842e9a91c6..f12b7a7844c946a70c88fe25a7a690033561ada6 100644 (file)
--- a/drivers/mtd/nand/raw/nand_timings.c
+++ b/drivers/mtd/nand/raw/nand_timings.c
@@ -314,10 +314,9 @@ int onfi_fill_data_interface(struct nand_chip *chip,
                /* microseconds -> picoseconds */
                timings->tPROG_max = 1000000ULL * ONFI_DYN_TIMING_MAX;
                timings->tBERS_max = 1000000ULL * ONFI_DYN_TIMING_MAX;
-               timings->tR_max = 1000000ULL * 200000000ULL;
 
-               /* nanoseconds -> picoseconds */
-               timings->tCCS_min = 1000UL * 500000;
+               timings->tR_max = 200000000;
+               timings->tCCS_min = 500000;
        }
 
        return 0;