fix(bindings): correct otRadioCaps type to u16 for riscv32imac

The riscv32imac bindings incorrectly defined otRadioCaps as u8, but
OT_RADIO_CAPS_RX_ON_WHEN_IDLE = 256 which doesn't fit in u8.

This caused the capability to be silently truncated to 0, preventing
OpenThread from calling otPlatRadioSetRxOnWhenIdle callback.

Changes:
- Change otRadioCaps from u8 to u16 in riscv32imac bindings
- Add missing OT_RADIO_CAPS_RX_ON_WHEN_IDLE constant (256)
- Use proper import in lib.rs instead of local const
- Fix otPlatRadioGetCaps return type to use otRadioCaps

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Aleksei Sviridkin <f@lex.la>

Author: lexfrei

openthread-sys/src/include/riscv32imac-unknown-none-elf.rs

@@ -2830,7 +2830,7 @@ pub type _bindgen_ty_4 = ::core::ffi::c_uint;
 /// Represents radio capabilities.
 ///
 /// The value is a bit-field indicating the capabilities supported by the radio. See `OT_RADIO_CAPS_*` definitions.
-pub type otRadioCaps = u8;
+pub type otRadioCaps = u16;
 ///< Radio supports no capability.
 pub const OT_RADIO_CAPS_NONE: _bindgen_ty_5 = 0;
 ///< Radio supports AckTime event.
@@ -2849,6 +2849,8 @@ pub const OT_RADIO_CAPS_TRANSMIT_SEC: _bindgen_ty_5 = 32;
 pub const OT_RADIO_CAPS_TRANSMIT_TIMING: _bindgen_ty_5 = 64;
 ///< Radio supports rx at specific time.
 pub const OT_RADIO_CAPS_RECEIVE_TIMING: _bindgen_ty_5 = 128;
+///< Radio supports RxOnWhenIdle setting.
+pub const OT_RADIO_CAPS_RX_ON_WHEN_IDLE: _bindgen_ty_5 = 256;
 /// Defines constants that are used to indicate different radio capabilities. See `otRadioCaps`.
 pub type _bindgen_ty_5 = ::core::ffi::c_uint;
 /// Represents the IEEE 802.15.4 PAN ID.

openthread/src/lib.rs

@@ -78,7 +78,7 @@ use sys::{
     otPlatRadioTxDone, otPlatRadioTxStarted, otRadioCaps, otRadioFrame, otSetStateChangedCallback,
     otTaskletsProcess, otThreadGetDeviceRole, otThreadGetExtendedPanId, otThreadSetEnabled,
     otThreadSetLinkMode, OT_RADIO_CAPS_ACK_TIMEOUT, OT_RADIO_CAPS_CSMA_BACKOFF,
-    OT_RADIO_FRAME_MAX_SIZE,
+    OT_RADIO_CAPS_RX_ON_WHEN_IDLE, OT_RADIO_FRAME_MAX_SIZE,
 };
 
 /// A newtype wrapper over the native OpenThread error type (`otError`).
@@ -1427,8 +1427,6 @@ impl<'a> OtContext<'a> {
     }
 
     fn plat_radio_caps(&mut self) -> otRadioCaps {
-        // OT_RADIO_CAPS_RX_ON_WHEN_IDLE = 256, not in all bindgen targets
-        const OT_RADIO_CAPS_RX_ON_WHEN_IDLE: u32 = 256;
         let caps =
             (OT_RADIO_CAPS_ACK_TIMEOUT | OT_RADIO_CAPS_CSMA_BACKOFF | OT_RADIO_CAPS_RX_ON_WHEN_IDLE)
                 as _;

openthread/src/platform.rs

@@ -7,7 +7,7 @@ use portable_atomic::AtomicUsize;
 
 use openthread_sys::otError_OT_ERROR_NONE;
 
-use crate::sys::{otError, otInstance, otLogLevel, otLogRegion, otRadioFrame};
+use crate::sys::{otError, otInstance, otLogLevel, otLogRegion, otRadioCaps, otRadioFrame};
 use crate::{IntoOtCode, OtActiveState, OtContext};
 
 /// A hack so that we can store a mutable reference to the active state in a global static variable
@@ -70,7 +70,7 @@ extern "C" fn otPlatRadioGetIeeeEui64(instance: *const otInstance, mac: *mut u8)
 }
 
 #[no_mangle]
-extern "C" fn otPlatRadioGetCaps(instance: *const otInstance) -> u8 {
+extern "C" fn otPlatRadioGetCaps(instance: *const otInstance) -> otRadioCaps {
     OtContext::callback(instance).plat_radio_caps()
 }