Mark more things unstable (#4811)

* esp-radio: Mark wifi::sta::eap unstable

* Mark RxControlInfo unstable, RxControlInfo::from_raw is no longer public and mark csi, event and sniffer modules unstable

* changelog

* review

* another review

Author: JurajSadel

esp-radio/CHANGELOG.md

@@ -35,6 +35,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - `esp-alloc` dependency no longer enables default features (#4721)
 - `sys-logs` feature has been renamed to `print-logs-from-driver` and has been marked unstable (#4810)
 - Unstable features now require the `unstable` feature flag to be enabled (#4810)
+- `RxControlInfo` is unstable, `RxControlInfo::from_raw()` is no longer public (#4811)
+- `event`, `sniffer`, and `csi` modules are marked unstable (#4811)
 
 ### Fixed
 

esp-radio/src/lib.rs

@@ -212,10 +212,13 @@ pub mod wifi;
 
 unstable_module! {
     #[cfg(feature = "esp-now")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "esp-now")))]
     pub mod esp_now;
     #[cfg(feature = "ble")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "ble")))]
     pub mod ble;
     #[cfg(feature = "ieee802154")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "ieee802154")))]
     pub mod ieee802154;
 }
 

esp-radio/src/wifi/mod.rs

@@ -42,12 +42,18 @@ use crate::{
     },
 };
 pub mod ap;
-#[cfg(all(feature = "csi", feature = "unstable"))]
-pub mod csi;
-pub mod event;
+
+unstable_module!(
+    #[cfg(feature = "csi")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "csi")))]
+    pub mod csi;
+    pub mod event;
+    #[cfg(feature = "sniffer")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "sniffer")))]
+    pub mod sniffer;
+);
+
 pub mod scan;
-#[cfg(all(feature = "sniffer", feature = "unstable"))]
-pub mod sniffer;
 pub mod sta;
 
 pub(crate) mod os_adapter;
@@ -1111,6 +1117,7 @@ pub enum Bandwidth {
 #[derive(Debug, Clone, Copy, Eq, PartialEq, Hash)]
 #[cfg_attr(feature = "defmt", derive(defmt::Format))]
 #[cfg(all(any(feature = "esp-now", feature = "sniffer"), feature = "unstable"))]
+#[instability::unstable]
 pub struct RxControlInfo {
     /// Received Signal Strength Indicator (RSSI) of the packet, in dBm.
     pub rssi: i32,
@@ -1169,6 +1176,7 @@ pub struct RxControlInfo {
 #[derive(Debug, Clone, Copy, Eq, PartialEq, Hash)]
 #[cfg_attr(feature = "defmt", derive(defmt::Format))]
 #[cfg(all(any(feature = "esp-now", feature = "sniffer"), feature = "unstable"))]
+#[instability::unstable]
 pub struct RxControlInfo {
     /// Received Signal Strength Indicator (RSSI) of the packet, in dBm.
     pub rssi: i32,
@@ -1219,7 +1227,7 @@ impl RxControlInfo {
     /// # Safety
     /// When calling this, you must ensure, that `rx_cntl` points to a valid
     /// instance of [wifi_pkt_rx_ctrl_t].
-    pub unsafe fn from_raw(rx_cntl: *const wifi_pkt_rx_ctrl_t) -> Self {
+    pub(super) unsafe fn from_raw(rx_cntl: *const wifi_pkt_rx_ctrl_t) -> Self {
         #[cfg(not(esp32c6))]
         let rx_control_info = unsafe {
             RxControlInfo {

esp-radio/src/wifi/sta/mod.rs

@@ -9,8 +9,11 @@ use procmacros::BuilderLite;
 use super::{AuthenticationMethod, Protocol, scan::ScanMethod};
 use crate::WifiError;
 
-#[cfg(feature = "wifi-eap")]
-pub mod eap;
+unstable_module!(
+    #[cfg(feature = "wifi-eap")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "wifi-eap")))]
+    pub mod eap;
+);
 
 /// Station configuration for a Wi-Fi connection.
 #[derive(BuilderLite, Clone, Eq, PartialEq, Hash)]