add possibility for XBARB

[lam-borg]

We are using the xbarb and it would be nice to have this possibility in the upstream repo aswell.
Thanks for considering

[lam-borg]

Thanks for reviewing and approval!

Now we just need one more with the right to merge it...

Literally one click away!

[dleach02]

@lam-borg, please ensure your commit messages are signed. While this is the NXP HAL, we follow the Zephyr contribution guidelines.

Also, you have two commits in this PR for a oneline change so clean up the commits.

[lam-borg]

@dleach02 Thanks for your input; I fixed it accordingly.

• cleaned up
• signed off
• commit message according guidelines

[dleach02]

@lam-borg just realized that we need a Zephyr side manifest PR to point to this

[lam-borg]

I made a pull request in zephyr as well: zephyrproject-rtos/zephyr#80225

Anything that i need to add there or here?

[danieldegrasse]

Yes, you need to update the west manifest in that PR to point to this HAL PR. Some documentation is available here: https://docs.zephyrproject.org/latest/develop/modules.html#process-for-submitting-changes-to-existing-modules

[lam-borg]

I wasn't aware, its an issue if my PR is behind main. But makes perfect sense, knowing its used for unit tests in the zephyr PR.

Now updated.

[lam-borg]

Somehow, my zephyr PR is waiting on this PR. Did I miss anything? What can I do to get it through?

[danieldegrasse]

Somehow, my zephyr PR is waiting on this PR. Did I miss anything? What can I do to get it through?

So the process we follow for HAL PRs is the following:

• wait until Zephyr PR is ready to merge (IE 2 project collaborators have approved it, and all CI is passing except do not merge check)
• Merge HAL side PR
• Contributor will update the Zephyr side PR to now use the latest SHA of the NXP HAL
• CI runs again, reviewers reapprove, and the Zephyr side PR merges

Since we are now in the 4.0 feature freeze this process will be delayed until after the 4.0 release, but those are the steps we will follow to get this merged.

[lam-borg]

@danieldegrasse ok, thanks for approval and the clarification!

[bia-bonobo]

@dleach02 I think your requested changes have been completed. Can you have a look at this again?

[dleach02]

yes, we need to now coordinate the merge with you zephyr side PR