SPI slave configuration & overlay

RE: SPI slave configuration & overlay

Sigurd — Thu, 13 Jan 2022 11:08:34 GMT

This new issue will be handled in this new case: https://devzone.nordicsemi.com/f/nordic-q-a/83583/spi-slave-always-returns--etimedout

Closing this case.

RE: SPI slave configuration & overlay

Ubaid_M — Thu, 13 Jan 2022 06:01:53 GMT

Hello Sigurd,

Thanks for the kind help,

Now the code execution is on track on bl653 running spi slave driver.

But the spi_read always returns -116, that is -ETIMEDOUT.

But I do not understand why slave driver always returns -ETIMEDOUT. Master board is sending data, i checked on USB logic analyzer for the same.

I traced the flow of execution all the way to the source of the error and here is what I found

spi_read( ) -> transceive (in file "spi_nrfx_spis.c") -> spi_context_wait_for_completion( in file "spi_context.h") -> and the source of error is as attached in the below snap in #else part

Without debug mode as well I am getting the same -ETIMEDOUT.

Kindly suggest on if I have missed out on configuring something.

adding my overlay file details for your reference:

&spi1 {
	compatible = "nordic,nrf-spis";
	status = "okay";
	sck-pin = <41>;
	mosi-pin = <40>;
	miso-pin = <4>;
	cs-gpios = <&gpio1 23 0>;
	csn-pin = <23>;
	def-char = <0xFF>;
};

Kindly suggest

RE: SPI slave configuration & overlay

Sigurd — Tue, 11 Jan 2022 10:27:25 GMT

[quote user="Ubaid_M"]1. what do csn-pin & def-char attributes do..?[/quote]

Here is the documentation for these attributes:

https://developer.nordicsemi.com/nRF_Connect_SDK/doc/1.8.0/zephyr/reference/devicetree/bindings/spi/nordic,nrf-spis.html

https://github.com/nrfconnect/sdk-zephyr/blob/v2.7.0-ncs1/dts/bindings/spi/nordic,nrf-spis.yaml

[quote user="Ubaid_M"]so is my definition right..?[/quote]

Looks correct

RE: SPI slave configuration & overlay

Ubaid_M — Tue, 11 Jan 2022 08:13:10 GMT

Hi Sigurd,

Sorry for the chain of updates, I was able to solve it,
Now execution is entering the slave driver file "spi_nrfx_spis.c"

I had to define extra properties in spi1 node in bl653_dvk.dts as:

&spi1 {
	compatible = "nordic,nrf-spis";
	status = "okay";
	sck-pin = <41>;
	mosi-pin = <40>;
	miso-pin = <4>;
	cs-gpios = <&gpio1 23 0>;
	csn-pin = <23>; #extra property defined
	def-char = <0xFF>;#extra property defined
};

Now Please help me with

1. what do csn-pin & def-char attributes do..?
2. I need to define slave chip select on gpio0, pin:23.,

csn-pin = <23>;

def-char = <0xFF>;

so is my definition right..?

RE: SPI slave configuration & overlay

Ubaid_M — Tue, 11 Jan 2022 07:58:08 GMT

Please find build errors when i include overlay file:

