Merge branch 'master' of https://github.com/project-chip/connectedhomeip into bump_CI_images

Author: jlatusek

examples/fabric-bridge-app/fabric-bridge-common/src/ZCLCallbacks.cpp

@@ -34,10 +34,9 @@ Protocols::InteractionModel::Status emberAfExternalAttributeReadCallback(Endpoin
                                                                          const EmberAfAttributeMetadata * attributeMetadata,
                                                                          uint8_t * buffer, uint16_t maxReadLength)
 {
-    uint16_t endpointIndex  = emberAfGetDynamicIndexFromEndpoint(endpoint);
     AttributeId attributeId = attributeMetadata->attributeId;
 
-    Device * dev = DeviceMgr().GetDevice(endpointIndex);
+    Device * dev = DeviceMgr().GetDevice(endpoint);
     if (dev != nullptr && clusterId == app::Clusters::BridgedDeviceBasicInformation::Id)
     {
         using namespace app::Clusters::BridgedDeviceBasicInformation::Attributes;

examples/thermostat/linux/thermostat-manager.cpp

@@ -61,12 +61,6 @@ ThermostatManager ThermostatManager::sThermostatMgr;
 
 namespace {
 
-CHIP_ERROR ChipErrorFromStatusCode(Status status)
-{
-    StatusIB statusIB(status);
-    return statusIB.ToChipError();
-}
-
 template <typename DecodableAttributeType>
 static void OnAttributeChangeReported(const ConcreteDataAttributePath & path, const DecodableAttributeType & value);
 
@@ -332,7 +326,7 @@ CHIP_ERROR ThermostatManager::SetSystemMode(SystemModeEnum systemMode)
     }
 
     ChipLogError(AppServer, "Setting system mode: %u (%s)", systemModeValue, SystemModeString(systemMode));
-    return ChipErrorFromStatusCode(SystemMode::Set(kThermostatEndpoint, systemMode));
+    return CHIP_ERROR_IM_GLOBAL_STATUS_VALUE(SystemMode::Set(kThermostatEndpoint, systemMode));
 }
 
 CHIP_ERROR ThermostatManager::SetRunningMode(ThermostatRunningModeEnum runningMode)
@@ -345,22 +339,22 @@ CHIP_ERROR ThermostatManager::SetRunningMode(ThermostatRunningModeEnum runningMo
     }
 
     ChipLogError(AppServer, "Setting running mode: %u (%s)", runningModeValue, RunningModeString(runningMode));
-    return ChipErrorFromStatusCode(ThermostatRunningMode::Set(kThermostatEndpoint, runningMode));
+    return CHIP_ERROR_IM_GLOBAL_STATUS_VALUE(ThermostatRunningMode::Set(kThermostatEndpoint, runningMode));
 }
 
 CHIP_ERROR ThermostatManager::SetCurrentTemperature(int16_t temperature)
 {
-    return ChipErrorFromStatusCode(LocalTemperature::Set(kThermostatEndpoint, temperature));
+    return CHIP_ERROR_IM_GLOBAL_STATUS_VALUE(LocalTemperature::Set(kThermostatEndpoint, temperature));
 }
 
 CHIP_ERROR ThermostatManager::SetCurrentHeatingSetPoint(int16_t heatingSetpoint)
 {
-    return ChipErrorFromStatusCode(OccupiedHeatingSetpoint::Set(kThermostatEndpoint, heatingSetpoint));
+    return CHIP_ERROR_IM_GLOBAL_STATUS_VALUE(OccupiedHeatingSetpoint::Set(kThermostatEndpoint, heatingSetpoint));
 }
 
 CHIP_ERROR ThermostatManager::SetCurrentCoolingSetPoint(int16_t coolingSetpoint)
 {
-    return ChipErrorFromStatusCode(OccupiedCoolingSetpoint::Set(kThermostatEndpoint, coolingSetpoint));
+    return CHIP_ERROR_IM_GLOBAL_STATUS_VALUE(OccupiedCoolingSetpoint::Set(kThermostatEndpoint, coolingSetpoint));
 }
 
 void ThermostatManager::EvalThermostatState()

src/lib/support/BUILD.gn

@@ -301,11 +301,9 @@ static_library("support") {
     "${chip_root}/src/inet:inet_config_header",
   ]
 
-  # Android has no 'platform' that provides logging
-  if (current_os == "android") {
-    public_deps += [ "${chip_root}/src/platform/android:logging" ]
-  }
-
+  # Platforms that utilize CHIP_SYSTEM_CONFIG_PLATFORM_LOG need to
+  # be pulled in here as public_deps since they hook into logging at
+  # the macro level rather than just providing a LogV implementation.
   if (current_os == "mac" || current_os == "ios") {
     public_deps += [ "${chip_root}/src/platform/Darwin:logging" ]
   }

src/platform/logging/BUILD.gn

@@ -76,10 +76,11 @@ source_set("default") {
       deps += [ ":stdio" ]
     } else if (chip_device_platform == "nuttx") {
       deps += [ "${chip_root}/src/platform/NuttX:logging" ]
+    } else if (chip_device_platform == "android") {
+      deps += [ "${chip_root}/src/platform/android:logging" ]
     } else {
-      assert(
-          chip_device_platform == "fake" || chip_device_platform == "android" ||
-          chip_device_platform == "external" || chip_device_platform == "none")
+      assert(chip_device_platform == "fake" ||
+             chip_device_platform == "external" || chip_device_platform == "none")
     }
   }
 }
@@ -88,11 +89,6 @@ source_set("headers") {
   public = [ "LogV.h" ]
 }
 
-# We need to reference the output file of ":stdio" at build time,
-# but get_target_outputs() does not work for binary targets. As a
-# workaround, define a reasonable path and make the target use it.
-stdio_archive = "$root_out_dir/liblogging-stdio.a"
-
 source_set("stdio") {
   sources = [ "impl/stdio/Logging.cpp" ]
 
@@ -104,10 +100,4 @@ source_set("stdio") {
     "${chip_root}/src/platform:platform_config_header",
     "${chip_root}/src/platform/logging:headers",
   ]
-
-  # Ensure we end up with the expected output file name
-  output_dir = get_path_info(stdio_archive, "dir")
-  output_name = get_path_info(stdio_archive, "name")
-  output_extension = get_path_info(stdio_archive, "extension")
-  output_prefix_override = true
 }