Apply suggestions from code review

Co-authored-by: Andrew Scheller <[email protected]>

Author: will-v-pi

src/rp2_common/pico_cyw43_driver/cyw43_firmware.py

@@ -19,7 +19,7 @@
         if "#define CYW43_WIFI_FW_LEN" in line:
             matches = re.search(r"#define\s+\S+\s+\((\S+)\)", line)
             cyw43_wifi_fw_len = int(matches[1])
-        if "#define CYW43_CLM_LEN" in line:
+        elif "#define CYW43_CLM_LEN" in line:
             matches = re.search(r"#define\s+\S+\s+\((\S+)\)", line)
             cyw43_clm_len = int(matches[1])
         if cyw43_wifi_fw_len > 0 and cyw43_clm_len > 0:

src/rp2_common/pico_cyw43_driver/include/pico/cyw43_driver.h

@@ -23,7 +23,8 @@
 #if CYW43_USE_FIRMWARE_PARTITION
 // PICO_CONFIG: CYW43_FIRMWARE_PARTITION_ID, ID of Wi-Fi firmware partition which must match the ID used in the partition table JSON, type=int, default=0x776966696669726d, group=pico_cyw43_driver
 #ifndef CYW43_FIRMWARE_PARTITION_ID
-#define CYW43_FIRMWARE_PARTITION_ID 0x776966696669726d // wififirm
+// The default 0x776966696669726d value is the ASCII encoding of "wififirm"
+#define CYW43_FIRMWARE_PARTITION_ID 0x776966696669726d
 #endif
 #endif