From fabeee6ed70f236cefb0014f487953f3872c96fd Mon Sep 17 00:00:00 2001
From: PhoenixCausesOof <63979557+PhoenixCausesOof@users.noreply.github.com>
Date: Sun, 29 Jun 2025 03:39:48 -0300
Subject: [PATCH] Fix `auto-cpufreq --monitor` reporting the "CPU frequency
 scaling" section wrong. (#855)

It used to always read the config file's "battery" section and report that, even if connected to a charger. I used a hacky solution (simply check the state of the battery, i.e., charging or not charging) and read the correct part of the config file based on that.
---
 auto_cpufreq/modules/system_info.py | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/auto_cpufreq/modules/system_info.py b/auto_cpufreq/modules/system_info.py
index 49bbf38..eed54a3 100644
--- a/auto_cpufreq/modules/system_info.py
+++ b/auto_cpufreq/modules/system_info.py
@@ -144,21 +144,23 @@ class SystemInfo:
             return None
 
     @staticmethod
-    def current_epp() -> str | None:
+    def current_epp(is_ac_plugged: bool) -> str | None:
         epp_path = "/sys/devices/system/cpu/cpu0/cpufreq/energy_performance_preference"
         if not Path(epp_path).exists():
             return None
-        return config.get_config().get(
-            "battery", "energy_performance_preference", fallback="balance_power"
+            
+        return config.get_config().get( 
+            "charger" if is_ac_plugged else "battery", "energy_performance_preference", fallback="balance_power"
         )
 
     @staticmethod
-    def current_epb() -> str | None:
+    def current_epb(is_ac_plugged: bool) -> str | None:
         epb_path = "/sys/devices/system/cpu/intel_pstate"
         if not Path(epb_path).exists():
             return None
+
         return config.get_config().get(
-            "battery", "energy_perf_bias", fallback="balance_power"
+            "charger" if is_ac_plugged else "battery", "energy_perf_bias", fallback="balance_power"
         )
 
     @staticmethod
@@ -315,6 +317,8 @@ class SystemInfo:
         return AVAILABLE_GOVERNORS_SORTED[-1]
 
     def generate_system_report(self) -> SystemReport:
+        battery_info = self.battery_info()
+
         return SystemReport(
             distro_name=self.distro_name,
             distro_ver=self.distro_version,
@@ -324,8 +328,8 @@ class SystemInfo:
             cpu_driver=self.cpu_driver,
             kernel_version=self.kernel_version,
             current_gov=self.current_gov(),
-            current_epp=self.current_epp(),
-            current_epb=self.current_epb(),
+            current_epp=self.current_epp(battery_info.is_ac_plugged),
+            current_epb=self.current_epb(battery_info.is_ac_plugged),
             cpu_fan_speed=self.cpu_fan_speed(),
             cpu_usage=self.cpu_usage(),
             cpu_max_freq=self.cpu_max_freq(),
@@ -334,7 +338,7 @@ class SystemInfo:
             avg_load=self.avg_load(),
             cores_info=self.get_cpu_info(),
             is_turbo_on=self.turbo_on(),
-            battery_info=self.battery_info(),
+            battery_info=battery_info,
         )