Merge branch 'project-chip:master' into master

project-chip · Apr 15, 2022 · e9b685d · e9b685d
2 parents d197cb5 + 0680f21
commit e9b685d
Show file tree

Hide file tree

Showing 443 changed files with 22,789 additions and 11,503 deletions.
diff --git a/.github/.wordlist.txt b/.github/.wordlist.txt
@@ -8,7 +8,6 @@
 21
 22
 AAAA
-aabbccddeeff
 aarch
 abcdef
 abfb
@@ -510,7 +509,6 @@ ffeebaefa
 FFF
 fffe
 fffff
-fHtcwcAAAAAAAAAwQAAAAAAXPMlAAAAAAA
 Fi
 filepath
 fini
@@ -620,6 +618,8 @@ IlluminanceMeasurement
 IM
 imager
 imagetool
+ImageURI
+imageUri
 img
 Impl
 ImplClass
@@ -1392,17 +1392,13 @@ xAAAA
 xab
 xaver
 xb
-xBEEF
 xbef
 xc
 xcd
 Xcode
 xcodeproj
 xcworkspace
 xd
-xDEAD
-xDEADBEEF
-xdeadbeefcafe
 xds
 xdsdfu
 xEA
@@ -1420,15 +1416,8 @@ xFF
 xFFF
 xFFFF
 xfffff
-xFFFFFFFD
-xffffffffe
-xfffffffff
-xffffffffffff
-xffffffffffffXXXX
 xtensa
 xwayland
-XXXX
-XXXXXXXX
 xyz
 xz
 xzvf
@@ -1439,7 +1428,6 @@ YNJV
 Yocto
 yoctoproject
 YourFolder
-yWsC
 zapt
 zaptool
 ZCL

diff --git a/.github/workflows/examples-cc13x2x7_26x2x7.yaml b/.github/workflows/examples-cc13x2x7_26x2x7.yaml
@@ -99,6 +99,13 @@ jobs:
                     cc13x2_26x2 LP_CC2652R7 pump-controller-app \
                     out/artifacts/cc13x2x7_26x2x7-pump-controller/chip-LP_CC2652R7-pump-controller-example.out \
                     /tmp/bloat_reports/
+            - name: Get All Clusters App size stats
+              timeout-minutes: 5
+              run: |
+                  .environment/pigweed-venv/bin/python3 scripts/tools/memory/gh_sizes.py \
+                    cc13x2_26x2 LP_CC2652R7 all-clusters-app \
+                    out/artifacts/cc13x2x7_26x2x7-all-clusters/chip-LP_CC2652R7-all-clusters-example.out \
+                    /tmp/bloat_reports/
             - name: Uploading Size Reports
               uses: actions/upload-artifact@v2
               if: ${{ !env.ACT }}

