From: Miquel Raynal <miquel.ray...@bootlin.com>

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.ray...@bootlin.com>
Reviewed-by: Boris Brezillon <boris.brezil...@collabora.com>
Link: 
https://lore.kernel.org/linux-mtd/20200428094302.14624-3-miquel.ray...@bootlin.com
Signed-off-by: Greg Kroah-Hartman <gre...@linuxfoundation.org>

---
 drivers/mtd/nand/raw/nand_timings.c |    5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

--- a/drivers/mtd/nand/raw/nand_timings.c
+++ b/drivers/mtd/nand/raw/nand_timings.c
@@ -331,10 +331,9 @@ int onfi_fill_data_interface(struct nand
                /* 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;


Reply via email to