spi: add handling for ClockData::Lsb* modes

Author: newAM

CHANGELOG.md

@@ -4,6 +4,10 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [Unreleased]
+### Fixed
+- Fixed SPI SCLK pin states for the `ClockData::MsbPosIn` and `ClockData::LsbPosIn` SPI modes.
+
 ## [0.23.1] - 2025-03-29
 ### Added
 - Added a `with_device` method to access device-specific features, such as the EEPROM.

src/spi.rs

@@ -501,15 +501,14 @@ where
         // assert the chip select pin
         let mut value_cs_asserted: u8 = lock.value & !self.cs_mask();
 
+        // drive pin 0, the clock pin according to the CPOL setting
+        // Reference: https://github.com/ftdi-rs/ftdi-embedded-hal/pull/69
         match self.pol.clk {
-            ClockData::MsbNegIn => {
+            ClockData::MsbNegIn | ClockData::LsbNegIn => {
                 value_cs_asserted |= 1;
             }
-            ClockData::MsbPosIn => {
-                value_cs_asserted &= !(1);
-            }
-            _ => {
-                unimplemented!()
+            ClockData::MsbPosIn | ClockData::LsbPosIn => {
+                value_cs_asserted &= !1;
             }
         }
 
@@ -555,17 +554,18 @@ where
 
         // deassert the chip select pin
         let mut value_cs_deasserted: u8 = lock.value | self.cs_mask();
+
+        // drive pin 0, the clock pin according to the CPOL setting
+        // Reference: https://github.com/ftdi-rs/ftdi-embedded-hal/pull/69
         match self.pol.clk {
-            ClockData::MsbNegIn => {
+            ClockData::MsbNegIn | ClockData::LsbNegIn => {
                 value_cs_deasserted |= 1;
             }
-            ClockData::MsbPosIn => {
-                value_cs_deasserted &= !(1);
-            }
-            _ => {
-                unimplemented!()
+            ClockData::MsbPosIn | ClockData::LsbPosIn => {
+                value_cs_deasserted &= !1;
             }
         }
+
         lock.ft.send(
             MpsseCmdBuilder::new()
                 .set_gpio_lower(value_cs_deasserted, direction)