Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Chime device type to zap-templates/zcl/data-model/chip/matter-devices.xml #37659

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

lboue
Copy link
Contributor

@lboue lboue commented Feb 19, 2025

Add Chime device type to zap-templates/zcl/data-model/chip/matter-devices.xml

Testing

./scripts/tools/zap/run_zaptool.sh examples/chef/devices/template.zap

Check that Chime is in the list:
Capture d’écran du 2025-02-19 16-58-48

Add Chime device type for ZAPtool.
Copy link

semanticdiff-com bot commented Feb 19, 2025

@github-actions github-actions bot added the app label Feb 19, 2025
@lboue lboue changed the title Update matter-devices.xml Add Chime device type to zap-templates/zcl/data-model/chip/matter-devices.xml Feb 19, 2025
@lboue lboue marked this pull request as ready for review February 19, 2025 16:02
Copy link

github-actions bot commented Feb 19, 2025

PR #37659: Size comparison from 86e0b85 to ee9bced

Full report (14 builds for cc13x4_26x4, cc32xx, nrfconnect, qpg, stm32, tizen)
platform target config section 86e0b85 ee9bced change % change
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 FLASH 815244 815244 0 0.0
RAM 120256 120256 0 0.0
lock-ftd LP_EM_CC1354P10_6 FLASH 824156 824156 0 0.0
RAM 125352 125352 0 0.0
pump-app LP_EM_CC1354P10_6 FLASH 771024 771024 0 0.0
RAM 113724 113724 0 0.0
pump-controller-app LP_EM_CC1354P10_6 FLASH 755300 755300 0 0.0
RAM 113932 113932 0 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 538730 538730 0 0.0
RAM 205112 205112 0 0.0
lock CC3235SF_LAUNCHXL FLASH 572886 572886 0 0.0
RAM 205360 205360 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 FLASH 907328 907328 0 0.0
RAM 142315 142315 0 0.0
nrf7002dk_nrf5340_cpuapp FLASH 901660 901660 0 0.0
RAM 124663 124663 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 FLASH 845948 845948 0 0.0
RAM 141243 141243 0 0.0
qpg lighting-app qpg6105+debug FLASH 662132 662132 0 0.0
RAM 105116 105116 0 0.0
lock-app qpg6105+debug FLASH 620240 620240 0 0.0
RAM 99664 99664 0 0.0
stm32 light STM32WB5MM-DK FLASH 459776 459776 0 0.0
RAM 141472 141472 0 0.0
tizen all-clusters-app arm unknown 5076 5076 0 0.0
FLASH 1750404 1750404 0 0.0
RAM 93260 93260 0 0.0
chip-tool-ubsan arm unknown 11476 11476 0 0.0
FLASH 18960222 18960222 0 0.0
RAM 8296776 8296776 0 0.0

@lboue lboue marked this pull request as draft February 19, 2025 16:16
Copy link

PR #37659: Size comparison from 86e0b85 to d9b947c

Full report (3 builds for cc32xx, stm32)
platform target config section 86e0b85 d9b947c change % change
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 538730 538746 16 0.0
RAM 205112 205112 0 0.0
lock CC3235SF_LAUNCHXL FLASH 572886 572902 16 0.0
RAM 205360 205360 0 0.0
stm32 light STM32WB5MM-DK FLASH 459776 459792 16 0.0
RAM 141472 141472 0 0.0

Copy link

PR #37659: Size comparison from 86e0b85 to f5f3e28

Full report (3 builds for cc32xx, stm32)
platform target config section 86e0b85 f5f3e28 change % change
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 538730 538802 72 0.0
RAM 205112 205112 0 0.0
lock CC3235SF_LAUNCHXL FLASH 572886 572950 64 0.0
RAM 205360 205360 0 0.0
stm32 light STM32WB5MM-DK FLASH 459776 459784 8 0.0
RAM 141472 141472 0 0.0

@bzbarsky-apple
Copy link
Contributor

It's a copy/paste mistake from MA-thermostatcontroller above. Fixed.

@lboue are you hand-writing this XML instead of generating it with https://github.com/project-chip/alchemy ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants