SPI chip select lines

RE: SPI chip select lines

dejans — Tue, 16 Aug 2022 07:43:53 GMT

Hi,

Thank you for additional information.

Parameter delay is defined in the spi_cs_control struct. It is defined as delay in microseconds to wait before starting the transmission and before releasing the CS line.

Best regards,
Dejan

RE: SPI chip select lines

jmilliken — Mon, 15 Aug 2022 14:47:05 GMT

Hi. I am referring to the macro SPI_CS_CONTROL_PTR_DT(DT_NODELABEL(irda_uart1), 2). Jumping to the definition of the macro gives me the following:

* @param node_id Devicetree node identifier for a device on a SPI bus

* @param delay_ The @p delay field to set in the @p spi_cs_control

* @return a pointer to the @p spi_cs_control structure

#define SPI_CS_CONTROL_PTR_DT(node_id, delay_) \

(&(struct spi_cs_control) { \

.gpio = DT_SPI_DEV_CS_GPIOS_DT_SPEC_GET(node_id), \

.delay = (delay_), \

})

So this delay does not seem to be used, or at least I don't understand how it is used. Changing the value does not seem to make a difference in the outcome.

RE: SPI chip select lines

dejans — Mon, 15 Aug 2022 14:00:09 GMT

Hi,

[quote user="jmilliken"]it might be worth mentioning that the delay time specified in the macro is completely ignored.[/quote]

Which macro do you refer to?

[quote user="jmilliken"]Regardless of what I put there - I've tried values from 1 to 10 - the timing on chip select is always the same.[/quote]

Could you please show where and how you specify delay values?

Best regards,
Dejan

RE: SPI chip select lines

jmilliken — Thu, 11 Aug 2022 23:57:35 GMT

Hi. I was able to make this work by changing the declaration of the spi_config struct a bit to just directly use the device tree macro SPI_CS_CONTROL_PTR_DT(). So I do the following:

static const struct spi_config spi_cfg0 = {

.operation = SPI_WORD_SET(8) | SPI_TRANSFER_MSB |

SPI_MODE_CPOL | SPI_MODE_CPHA,

.frequency = 2000000,

.slave = 0,

.cs = SPI_CS_CONTROL_PTR_DT(DT_NODELABEL(irda_uart0), 2),

};

I can also target the other CS pin by creating a second struct:

static const struct spi_config spi_cfg1 = {

.operation = SPI_WORD_SET(8) | SPI_TRANSFER_MSB |

SPI_MODE_CPOL | SPI_MODE_CPHA,

.frequency = 2000000,

.slave = 0,

.cs = SPI_CS_CONTROL_PTR_DT(DT_NODELABEL(irda_uart1), 2),

};

I test this with the following:

if(ping_pong) {

ping_pong = false;

err = spi_transceive(spi_dev, &spi_cfg0, &tx, &rx);

} else {

ping_pong = true;

err = spi_transceive(spi_dev, &spi_cfg1, &tx, &rx);

}

And can see on the scope, triggering off the clock line, that the chip select pins alternate. So all good there. However, it might be worth mentioning that the delay time specified in the macro is completely ignored. Regardless of what I put there - I've tried values from 1 to 10 - the timing on chip select is always the same. CS line is pulled low 10usec before the clock starts and then pulled high 30usec after clock ends. This timing is fine and fact specifying a chip select delay for a SPI in msec is somewhat ludicrous in the first place. But I'm curious what that delay parameter actually controls.

-Jamie

RE: SPI chip select lines

dejans — Mon, 01 Aug 2022 14:57:00 GMT

Hi,

Could you provide zephyr.dts file which is located in <your_application_folder>\build\zephyr?

Could you please provide more information about "spi-device" that is present in your overlay file?

Best regards,
Dejan

RE: SPI chip select lines

jmilliken — Sun, 31 Jul 2022 16:08:50 GMT

One thing I noticed - when I hover over the &spi1 in my overlay file to expand its contents, I see that it has added the cs-gpios information and pin assignments from the overlay. But the irda_uart nodes are showing that content automatically commented out. It seems that everything I define in there is being discarded, yet I'm not getting errors or warnings. See below.