Minor fixes to attr writes

asusctl/src/main.rs

@@ -31,15 +31,11 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
 
     let missing_argument_k = gumdrop::Error::missing_argument(Opt::Short('k'));
     let parsed = match CliStart::parse_args_default(&args) {
-        Ok(p) => {
+        Ok(p) => p,
-            p
+        Err(err) if err.to_string() == missing_argument_k.to_string() => CliStart {
-        }
+            kbd_bright: Some(LedBrightness::new(None)),
-        Err(err) if err.to_string() == missing_argument_k.to_string() => {
+            ..Default::default()
-            CliStart {
+        },
-                kbd_bright: Some(LedBrightness::new(None)),
-                ..Default::default()
-            }
-        }
         Err(err) => {
             eprintln!("source {}", err);
             std::process::exit(2);

daemon/src/ctrl_aura/controller.rs

@@ -422,10 +422,7 @@ mod tests {
 
         controller.supported_modes.multizone.push(AuraZone::Key2);
         assert_eq!(
-            controller
+            controller.set_effect(effect).unwrap_err().to_string(),
-                .set_effect(effect)
-                .unwrap_err()
-                .to_string(),
             "No supported Aura keyboard"
         );
     }

daemon/src/ctrl_platform.rs

@@ -78,7 +78,7 @@ impl CtrlPlatform {
     }
 
     fn set_gfx_mode(&self, mode: GpuMode) -> Result<(), RogError> {
-        self.platform.set_gpu_mux_mode(mode.to_mux())?;
+        self.platform.set_gpu_mux_mode(mode.to_mux_attr())?;
         // self.update_initramfs(enable)?;
         if mode == GpuMode::Discrete {
             info!("Set system-level graphics mode: Dedicated Nvidia");

rog-aura/src/sequencer/effects.rs

@@ -142,11 +142,7 @@ impl EffectState for Breathe {
             *use_colour1 = !*use_colour1;
         }
 
-        let colour = if !*use_colour1 {
+        let colour = if !*use_colour1 { colour2 } else { colour1 };
-            colour2
-        } else {
-            colour1
-        };
 
         let r1_scale = colour.0 / speed / 2;
         let g1_scale = colour.1 / speed / 2;

rog-platform/src/macros.rs

@@ -85,6 +85,7 @@ macro_rules! get_attr_u8 {
     };
 }
 
+/// Most attributes expect `u8` as a char, so `1` should be written as `b'1'`.
 #[macro_export]
 macro_rules! set_attr_u8 {
     ($(#[$doc_comment:meta])? $attr_name:literal $item:ident) => {

rog-platform/src/platform.rs

@@ -76,25 +76,26 @@ pub enum GpuMode {
 }
 
 impl GpuMode {
-    pub fn to_mux(&self) -> u8 {
+    /// For writing to `gpu_mux_mode` attribute
+    pub fn to_mux_attr(&self) -> u8 {
         if *self == Self::Discrete {
-            return 0;
+            return b'0';
         }
-        1
+        b'1'
     }
 
-    pub fn to_dgpu(&self) -> u8 {
+    pub fn to_dgpu_attr(&self) -> u8 {
         if *self == Self::Integrated {
-            return 1;
+            return b'1';
         }
-        0
+        b'0'
     }
 
-    pub fn to_egpu(&self) -> u8 {
+    pub fn to_egpu_attr(&self) -> u8 {
         if *self == Self::Egpu {
-            return 1;
+            return b'1';
         }
-        0
+        b'0'
     }
 
     pub fn from_mux(num: u8) -> Self {