Zephyr peripheral pin remap with overlay file

RE: Zephyr peripheral pin remap with overlay file

Håkon Alseth — Tue, 14 Sep 2021 08:34:01 GMT

Hi,

[quote user="Arturo H"]The macros solved the problem with the chip select issue, thanks.[/quote]

I'm glad to hear this!

[quote user="Arturo H"]

Each time I'm making a change on the overlay file, should I run CMake again?

On SES does build can be configured to run CMake each build?

[/quote]

.overlay changes isn't always picked up by the build system, so it is recommended that you re-import the project to ensure that the changes have been applied.

Kind regards,

Håkon

RE: Zephyr peripheral pin remap with overlay file

Arturo Hache — Tue, 14 Sep 2021 07:32:37 GMT

Each time I'm making a change on the overlay file, should I run CMake again?

On SES does build can be configured to run CMake each build?

The macros solved the problem with the chip select issue, thanks.

RE: Zephyr peripheral pin remap with overlay file

Håkon Alseth — Mon, 13 Sep 2021 09:27:33 GMT

Hi,

This is not the correct format for the cs-gpios member:

cs-gpios = <31>; //P0.31

You can see the expected format in the yml file for the generic spi controller:

https://github.com/nrfconnect/sdk-zephyr/blob/v2.6.99-ncs1-rc1/dts/bindings/spi/spi-controller.yaml#L24-L63

For your scenario, where you want to use P0.31, it should be:

cs-gpios = <&gpio0 31 GPIO_ACTIVE_LOW>;

[quote user=""]I tested this modification running some dupont cables to the appropiate pins, still with the nRF52833. What does work is changing the pin assignment on the code, for example using a custom structure like:[/quote]

const struct spi_cs_control ctrl =
{
.gpio_dev = DEVICE_DT_GET(DT_NODELABEL(gpio0)),
.delay = 0,
.gpio_pin = 31,
.gpio_dt_flags = (GPIO_ACTIVE_LOW)
};

[quote user=""]Which is the best way to get this to work?[/quote]

The code has omitted the defines that were emitted by the compiled device tree, as you're not fetching the cs-gpios member, which is just fine (you end up in the same scenario), but if you change it in dts, nothing will happen on the application level.

You can use these helper macros:

https://github.com/nrfconnect/sdk-zephyr/blob/v2.4.99-ncs2/subsys/disk/disk_access_spi_sdhc.c#L844-L848

As long as you set this in the top of main:

https://github.com/sigurdnev/ncs-playground/blob/master/samples/spi_test/src/main.c#L11

Kind regards,

Håkon