Add comment regarding why platforms depend on address resolve config (p…

…roject-chip#30597)

* Add comment regarding why platforms depend on address resolve config

* Restyle

---------

Co-authored-by: Andrei Litvin <andreilitvin@google.com>

src/platform/Infineon/CYW30739/BUILD.gn

@@ -82,6 +82,11 @@ static_library("CYW30739") {
       "ThreadStackManagerImpl.cpp",
       "ThreadStackManagerImpl.h",
     ]
+
+    # TODO: platform should NOT depend on default_address_resolve_config. This should
+    #       be removed. See https://github.com/project-chip/connectedhomeip/issues/30596
+    #
+    #       Currently this exists because OpenThread platform includes src/app/Server.h
     public_configs = [
       "${chip_root}/src/lib/address_resolve:default_address_resolve_config",
     ]

src/platform/bouffalolab/BL702/BUILD.gn

@@ -101,6 +101,14 @@ static_library("BL702") {
       ]
       deps += [ "${chip_root}/src/lib/dnssd:platform_header" ]
     }
+
+    # TODO: platform should NOT depend on default_address_resolve_config. This should
+    #       be removed. See https://github.com/project-chip/connectedhomeip/issues/30596
+    #
+    #       Currently this exists because OpenThread platform includes src/app/Server.h
+    public_configs = [
+      "${chip_root}/src/lib/address_resolve:default_address_resolve_config",
+    ]
   }
 
   if (chip_enable_ethernet) {
@@ -115,7 +123,4 @@ static_library("BL702") {
 
   deps += [ "${chip_root}/src/credentials:credentials_header" ]
   public_deps = [ "${chip_root}/src/platform:platform_base" ]
-
-  public_configs =
-      [ "${chip_root}/src/lib/address_resolve:default_address_resolve_config" ]
 }

src/platform/bouffalolab/BL702L/BUILD.gn

@@ -88,11 +88,16 @@ static_library("BL702L") {
       ]
       deps += [ "${chip_root}/src/lib/dnssd:platform_header" ]
     }
+
+    # TODO: platform should NOT depend on default_address_resolve_config. This should
+    #       be removed. See https://github.com/project-chip/connectedhomeip/issues/30596
+    #
+    #       Currently this exists because OpenThread platform includes src/app/Server.h
+    public_configs = [
+      "${chip_root}/src/lib/address_resolve:default_address_resolve_config",
+    ]
   }
 
   deps += [ "${chip_root}/src/credentials:credentials_header" ]
   public_deps = [ "${chip_root}/src/platform:platform_base" ]
-
-  public_configs =
-      [ "${chip_root}/src/lib/address_resolve:default_address_resolve_config" ]
 }

src/platform/cc13xx_26xx/cc13x2_26x2/BUILD.gn

@@ -52,6 +52,11 @@ static_library("cc13x2_26x2") {
     "${chip_root}/src/crypto",
     "${chip_root}/src/platform:platform_base",
   ]
+
+  # TODO: platform should NOT depend on default_address_resolve_config. This should
+  #       be removed. See https://github.com/project-chip/connectedhomeip/issues/30596
+  #
+  #       Currently this exists because OpenThread platform includes src/app/Server.h
   public_configs =
       [ "${chip_root}/src/lib/address_resolve:default_address_resolve_config" ]
 

src/platform/cc13xx_26xx/cc13x4_26x4/BUILD.gn

@@ -52,6 +52,11 @@ static_library("cc13x4_26x4") {
     "${chip_root}/src/crypto",
     "${chip_root}/src/platform:platform_base",
   ]
+
+  # TODO: platform should NOT depend on default_address_resolve_config. This should
+  #       be removed. See https://github.com/project-chip/connectedhomeip/issues/30596
+  #
+  #       Currently this exists because OpenThread platform includes src/app/Server.h
   public_configs =
       [ "${chip_root}/src/lib/address_resolve:default_address_resolve_config" ]
 

src/platform/nxp/k32w/k32w0/BUILD.gn

@@ -140,10 +140,15 @@ static_library("k32w0") {
       ]
       deps += [ "${chip_root}/src/lib/dnssd:platform_header" ]
     }
+
+    # TODO: platform should NOT depend on default_address_resolve_config. This should
+    #       be removed. See https://github.com/project-chip/connectedhomeip/issues/30596
+    #
+    #       Currently this exists because OpenThread platform includes src/app/Server.h
+    public_configs = [
+      "${chip_root}/src/lib/address_resolve:default_address_resolve_config",
+    ]
   }
 
   public_deps += [ "${chip_root}/src/crypto" ]
-
-  public_configs =
-      [ "${chip_root}/src/lib/address_resolve:default_address_resolve_config" ]
 }

src/platform/nxp/k32w/k32w1/BUILD.gn

@@ -107,6 +107,10 @@ static_library("k32w1") {
 
   public_deps += [ "${chip_root}/src/crypto" ]
 
+  # TODO: platform should NOT depend on default_address_resolve_config. This should
+  #       be removed. See https://github.com/project-chip/connectedhomeip/issues/30596
+  #
+  #       Currently this exists because OpenThread platform includes src/app/Server.h
   public_configs =
       [ "${chip_root}/src/lib/address_resolve:default_address_resolve_config" ]
 }

src/platform/nxp/rt/rw61x/BUILD.gn

@@ -147,7 +147,10 @@ static_library("nxp_platform") {
   public_configs = [
     ":nxp_platform_config",
 
-    # address_resolve config needed by the include of server.h in ConfigurationManagerImpl.cpp
+    # TODO: platform should NOT depend on default_address_resolve_config. This should
+    #       be removed. See https://github.com/project-chip/connectedhomeip/issues/30596
+    #
+    #       Currently this exists because OpenThread platform includes src/app/Server.h
     "${chip_root}/src/lib/address_resolve:default_address_resolve_config",
   ]
 }

src/platform/qpg/BUILD.gn

@@ -70,6 +70,10 @@ static_library("qpg") {
       public_deps += [ "${openthread_root}:libopenthread-mtd" ]
     }
 
+    # TODO: platform should NOT depend on default_address_resolve_config. This should
+    #       be removed. See https://github.com/project-chip/connectedhomeip/issues/30596
+    #
+    #       Currently this exists because OpenThread platform includes src/app/Server.h
     public_configs += [
       "${chip_root}/third_party/openthread/platforms/qpg:openthread_qpg_config",
       "${chip_root}/src/lib/address_resolve:default_address_resolve_config",

src/platform/silabs/efr32/BUILD.gn

@@ -124,6 +124,10 @@ static_library("efr32") {
       deps += [ "${chip_root}/src/lib/dnssd:platform_header" ]
     }
 
+    # TODO: platform should NOT depend on default_address_resolve_config. This should
+    #       be removed. See https://github.com/project-chip/connectedhomeip/issues/30596
+    #
+    #       Currently this exists because OpenThread platform includes src/app/Server.h
     public_configs = [
       "${chip_root}/src/lib/address_resolve:default_address_resolve_config",
     ]

src/platform/stm32/BUILD.gn

@@ -90,6 +90,10 @@ static_library("stm32") {
 
     deps += [ "${chip_root}/src/lib/dnssd:platform_header" ]
 
+    # TODO: platform should NOT depend on default_address_resolve_config. This should
+    #       be removed. See https://github.com/project-chip/connectedhomeip/issues/30596
+    #
+    #       Currently this exists because OpenThread platform includes src/app/Server.h
     public_configs = [
       "${chip_root}/src/lib/address_resolve:default_address_resolve_config",
     ]