Skip to content

Commit 401d56f

Browse files
samples: matter: Addressed review comments to the configuration
Introduced several fixes for the configuration recommended during code review. Signed-off-by: Kamil Kasperczyk <[email protected]>
1 parent f744d93 commit 401d56f

11 files changed

+15
-39
lines changed

samples/matter/contact_sensor/Kconfig.sysbuild

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,13 +60,13 @@ config DFU_MULTI_IMAGE_PACKAGE_NET
6060

6161
endif # SOC_SERIES_NRF53X
6262

63-
if BOARD_NRF54L15DK
63+
if BOARD_NRF54L15DK || BOARD_NRF54LM20DK
6464

65-
# Disable checking the external drivers for nRF54L15 DKs.
65+
# Disable checking the external drivers for nRF54L15 and nRF54LM20 DKs.
6666
config PM_OVERRIDE_EXTERNAL_DRIVER_CHECK
6767
default y
6868

69-
endif # BOARD_NRF54L15DK
69+
endif # BOARD_NRF54L15DK || BOARD_NRF54LM20DK
7070

7171
endif # BOOTLOADER_MCUBOOT
7272

samples/matter/contact_sensor/boards/nrf52840dk_nrf52840.overlay

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,18 +14,23 @@
1414
&adc {
1515
status = "disabled";
1616
};
17+
1718
&uart1 {
1819
status = "disabled";
1920
};
21+
2022
&i2c0 {
2123
status = "disabled";
2224
};
25+
2326
&spi1 {
2427
status = "disabled";
2528
};
29+
2630
&spi3 {
2731
status = "disabled";
2832
};
33+
2934
&usbd {
3035
status = "disabled";
3136
};

samples/matter/contact_sensor/boards/nrf5340dk_nrf5340_cpuapp.overlay

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,15 @@
2121
&adc {
2222
status = "disabled";
2323
};
24+
2425
&i2c1 {
2526
status = "disabled";
2627
};
28+
2729
&spi2 {
2830
status = "disabled";
2931
};
32+
3033
&usbd {
3134
status = "disabled";
3235
};

samples/matter/contact_sensor/sysbuild/mcuboot/boards/nrf5340dk_nrf5340_cpuapp.conf

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -7,19 +7,3 @@
77
CONFIG_NORDIC_QSPI_NOR=y
88
CONFIG_NORDIC_QSPI_NOR_FLASH_LAYOUT_PAGE_SIZE=4096
99
CONFIG_NORDIC_QSPI_NOR_STACK_WRITE_BUFFER_SIZE=16
10-
11-
# The following configurations are required to support simultaneous multi image update
12-
CONFIG_PCD_APP=y
13-
CONFIG_UPDATEABLE_IMAGE_NUMBER=2
14-
15-
CONFIG_BOOT_SWAP_USING_MOVE=n
16-
# Multi-image updates do not support image swapping yet.
17-
CONFIG_BOOT_UPGRADE_ONLY=y
18-
19-
# The network core cannot access external flash directly. The flash simulator must be used to
20-
# provide a memory region that is used to forward the new firmware to the network core.
21-
CONFIG_FLASH_SIMULATOR=y
22-
CONFIG_FLASH_SIMULATOR_DOUBLE_WRITES=y
23-
CONFIG_FLASH_SIMULATOR_STATS=n
24-
25-
CONFIG_ZCBOR=y

samples/matter/contact_sensor/sysbuild/mcuboot/boards/nrf5340dk_nrf5340_cpuapp.overlay

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
};
1212
};
1313

14-
// Disable GPIO forwarder for nRF7002 EK compatibility
14+
/* Disable GPIO forwarder for nRF7002 EK compatibility */
1515
&gpio_fwd {
1616
status = "disabled";
1717
};

samples/matter/contact_sensor/sysbuild/mcuboot/boards/nrf54l15dk_nrf54l15_cpuapp.conf

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,6 @@ CONFIG_SPI_NOR=y
1616
CONFIG_SPI_NOR_SFDP_DEVICETREE=y
1717
CONFIG_SPI_NOR_FLASH_LAYOUT_PAGE_SIZE=4096
1818

19-
CONFIG_PM_OVERRIDE_EXTERNAL_DRIVER_CHECK=y
20-
2119
# Adjust the maximum sectors to the app image size of ~1.4MB
2220
CONFIG_BOOT_MAX_IMG_SECTORS=512
2321

samples/matter/contact_sensor/sysbuild/mcuboot/boards/nrf54l15dk_nrf54l15_cpuapp.overlay

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
};
1111
};
1212

13-
// restore full RRAM and SRAM space - by default some parts are dedicated to FLRP
13+
/* restore full RRAM and SRAM space - by default some parts are dedicated to FLRP */
1414
&cpuapp_rram {
1515
reg = <0x0 DT_SIZE_K(1524)>;
1616
};

samples/matter/contact_sensor/sysbuild/mcuboot/boards/nrf54lm20dk_nrf54lm20a_cpuapp.conf

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ CONFIG_SPI=y
1313
CONFIG_SPI_NOR=y
1414
CONFIG_SPI_NOR_SFDP_DEVICETREE=y
1515
CONFIG_SPI_NOR_FLASH_LAYOUT_PAGE_SIZE=4096
16-
CONFIG_PM_OVERRIDE_EXTERNAL_DRIVER_CHECK=y
1716

1817
# Increase the maximum number of sectors to 512 to fit the big image size (> 1024 kB).
1918
CONFIG_BOOT_MAX_IMG_SECTORS=512

samples/matter/contact_sensor/sysbuild/mcuboot/boards/nrf54lm20dk_nrf54lm20a_cpuapp.overlay

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
};
1111
};
1212

13-
// restore full RRAM and SRAM space - by default some parts are dedicated to FLRP
13+
/* restore full RRAM and SRAM space - by default some parts are dedicated to FLRP */
1414
&cpuapp_rram {
1515
reg = <0x0 DT_SIZE_K(2036)>;
1616
};

samples/matter/contact_sensor/sysbuild/mcuboot/boards/nrf54lm20dk_nrf54lm20a_cpuapp_internal.conf

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -8,29 +8,16 @@ CONFIG_HW_STACK_PROTECTION=n
88
CONFIG_BOOT_WATCHDOG_FEED=n
99

1010
# Disable all debug features
11-
CONFIG_USE_SEGGER_RTT=n
12-
CONFIG_SHELL=n
13-
CONFIG_OPENTHREAD_SHELL=n
14-
CONFIG_CONSOLE=n
15-
CONFIG_UART_CONSOLE=n
16-
CONFIG_SERIAL=n
17-
CONFIG_LOG=n
18-
CONFIG_LOG_MODE_MINIMAL=n
1911
CONFIG_ASSERT_VERBOSE=n
2012
CONFIG_ASSERT_NO_FILE_INFO=y
2113
CONFIG_PRINTK=n
22-
CONFIG_PRINTK_SYNC=n
23-
CONFIG_THREAD_NAME=n
24-
CONFIG_BOOT_BANNER=n
2514

2615
# Bootloader size optimization to fit into 28 KB partition.
2716
# These options can be safely disabled because in this configuration,
2817
# the secondary slot resides in the internal RRAM, so features
2918
# needed to handle the external flash are not needed.
30-
CONFIG_GPIO=n
3119
CONFIG_SPI_NOR=n
3220
CONFIG_SPI=n
33-
CONFIG_MULTITHREADING=n
3421

3522
# Increase the maximum number of sectors to 512 to fit the big image size (> 1024 kB).
3623
CONFIG_BOOT_MAX_IMG_SECTORS=512

0 commit comments

Comments
 (0)