Implement ST7306 display

Signed-off-by: Daniel Schaefer <[email protected]>

Author: JohnAZoidberg

Cargo.lock

@@ -26,8 +26,7 @@ usbd-serial = "0.1.1"
 usbd-hid = "0.5.1"
 fugit = "0.3.6"
 
-display-interface-spi = "0.4.1"
-mipidsi = "0.6.0"
+st7306-lcd = { git = "ssh://[email protected]/FrameworkComputer/st7306.git" }
 embedded-graphics = "0.7"
 tinybmp = "0.4.0"
 

b1display/Cargo.toml

@@ -9,7 +9,6 @@ use cortex_m::delay::Delay;
 use defmt_rtt as _;
 use embedded_hal::digital::v2::{InputPin, OutputPin};
 
-use mipidsi::Orientation;
 use rp2040_hal::gpio::bank0::Gpio18;
 use rp2040_hal::gpio::{Output, Pin, PushPull};
 //#[cfg(debug_assertions)]
@@ -18,6 +17,9 @@ use rp2040_panic_usb_boot as _;
 
 use embedded_graphics::pixelcolor::Rgb565;
 use embedded_graphics::prelude::*;
+use embedded_graphics::primitives::*;
+use embedded_hal::blocking::spi;
+use st7306_lcd::{Orientation, ST7306};
 
 // Provide an alias for our BSP so we can switch targets quickly.
 // Uncomment the BSP you included in Cargo.toml, the rest of the code does not need to change.
