Skip to content

Conversation

@lmaciejonczyk
Copy link
Contributor

Pass enh ack data injection arguments to the nRF5 radio driver.

Signed-off-by: Lukasz Maciejonczyk [email protected]

@lmaciejonczyk lmaciejonczyk changed the title [nrf toup] drivers: ieee802154: configure IE header injection for nRF5 drivers: ieee802154: configure IE header injection for nRF5 Jun 7, 2021
@lmaciejonczyk lmaciejonczyk force-pushed the ot/configure_IE_header_injection_for_nRF5 branch from 2ff67a0 to 4ca1e86 Compare June 7, 2021 12:54
@github-actions github-actions bot added manifest west manifest-hal_nordic DNM This PR should not be merged (Do Not Merge) labels Jun 7, 2021
@github-actions
Copy link

github-actions bot commented Jun 7, 2021

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
hal_nordic zephyrproject-rtos/hal_nordic@574493f (master) zephyrproject-rtos/hal_nordic#96 zephyrproject-rtos/hal_nordic#96/files

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@lmaciejonczyk lmaciejonczyk force-pushed the ot/configure_IE_header_injection_for_nRF5 branch from 4ca1e86 to 8375261 Compare June 7, 2021 13:46
@lmaciejonczyk lmaciejonczyk requested a review from nashif as a code owner June 7, 2021 13:46
@lmaciejonczyk lmaciejonczyk force-pushed the ot/configure_IE_header_injection_for_nRF5 branch from 8375261 to 47b9f34 Compare June 7, 2021 13:57
@jukkar jukkar requested a review from rlubos June 7, 2021 13:59
@lmaciejonczyk lmaciejonczyk force-pushed the ot/configure_IE_header_injection_for_nRF5 branch 2 times, most recently from 4ba5032 to e49c75b Compare June 8, 2021 11:09
Pass enh ack data injection arguments to the nRF5 radio driver.

Signed-off-by: Lukasz Maciejonczyk <[email protected]>
@lmaciejonczyk lmaciejonczyk force-pushed the ot/configure_IE_header_injection_for_nRF5 branch from e49c75b to 888b949 Compare June 9, 2021 13:24
@edmont edmont removed the DNM This PR should not be merged (Do Not Merge) label Jul 2, 2021
@nashif nashif merged commit bde592e into zephyrproject-rtos:main Jul 2, 2021
@ppryga-nordic
Copy link
Contributor

Code from this PR was merged after this one: #36455.
Nevertheless in file drivers/ieee802154/ieee802154_nrf5.c the diff does not show code originated in #36455.
Due to that, the PR was merged and causes issue during build process (mentioned by @CristXu
bde592e#commitcomment-53003227).

After merge of this PR, CI starts to fail on samples that compile in code from file: drivers/ieee802154/ieee802154_nrf5.c, e.g. #36705

nashif added a commit to nashif/zephyr that referenced this pull request Jul 5, 2021
This reverts commit bde592e.

This is causing build failures. Original PR zephyrproject-rtos#36008.

Signed-off-by: Anas Nashif <[email protected]>
nashif added a commit that referenced this pull request Jul 5, 2021
This reverts commit bde592e.

This is causing build failures. Original PR #36008.

Signed-off-by: Anas Nashif <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants