Fixes to get GitHub Actions passing again

- Includes a temporary hidio_test() function to debug hid-io-kiibohd

Author: haata

.github/workflows/rust_linux.yml

@@ -151,7 +151,7 @@ jobs:
       - uses: actions-rs/cargo@v1
         with:
           command: build
-          args: --manifest-path hid-io-protocol/Cargo.toml --target thumbv7em-none-eabi --no-default-features --features device --release
+          args: --manifest-path hid-io-protocol/Cargo.toml --no-default-features --features device --release
 
   fmt:
     name: Rustfmt

hid-io-kiibohd/src/lib.rs

@@ -196,6 +196,38 @@ pub extern "C" fn hidio_txbyte_bufsize() -> u16 {
     <TxBuf as Unsigned>::to_u16()
 }
 
+#[no_mangle]
+pub extern "C" fn hidio_test() {
+    let ids = [
+        HidIoCommandID::SupportedIDs,
+        HidIoCommandID::GetInfo,
+        HidIoCommandID::TestPacket,
+        HidIoCommandID::ManufacturingTest,
+    ];
+
+    let config = HidioConfig {
+        device_name: String::<U256>::from("Test").as_ptr() as *mut c_char,
+        device_mcu: String::<U256>::from("Test").as_ptr() as *mut c_char,
+        device_serial_number: String::<U256>::from("Test").as_ptr() as *mut c_char,
+        firmware_version: String::<U256>::from("Test").as_ptr() as *mut c_char,
+        firmware_vendor: String::<U256>::from("Test").as_ptr() as *mut c_char,
+    };
+
+    unsafe {
+        INTF = Some(match CommandInterface::<TxBuf, RxBuf, BufChunk, MessageLen, SerializationLen, IdLen>::new(&ids, config) {
+            Ok(intf) => intf,
+            Err(CommandError::IdVecTooSmall) => {
+                return;
+            }
+            Err(_) => {
+                // TODO Convert errors properly
+                //HIDIO_ERROR.copy_from_slice("ERROR TODO".as_bytes());
+                return;
+            }
+        });
+    }
+}
+
 /// Initialized the hid-io CommandInterface
 #[no_mangle]
 pub extern "C" fn hidio_init(config: HidioConfig) -> HidioStatus {

hid-io-protocol/Cargo.toml

@@ -18,7 +18,6 @@ edition       = "2018"
 
 [lib]
 name = "hid_io_protocol"
-crate-type = ["rlib", "staticlib"]
 
 
 [features]
@@ -36,7 +35,7 @@ server = ["log", "bincode_core/std"]
 
 [dependencies]
 arraydeque      = { version = "^0.4", default-features = false }
-bincode_core    = { git = "https://github.com/bincode-org/bincode-core.git", default-features = false }
+bincode_core    = { git = "https://github.com/bincode-org/bincode-core.git" }
 heapless        = { version = "^0.5" }
 log             = { version = "^0.4", default-features = false, optional = true }
 num_enum        = { version = "^0.5", default-features = false }

hid-io-protocol/src/lib.rs

@@ -40,6 +40,7 @@ use heapless::Vec;
 use num_enum::{IntoPrimitive, TryFromPrimitive};
 use serde::ser::{self, Serialize, SerializeSeq, Serializer};
 
+#[cfg(all(not(test), target_feature = "thumb-mode"))]
 #[cfg(feature = "device")]
 use core::panic::PanicInfo;
 
@@ -685,7 +686,9 @@ impl<H: ArrayLength<u8>> HidIoPacketBuffer<H> {
 
         // Make sure serialization worked
         len = writer.written_len();
-        if self.ptype == HidIoPacketType::Sync && len < 2 || self.ptype != HidIoPacketType::Sync && len < 5 {
+        if self.ptype == HidIoPacketType::Sync && len < 2
+            || self.ptype != HidIoPacketType::Sync && len < 5
+        {
             error!(
                 "Serialization too small: {} -> {:02X?}",
                 len,
@@ -943,11 +946,12 @@ impl<H: ArrayLength<u8>> fmt::Display for HidIoPacketBuffer<H> {
     }
 }
 
-//#[cfg(all(not(test), target_feature = "thumb-mode"))]
+#[cfg(all(not(test), target_feature = "thumb-mode"))]
 #[cfg(all(not(test), feature = "device"))]
 #[lang = "eh_personality"]
 fn eh_personality() {}
 
+#[cfg(all(not(test), target_feature = "thumb-mode"))]
 #[cfg(all(not(test), feature = "device"))]
 #[panic_handler]
 fn panic(_info: &PanicInfo) -> ! {