korken69's J-Link fix behind a feature flag

Author: nickray

Cargo.toml

@@ -14,6 +14,7 @@ version = "0.3.4"
 
 [features]
 inline-asm = []
+jlink-quirks = []
 
 [dependencies]
 cortex-m = ">= 0.5.8, < 0.7"

src/hio.rs

@@ -68,12 +68,16 @@ fn write_all(fd: usize, mut buffer: &[u8]) -> Result<(), ()> {
             // Done
             0 => return Ok(()),
             // `n` bytes were not written
-            n if n <= buffer.len() => {
+            n if n <= buffer.len() && n > 0 => {
                 let offset = (buffer.len() - n) as isize;
                 buffer = unsafe {
                     slice::from_raw_parts(buffer.as_ptr().offset(offset), n)
                 }
             }
+            #[cfg(feature = "jlink-quirks")]
+            // Error (-1) - should be an error but JLink can return -1, -2, -3,...
+            // For good measure, we allow up to negative 15.
+            n if n > 0xfffffff0 => return Ok(()),
             // Error
             _ => return Err(()),
         }

src/lib.rs

@@ -133,6 +133,13 @@
 //! When this feature is disabled semihosting is implemented using FFI calls into an external
 //! assembly file and compiling this crate works on stable and beta.
 //!
+//! ## `jlink-quirks`
+//!
+//! When this feature is enabled, return values above `0xfffffff0` from semihosting operation
+//! `SYS_WRITE` (0x05) are interpreted as if the entire buffer had been written. The current
+//! latest version 6.48b of J-Link exhibits such behaviour, causing a panic if this feature
+//! is not enabled.
+//!
 //! # Reference
 //!
 //! For documentation about the semihosting operations, check: