Add Ally X config

CHANGELOG.md

@@ -2,6 +2,9 @@
 
 ## [Unreleased]
 
+### Changed
+- Add Ally X aura config
+
 ## [v6.0.11]
 
 ### Changed

asusd/src/ctrl_platform.rs

@@ -132,13 +132,12 @@ impl CtrlPlatform {
                             | inotify::WatchMask::ATTRIB
                             | inotify::WatchMask::CREATE,
                     )
-                    .map_err(|e| {
+                    .inspect_err(|e| {
                         if e.kind() == std::io::ErrorKind::NotFound {
                             error!("Not found: {:?}", config_path);
                         } else {
                             error!("Could not set asusd config inotify: {:?}", config_path);
                         }
-                        e
                     })
                     .ok();
                 let mut events = inotify.into_event_stream(&mut buffer).unwrap();

config-traits/src/lib.rs

@@ -270,6 +270,8 @@ mod tests {
             }
         }
 
+        let _ = Test {};
+
         impl crate::StdConfigLoad1<Old1> for Test {}
     }
 
@@ -319,6 +321,8 @@ mod tests {
             }
         }
 
+        let _ = Test {};
+
         impl crate::StdConfigLoad3<Old1, Old2, Old3> for Test {}
     }
 }

rog-aura/data/aura_support.ron

@@ -890,4 +890,13 @@
         advanced_type: None,
         power_zones: [Keyboard],
     ),
+    (
+        device_name: "RC72L",
+        product_id: "",
+        layout_name: "ga401q",
+        basic_modes: [Static, Breathe, Pulse],
+        basic_zones: [],
+        advanced_type: None,
+        power_zones: [Keyboard],
+    ),
 ])

rog-aura/src/effects/base.rs

@@ -2,6 +2,7 @@ use super::{EffectState, InputForEffect};
 use crate::keyboard::{KeyLayout, LedCode};
 use crate::Colour;
 
