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.

Sign up for GitHub

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

Jump to bottom

Fix Chef MaxMeasuredValue of PressureMeasurement #36434

Merged
merged 1 commit into from
Nov 8, 2024
Merged

Fix Chef MaxMeasuredValue of PressureMeasurement #36434

merged 1 commit into from
Nov 8, 2024

Conversation

erwinpan1
Copy link
Contributor

The MaxMeasuredValue in Pressure Measurement cluster are incorrectly set to 0xFFFE (or -2) which is not compatible with the the spec (max 32767)

@semanticdiff-com SemanticDiff.com
Copy link

Review changes with  SemanticDiff

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 8, 2024 •
edited
Loading

PR #36434: Size comparison from 5ace630 to d098c46

Full report (68 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
platform target config section 5ace630 d098c46 change % change
bl602 lighting-app bl602+mfd+littlefs+rpc FLASH 1349426 1349426 0 0.0
RAM 104080 104080 0 0.0
bl702 lighting-app bl702+eth FLASH 649386 649386 0 0.0
RAM 25321 25321 0 0.0
bl702+wifi FLASH 826966 826966 0 0.0
RAM 14053 14053 0 0.0
bl706+mfd+rpc+littlefs FLASH 1055164 1055164 0 0.0
RAM 23893 23893 0 0.0
bl702l lighting-app bl702l+mfd+littlefs FLASH 976530 976530 0 0.0
RAM 16556 16556 0 0.0
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 FLASH 837876 837876 0 0.0
RAM 123628 123628 0 0.0
lock-ftd LP_EM_CC1354P10_6 FLASH 823792 823792 0 0.0
RAM 125516 125516 0 0.0
pump-app LP_EM_CC1354P10_6 FLASH 770532 770532 0 0.0
RAM 113984 113984 0 0.0
pump-controller-app LP_EM_CC1354P10_6 FLASH 754736 754736 0 0.0
RAM 114192 114192 0 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 625926 625926 0 0.0
RAM 205780 205780 0 0.0
lock CC3235SF_LAUNCHXL FLASH 667238 667238 0 0.0
RAM 205932 205932 0 0.0
cyw30739 light CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 679513 679513 0 0.0
RAM 78684 78684 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 699357 699357 0 0.0
RAM 81324 81324 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 699357 699357 0 0.0
RAM 81324 81324 0 0.0
CYW930739M2EVB-02 unknown 2040 2040 0 0.0
FLASH 656301 656301 0 0.0
RAM 73752 73752 0 0.0
light-switch CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 616361 616361 0 0.0
RAM 71644 71644 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 635989 635989 0 0.0
RAM 74188 74188 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 635989 635989 0 0.0
RAM 74188 74188 0 0.0
lock CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 635817 635817 0 0.0
RAM 74692 74692 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 655525 655525 0 0.0
RAM 77236 77236 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 655525 655525 0 0.0
RAM 77236 77236 0 0.0
thermostat CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 611213 611213 0 0.0
RAM 68772 68772 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 631065 631065 0 0.0
RAM 71412 71412 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 631065 631065 0 0.0
RAM 71412 71412 0 0.0
efr32 lock-app BRD4187C FLASH 927628 927628 0 0.0
RAM 160160 160160 0 0.0
BRD4338a FLASH 743696 743696 0 0.0
RAM 233288 233288 0 0.0
window-app BRD4187C FLASH 1018816 1018808 -8 -0.0
RAM 128260 128260 0 0.0
esp32 all-clusters-app c3devkit DRAM 95328 95328 0 0.0
FLASH 1540466 1540466 0 0.0
IRAM 82542 82542 0 0.0
m5stack DRAM 116256 116256 0 0.0
FLASH 1549282 1549282 0 0.0
IRAM 117039 117039 0 0.0
linux air-purifier-app debug unknown 4712 4712 0 0.0
FLASH 2698879 2698879 0 0.0
RAM 129648 129648 0 0.0
all-clusters-app debug unknown 5552 5552 0 0.0
FLASH 5992608 5992608 0 0.0
RAM 523968 523968 0 0.0
all-clusters-minimal-app debug unknown 5448 5448 0 0.0
FLASH 5328216 5328216 0 0.0
RAM 242480 242480 0 0.0
bridge-app debug unknown 5432 5432 0 0.0
FLASH 4670954 4670954 0 0.0
RAM 218416 218416 0 0.0
chip-tool debug unknown 5984 5984 0 0.0
FLASH 12872194 12872194 0 0.0
RAM 584466 584466 0 0.0
chip-tool-ipv6only arm64 unknown 21400 21400 0 0.0
FLASH 11002704 11002704 0 0.0
RAM 635368 635368 0 0.0
fabric-admin debug unknown 5808 5808 0 0.0
FLASH 11277239 11277239 0 0.0
RAM 584826 584826 0 0.0
fabric-bridge-app debug unknown 4688 4688 0 0.0
FLASH 4495222 4495222 0 0.0
RAM 205400 205400 0 0.0
fabric-sync debug unknown 4880 4880 0 0.0
FLASH 5309077 5309077 0 0.0
RAM 466872 466872 0 0.0
lighting-app debug+rpc+ui unknown 6096 6096 0 0.0
FLASH 5606881 5606881 0 0.0
RAM 228552 228552 0 0.0
lock-app debug unknown 5368 5368 0 0.0
FLASH 4720428 4720428 0 0.0
RAM 204552 204552 0 0.0
ota-provider-app debug unknown 4744 4744 0 0.0
FLASH 4345860 4345860 0 0.0
RAM 198272 198272 0 0.0
ota-requestor-app debug unknown 4680 4680 0 0.0
FLASH 4484262 4484262 0 0.0
RAM 202840 202840 0 0.0
shell debug unknown 4240 4240 0 0.0
FLASH 3011293 3011293 0 0.0
RAM 160368 160368 0 0.0
thermostat-no-ble arm64 unknown 9440 9440 0 0.0
FLASH 4089952 4089952 0 0.0
RAM 242936 242936 0 0.0
tv-app debug unknown 5648 5648 0 0.0
FLASH 5942565 5942565 0 0.0
RAM 596336 596336 0 0.0
tv-casting-app debug unknown 5232 5232 0 0.0
FLASH 11079389 11079389 0 0.0
RAM 694536 694536 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 FLASH 915372 915372 0 0.0
RAM 143281 143281 0 0.0
nrf7002dk_nrf5340_cpuapp FLASH 887588 887588 0 0.0
RAM 141468 141468 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 FLASH 847628 847628 0 0.0
RAM 142189 142189 0 0.0
nxp contact k32w0+release FLASH 584144 584144 0 0.0
RAM 71044 71044 0 0.0
mcxw71+release FLASH 598688 598688 0 0.0
RAM 63144 63144 0 0.0
light k32w0+release FLASH 611220 611220 0 0.0
RAM 70436 70436 0 0.0
k32w1+release FLASH 685016 685016 0 0.0
RAM 48776 48776 0 0.0
lock mcxw71+release FLASH 748544 748544 0 0.0
RAM 67300 67300 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 FLASH 1642980 1642980 0 0.0
RAM 212056 212056 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 FLASH 1550180 1550180 0 0.0
RAM 208856 208856 0 0.0
light cy8ckit_062s2_43012 FLASH 1466620 1466620 0 0.0
RAM 200840 200840 0 0.0
lock cy8ckit_062s2_43012 FLASH 1464796 1464796 0 0.0
RAM 225200 225200 0 0.0
qpg lighting-app qpg6105+debug FLASH 662184 662184 0 0.0
RAM 105380 105380 0 0.0
lock-app qpg6105+debug FLASH 620284 620284 0 0.0
RAM 99832 99832 0 0.0
stm32 light STM32WB5MM-DK FLASH 482984 482984 0 0.0
RAM 144844 144844 0 0.0
telink bridge-app tlsr9258a FLASH 684684 684684 0 0.0
RAM 91528 91528 0 0.0
contact-sensor-app tlsr9528a_retention FLASH 622696 622696 0 0.0
RAM 50464 50464 0 0.0
light-switch-app-ota-compress-lzma-shell-factory-data tlsr9528a FLASH 710622 710622 0 0.0
RAM 73804 73804 0 0.0
lighting-app-ota-factory-data tlsr9118bdk40d FLASH 628990 628990 0 0.0
RAM 145384 145384 0 0.0
lighting-app-ota-rpc-factory-data-4mb tlsr9518adk80d FLASH 814878 814878 0 0.0
RAM 100004 100004 0 0.0
tizen all-clusters-app arm unknown 4948 4948 0 0.0
FLASH 1722580 1722580 0 0.0
RAM 90656 90656 0 0.0
chip-tool-ubsan arm unknown 10832 10832 0 0.0
FLASH 18007550 18007550 0 0.0
RAM 7857676 7857676 0 0.0

@andy31415 andy31415 merged commit cf50115 into project-chip:master Nov 8, 2024
64 of 67 checks passed
erwinpan1 added a commit to erwinpan1/connectedhomeip that referenced this pull request Nov 11, 2024
@erwinpan1
Fix Chef MaxMeasuredValue of PressureMeasurement (project-chip#36434)
3efd1a0
@erwinpan1 erwinpan1 mentioned this pull request Nov 11, 2024
Merged
@erwinpan1 erwinpan1 deleted the fix_chef_pressure_measurement_max_value branch November 11, 2024 02:20
GinyWang added a commit to erwinpan1/connectedhomeip that referenced this pull request Nov 11, 2024
@GinyWang
Merge pull request #25 from erwinpan1/fix_max_of_pressure_cluster
3a2f6a1 
Fix Chef MaxMeasuredValue of PressureMeasurement (project-chip#36434)
yyzhong-g pushed a commit to yyzhong-g/connectedhomeip that referenced this pull request Dec 12, 2024
@erwinpan1 @yyzhong-g
Fix Chef MaxMeasuredValue of PressureMeasurement (project-chip#36434)
8fbd883
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andy31415 andy31415 andy31415 approved these changes

Assignees
No one assigned
Labels
examples fast track review - pending
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@erwinpan1 @andy31415