GPU: Apparently, some devices store the freqs in khz rather than hz.

This results in 0Mhz being displayed in the dropdown list, so account for this.
JoeSchubert · Mar 23, 2016 · ca667a3 · ca667a3
1 parent 2d86522
commit ca667a3
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 5 deletions.
diff --git a/app/src/main/java/com/grarak/kerneladiutor/fragments/kernel/GPUFragment.java b/app/src/main/java/com/grarak/kerneladiutor/fragments/kernel/GPUFragment.java
@@ -17,6 +17,7 @@
 package com.grarak.kerneladiutor.fragments.kernel;
 
 import android.os.Bundle;
+import android.util.Log;
 
 import com.grarak.kerneladiutor.R;
 import com.grarak.kerneladiutor.elements.DDivider;
@@ -99,8 +100,13 @@ private void maxFreqInit() {
 
         if (GPU.hasGpuMaxFreq() && GPU.hasGpuFreqs()) {
             List<String> freqs = new ArrayList<>();
-            for (int freq : GPU.getGpuFreqs())
-                freqs.add(freq / 1000000 + getString(R.string.mhz));
+            for (int freq : GPU.getGpuFreqs()) {
+                if ((freq / 1000000) == 0) {
+                    freqs.add(freq / 1000 + getString(R.string.mhz));
+                } else {
+                    freqs.add(freq / 1000000 + getString(R.string.mhz));
+                }
+            }
 
             mMaxFreqCard = new PopupCardView.DPopupCard(freqs);
             mMaxFreqCard.setTitle(getString(R.string.gpu_max_freq));
@@ -115,8 +121,14 @@ private void maxFreqInit() {
     private void minFreqInit() {
         if (GPU.hasGpuMinFreq() && GPU.hasGpuFreqs()) {
             List<String> freqs = new ArrayList<>();
-            for (int freq : GPU.getGpuFreqs())
-                freqs.add(freq / 1000000 + getString(R.string.mhz));
+            for (int freq : GPU.getGpuFreqs()) {
+                if ((freq/1000000) == 0) {
+                    freqs.add(freq / 1000 + getString(R.string.mhz));
+                }
+                else {
+                    freqs.add(freq / 1000000 + getString(R.string.mhz));
+                }
+            }
 
             mMinFreqCard = new PopupCardView.DPopupCard(freqs);
             mMinFreqCard.setTitle(getString(R.string.gpu_min_freq));

diff --git a/app/src/main/java/com/grarak/kerneladiutor/utils/kernel/GPU.java b/app/src/main/java/com/grarak/kerneladiutor/utils/kernel/GPU.java
@@ -262,8 +262,9 @@ public static List<Integer> getGpuFreqs() {
                     if (value != null) {
                         String[] freqs = value.split(" ");
                         mGpuFreqs = new Integer[freqs.length];
-                        for (int i = 0; i < mGpuFreqs.length; i++)
+                        for (int i = 0; i < mGpuFreqs.length; i++) {
                             mGpuFreqs[i] = Utils.stringToInt(freqs[i]);
+                        }
                     }
                 }
         return new ArrayList<>(Arrays.asList(mGpuFreqs));