-
Notifications
You must be signed in to change notification settings - Fork 624
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
wifi: hostap: Pull external crypto support #2253
base: main
Are you sure you want to change the base?
Conversation
be3b9ac
to
daa6222
Compare
This reverts commit 86bff0f. Signed-off-by: Chaitanya Tata <[email protected]>
This reverts commit 371c48b. Signed-off-by: Chaitanya Tata <[email protected]>
This reverts commit 34010be. Signed-off-by: Chaitanya Tata <[email protected]>
This reverts commit 2635f7e. Signed-off-by: Chaitanya Tata <[email protected]>
This reverts commit 01ba392. Signed-off-by: Chaitanya Tata <[email protected]>
…security" This reverts commit d49b489. Signed-off-by: Chaitanya Tata <[email protected]>
This reverts commit b8406eb. Signed-off-by: Chaitanya Tata <[email protected]>
Add an option for platforms or forks to provide their own hostap compatible crypto implementation. This may include proprietary or platform specific stuff that may or may not be upstreamed to Zephyr. Upstream PR #: 81243 Signed-off-by: Chaitanya Tata <[email protected]>
@carlescufi @rlubos the OSS history on the manifest PR is failing due to a bug in the history, I don't see how it can be fixed without a rewrite, WDYT? |
daa6222
to
291957f
Compare
Do I understand correctly that the problem here is that we have two commits with the same title Would like to hear from @carlescufi first, but I don't really see a way around this other than history rewrite... For sure we cannot merge like this as it'll break the check for every other PR. |
Yes that is the issue.
Sure, let's resolve this now. |
Pulls in external crypto support that will be used in NCS to manage hostap dependencies and help use revert the
noup
patches (they will be removed during history rewrite in upcoming upmerge).