From d7bc1b51196c346978268611621d55f4728dc522 Mon Sep 17 00:00:00 2001
From: Finomnis <finomnis@gmail.com>
Date: Sat, 18 Nov 2023 23:08:12 +0100
Subject: [PATCH] Minor comment

---
 src/common/lpspi/bus.rs | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/common/lpspi/bus.rs b/src/common/lpspi/bus.rs
index 9dcde6e3..7d42e3cb 100644
--- a/src/common/lpspi/bus.rs
+++ b/src/common/lpspi/bus.rs
@@ -1,5 +1,4 @@
 use eh1::{delay::DelayUs, spi::MODE_0};
-use imxrt_dma::channel::Channel;
 
 use super::{
     Arbiter, LpspiBus, LpspiData, LpspiDataInner, LpspiDevice, LpspiDma, LpspiDriver, LpspiError,
@@ -17,6 +16,8 @@ impl<const N: u8> LpspiBus<N> {
     /// TODO
     pub fn new<SDO, SDI, SCK>(
         lpspi: ral::lpspi::Instance<N>,
+        // TODO: Open question: How to make those pins optional? (For example, WS2812 driver only uses SDO pin)
+        //       Or should we simply do a `new_without_pins` again?
         mut pins: Pins<SDO, SDI, SCK>,
         data_storage: &'static mut Option<LpspiData<N>>,
         clk_frequency: u32,