Address review comment

project-chip · Jul 23, 2022 · cb966e7 · cb966e7
1 parent 9e7b8a8
commit cb966e7
Showing 1 changed file with 4 additions and 8 deletions.
diff --git a/src/platform/Darwin/DiagnosticDataProviderImpl.cpp b/src/platform/Darwin/DiagnosticDataProviderImpl.cpp
@@ -72,16 +72,12 @@ CHIP_ERROR DiagnosticDataProviderImpl::GetTotalOperationalHours(uint32_t & total
 CHIP_ERROR DiagnosticDataProviderImpl::GetBootReason(BootReasonType & bootReason)
 {
     uint32_t reason = 0;
+    ReturErrorOnFailure(ConfigurationMgr().GetBootReason(reason));
 
-    CHIP_ERROR err = ConfigurationMgr().GetBootReason(reason);
+    VerifyOrReturnError(CanCastTo<BootReasonType>(reason), CHIP_ERROR_INVALID_INTEGER_VALUE);
+    bootReason = static_cast<BootReasonType>(reason);
 
-    if (err == CHIP_NO_ERROR)
-    {
-        VerifyOrReturnError(CanCastTo<BootReasonType>(reason), CHIP_ERROR_INVALID_INTEGER_VALUE);
-        bootReason = static_cast<BootReasonType>(reason);
-    }
-
-    return err;
+    return CHIP_NO_ERROR;
 }
 
 CHIP_ERROR DiagnosticDataProviderImpl::ResetWatermarks()