+#[allow(dead_code)]
 pub struct InputBased {
     led: LedCode,
     colour: Colour,

rog-control-center/src/main.rs

@@ -74,13 +74,7 @@ async fn main() -> Result<()> {
         .format_timestamp(None)
         .init();
 
-    let supported_properties = match proxy.supported_properties() {
-        Ok(s) => s,
-        Err(_e) => {
-            // TODO: show an error window
-            Vec::default()
-        }
-    };
+    let supported_properties = proxy.supported_properties().unwrap_or_default();
 
     // Startup
     let mut config = Config::new().load();

rog-control-center/src/notify.rs

@@ -191,17 +191,14 @@ pub fn start_notifications(
     let enabled_notifications_copy = config.clone();
     // GPU Mode change/action notif
     tokio::spawn(async move {
-        let conn = zbus::Connection::system().await.map_err(|e| {
-            no_supergfx(&e);
-            e
+        let conn = zbus::Connection::system().await.inspect_err(|e| {
+            no_supergfx(e);
         })?;
-        let proxy = SuperProxy::builder(&conn).build().await.map_err(|e| {
-            no_supergfx(&e);
-            e
+        let proxy = SuperProxy::builder(&conn).build().await.inspect_err(|e| {
+            no_supergfx(e);
         })?;
-        let _ = proxy.mode().await.map_err(|e| {
-            no_supergfx(&e);
-            e
+        let _ = proxy.mode().await.inspect_err(|e| {
+            no_supergfx(e);
         })?;
 
         let proxy_copy = proxy.clone();

rog-control-center/translations/en/rog-control-center.po

@@ -2,7 +2,7 @@
 msgid ""
 msgstr ""
 "Project-Id-Version: PACKAGE VERSION\n"
-"POT-Creation-Date: 2024-06-09 00:20+0000\n"
+"POT-Creation-Date: 2024-07-25 10:03+0000\n"
 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
 "Language-Team: LANGUAGE <LL@li.org>\n"

rog-platform/src/firmware_attributes.rs

@@ -0,0 +1,289 @@
+// Firmware attribute interfaces
+// - current_value
+// - default_value
+// - display_name
+// - default_value
+// - possible_values
+// - max_value
+// - min_value
+// - scalar_increment
+// - type
+
+use std::{
+    fs::{read_dir, File},
+    io::Read,
+    path::{Path, PathBuf},
+};
+
+use log::error;
+
+use crate::error::PlatformError;
+
+const BASE_DIR: &str = "/sys/class/firmware-attributes/asus-bioscfg/attributes/";
+
+fn read_i32(path: &Path) -> Result<i32, PlatformError> {
+    if let Ok(mut f) = File::open(path) {
+        let mut buf = String::new();
+        f.read_to_string(&mut buf)?;
+        buf = buf.trim_end().to_string();
+        if let Ok(int) = buf.parse::<i32>() {
+            return Ok(int);
+        }
+    }
+    Err(PlatformError::ParseNum)
+}
+
+fn read_string(path: &Path) -> Result<String, PlatformError> {
+    if let Ok(mut f) = File::open(path) {
+        let mut buf = String::new();
+        f.read_to_string(&mut buf)?;
+        buf = buf.trim_end().to_string();
+        return Ok(buf);
+    }
+    Err(PlatformError::ParseNum)
+}
+
+fn attr_path_if_exists(mut base_path: PathBuf, attr: &str) -> Option<PathBuf> {
+    base_path.push(attr);
+    if base_path.exists() {
+        return Some(base_path.clone());
+    }
+    None
+}
+
+#[derive(Debug, Default)]
+pub struct AttrInteger {
+    current: PathBuf,
+    default: Option<PathBuf>,
+    min: Option<PathBuf>,
+    max: Option<PathBuf>,
+    scalar_inc: Option<PathBuf>,
+}
+
+impl AttrInteger {
+    pub fn current_value(&self) -> Result<i32, PlatformError> {
+        read_i32(&self.current)
+    }
+
+    fn read_i32(path: Option<&PathBuf>) -> Result<Option<i32>, PlatformError> {
+        if let Some(path) = path {
+            let int = read_i32(path)?;
+            return Ok(Some(int));
+        }
+        Ok(None)
+    }
+
+    pub fn default_value(&self) -> Result<Option<i32>, PlatformError> {
+        Self::read_i32(self.default.as_ref())
+    }
+
+    pub fn min_value(&self) -> Result<Option<i32>, PlatformError> {
+        Self::read_i32(self.min.as_ref())
+    }
+
+    pub fn max_value(&self) -> Result<Option<i32>, PlatformError> {
+        Self::read_i32(self.max.as_ref())
+    }
+
+    pub fn scalar_increment(&self) -> Result<Option<i32>, PlatformError> {
+        Self::read_i32(self.scalar_inc.as_ref())
+    }
+}
+
+#[derive(Debug, Default)]
+pub struct AttEnumInteger {
+    current: PathBuf,
+    default: Option<PathBuf>,
+    possible: Option<PathBuf>,
+}
+
+impl AttEnumInteger {
+    pub fn current_value(&self) -> Result<i32, PlatformError> {
+        read_i32(&self.current)
+    }
+
+    pub fn default_value(&self) -> Result<Option<i32>, PlatformError> {
+        if let Some(path) = self.default.as_ref() {
+            let int = read_i32(path)?;
+            return Ok(Some(int));
+        }
+        Ok(None)
+    }
+
+    pub fn possible_values(&self) -> Vec<i32> {
+        let mut output = Vec::new();
+        if let Some(path) = self.possible.as_ref() {
+            if let Ok(string) = read_string(path) {
+                for n in string.split(';') {
+                    match n.parse::<i32>() {
+                        Ok(n) => output.push(n),
+                        Err(e) => error!("Couldn't parse num: {e:?}"),
+                    }
+                }
+            }
+        }
+
+        output
+    }
+}
+
+#[derive(Debug, Default)]
+pub struct AttEnumString {
+    current: PathBuf,
+    default: Option<PathBuf>,
+    possible: Option<PathBuf>,
+}
+
+impl AttEnumString {
+    pub fn current_value(&self) -> Result<String, PlatformError> {
+        read_string(&self.current)
+    }
+
+    pub fn default_value(&self) -> Result<Option<String>, PlatformError> {
+        if let Some(path) = self.default.as_ref() {
+            let string = read_string(path)?;
+            return Ok(Some(string));
+        }
+        Ok(None)
+    }
+
+    pub fn possible_values(&self) -> Vec<String> {
+        let mut output = Vec::new();
+        if let Some(path) = self.possible.as_ref() {
+            if let Ok(string) = read_string(path) {
+                for n in string.split(';') {
+                    if !n.is_empty() {
+                        output.push(n.to_owned());
+                    }
+                }
+            }
+        }
+
+        output
+    }
+}
+
+#[derive(Debug, Default)]
+pub enum AttrType {
+    Integer(AttrInteger),
+    EnumInt(AttEnumInteger),
+    EnumStr(AttEnumString),
+    #[default]
+    Unknown,
+}
+
+#[derive(Debug, Default)]
+pub struct Attribute {
+    name: String,
+    help: String,
+    _base_path: PathBuf,
+    attr_type: AttrType,
+}
+
+impl Attribute {
+    pub fn name(&self) -> &str {
+        self.name.as_str()
+    }
+
+    pub fn help(&self) -> &str {
+        self.help.as_str()
+    }
+}
+
+pub fn get_attributes() -> Vec<Attribute> {
+    let mut attrs = Vec::new();
+
+    let dir = read_dir(BASE_DIR).unwrap();
+    dir.for_each(|d| {
+        if let Ok(base_dir) = d {
+            let mut attr_path = base_dir.path();
+            let mut attr = Attribute {
+                _base_path: base_dir.path(),
+                attr_type: AttrType::Unknown,
+                ..Default::default()
+            };
+
+            // TYPE
+            attr_path.push("type");
+            let mut buf = String::new();
+            if let Ok(mut f) = File::open(&attr_path) {
+                f.read_to_string(&mut buf).unwrap();
+                buf = buf.trim_end().to_string();
+
+                attr_path.pop();
+                let mut current = attr_path.clone();
+                current.push("current_value");
+
+                match buf.to_lowercase().as_str() {
+                    "integer" => {
+                        attr.attr_type = AttrType::Integer(AttrInteger {
+                            current,
+                            default: attr_path_if_exists(attr_path.clone(), "default_value"),
+                            min: attr_path_if_exists(attr_path.clone(), "min_value"),
+                            max: attr_path_if_exists(attr_path.clone(), "max_value"),
+                            scalar_inc: attr_path_if_exists(attr_path.clone(), "scalar_increment"),
+                        })
+                    }
+                    _ => {
+                        // Check what the current_value type is
+                        if let Ok(mut f) = File::open(&current) {
+                            let mut buf = String::new();
+                            f.read_to_string(&mut buf).unwrap();
+                            buf = buf.trim_end().to_string();
+                            if buf.parse::<i32>().is_ok() {
+                                attr.attr_type = AttrType::EnumInt(AttEnumInteger {
+                                    current,
+                                    default: attr_path_if_exists(
+                                        attr_path.clone(),
+                                        "default_value",
+                                    ),
+                                    possible: attr_path_if_exists(
+                                        attr_path.clone(),
+                                        "possible_values",
+                                    ),
+                                });
+                            } else {
+                                attr.attr_type = AttrType::EnumStr(AttEnumString {
+                                    current,
+                                    default: attr_path_if_exists(
+                                        attr_path.clone(),
+                                        "default_value",
+                                    ),
+                                    possible: attr_path_if_exists(
+                                        attr_path.clone(),
+                                        "possible_values",
+                                    ),
+                                });
+                            }
+                        }
+                    }
+                }
+            }
+            // DISPLAY_NAME
+            attr_path.push("display_name");
+            if let Ok(res) = read_string(&attr_path) {
+                attr.help = res;
+            }
+            // DISPLAY_NAME
+            attr_path.pop();
+            attr.name = attr_path.file_name().unwrap().to_string_lossy().to_string();
+
+            attrs.push(attr);
+        }
+    });
+
+    attrs
+}
+
+#[cfg(test)]
+mod tests {
+    use super::get_attributes;
+
+    #[test]
+    fn find_attributes() {
+        let attrs = get_attributes();
+        for a in attrs {
+            dbg!(&a);
+        }
+    }
+}

rog-platform/src/lib.rs

@@ -44,7 +44,7 @@ pub fn read_attr_bool(device: &Device, attr_name: &str) -> Result<bool> {
 
 pub fn write_attr_bool(device: &mut Device, attr: &str, value: bool) -> Result<()> {
     device
-        .set_attribute_value(attr, &(value as u8).to_string())
+        .set_attribute_value(attr, value.to_string())
         .map_err(|e| PlatformError::IoPath(attr.into(), e))
 }
 
@@ -58,7 +58,7 @@ pub fn read_attr_u8(device: &Device, attr_name: &str) -> Result<u8> {
 
 pub fn write_attr_u8(device: &mut Device, attr: &str, value: u8) -> Result<()> {
     device
-        .set_attribute_value(attr, &(value).to_string())
+        .set_attribute_value(attr, value.to_string())
         .map_err(|e| PlatformError::IoPath(attr.into(), e))
 }
 

rog-profiles/src/fan_curve_set.rs

@@ -91,7 +91,7 @@ impl std::str::FromStr for CurveData {
         }
 
         for (index, value) in input.split(',').enumerate() {
-            for (select, num) in value.splitn(2, |c| c == 'c' || c == ':').enumerate() {
+            for (select, num) in value.splitn(2, ['c', ':']).enumerate() {
                 if num.contains('%') {
                     percentages = true;
                 }
@@ -173,13 +173,13 @@ impl CurveData {
         for (index, out) in self.pwm.iter().enumerate() {
             let pwm = pwm_str(pwm_num, index);
             trace!("writing {pwm}");
-            device.set_attribute_value(&pwm, &out.to_string())?;
+            device.set_attribute_value(&pwm, out.to_string())?;
         }
 
         for (index, out) in self.temp.iter().enumerate() {
             let temp = temp_str(pwm_num, index);
             trace!("writing {temp}");
-            device.set_attribute_value(&temp, &out.to_string())?;
+            device.set_attribute_value(&temp, out.to_string())?;
         }
 
         // Enable must be done *after* all points are written pwm3_enable