@@ -29,6 +31,7 @@ use bsp::hal::{
     clocks::{init_clocks_and_plls, Clock},
     pac,
     sio::Sio,
+    gpio,
     usb,
     watchdog::Watchdog,
     Timer,
@@ -42,7 +45,7 @@ use usb_device::{class_prelude::*, prelude::*};
 use usbd_serial::{SerialPort, USB_CLASS_CDC};
 
 // Used to demonstrate writing formatted strings
-use core::fmt::Write;
+use core::fmt::{Debug, Write};
 use heapless::String;
 
 use lotus_inputmodules::control::*;
@@ -126,10 +129,12 @@ fn main() -> ! {
     let _spi_sclk = pins.scl.into_mode::<bsp::hal::gpio::FunctionSpi>();
     let _spi_mosi = pins.sda.into_mode::<bsp::hal::gpio::FunctionSpi>();
     let _spi_miso = pins.miso.into_mode::<bsp::hal::gpio::FunctionSpi>();
-    let spi = bsp::hal::Spi::<_, _, 8>::new(pac.SPI1);
+    let spi = bsp::hal::Spi::<_, _, 8>::new(pac.SPI0);
     // Display control pins
     let dc = pins.dc.into_push_pull_output();
-    let mut lcd_led = pins.backlight.into_push_pull_output();
+    //let mut lcd_led = pins.backlight.into_push_pull_output();
+    let mut cs = pins.cs.into_push_pull_output();
+    cs.set_low().unwrap();
     let rst = pins.rstb.into_push_pull_output();
 
     let spi = spi.init(
@@ -139,27 +144,46 @@ fn main() -> ! {
         &embedded_hal::spi::MODE_0,
     );
 
-    // Create a DisplayInterface from SPI and DC pin, with no manual CS control
-    let di = display_interface_spi::SPIInterfaceNoCS::new(spi, dc);
-    let mut disp = mipidsi::Builder::st7735s(di)
-        .with_invert_colors(true) // Looks cooler. TODO: Should invert image not entire screen
-        .with_orientation(Orientation::PortraitInverted(false))
-        .init(&mut delay, Some(rst))
-        .unwrap();
-    disp.clear(Rgb565::WHITE).unwrap();
+    let mut disp: ST7306<
+        rp2040_hal::Spi<rp2040_hal::spi::Enabled, pac::SPI0, 8>,
+        Pin<gpio::bank0::Gpio20, Output<PushPull>>,
+        Pin<gpio::bank0::Gpio17, Output<PushPull>>,
+        Pin<gpio::bank0::Gpio21, Output<PushPull>>,
+        25,
+        200,
+    > = ST7306::new(spi, dc, cs, rst, false, 300, 460);
+    disp.init(&mut delay).unwrap();
+
+    disp.clear(Rgb565::BLACK).unwrap();
 
     let logo_rect = draw_logo(&mut disp).unwrap();
+    Rectangle::new(Point::new(10, 10), Size::new(10, 10))
+        .into_styled(PrimitiveStyle::with_fill(Rgb565::WHITE))
+        .draw(&mut disp)
+        .unwrap();
+    Rectangle::new(Point::new(20, 20), Size::new(10, 10))
+        .into_styled(PrimitiveStyle::with_fill(Rgb565::WHITE))
+        .draw(&mut disp)
+        .unwrap();
+    Rectangle::new(Point::new(30, 30), Size::new(10, 10))
+        .into_styled(PrimitiveStyle::with_fill(Rgb565::WHITE))
+        .draw(&mut disp)
+        .unwrap();
+    Rectangle::new(Point::new(40, 40), Size::new(10, 10))
+        .into_styled(PrimitiveStyle::with_fill(Rgb565::WHITE))
+        .draw(&mut disp)
+        .unwrap();
+    Rectangle::new(Point::new(50, 50), Size::new(10, 10))
+        .into_styled(PrimitiveStyle::with_fill(Rgb565::WHITE))
+        .draw(&mut disp)
+        .unwrap();
     draw_text(
         &mut disp,
         "Framework",
-        Point::new(0, LOGO_OFFSET + logo_rect.size.height as i32),
+        Point::new(LOGO_OFFSET_X, LOGO_OFFSET_Y + logo_rect.size.height as i32),
     )
     .unwrap();
 
-    // Wait until the background and image have been rendered otherwise
-    // the screen will show random pixels for a brief moment
-    lcd_led.set_high().unwrap();
-
     let sleep = pins.sleep.into_pull_down_input();
 
     let mut state = State {
@@ -172,10 +196,9 @@ fn main() -> ! {
     let mut prev_timer = timer.get_counter().ticks();
 
     loop {
-        // TODO: Current hardware revision does not have the sleep pin wired up :(
         // Go to sleep if the host is sleeping
-        let _host_sleeping = sleep.is_low().unwrap();
-        //handle_sleep(host_sleeping, &mut state, &mut matrix, &mut delay);
+        let host_sleeping = sleep.is_low().unwrap();
+        handle_sleep(host_sleeping, &mut state, &mut delay, &mut disp);
 
         // Handle period display updates. Don't do it too often
         if timer.get_counter().ticks() > prev_timer + 20_000 {
@@ -212,7 +235,7 @@ fn main() -> ! {
                 Ok(count) => {
                     if let Some(command) = parse_command(count, &buf) {
                         if let Command::Sleep(go_sleeping) = command {
-                            handle_sleep(go_sleeping, &mut state, &mut delay, &mut lcd_led);
+                            handle_sleep(host_sleeping, &mut state, &mut delay, &mut disp);
                         } else if let SleepState::Awake = state.sleeping {
                             // While sleeping no command is handled, except waking up
                             //handle_command(&command, &mut disp, logo_rect);
@@ -227,20 +250,26 @@ fn main() -> ! {
     }
 }
 
-fn handle_sleep(
+fn handle_sleep<SPI, DC, CS, RST, const COLS: usize, const ROWS: usize>(
     go_sleeping: bool,
     state: &mut State,
     _delay: &mut Delay,
-    lcd_led: &mut Pin<Gpio18, Output<PushPull>>,
-) {
+    disp: &mut ST7306<SPI, DC, CS, RST, COLS, ROWS>,
+) where
+    SPI: spi::Write<u8>,
+    DC: OutputPin,
+    CS: OutputPin,
+    RST: OutputPin,
+    <SPI as spi::Write<u8>>::Error: Debug,
+{
     match (state.sleeping.clone(), go_sleeping) {
         (SleepState::Awake, false) => (),
         (SleepState::Awake, true) => {
             state.sleeping = SleepState::Sleeping;
             //state.grid = display_sleep();
 
             // Turn off backlight
-            lcd_led.set_low().unwrap();
+            disp.on_off(false);
 
             // TODO: Power Display controller down
 
@@ -253,9 +282,7 @@ fn handle_sleep(
             state.sleeping = SleepState::Awake;
 
             // TODO: Power display controller back on
-
-            // Turn backlight back on
-            lcd_led.set_high().unwrap();
+            disp.on_off(true);
         }
     }
 }

b1display/src/main.rs

@@ -30,8 +30,7 @@ fugit = "0.3.6"
 is31fl3741 = { git = "https://github.com/JohnAZoidberg/is31fl3741", branch = "all-at-once", optional = true }
 
 # B1 Display
-display-interface-spi = { version = "0.4.1", optional = true }
-mipidsi = { version = "0.6.0", optional = true }
+st7306-lcd = { git = "ssh://[email protected]/FrameworkComputer/st7306.git", optional = true }
 embedded-graphics = { version = "0.7", optional = true }
 tinybmp = { version = "0.4.0", optional = true }
 
@@ -42,5 +41,5 @@ ws2812-pio = { version = "0.5.0", optional = true }
 [features]
 default = []
 ledmatrix = [ "is31fl3741" ]
-b1display = [ "display-interface-spi", "mipidsi", "embedded-graphics", "tinybmp" ]
+b1display = [ "st7306-lcd", "embedded-graphics", "tinybmp" ]
 c1minimal = ["smart-leds", "ws2812-pio" ]