diff --git a/.github/workflows/examples-linux-arm.yaml b/.github/workflows/examples-linux-arm.yaml
@@ -65,7 +65,7 @@ jobs:
                       .environment/gn_out/.ninja_log
                       .environment/pigweed-venv/*.log
             - name: Build Some samples
-              timeout-minutes: 20
+              timeout-minutes: 45
               run: |
                   ./scripts/run_in_build_env.sh \
                      "./scripts/build/build_examples.py \

diff --git a/.github/workflows/examples-nrfconnect.yaml b/.github/workflows/examples-nrfconnect.yaml
@@ -34,7 +34,7 @@ jobs:
         if: github.actor != 'restyled-io[bot]'
 
         container:
-            image: connectedhomeip/chip-build-nrf-platform:0.5.58
+            image: connectedhomeip/chip-build-nrf-platform:0.5.64
             volumes:
                 - "/tmp/bloat_reports:/tmp/bloat_reports"
 

diff --git a/.github/workflows/examples-qpg.yaml b/.github/workflows/examples-qpg.yaml
@@ -67,12 +67,12 @@ jobs:
                       .environment/gn_out/.ninja_log
                       .environment/pigweed-venv/*.log
             - name: Build QPG6105 example apps
-              timeout-minutes: 5
+              timeout-minutes: 20
               run: |
                   ./scripts/run_in_build_env.sh \
                      "./scripts/build/build_examples.py \
                         --enable-flashbundle \
-                        --target-glob 'qpg6105-{lock,light,shell,persistent-storage}' \
+                        --target-glob 'qpg-{lock,light,shell,persistent-storage}' \
                         build \
                         --copy-artifacts-to out/artifacts \
                      "

diff --git a/.github/workflows/tests.yaml b/.github/workflows/tests.yaml
@@ -270,6 +270,7 @@ jobs:
                     ./scripts/run_in_build_env.sh \
                      "./scripts/build/build_examples.py \
                         --target linux-x64-all-clusters-${BUILD_VARIANT} \
+                        --target linux-x64-python-bindings \
                         build \
                         --copy-artifacts-to objdir-clone \
                      "

diff --git a/.spellcheck.yml b/.spellcheck.yml
@@ -12,12 +12,53 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+# To run locally:
+#   Prerequisits (only once):
+#
+#      apt-get install aspell aspell-en
+#      pip install pyspelling
+#
+#   Actual run:
+#
+#      pyspelling pyspelling --config .spellcheck.yml
+
 matrix:
   - name: markdown
     dictionary:
       wordlists:
         - .github/.wordlist.txt
     pipeline:
+      # See https://facelessuser.github.io/pyspelling/configuration/#pipeline
+      # and https://facelessuser.github.io/pyspelling/filters/context/
+
+      # context-aware logic: we have a LOT of code examples in markdown,
+      # avoid trying to spellcheck code.
+      - pyspelling.filters.context:
+          context_visible_first: true
+          escapes: \\[\\`~]
+          delimiters:
+          # Ignore multiline content between fences (fences can have 3 or more back ticks)
+          # ```
+          # content
+          # ```
+          #
+          # Allows language marker (since github allows it) like:
+          # ```python
+          # content
+          # ```
+          - open: '(?s)^(?P<open> *`{3,})[a-z]*$'
+            close: '^(?P=open)$'
+          # Ignore text between inline back ticks
+          - open: '(?P<open>`+)'
+            close: '(?P=open)'
+          # Ignore URL in hyperlinks [title](url)
+          - open: '\[[^]]*\]\('
+            close: '\)'
+          # Ignore code markers
+          - open: '<code>'
+            close: '</code>'
+
+      # converts markdown to HTML
       - pyspelling.filters.markdown:
     sources:
       - '**/*.md|!third_party/**|!examples/common/**/repo/**'

diff --git a/BUILD.gn b/BUILD.gn
@@ -253,6 +253,9 @@ if (current_toolchain != "${dir_pw_toolchain}/default:default") {
     # Build the cc13x2x7_26x2x7 lock app example.
     enable_cc13x2x7_26x2x7_lock_app_build = enable_ti_simplelink_builds
 
+    # Build the cc13x2x7_26x2x7 all-clusters app example.
+    enable_cc13x2x7_26x2x7_all_clusters_app_build = enable_ti_simplelink_builds
+
     # Build the efr32 lock app example.
     enable_efr32_lock_app_build = enable_efr32_builds
 
@@ -353,6 +356,12 @@ if (current_toolchain != "${dir_pw_toolchain}/default:default") {
     }
   }
 
+  if (enable_cc13x2x7_26x2x7_all_clusters_app_build) {
+    group("cc13x2x7_26x2x7_all_clusters_app") {
+      deps = [ "${chip_root}/examples/all-clusters-app/cc13x2x7_26x2x7(${chip_root}/config/cc13x2_26x2/toolchain:cc13x2x7_26x2x7_all_clusters_app)" ]
+    }
+  }
+
   if (enable_standalone_shell_build) {
     group("standalone_shell") {
       deps =
@@ -539,6 +548,9 @@ if (current_toolchain != "${dir_pw_toolchain}/default:default") {
     if (enable_cc13x2x7_26x2x7_lock_app_build) {
       deps += [ ":cc13x2x7_26x2x7_lock_app" ]
     }
+    if (enable_cc13x2x7_26x2x7_all_clusters_app_build) {
+      deps += [ ":cc13x2x7_26x2x7_all_clusters_app" ]
+    }
   }
 
   group("check") {

diff --git a/config/cc13x2_26x2/toolchain/BUILD.gn b/config/cc13x2_26x2/toolchain/BUILD.gn
@@ -22,4 +22,10 @@ arm_toolchain("cc13x2x7_26x2x7_lock_app") {
     current_os = "freertos"
     import("${chip_root}/examples/lock-app/cc13x2x7_26x2x7/args.gni")
   }
+  arm_toolchain("cc13x2x7_26x2x7_all_clusters_app") {
+    toolchain_args = {
+      current_os = "freertos"
+      import("${chip_root}/examples/all-clusters-app/cc13x2x7_26x2x7/args.gni")
+    }
+  }
 }
diff --git a/config/esp32/components/chip/CMakeLists.txt b/config/esp32/components/chip/CMakeLists.txt
@@ -103,6 +103,10 @@ if (CONFIG_ENABLE_CHIP_SHELL)
     chip_gn_arg_append("chip_build_libshell"                "true")
 endif()
 
+if (CONFIG_IDF_TARGET_ESP32H2)
+    chip_gn_arg_append("chip_enable_wifi"                       "false")
+endif()
+
 if (CONFIG_OPENTHREAD_ENABLED)
     chip_gn_arg_append("chip_enable_openthread"                 "true")
 endif()

diff --git a/config/nrfconnect/.nrfconnect-recommended-revision b/config/nrfconnect/.nrfconnect-recommended-revision
@@ -1 +1 @@
-v1.9.1
+5ea8f7fa91d7315fcc6cd9eb3aa74f9640d0abac
diff --git a/config/qpg/chip-gn/.gn b/config/qpg/chip-gn/.gn
@@ -19,7 +19,9 @@ import("//build_overrides/chip.gni")
 buildconfig = "//build/config/BUILDCONFIG.gn"
 
 # CHIP uses angle bracket includes.
-check_system_includes = true
+# DISABLED since CI failed on PR 17352
+# Build error out for unused/unavailble include file (IPAddress.h : <openthread/ip6.h>)
+check_system_includes = false
 
 default_args = {
   target_cpu = "arm"

diff --git a/docs/api/device_runner.md b/docs/api/device_runner.md
@@ -99,5 +99,5 @@ enable reusing tests between different devices.
 The TSG is targeting the following platforms/boards for early bringup:
 
 -   Nordic nRF52 board <TODO: REF>
--   SiLabs XXXX board <TODO:REF>
--   Espressif ESP32 XXXX board <TODO:REF>
+-   SiLabs `XXXX` board <TODO:REF>
+-   Espressif ESP32 `XXXX` board <TODO:REF>
diff --git a/docs/discussion/lwip_ipv6.md b/docs/discussion/lwip_ipv6.md
@@ -27,8 +27,8 @@ addresses for communication to nodes on the same network (wifi or thread). When
 there is more than one netif in the system (ex. loopback, softAP, STA), the link
 local address needs more information to determine which link the address is
 local to. This is normally added as the link local scope and can be seen on
-addresses ex. FE80::xxxx:xxxx:xxxx:xxxx%<scope>, where the <scope> identifies
-the netif (something like %wlan0 or %eno1 etc.).
+addresses ex. `FE80::xxxx:xxxx:xxxx:xxxx%<scope>`, where the <scope> identifies
+the netif (something like `%wlan0` or `%eno1` etc.).
 
 Without this indicator, the link local address can only be resolved if there is
 one netif. LwIP will also allow a direct address match to the netif source
@@ -122,7 +122,7 @@ Instead, it might be better to build this into the ICMP layer itself.
 LwIP's DNS handling isn’t great and breaks down when the router supports
 IPv4/IPv6. There is a single list of DNS servers, DHCP, SLAAC and DHCPv6 all
 update the list without locks. Basically, whatever wrote to the list last gets
-to set the list. Although there is handling for IP type (requesting A or AAAA
+to set the list. Although there is handling for IP type (requesting A or `AAAA`
 records), there isn’t handling to specify an IPv6 or IPv4 server specifically,
 which can be challenging since not all servers serve all record types.
 

diff --git a/docs/examples/discussion/PID_allocation_for_example_apps.md b/docs/examples/discussion/PID_allocation_for_example_apps.md
@@ -3,34 +3,34 @@
 Unless specifically overridden by the platform, example apps in this SDK use the
 Example credentials implementation in `DeviceAttestationCredsExample.cpp`.
 
-The SDK holds example certificates for VID '0xFFF1' and any PID in
-'0x8000-0x801F'. The device VID and PID supplied by the basic information
+The SDK holds example certificates for VID `0xFFF1` and any PID in
+`0x8000-0x801F`. The device VID and PID supplied by the basic information
 cluster must correspond to the VID/PID given in the certificate for the device
 to pass verification.
 
 Certificates are selected using the value in
 CHIP_DEVICE_CONFIG_DEVICE_PRODUCT_ID. The vendor ID for every example app is the
-same because they are all signed by the same PAI (vendor id '0xFFF1').
+same because they are all signed by the same PAI (vendor id `0xFFF1`).
 
 In order to allow some differentiation between the various example apps, each
 app is assigned a PID from the list below:
 
-| App                     | PID    |
-| ----------------------- | ------ |
-| All Clusters            | 0x8001 |
-| Bridge                  | 0x8002 |
-| Door Lock               | 0x8003 |
-| Light switch            | 0x8004 |
-| Lighting                | 0x8005 |
-| Lock                    | 0x8006 |
-| OTA provider            | 0x8007 |
-| OTA requestor           | 0x8008 |
-| Persistent Storage      | 0x8009 |
-| Pigweed                 | 0x800B |
-| Pump                    | 0x800A |
-| Pump Controller         | 0x8011 |
-| Shell                   | 0x8012 |
-| Temperature measurement | 0x800D |
-| Thermostat              | 0x800E |
-| TV                      | 0x800F |
-| Window                  | 0x8010 |
+| App                     | PID      |
+| ----------------------- | -------- |
+| All Clusters            | `0x8001` |
+| Bridge                  | `0x8002` |
+| Door Lock               | `0x8003` |
+| Light switch            | `0x8004` |
+| Lighting                | `0x8005` |
+| Lock                    | `0x8006` |
+| OTA provider            | `0x8007` |
+| OTA requestor           | `0x8008` |
+| Persistent Storage      | `0x8009` |
+| Pigweed                 | `0x800B` |
+| Pump                    | `0x800A` |
+| Pump Controller         | `0x8011` |
+| Shell                   | `0x8012` |
+| Temperature measurement | `0x800D` |
+| Thermostat              | `0x800E` |
+| TV                      | `0x800F` |
+| Window                  | `0x8010` |
diff --git a/docs/guides/BUILDING.md b/docs/guides/BUILDING.md
@@ -182,7 +182,7 @@ To run all tests, run:
 ninja -C out/host check
 ```
 
-To run only the tests in src/inet/tests, you can run:
+To run only the tests in `src/inet/tests`, you can run:
 
 ```
 ninja -C out/host src/inet/tests:tests_run

diff --git a/docs/guides/matter-repl.md b/docs/guides/matter-repl.md
@@ -149,9 +149,7 @@ cloud-hosted playground.
 The following icon is present at the top of applicable guides that can be
 launched into the playground:
 
-<a href="https://www.w3schools.com">
-<img src="https://i.ibb.co/hR3yWsC/launch-playground.png" alt="drawing" width="130"/>
-</a>
+![Launch playground icon](https://i.ibb.co/hR3yWsC/launch-playground.png")
 <br></br>
 
 ## Guides

diff --git a/docs/guides/mbedos_add_new_target.md b/docs/guides/mbedos_add_new_target.md
@@ -1,6 +1,4 @@
-<p align="center">
-  <img src="https://raw.githubusercontent.com/ARMmbed/mbed-os/master/logo.png" alt="ARM Mbed-OS logo"/>
-</p>
+![ARM Mbed-OS logo](https://raw.githubusercontent.com/ARMmbed/mbed-os/master/logo.png)
 
 <h1> Mbed-OS add new hardware target </h1>
 

diff --git a/docs/guides/mbedos_commissioning.md b/docs/guides/mbedos_commissioning.md
@@ -1,6 +1,4 @@
-<p align="center">
-    <img src="https://raw.githubusercontent.com/ARMmbed/mbed-os/master/logo.png" alt="ARM Mbed-OS logo"/>
-</p>
+![ARM Mbed-OS logo](https://raw.githubusercontent.com/ARMmbed/mbed-os/master/logo.png)
 
 <h1> Matter Arm Mbed OS provisioning guide </h1>
 
@@ -67,7 +65,9 @@ After building, install the application by completing the following steps:
 1.  Install the Android Debug Bridge (adb) package by running the following
     command:
 
+        ```
         $ sudo apt install android-tools-adb
+        ```
 
 2.  Enable **USB debugging** on your smartphone. See the
     [Configure on-device developer options](https://developer.android.com/studio/debug/dev-options)
@@ -94,7 +94,9 @@ complete the following steps:
     accessory device. You can use **mbed-tools** for this purpose
     ([mbed-tools](https://github.com/ARMmbed/mbed-tools)):
 
+        ```
         mbed-tools sterm -p /dev/ttyACM0 -b 115200 -e off
+        ```
 
 To start the rendezvous, CHIPTool must get the commissioning information from
 the Matter device. The data payload is encoded within a QR code and is printed
@@ -104,8 +106,10 @@ to the UART console.
 
 -   Find a message similar to the following one in the application logs:
 
+        ```
         [INFO][CHIP]: [SVR]Copy/paste the below URL in a browser to see the QR Code:
         [INFO][CHIP]: [SVR]https://dhrishi.github.io/connectedhomeip/qrcode.html?data=MT%3AYNJV7VSC00CMVH7SR00
+        ```
 
 -   Open URL from the console to display the QR in a web browser.