search

3mdeb / meta-rte

Yocto meta layer for the RTE https://3mdeb.com/products/open-source-hardware/rte/ board
MIT License
7 stars 3 forks source link

RTE dmesg show warning related to "buggy DT: spidev listed directly in DT" #51

Closed bdelgado1995 closed 8 months ago

bdelgado1995 commented 11 months ago

meta-rte version

v1.1.0

Question text My two RTEs show the warning below in the system dmesg for the spidev. Is this harmless or a sign of a problem? It looks like the spidev0.0 detects a chip but I haven't been able to get the spidev1.0 able to detect the chip, but that could also be for some other issue. But for now, just wanted to see if this warning indicates a problem or if it's harmless.

0.942965] WARNING: CPU: 0 PID: 1 at drivers/spi/spidev.c:737 spidev_probe+0xf0/0x1ac [ 0.950930] /soc/spi@1c68000/spidev@0: buggy DT: spidev listed directly in DT

The log is attached. spidev-rte.txt

artur-rs commented 10 months ago

@bdelgado1995 thank you for raising this issue. We've scheduled the analysis for this week. The result should be added to the FAQ/Troubleshooting section in the README/Dasharo documentation. I'll keep you updated.

PLangowski commented 10 months ago

@bdelgado1995 Hi, the warning was added to spidev.c to deter people from setting the compatible attributes in the device tree to spidev (source). The warning is harmless. We actually tried to work around it before, but there was a slight issue with the way we did it. We have fixed the problem in https://github.com/3mdeb/meta-rte/pull/55. It will soon be merged into develop. Thank you for raising the issue.

PLangowski commented 8 months ago

@bdelgado1995 Hi again, just letting you know that the problem is now fixed in release v0.7.5

bdelgado1995 commented 8 months ago

That's great to hear! Thanks for the closure.