search

zephyrproject-rtos / zephyr

Primary Git Repository for the Zephyr Project. Zephyr is a new generation, scalable, optimized, secure RTOS for multiple hardware architectures.
https://docs.zephyrproject.org
Apache License 2.0
10.7k stars 6.54k forks source link

nrf9131ek/nrf9131 CI failure with Device initialization priority validation failed #79772

Open nordicjm opened 4 days ago

nordicjm commented 4 days ago

Describe the bug Board CI failing, see: https://github.com/zephyrproject-rtos/zephyr/actions/runs/11322012174/job/31482120728?pr=78621

Expected behavior Board to work

Impact Showstopper

@maxd-nordic @aasinclair

maxd-nordic commented 4 days ago

These tests are failing: drivers.eeprom.emul.build drivers.eeprom.build

Now, what makes eeprom tests fail on a board without eeprom?

maxd-nordic commented 4 days ago

/soc/peripheral@50000000/i2c@a000/npm1300@6b/charger is initialized before its dependency /soc/peripheral@50000000/i2c@a000/npm1300@6b

maxd-nordic commented 4 days ago

charger is sensor priority and the mfd device has its own prio.

maxd-nordic commented 4 days ago

This patch could fix it:

diff --git a/tests/drivers/build_all/eeprom/boards/nrf9131ek_nrf9131.conf b/tests/drivers/build_all/eeprom/boards/nrf9131ek_nrf9131.conf
new file mode 100644
index 00000000000..cbe703cf555
--- /dev/null
+++ b/tests/drivers/build_all/eeprom/boards/nrf9131ek_nrf9131.conf
@@ -0,0 +1 @@
+CONFIG_MFD_INIT_PRIORITY=55
maxd-nordic commented 4 days ago

@nordicjm This fails because this PR https://github.com/zephyrproject-rtos/zephyr/pull/79138 messes with the sensor priority. Maybe you can add the above change to https://github.com/zephyrproject-rtos/zephyr/pull/78621 to fix CI since you are already changing nrf9131ek.

nordicjm commented 4 days ago

@masz-nordic now a new build error: https://github.com/zephyrproject-rtos/zephyr/actions/runs/11324295360/job/31492031631?pr=78621