From 95e0d9bdb91f89052f82f0869701ca50aab74dba Mon Sep 17 00:00:00 2001
From: Adam Shih <adamshih@google.com>
Date: Thu, 10 Nov 2022 15:00:00 +0800
Subject: [PATCH] move sensors dump to gs-common

Bug: 250475720
Test: adb bugreport
Change-Id: I500bfba706b74a872d4ba07b9b58efaed015772e
---
 device.mk               |  2 +-
 dumpstate/Dumpstate.cpp | 23 -----------------------
 dumpstate/Dumpstate.h   |  1 -
 3 files changed, 1 insertion(+), 25 deletions(-)

diff --git a/device.mk b/device.mk
index 2d920b5e..ab3f35c9 100644
--- a/device.mk
+++ b/device.mk
@@ -936,7 +936,7 @@ endif
 #$(call inherit-product-if-exists, hardware/google_devices/gs101/gs101.mk)
 #$(call inherit-product-if-exists, vendor/google_devices/common/exynos-vendor.mk)
 #$(call inherit-product-if-exists, hardware/broadcom/wlan/bcmdhd/firmware/bcm4375/device-bcm.mk)
-$(call inherit-product-if-exists, vendor/google/sensors/usf/android/usf_efw_product.mk)
+include device/google/gs-common/sensors/sensors.mk
 $(call inherit-product-if-exists, vendor/google/services/LyricCameraHAL/src/build/device-vendor.mk)
 $(call inherit-product-if-exists, vendor/google/camera/devices/whi/device-vendor.mk)
 
diff --git a/dumpstate/Dumpstate.cpp b/dumpstate/Dumpstate.cpp
index fdc32922..a48f1e6c 100644
--- a/dumpstate/Dumpstate.cpp
+++ b/dumpstate/Dumpstate.cpp
@@ -259,7 +259,6 @@ Dumpstate::Dumpstate()
         { "thermal", [this](int fd) { dumpThermalSection(fd); } },
         { "touch", [this](int fd) { dumpTouchSection(fd); } },
         { "display", [this](int fd) { dumpDisplaySection(fd); } },
-        { "sensors-usf", [this](int fd) { dumpSensorsUSFSection(fd); } },
         { "misc", [this](int fd) { dumpMiscSection(fd); } },
         { "camera", [this](int fd) { dumpCameraSection(fd); } },
     } {
@@ -913,28 +912,6 @@ void Dumpstate::dumpDisplaySection(int fd) {
     }
 }
 
-// Dump items related to sensors usf.
-void Dumpstate::dumpSensorsUSFSection(int fd) {
-    CommandOptions options = CommandOptions::WithTimeout(2).Build();
-    RunCommandToFd(fd, "USF statistics",
-                   {"/vendor/bin/sh", "-c", "usf_stats get --all"},
-                   options);
-    if (!PropertiesHelper::IsUserBuild()) {
-        // Not a user build, if this is also not a production device dump the USF registry.
-        std::string hwRev = ::android::base::GetProperty(HW_REVISION, "");
-        if (hwRev.find("PROTO") != std::string::npos ||
-            hwRev.find("EVT") != std::string::npos ||
-            hwRev.find("DVT") != std::string::npos) {
-            RunCommandToFd(fd, "USF Registry",
-                           {"/vendor/bin/sh", "-c", "usf_reg_edit save -"},
-                           options);
-            RunCommandToFd(fd, "USF Last Stat Buffer",
-                           {"/vendor/bin/sh", "-c", "cat /data/vendor/sensors/debug/stats.history"},
-                           options);
-        }
-    }
-}
-
 // Dump items that don't fit well into any other section
 void Dumpstate::dumpMiscSection(int fd) {
     RunCommandToFd(fd, "VENDOR PROPERTIES", {"/vendor/bin/getprop"});
diff --git a/dumpstate/Dumpstate.h b/dumpstate/Dumpstate.h
index 12497d5c..4c2d9cb6 100644
--- a/dumpstate/Dumpstate.h
+++ b/dumpstate/Dumpstate.h
@@ -55,7 +55,6 @@ class Dumpstate : public BnDumpstateDevice {
     void dumpDevfreqSection(int fd);
     void dumpMemorySection(int fd);
     void dumpDisplaySection(int fd);
-    void dumpSensorsUSFSection(int fd);
     void dumpMiscSection(int fd);
     void dumpCameraSection(int fd);
 };