[0/1] Re-running CMake...
Including boilerplate (Zephyr base (cached)): D:/Nordic_Sample_App/Nordic_SDK/v1.8.0/zephyr/cmake/app/boilerplate.cmake
-- Application: D:/PXZ_Sample_App
-- Zephyr version: 2.7.0 (D:/Nordic_Sample_App/Nordic_SDK/v1.8.0/zephyr), build: v2.7.0-ncs1
-- Found west (found suitable version "0.12.0", minimum required is "0.7.1")
-- Board: bl653_dvk
-- Cache files will be written to: D:/Nordic_Sample_App/Nordic_SDK/v1.8.0/zephyr/.cache
-- Found dtc: D:/Nordic_Sample_App/Nordic_SDK/v1.8.0/toolchain/opt/bin/dtc.exe (found suitable version "1.4.7", minimum required is "1.4.6")
-- Found toolchain: gnuarmemb (d:/Nordic_Sample_App/Nordic_SDK/v1.8.0/toolchain/opt)
-- Found BOARD.dts: D:/Nordic_Sample_App/Nordic_SDK/v1.8.0/zephyr/boards/arm/bl653_dvk/bl653_dvk.dts
-- Found devicetree overlay: D:/PXZ_Sample_App/bl653_dvk.overlay
devicetree error: 'def-char' is marked as required in 'properties:' in D:/Nordic_Sample_App/Nordic_SDK/v1.8.0/zephyr/dts/bindings\spi\nordic,nrf-spis.yaml, but does not appear in <Node /soc/spi@40004000 in 'bl653_dvk.dts.pre.tmp'>
CMake Error at D:\Nordic_Sample_App\Nordic_SDK\v1.8.0\zephyr\cmake\dts.cmake:251 (message):
  gen_defines.py failed with return code: 1
Call Stack (most recent call first):
  D:\Nordic_Sample_App\Nordic_SDK\v1.8.0\zephyr\cmake\app\boilerplate.cmake:545 (include)
  D:\Nordic_Sample_App\Nordic_SDK\v1.8.0\zephyr\share\zephyr-package\cmake\ZephyrConfig.cmake:24 (include)
  D:\Nordic_Sample_App\Nordic_SDK\v1.8.0\zephyr\share\zephyr-package\cmake\ZephyrConfig.cmake:40 (include_boilerplate)
  d:\PXZ_Sample_App\build\CMakeLists.txt:4 (find_package)

RE: SPI slave configuration & overlay

Ubaid_M — Tue, 11 Jan 2022 07:46:52 GMT

Also, I am wondering about including overlay in CMakeLists.txt, Do i need to add "DTC_OVERLAY_FILE"..?
Because whenever I am trying to include overlay file, I am getting boiler plate error,

RE: SPI slave configuration & overlay

Ubaid_M — Tue, 11 Jan 2022 06:17:36 GMT

Hello Sigurd,

[quote userid="15146" url="~/f/nordic-q-a/83514/spi-slave-configuration-overlay/347008#347008"].slave = 1 in spi_config ?[/quote]

.slave =1 i have set.

Yet it execution is entering to the same file "spi_nrfx_spi.c":

[quote userid="108934" url="~/f/nordic-q-a/83514/spi-slave-configuration-overlay"]xecution enters the same file "spi_nrfx_spi.c[/quote]

I also noticed that, maybe something is wrong with my overlay, my overlay name is "bl653_dvk.overlay" & it is placed in project directory with the configuration as:

&spi1 {
	compatible = "nordic,nrf-spis";
	status = "okay";
	sck-pin = <41>;
	mosi-pin = <40>;
	miso-pin = <4>;
};

1. I am saying this as I do not see overlay being included, in project dts structure as in below, is the name "bl653_dvk.overlay" proper..?

2. In "prj.conf", I have the following configurations, do I need to add anything more..?

#SPI
CONFIG_SPI=y
CONFIG_SPI_ASYNC=y
CONFIG_SPI_SLAVE=y

Kindly suggest where I am mistaken.

Attaching the bl653_dvk.dts & bl653_dvk.overlay for your reference.devzone.nordicsemi.com/.../bl653_5F00_dvk.dts

devzone.nordicsemi.com/.../bl653_5F00_dvk.overlay

Thanks,

Ubaid

RE: SPI slave configuration & overlay

Sigurd — Mon, 10 Jan 2022 17:23:20 GMT

Hi,

Did you set this?

CONFIG_SPI_SLAVE=y

and

.slave = 1 in spi_config ? Snippet:

static const struct spi_config spi_cfg = {
	.operation = SPI_WORD_SET(8) | SPI_TRANSFER_MSB | SPI_OP_MODE_SLAVE |
		     SPI_MODE_CPOL | SPI_MODE_CPHA,
	.frequency = 4000000,
	.slave = 1,
};

Some sample code: https://github.com/sigurdnev/ncs-playground/blob/master/samples/spis_thread_test/src/main.c