Skip to content

Conversation

@vaishnavachath
Copy link
Contributor

Fixes #12
Changes as per discussion in https://github.com/cfriedt/zephyr/pull/5

Currently priorities are assigned in the following manner:

CONFIG_GREYBUS_SERVICE_PRIORITY N-5
CONFIG_GREYBUS_STRING_PRIORITY N-4
CONFIG_GREYBUS_INTERFACE_PRIORITY N-3
CONFIG_GREYBUS_BUNDLE_PRIORITY N-2
CONFIG_GREYBUS_CPORT_PRIORITY N-1

where N = CONFIG_APPLICATION_INIT_PRIORITY

Tested GPIO, I2C and SPI on CC1352R SensorTag

Fixes cfriedt/greybus-for-zephyr#12
Changes as per discussion in  https://github.com/cfriedt/zephyr/pull/5

Currently priorities are assigned in the following manner:

CONFIG_GREYBUS_SERVICE_PRIORITY N-5
CONFIG_GREYBUS_STRING_PRIORITY N-4
CONFIG_GREYBUS_INTERFACE_PRIORITY N-3
CONFIG_GREYBUS_BUNDLE_PRIORITY N-2
CONFIG_GREYBUS_CPORT_PRIORITY N-1

where N = CONFIG_APPLICATION_INIT_PRIORITY

Tested GPIO, I2C and SPI on CC1352R SensorTag

Signed-off-by: Vaishnav M A <[email protected]>
@cfriedt cfriedt merged this pull request into cfriedt:master Dec 21, 2020
@cfriedt
Copy link
Owner

cfriedt commented Dec 21, 2020

Feel free to make another PR, but this one had to be dropped because (somehow) it broke the build after it was merged into master.

I see where it was:
https://github.com/cfriedt/greybus-for-zephyr/runs/1589262942?check_suite_focus=true

variable r needed to be initialized.

Very odd that it didn't trigger in the PR though 🤨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

subsys: greybus: remove deferred init

2 participants