feat(cli): Added option to inc and dec brightness

Author: uttarayan21

flake.lock

@@ -6,17 +6,34 @@ pub struct Args {
     pub op: Op,
     #[arg(short, long, action = clap::ArgAction::Count)]
     pub verbosity: u8,
-    #[arg(short, long)]
-    pub bus: Option<u8>,
 }
 #[derive(Debug, Subcommand)]
 pub enum Op {
     #[clap(name = "set")]
     SetBrightness {
         brightness: u8,
+        #[arg(short, long)]
+        bus: Option<u8>,
     },
     #[clap(name = "get")]
-    GetBrightnexs,
+    GetBrightness {
+        #[arg(short, long)]
+        bus: Option<u8>,
+    },
+    #[clap(name = "inc")]
+    IncreaseBrightness {
+        #[arg(default_value = "10")]
+        amount: u8,
+        #[arg(short, long)]
+        bus: Option<u8>,
+    },
+    #[clap(name = "dec")]
+    DecreaseBrightness {
+        #[arg(default_value = "10")]
+        amount: u8,
+        #[arg(short, long)]
+        bus: Option<u8>,
+    },
     SetInput {
         bus: u8,
         input: crate::ddc::Input,

src/cli.rs

@@ -31,9 +31,12 @@ fn main() -> Result<()> {
     }
 
     match cli.op {
-        Op::GetBrightnexs => {
+        Op::GetBrightness { bus } => {
             let list = DisplayList::probe(true)?;
-            for dinfo in list.iter() {
+            for dinfo in list.iter().filter(|info| {
+                bus.map(|bus| info.io_path() == IOPath::I2C(bus as i32))
+                    .unwrap_or(true)
+            }) {
                 tracing::info!("Found display: {}", dinfo.model());
                 let display = dinfo.open()?;
                 let backlight = display.backlight_get()?;
@@ -45,9 +48,12 @@ fn main() -> Result<()> {
                 );
             }
         }
-        Op::SetBrightness { brightness } => {
+        Op::SetBrightness { brightness, bus } => {
             let list = DisplayList::probe(true)?;
-            for dinfo in list.iter() {
+            for dinfo in list.iter().filter(|info| {
+                bus.map(|bus| info.io_path() == IOPath::I2C(bus as i32))
+                    .unwrap_or(true)
+            }) {
                 tracing::info!("Found display: {}", dinfo.model());
                 let display = dinfo.open()?;
                 display.backlight_set(brightness.into())?;
@@ -60,6 +66,48 @@ fn main() -> Result<()> {
                 );
             }
         }
+        Op::IncreaseBrightness { amount, bus } => {
+            let list = DisplayList::probe(true)?;
+            for dinfo in list.iter().filter(|info| {
+                bus.map(|bus| info.io_path() == IOPath::I2C(bus as i32))
+                    .unwrap_or(true)
+            }) {
+                tracing::info!("Found display: {}", dinfo.model());
+                let display = dinfo.open()?;
+                let current_backlight = display.backlight_get()?;
+                let new_brightness = std::cmp::min(100, current_backlight.current + amount as u16);
+                display.backlight_set(new_brightness)?;
+                let backlight = display.backlight_get()?;
+                println!(
+                    "{}: {}/{} (increased by {})",
+                    dinfo.model().blue(),
+                    backlight.current,
+                    backlight.max,
+                    amount
+                );
+            }
+        }
+        Op::DecreaseBrightness { amount, bus } => {
+            let list = DisplayList::probe(true)?;
+            for dinfo in list.iter().filter(|info| {
+                bus.map(|bus| info.io_path() == IOPath::I2C(bus as i32))
+                    .unwrap_or(true)
+            }) {
+                tracing::info!("Found display: {}", dinfo.model());
+                let display = dinfo.open()?;
+                let current_backlight = display.backlight_get()?;
+                let new_brightness = current_backlight.current.saturating_sub(amount as u16);
+                display.backlight_set(new_brightness)?;
+                let backlight = display.backlight_get()?;
+                println!(
+                    "{}: {}/{} (decreased by {})",
+                    dinfo.model().blue(),
+                    backlight.current,
+                    backlight.max,
+                    amount
+                );
+            }
+        }
         Op::GetInput { bus } => {
             let list = DisplayList::probe(true)?;
             for dinfo in list.iter().filter(|info| {