mtd: rawnand: Add onfi_fill_nvddr_interface_config() helper
authorMiquel Raynal <miquel.raynal@bootlin.com>
Wed, 5 May 2021 21:37:39 +0000 (23:37 +0200)
committerMiquel Raynal <miquel.raynal@bootlin.com>
Wed, 26 May 2021 08:43:50 +0000 (10:43 +0200)
Same logic as for the SDR path, let's create a
onfi_fill_nvddr_interface_config() helper to fill an interface
configuration structure with NV-DDR timings, given a specific ONFI mode.

There is one additional thing to do compared to SDR mode: tCAD timing
can be fast or slow and this depends on an ONFI parameter page bit. By
default the slow value is declared in the timings structure definition,
but this helper can shrink it down if necessary.

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Link: https://lore.kernel.org/linux-mtd/20210505213750.257417-12-miquel.raynal@bootlin.com
drivers/mtd/nand/raw/nand_onfi.c
drivers/mtd/nand/raw/nand_timings.c
include/linux/mtd/onfi.h

index f7a4a05..8e4677f 100644 (file)
@@ -315,6 +315,7 @@ int nand_onfi_detect(struct nand_chip *chip)
        onfi->tBERS = le16_to_cpu(p->t_bers);
        onfi->tR = le16_to_cpu(p->t_r);
        onfi->tCCS = le16_to_cpu(p->t_ccs);
+       onfi->fast_tCAD = p->nvddr_nvddr2_features & BIT(0);
        onfi->sdr_timing_modes = le16_to_cpu(p->sdr_timing_modes);
        if (p->features & ONFI_FEATURE_NV_DDR)
                onfi->nvddr_timing_modes = p->nvddr_timing_modes;
index fb483e6..8eaa132 100644 (file)
@@ -637,6 +637,45 @@ static void onfi_fill_sdr_interface_config(struct nand_chip *chip,
 }
 
 /**
+ * onfi_fill_nvddr_interface_config - Initialize a NVDDR interface config from a
+ *                                    given ONFI mode
+ * @chip: The NAND chip
+ * @iface: The interface configuration to fill
+ * @timing_mode: The ONFI timing mode
+ */
+static void onfi_fill_nvddr_interface_config(struct nand_chip *chip,
+                                            struct nand_interface_config *iface,
+                                            unsigned int timing_mode)
+{
+       struct onfi_params *onfi = chip->parameters.onfi;
+
+       if (WARN_ON(timing_mode >= ARRAY_SIZE(onfi_nvddr_timings)))
+               return;
+
+       *iface = onfi_nvddr_timings[timing_mode];
+
+       /*
+        * Initialize timings that cannot be deduced from timing mode:
+        * tPROG, tBERS, tR, tCCS and tCAD.
+        * These information are part of the ONFI parameter page.
+        */
+       if (onfi) {
+               struct nand_nvddr_timings *timings = &iface->timings.nvddr;
+
+               /* microseconds -> picoseconds */
+               timings->tPROG_max = 1000000ULL * onfi->tPROG;
+               timings->tBERS_max = 1000000ULL * onfi->tBERS;
+               timings->tR_max = 1000000ULL * onfi->tR;
+
+               /* nanoseconds -> picoseconds */
+               timings->tCCS_min = 1000UL * onfi->tCCS;
+
+               if (onfi->fast_tCAD)
+                       timings->tCAD_min = 25000;
+       }
+}
+
+/**
  * onfi_fill_interface_config - Initialize an interface config from a given
  *                              ONFI mode
  * @chip: The NAND chip
@@ -651,4 +690,6 @@ void onfi_fill_interface_config(struct nand_chip *chip,
 {
        if (type == NAND_SDR_IFACE)
                return onfi_fill_sdr_interface_config(chip, iface, timing_mode);
+       else
+               return onfi_fill_nvddr_interface_config(chip, iface, timing_mode);
 }
index 14e66a4..a9677bf 100644 (file)
@@ -162,6 +162,7 @@ struct onfi_ext_param_page {
  * @tBERS: Block erase time
  * @tR: Page read time
  * @tCCS: Change column setup time
+ * @fast_tCAD: Command/Address/Data slow or fast delay (NV-DDR only)
  * @sdr_timing_modes: Supported asynchronous/SDR timing modes
  * @nvddr_timing_modes: Supported source synchronous/NV-DDR timing modes
  * @vendor_revision: Vendor specific revision number
@@ -173,6 +174,7 @@ struct onfi_params {
        u16 tBERS;
        u16 tR;
        u16 tCCS;
+       bool fast_tCAD;
        u16 sdr_timing_modes;
        u16 nvddr_timing_modes;
        u16 vendor_revision;