From c091c176cbc93044c4d70d7f733fef4c59672f96 Mon Sep 17 00:00:00 2001 From: David Anderson <dvander@google.com> Date: Tue, 4 Sep 2018 18:11:03 -0700 Subject: [PATCH] fastbootd: Implement getvar hw-revision. Bug: 78793464 Test: fastboot getvar hw-revision works fuzzy_fastboot Conformance.GetVarRevision passes Change-Id: I5a3e9893d61e18567f2f818ef06cad8e862d637f --- fastboot/constants.h | 1 + fastboot/device/commands.cpp | 3 ++- fastboot/device/variables.cpp | 6 ++++++ fastboot/device/variables.h | 2 ++ 4 files changed, 11 insertions(+), 1 deletion(-) diff --git a/fastboot/constants.h b/fastboot/constants.h index 063cd40b41..8a425ae0d1 100644 --- a/fastboot/constants.h +++ b/fastboot/constants.h @@ -57,3 +57,4 @@ #define FB_VAR_SLOT_UNBOOTABLE "slot-unbootable" #define FB_VAR_IS_LOGICAL "is-logical" #define FB_VAR_IS_USERSPACE "is-userspace" +#define FB_VAR_HW_REVISION "hw-revision" diff --git a/fastboot/device/commands.cpp b/fastboot/device/commands.cpp index b1c2958fef..771c28892c 100644 --- a/fastboot/device/commands.cpp +++ b/fastboot/device/commands.cpp @@ -89,7 +89,8 @@ bool GetVarHandler(FastbootDevice* device, const std::vector<std::string>& args) {FB_VAR_SLOT_UNBOOTABLE, {GetSlotUnbootable, nullptr}}, {FB_VAR_PARTITION_SIZE, {GetPartitionSize, GetAllPartitionArgsWithSlot}}, {FB_VAR_IS_LOGICAL, {GetPartitionIsLogical, GetAllPartitionArgsWithSlot}}, - {FB_VAR_IS_USERSPACE, {GetIsUserspace, nullptr}}}; + {FB_VAR_IS_USERSPACE, {GetIsUserspace, nullptr}}, + {FB_VAR_HW_REVISION, {GetHardwareRevision, nullptr}}}; if (args.size() < 2) { return device->WriteFail("Missing argument"); diff --git a/fastboot/device/variables.cpp b/fastboot/device/variables.cpp index 68efa08c35..a9601896df 100644 --- a/fastboot/device/variables.cpp +++ b/fastboot/device/variables.cpp @@ -256,3 +256,9 @@ std::vector<std::vector<std::string>> GetAllPartitionArgsNoSlot(FastbootDevice* } return args; } + +bool GetHardwareRevision(FastbootDevice* /* device */, const std::vector<std::string>& /* args */, + std::string* message) { + *message = android::base::GetProperty("ro.revision", ""); + return true; +} diff --git a/fastboot/device/variables.h b/fastboot/device/variables.h index c3a64cfab8..a44e72928c 100644 --- a/fastboot/device/variables.h +++ b/fastboot/device/variables.h @@ -48,6 +48,8 @@ bool GetPartitionIsLogical(FastbootDevice* device, const std::vector<std::string std::string* message); bool GetIsUserspace(FastbootDevice* device, const std::vector<std::string>& args, std::string* message); +bool GetHardwareRevision(FastbootDevice* device, const std::vector<std::string>& args, + std::string* message); // Helpers for getvar all. std::vector<std::vector<std::string>> GetAllPartitionArgsWithSlot(FastbootDevice* device); -- GitLab