Updated to nrfxlib v1.4.2

Author: Wassasin

Cargo.toml

@@ -9,8 +9,11 @@ repository = "https://github.com/42-technology-ltd/nrfxlib"
 description = "Rust driver for the LTE stack on the Nordic nRF9160"
 
 [dependencies]
-nrfxlib-sys = "1.2.0"
 nrf9160-pac = "0.2.1"
 cortex-m = "0.6"
 heapless = "0.5"
 log = "0.4"
+
+# Note: when v1.4.2 is upstreamed we can merge this again.
+# nrfxlib-sys = "1.2.0"
+nrfxlib-sys = { git = "https://github.com/Wassasin/nrfxlib-sys", branch = "nrfxlib-v1.4.2" }

src/at.rs

@@ -53,7 +53,7 @@ pub struct AtSocket(Socket);
 impl AtSocket {
 	/// Create a new AT socket.
 	pub fn new() -> Result<AtSocket, Error> {
-		let skt = Socket::new(SocketDomain::Lte, SocketType::None, SocketProtocol::At)?;
+		let skt = Socket::new(SocketDomain::Lte, SocketType::Datagram, SocketProtocol::At)?;
 		Ok(AtSocket(skt))
 	}
 

src/lib.rs

@@ -53,8 +53,8 @@ pub use ffi::get_last_error;
 pub use raw::{poll, PollEntry, PollFlags, PollResult, Pollable};
 
 use log::{debug, trace};
-use nrfxlib_sys as sys;
 use nrf9160_pac as cpu;
+use nrfxlib_sys as sys;
 
 //******************************************************************************
 // Types
@@ -132,12 +132,28 @@ pub enum Error {
 //******************************************************************************
 
 /// Start the BSD library
-pub fn init() {
+pub fn init(trace_on: bool) -> Result<(), Error> {
 	debug!("nrfxlib init");
-	unsafe {
-		sys::bsd_init();
+	let bsd_memory_size = if trace_on {
+		sys::BSD_RESERVED_MEMORY_SIZE
+	} else {
+		sys::BSD_RESERVED_MEMORY_SIZE_TRACE_DISABLED
+	};
+
+	let result = unsafe {
+		sys::bsd_init(&sys::bsd_init_params_t {
+			trace_on,
+			bsd_memory_address: sys::BSD_RESERVED_MEMORY_ADDRESS,
+			bsd_memory_size,
+		})
+	};
+
+	if result < 0 {
+		Err(Error::Nordic("init", result, ffi::get_last_error()))
+	} else {
+		trace!("nrfxlib init complete");
+		Ok(())
 	}
-	trace!("nrfxlib init complete");
 }
 
 /// Stop the BSD library

src/raw.rs

@@ -71,8 +71,6 @@ pub(crate) enum SocketDomain {
 /// The type of socket (Stream, Datagram, or neither)
 #[derive(Debug, Copy, Clone, PartialEq, Eq)]
 pub(crate) enum SocketType {
-	/// Used with `SocketDomain::Lte`
-	None,
 	/// Used with `SocketDomain::Inet` for TCP and TLS streams
 	Stream,
 	/// Used with UDP sockets, and for GPS
@@ -323,7 +321,6 @@ impl Into<i32> for SocketType {
 	fn into(self) -> i32 {
 		use SocketType::*;
 		match self {
-			None => 0,
 			Stream => sys::NRF_SOCK_STREAM as i32,
 			Datagram => sys::NRF_SOCK_DGRAM as i32,
 		}
@@ -434,14 +431,14 @@ pub fn poll(poll_list: &mut [PollEntry], timeout_ms: u16) -> Result<i32, Error>
 	}
 
 	let mut poll_fds: [sys::nrf_pollfd; MAX_SOCKETS_POLL] = [sys::nrf_pollfd {
-		handle: 0,
-		requested: 0,
-		returned: 0,
+		fd: 0,
+		events: 0,
+		revents: 0,
 	}; MAX_SOCKETS_POLL];
 
 	for (poll_entry, pollfd) in poll_list.iter_mut().zip(poll_fds.iter_mut()) {
-		pollfd.handle = poll_entry.socket.get_fd();
-		pollfd.requested = poll_entry.flags as i16;
+		pollfd.fd = poll_entry.socket.get_fd();
+		pollfd.events = poll_entry.flags as i16;
 		count += 1;
 	}
 
@@ -452,7 +449,7 @@ pub fn poll(poll_list: &mut [PollEntry], timeout_ms: u16) -> Result<i32, Error>
 		0 => Ok(0),
 		n => {
 			for (poll_entry, pollfd) in poll_list.iter_mut().zip(poll_fds.iter()) {
-				poll_entry.result = PollResult(pollfd.returned as u32);
+				poll_entry.result = PollResult(pollfd.revents as u32);
 			}
 			Ok(n)
 		}