drivers: spi: nxp_lpspi: Use spi_config timing params

Use the timing params from spi_config that are specific to the slave
instead of using the same timing for the controller for all slaves.
Remove these properties from the LPSPI DT binding.

Signed-off-by: Declan Snyder <[email protected]>

Author: decsny

drivers/spi/spi_nxp_lpspi/spi_nxp_lpspi_common.c

@@ -149,16 +149,22 @@ static uint8_t lpspi_calc_delay_scaler(uint32_t desired_delay_ns,
 }
 
 /* returns CCR mask of the bits 8-31 */
-static inline uint32_t lpspi_set_delays(const struct device *dev, uint32_t prescaled_clock)
+static inline uint32_t lpspi_set_delays(const struct spi_config *spi_cfg,
+					uint32_t prescaled_clock)
 {
-	const struct lpspi_config *config = dev->config;
+	uint32_t val = 0;
+
+	if (!spi_cs_is_gpio(spi_cfg)) {
+		val |= LPSPI_CCR_PCSSCK(lpspi_calc_delay_scaler(spi_cfg->cs.lead,
+								prescaled_clock, 1)) |
+		       LPSPI_CCR_SCKPCS(lpspi_calc_delay_scaler(spi_cfg->cs.lag,
+								prescaled_clock, 1));
+	}
+
+       val |= LPSPI_CCR_DBT(lpspi_calc_delay_scaler(spi_get_word_delay(spi_cfg),
+						    prescaled_clock, 2));
 
-	return LPSPI_CCR_PCSSCK(lpspi_calc_delay_scaler(config->pcs_sck_delay,
-							prescaled_clock, 1)) |
-	       LPSPI_CCR_SCKPCS(lpspi_calc_delay_scaler(config->sck_pcs_delay,
-							prescaled_clock, 1)) |
-	       LPSPI_CCR_DBT(lpspi_calc_delay_scaler(config->transfer_delay,
-							prescaled_clock, 2));
+       return val;
 }
 
 /* This is the equation for the sck frequency given a div and prescaler. */
@@ -315,7 +321,7 @@ int lpspi_configure(const struct device *dev, const struct spi_config *spi_cfg)
 
 		/* sckdiv algorithm must run *before* delays are set in order to know prescaler */
 		ccr |= lpspi_set_sckdiv(spi_cfg->frequency, clock_freq, &prescaler);
-		ccr |= lpspi_set_delays(dev, clock_freq / TWO_EXP(prescaler));
+		ccr |= lpspi_set_delays(spi_cfg, clock_freq / TWO_EXP(prescaler));
 
 		/* note that not all bits of the register are readable on some platform,
 		 * that's why we update it on one write

drivers/spi/spi_nxp_lpspi/spi_nxp_lpspi_priv.h

@@ -97,9 +97,6 @@ int lpspi_wait_tx_fifo_empty(const struct device *dev);
 		.clock_dev = DEVICE_DT_GET(DT_INST_CLOCKS_CTLR(n)),                                \
 		.clock_subsys = (clock_control_subsys_t)DT_INST_CLOCKS_CELL(n, name),              \
 		.irq_config_func = lpspi_config_func_##n,                                          \
-		.pcs_sck_delay = DT_INST_PROP_OR(n, pcs_sck_delay, 0),                             \
-		.sck_pcs_delay = DT_INST_PROP_OR(n, sck_pcs_delay, 0),                             \
-		.transfer_delay = DT_INST_PROP_OR(n, transfer_delay, 0),                           \
 		.pincfg = PINCTRL_DT_INST_DEV_CONFIG_GET(n),                                       \
 		.data_pin_config = (uint8_t)DT_INST_ENUM_IDX(n, data_pin_config),                  \
 		.tristate_output = DT_INST_PROP(n, tristate_output),                               \

dts/bindings/spi/nxp,lpspi.yaml

@@ -14,27 +14,6 @@ properties:
   clocks:
     required: true
 
-  pcs-sck-delay:
-    type: int
-    description: |
-      Delay in nanoseconds from the chip select assert to the first clock
-      edge.
-      If not set, the minimum supported delay is used.
-
-  sck-pcs-delay:
-    type: int
-    description: |
-      Delay in nanoseconds from the last clock edge to the chip select
-      deassert.
-      If not set, the minimum supported delay is used.
-
-  transfer-delay:
-    type: int
-    description: |
-      Delay in nanoseconds between last SCK edge of a transfer word and the
-      first SCK edge of the next transfer word.
-      If not set, the minimum supported delay is used.
-
   data-pin-config:
     type: string
     enum: