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.89k stars 6.63k forks source link

External module request: nRF Wi-Fi (split off of hal_nordic) #81002

Open carlescufi opened 1 week ago

carlescufi commented 1 week ago

Origin

nRF Wi-Fi is a cross-platform, RTOS-agnostic Wi-Fi framework for nRF Wi-Fi ICs.

The current repo is here: https://github.com/nrfconnect/nrf_wifi

Note that this code is currently in hal_nordic

Purpose

The purpose of this request is to split up the nRF Wi-Fi source code from hal_nordic, for a couple of reasons:

  1. The releases of nrfx (our HAL) and nrf_wifi are completely independent, and having them in the same repo brings issues when updating the SHA of it in the main manifest
  2. While nrfx belongs to Zephyr as a built-in module, we think that nrf_wifi could be entirely optional
  3. This aligns hal_nordic to what other vendors do, such as ST with hal_st and hal_stm32

Mode of integration

As a optional module, with the name nrf_wifi.

Maintainership

@sachinthegreen @krish2718

Pull Request

https://github.com/zephyrproject-rtos/zephyr/pull/81101

Description

This is an existing component, it essentially contains the software framework necessary for the Wi-Fi functionality in the nRF Wi-Fi ICs.

Dependencies

None

Revision

1c85b3c927897c752c155560e263d4947ddcc6e4

License

SPDX-License-Identifier: BSD-3-Clause

nashif commented 5 days ago

https://github.com/zephyrproject-rtos/zephyr/pull/81101/commits/c4c2b58e29ea6677008329a3069a8435dd1831c7 is not adding this as optional module, it is in the default manifest.

krish2718 commented 5 days ago

c4c2b58 is not adding this as optional module, it is in the default manifest.

Sorry, https://github.com/zephyrproject-rtos/zephyr/pull/81101/commits/ca5bcc8f2fce547eb3fa683edd73ad87d492b80e reverts that change, I will fix the commits, but overally that PR still adds it to the optional manifests.

krish2718 commented 5 days ago

Fixed and updated the PR.