review changes

src/app/codegen-data-model-provider/CodegenDataModelProvider_Write.cpp

@@ -270,6 +270,7 @@ void CodegenDataModelProvider::Temporary_ReportAttributeChanged(const AttributeP
     }
     else
     {
+        // If cluster Id is invalid, mark the endpoint dirty using the path. This can happen when enabling/disabling the endpoint.
         change_listener.MarkDirty(path);
     }
 }

src/app/data-model-provider/Provider.h

@@ -111,7 +111,7 @@ class Provider : public ProviderMetadataTree
     /// should be marked as `dirty` by the data model provider listener so that the reporter can notify the subscriber
     /// of attribute changes.
     /// This function should be invoked when attribute managed by attribute access interface is modified but not
-    /// through am actual Write interaction.
+    /// through an actual Write interaction.
     /// For example, if the LastNetworkingStatus attribute changes because the NetworkCommissioning driver detects a
     /// network connection status change and calls SetLastNetworkingStatusValue(). The data model provider can recognize
     /// this change by invoking this function at the point of change.

src/app/reporting/reporting.cpp

@@ -24,6 +24,12 @@
 using namespace chip;
 using namespace chip::app;
 
+#if defined(__GNUC__) || defined(__clang__)
+// Disable the deprecated warning only for this file
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wdeprecated-declarations"
+#endif
+
 void MatterReportingAttributeChangeCallback(EndpointId endpoint, ClusterId clusterId, AttributeId attributeId)
 {
     // Attribute writes have asserted this already, but this assert should catch
@@ -52,3 +58,6 @@ void MatterReportingAttributeChangeCallback(EndpointId endpoint)
 
     InteractionModelEngine::GetInstance()->GetDataModelProvider()->Temporary_ReportAttributeChanged(AttributePathParams(endpoint));
 }
+#if defined(__GNUC__) || defined(__clang__)
+#pragma GCC diagnostic pop
+#endif