aokzoe A1 not detected as a generic AMD handheld

drokath commented

2023-11-25 21:34:52 +00:00

Contributor

Describe what you'd like to be able to do

On bazzite with DeckyLoader and PowerTools 1.4.0 installed: the Aokzoe A1 handheld uses the AMD Ryzen 7 6800U APU and should be detected as a Generic AMD device.

Describe alternatives you've considered

No alternative considered

Anything else?

In powertools.log

model name      : AMD Ryzen 7 6800U with Radeon Graphics

[...]

[INFO] Detected device automatically Unknown, using driver: Unknown (This can be overriden)

Content of /proc/cpuinfo:

--
cpu family	: 25
model		: 68
model name	: AMD Ryzen 7 6800U with Radeon Graphics
stepping	: 1
microcode	: 0xa404102

I have tried to patch the following, without success:

diff --git a/backend/limits_core/src/json/base.rs b/backend/limits_core/src/json/base.rs
index cbb77fc..323f426 100644
--- a/backend/limits_core/src/json/base.rs
+++ b/backend/limits_core/src/json/base.rs
@@ -1,5 +1,5 @@
-use std::default::Default;
 use serde::{Deserialize, Serialize};
+use std::default::Default;
 
 /// Base JSON limits information
 #[derive(Serialize, Deserialize, Debug, Clone)]
@@ -131,7 +131,7 @@ impl Default for Base {
                     name: "AMD R7 6800U".to_owned(),
                     conditions: super::Conditions {
                         dmi: None,
-                        cpuinfo: Some("model name\t+: AMD Ryzen 7 6800U\n".to_owned()),
+                        cpuinfo: Some("model name\t+: AMD Ryzen 7 6800U( with Radeon Graphics)?\n".to_owned()),
                         os: None,
                         command: None,
                         file_exists: None,
diff --git a/backend/limits_srv/pt_limits.json b/backend/limits_srv/pt_limits.json
index 2b34975..2cea737 100644
--- a/backend/limits_srv/pt_limits.json
+++ b/backend/limits_srv/pt_limits.json
@@ -214,7 +214,7 @@
       "name": "AMD R7 6800U",
       "conditions": {
         "dmi": null,
-        "cpuinfo": "model name\t+: AMD Ryzen 7 6800U\n",
+        "cpuinfo": "model name\t+: AMD Ryzen 7 6800U( with Radeon Graphics)?\n",
         "os": null,
         "command": null,
         "file_exists": null

### Describe what you'd like to be able to do On bazzite with DeckyLoader and PowerTools 1.4.0 installed: the Aokzoe A1 handheld uses the AMD Ryzen 7 6800U APU and should be detected as a Generic AMD device. ### Describe alternatives you've considered No alternative considered ### Anything else? In powertools.log ``` model name : AMD Ryzen 7 6800U with Radeon Graphics [...] [INFO] Detected device automatically Unknown, using driver: Unknown (This can be overriden) ``` Content of /proc/cpuinfo: ``` -- cpu family : 25 model : 68 model name : AMD Ryzen 7 6800U with Radeon Graphics stepping : 1 microcode : 0xa404102 ``` I have tried to patch the following, without success: ``` diff --git a/backend/limits_core/src/json/base.rs b/backend/limits_core/src/json/base.rs index cbb77fc..323f426 100644 --- a/backend/limits_core/src/json/base.rs +++ b/backend/limits_core/src/json/base.rs @@ -1,5 +1,5 @@ -use std::default::Default; use serde::{Deserialize, Serialize}; +use std::default::Default; /// Base JSON limits information #[derive(Serialize, Deserialize, Debug, Clone)] @@ -131,7 +131,7 @@ impl Default for Base { name: "AMD R7 6800U".to_owned(), conditions: super::Conditions { dmi: None, - cpuinfo: Some("model name\t+: AMD Ryzen 7 6800U\n".to_owned()), + cpuinfo: Some("model name\t+: AMD Ryzen 7 6800U( with Radeon Graphics)?\n".to_owned()), os: None, command: None, file_exists: None, diff --git a/backend/limits_srv/pt_limits.json b/backend/limits_srv/pt_limits.json index 2b34975..2cea737 100644 --- a/backend/limits_srv/pt_limits.json +++ b/backend/limits_srv/pt_limits.json @@ -214,7 +214,7 @@ "name": "AMD R7 6800U", "conditions": { "dmi": null, - "cpuinfo": "model name\t+: AMD Ryzen 7 6800U\n", + "cpuinfo": "model name\t+: AMD Ryzen 7 6800U( with Radeon Graphics)?\n", "os": null, "command": null, "file_exists": null ```

drokath added the

enhancement

label 2023-11-25 21:34:52 +00:00

drokath commented

2023-11-25 21:38:30 +00:00

Author

Contributor

Nevermind after cleaning ~/.config/powertools and rebooting the device it is now properly detected. Will open a PR.

drokath commented

2023-11-26 09:55:03 +00:00

Author

Contributor

Can not fork repo nor push a new branch unfortunately.

NGnius commented

2023-11-26 15:39:12 +00:00

Owner

Can not fork repo nor push a new branch unfortunately.

You should be able to now -- I have disabled new users' ability to create repositories by default to prevent people from filling up the limited storage. I have now increased your repo limit from none to 10.

> Can not fork repo nor push a new branch unfortunately. You should be able to now -- I have disabled new users' ability to create repositories by default to prevent people from filling up the limited storage. I have now increased your repo limit from none to 10.

drokath referenced this issue

2023-11-26 18:14:49 +00:00

Improve AMD 6800 handheld detection #132

drokath commented

2023-11-26 18:15:49 +00:00

Author

Contributor

Great thank you!