Only spi nodes accepted in /soc/spi@40023000/

RE: Only spi nodes accepted in /soc/spi@40023000/

Einarh — Tue, 18 Jan 2022 12:13:03 GMT

Glad to see you figured it out!

Good luck on your project, best regards,

Einar

RE: Only spi nodes accepted in /soc/spi@40023000/

glenn-ide — Tue, 18 Jan 2022 11:17:14 GMT

Sorry, I didn't look close enough at the messages. You're right the second message is generated by a different bit of code.

Upon closer inspection it seems my spi code was doing something funky. I've now fixed it and it does write data:

Device ADS1291 is not ready

[00:00:00.000,366] [0m<dbg> ADS1291.ads1291_bus_access: REG Read for register 0x00 resulting in 0x20 sent[0m

[00:00:00.000,366] [0m<dbg> ADS1291.ads1291_bus_access: Number of registers to read/write 2[0m

[00:00:00.000,457] [1;31m<err> ADS1291: Failed to read id. Expected 0x2A got 0x00[0m

The error message which started this post still remains but at least I can now continue development.

Thanks for making me look deeper in to this.

RE: Only spi nodes accepted in /soc/spi@40023000/

Einarh — Tue, 18 Jan 2022 10:16:48 GMT

I see

[quote user="glenn-ide"]

Device ADS1291 is not ready

[00:00:00.000,427] [1;31m<err> ADS1291: Failed to read id. Expected 0x2A got 0x0

[/quote]

I suspect dealing with these errors would solve your problem, maybe there should be a difference in the SPI config that you've forgotten?

Any idea where "Expected 0x2A" comes from?

Have you tried putting small sleeps at critical points of your initialization, such as between DEVICE_DT_GET_ANY and device_is_ready?

-Einar

RE: Only spi nodes accepted in /soc/spi@40023000/

glenn-ide — Mon, 17 Jan 2022 15:39:19 GMT

I see the following:

Device ADS1291 is not ready

[00:00:00.000,427] [1;31m<err> ADS1291: Failed to read id. Expected 0x2A got 0x0[0m

The first log line is generated by

const struct device *afe_dev = DEVICE_DT_GET_ANY(ti_ads1291);
if (!device_is_ready(afe_dev)) {
	printk("Device %s is not ready\n", afe_dev->name);
}

The second log line is generated by:

static int ads1291_init(const struct device *dev)
{
    struct ads1291_data *data = dev->data;
    const struct ads1291_dev_config *cfg = dev->config;
    int ret;

    data->bus = device_get_binding(cfg->spi_port);
    if (!data->bus) {
        LOG_ERR("SPI device not found: %s", cfg->spi_port);
        return -EINVAL;
    }

This code uses the following SPI config in the config struct which I copied from the ADXL372 example:

static const struct ads1291_dev_config ads1291_config = {

    .spi_port = DT_INST_BUS_LABEL(0),
    .spi_slave = DT_INST_REG_ADDR(0),
    .spi_max_frequency = DT_INST_PROP(0, spi_max_frequency),
#if DT_INST_SPI_DEV_HAS_CS_GPIOS(0)
    .gpio_cs_port = DT_INST_SPI_DEV_CS_GPIOS_LABEL(0),
    .cs_gpio = DT_INST_SPI_DEV_CS_GPIOS_PIN(0),
    .cs_flags = DT_INST_SPI_DEV_CS_GPIOS_FLAGS(0),
#endif

I've had a second look with the logic analyser and it does seem to do something, but I also expect to see MOSI data. It does seem to recognize the 'spi-max-frequency = 2 MHz' which I set in the device tree under the sensor.

For example, this is how the ADXL372 example looks with the same pin configuration as my project:

RE: Only spi nodes accepted in /soc/spi@40023000/

Einarh — Mon, 17 Jan 2022 15:03:53 GMT

Hm, I cannot see anything obvious missing.

I assume you've already double and triple checked that the gpio pins you've defined in the devicetree correspond with how your sensor is connected?

I find it strange that you don't see anything when measuring the pins. Are any of your function calls returning error codes that could give you hints?

-Einar

RE: Only spi nodes accepted in /soc/spi@40023000/

glenn-ide — Mon, 17 Jan 2022 12:26:25 GMT

Of course:

CONFIG_SPI=y
CONFIG_SERIAL=y
CONFIG_MAIN_STACK_SIZE=2000
CONFIG_PPI_TRACE=y
CONFIG_THREAD_RUNTIME_STATS=y
CONFIG_CONSOLE=y
CONFIG_UART_CONSOLE=y
CONFIG_PWM=y
CONFIG_COUNTER=y
CONFIG_NEWLIB_LIBC=y
CONFIG_CBPRINTF_FP_SUPPORT=y
CONFIG_BT=y
CONFIG_BT_PERIPHERAL=y
CONFIG_BT_DIS=y
CONFIG_BT_DIS_MODEL="ABC"
CONFIG_BT_DIS_MANUF="IDE Group"
CONFIG_BT_DIS_PNP=n
CONFIG_BT_BAS=y
CONFIG_BT_DEBUG_LOG=y
CONFIG_BT_SMP=y
CONFIG_BT_DEVICE_NAME="ABC"
CONFIG_BT_DEVICE_APPEARANCE=833
CONFIG_BT_EMGS=y
CONFIG_ADS1291=y
CONFIG_ADS1291_ODR_2000HZ=y

RE: Only spi nodes accepted in /soc/spi@40023000/

Einarh — Mon, 17 Jan 2022 12:23:49 GMT

Hm, that is strange.

Since your code is compiling but the behavior isn't as expected, the first thing I would do is double check the contents of the prj.conf file, maybe some module is missing there.

Can you show me the content of the file?

-Einar

RE: Only spi nodes accepted in /soc/spi@40023000/

glenn-ide — Mon, 17 Jan 2022 10:59:18 GMT

Hi,

I'm aware of the ticket. I've tried restarting VSCode and my laptop multiple times but no success. The main difference between their code and mine is they specifiy "on-bus: spi" in their yaml but this is not neccesary since it is included in spi-device.yaml (also tested this doesn't fix my issue).

Also, the order of their CMakeList is a bit different which I've tried copying but no success.

You're right; my code compiles fine but gives an error in VSCode. Also, I've tried measuring the physical signals but I don't see anything.

RE: Only spi nodes accepted in /soc/spi@40023000/

Einarh — Mon, 17 Jan 2022 10:20:55 GMT

Hello again,

This issue seems to be very similar to your problem:

https://devzone.nordicsemi.com/f/nordic-q-a/81697/only-spi-nodes-accepted-in-soc-peripheral-50000000-spi-b000-soc-peripheral-50000000-spi-b000-ads129xr-0

maybe looking over his config could help you?

Also, I assume your code compiles properly, and the error message appears similarly as in this other issue?

Best regards,

Einar

RE: Only spi nodes accepted in /soc/spi@40023000/

glenn-ide — Fri, 14 Jan 2022 13:10:44 GMT

Hi, thanks for looking in to this.

For my custom project it looks like this:

		spi2: arduino_spi: spi@40023000 {
			#address-cells = < 0x1 >;
			#size-cells = < 0x0 >;
			reg = < 0x40023000 0x1000 >;
			interrupts = < 0x23 0x1 >;
			status = "okay";
			label = "SPI_2";
			compatible = "nordic,nrf-spi";
			sck-pin = < 0x19 >;
			mosi-pin = < 0x17 >;
			miso-pin = < 0x18 >;
			cs-gpios = < &gpio0 0x4 0x1 >;
			ads1291@0 {
				compatible = "ti,ads1291";
				reg = < 0x0 >;
				spi-max-frequency = < 0x1e8480 >;
				label = "ADS1291";
				int1-gpios = < &gpio0 0x1f 0x1 >;
			};
		};

The example looks similar to yours but some of the gpios are different.

		spi2: arduino_spi: spi@40023000 {
			#address-cells = < 0x1 >;
			#size-cells = < 0x0 >;
			reg = < 0x40023000 0x1000 >;
			interrupts = < 0x23 0x1 >;
			status = "okay";
			label = "SPI_2";
			compatible = "nordic,nrf-spi";
			sck-pin = < 0x19 >;
			mosi-pin = < 0x17 >;
			miso-pin = < 0x18 >;
			cs-gpios = < &gpio0 0x16 0x1 >;
			adxl372@0 {
				compatible = "adi,adxl372";
				reg = < 0x0 >;
				spi-max-frequency = < 0x7a1200 >;
				label = "ADXL372";
				int1-gpios = < &gpio0 0xb 0x0 >;
			};
		};

I don't see any significant differences between my custom one and the example..

RE: Only spi nodes accepted in /soc/spi@40023000/

Einarh — Fri, 14 Jan 2022 13:03:38 GMT

Hello!

Can you show me how the spi2 node looks in your devicetree (in the build/zephyr/zephyr.dts file) when you have compiled the ADXL372 and the ADS1291 projects?

If there is a difference it might help you spot what you have missed.

On my end it looks like this in the ADXL372 project:

spi2: arduino_spi: spi@40023000 {
	#address-cells = < 0x1 >;
	#size-cells = < 0x0 >;
	reg = < 0x40023000 0x1000 >;
	interrupts = < 0x23 0x1 >;
	status = "okay";
	label = "SPI_2";
	compatible = "nordic,nrf-spi";
	sck-pin = < 0x19 >;
	mosi-pin = < 0x17 >;
	miso-pin = < 0x18 >;
	cs-gpios = < &gpio0 0x16 0x1 >;
	adxl372@0 {
		compatible = "adi,adxl372";
		reg = < 0x0 >;
		spi-max-frequency = < 0x7a1200 >;
		label = "ADXL372";
		int1-gpios = < &gpio0 0xb 0x0 >;
	};
};

Best regards,

Einar