In emberAfEndpointConfigure() change endpoint ID to a uint16_t #22998

selissia · 2022-10-03T14:54:52Z

Problem

In emberAfEndpointConfigure() a uint8_t is used for the endpoint ID. According to the spec (7.18.2. Derived Data Types) a uint16_t must be used.

Separate minor issue: wrong type is being passed to OnOffServer::setOnOffValue()

Change overview

In emberAfEndpointConfigure() use uint16_t for the endpoint ID.

Change the type of the second parameter of OnOffServer::setOnOffValue() to chip::CommandId

Testing

Tested with Silabs Unify Matter bridge on Ubuntu

* Added new manual scripts * Added Auto generated File

* [ota] Store Default OTA Providers in flash Store Default OTA Providers in flash each time the attribute is modified and load it back on the application startup. * Restyled by clang-format * Fix build and reduce flash usage Co-authored-by: Restyled.io <[email protected]>

github-actions · 2022-10-03T15:50:21Z

PR #22998: Size comparison from 5056270 to 2d17db0

Full report (2 builds for bl702)

platform	target	config	section	`5056270`	`2d17db0`
bl702	lighting-app	bl702		0	0
			(read only)	3262	3262
			(read/write)	1188027	1188027
			.bleromro	6296	6296
			.bleromrw	124	124
			.boot2	688	688
			.bss	67094	67094
			.bss_psram	29696	29696
			.comment	48	48
			.data	4272	4272
			.debug_abbrev	`1506903`	`1506903`
			.debug_aranges	133080	133080
			.debug_frame	486388	486388
			.debug_info	37900563	37900563
			.debug_line	5252986	5252986
			.debug_loc	3364667	3364667
			.debug_ranges	359016	359016
			.debug_str	3456190	3456190
			.hbn	509	509
			.hbn_noinit	260	260
			.init	342	342
			.init_array	144	144
			.psram	0	0
			.riscv.attributes	47	47
			.rodata	116536	116536
			.rsvd	3188	3188
			.shstrtab	293	293
			.stack	2048	2048
			.strtab	564835	564835
			.symtab	171584	171584
			.tcm_data	36	36
			.tcmcode	3262	3262
			.text	956624	956624
		bl702+rpc		0	0
			(read only)	3262	3262
			(read/write)	1283963	1283963
			.bleromro	6296	6296
			.bleromrw	124	124
			.boot2	688	688
			.bss	75142	75142
			.bss_psram	29936	29936
			.comment	48	48
			.data	4800	4800
			.debug_abbrev	1644448	1644448
			.debug_aranges	140584	140584
			.debug_frame	511764	511764
			.debug_info	41807174	41807174
			.debug_line	5627521	5627521
			.debug_loc	3557370	3557370
			.debug_ranges	381472	381472
			.debug_str	3852090	3852090
			.hbn	509	509
			.hbn_noinit	260	260
			.init	342	342
			.init_array	160	160
			.psram	0	0
			.riscv.attributes	47	47
			.rodata	129928	129928
			.rsvd	3188	3188
			.shstrtab	293	293
			.stack	2048	2048
			.strtab	624008	624008
			.symtab	189392	189392
			.tcm_data	36	36
			.tcmcode	3262	3262
			.text	1030328	1030328

src/app/clusters/level-control/level-control.cpp

github-actions · 2022-10-03T19:02:12Z

PR #22998: Size comparison from 5056270 to 1672f76

Increases (15 builds for bl602, bl702, cc13x2_26x2, esp32, linux, psoc6, telink)

platform	target	config	section	`5056270`	`1672f76`	change	% change
bl602	lighting-app	bl602	.text	`1068426`	1068428	2	0.0
		bl602+rpc	.text	1099774	1099776	2	0.0
bl702	lighting-app	bl702	.debug_aranges	133080	133088	8	0.0
			.debug_frame	486388	486444	56	0.0
			.debug_info	37900563	37902575	2012	0.0
			.debug_line	5252986	`5255713`	2727	0.1
			.debug_loc	3364667	3366207	1540	0.0
			.debug_ranges	359016	359136	120	0.0
			.debug_str	3456190	3456572	382	0.0
		bl702+rpc	.debug_aranges	140584	140592	8	0.0
			.debug_frame	511764	511820	56	0.0
			.debug_info	41807174	41809185	2011	0.0
			.debug_line	5627521	5630248	2727	0.0
			.debug_loc	3557370	3558839	1469	0.0
			.debug_ranges	381472	381592	120	0.0
			.debug_str	3852090	3852472	382	0.0
cc13x2_26x2	lock-mtd	LP_CC2652R7	(read/write)	185276	185284	8	0.0
	pump-controller-app	LP_CC2652R7	(read/write)	177616	177624	8	0.0
esp32	all-clusters-app	c3devkit	(read only)	1223036	`1223038`	2	0.0
			.flash.text	1223036	`1223038`	2	0.0
		m5stack	(read/write)	564004	564012	8	0.0
			.flash.rodata	314736	314744	8	0.0
linux	chip-tool-ipv6only	arm64	(read only)	10422652	10427436	4784	0.0
			(read/write)	706257	706401	144	0.0
			.data.rel.ro	650560	650584	24	0.0
			.rodata	517404	517436	32	0.0
			.text	8251268	8254692	3424	0.0
	thermostat-no-ble	arm64	(read only)	2387620	2388164	544	0.0
			(read/write)	143649	143681	32	0.0
			.text	2001648	2001840	192	0.0
psoc6	all-clusters	cy8ckit_062s2_43012	.debug_aranges	111816	111824	8	0.0
			.debug_frame	373332	373372	40	0.0
			.debug_info	26823182	26826796	3614	0.0
			.debug_line	3668940	3670548	1608	0.0
			.debug_loc	`3583338`	3585589	2251	0.1
			.debug_ranges	338840	338936	96	0.0
			.debug_str	3439862	3440244	382	0.0
	all-clusters-minimal	cy8ckit_062s2_43012	.debug_aranges	111288	111296	8	0.0
			.debug_frame	376412	376452	40	0.0
			.debug_info	26559963	26563578	3615	0.0
			.debug_line	`3689656`	3691264	1608	0.0
			.debug_loc	3570975	3573226	2251	0.1
			.debug_ranges	337456	337552	96	0.0
			.debug_str	`3428875`	`3429257`	382	0.0
	light	cy8ckit_062s2_43012	.debug_aranges	103496	103504	8	0.0
			.debug_frame	346740	346780	40	0.0
			.debug_info	22023599	22027215	3616	0.0
			.debug_line	`3259572`	`3261180`	1608	0.0
			.debug_loc	`3268950`	`3271203`	2253	0.1
			.debug_ranges	302784	302880	96	0.0
			.debug_str	3234407	3234789	382	0.0
	lock	cy8ckit_062s2_43012	.debug_aranges	103896	103904	8	0.0
			.debug_frame	348632	348672	40	0.0
			.debug_info	22262269	22265884	3615	0.0
			.debug_line	`3257434`	`3259041`	1607	0.0
			.debug_loc	3297544	`3299781`	2237	0.1
			.debug_ranges	304728	304824	96	0.0
			.debug_str	`3253921`	3254303	382	0.0
telink	lighting-app	tlsr9518adk80d	(read/write)	810908	810916	8	0.0
			text	570996	570998	2	0.0

Decreases (10 builds for bl702, cc13x2_26x2, esp32, psoc6, telink)

platform	target	config	section	`5056270`	`1672f76`	change	% change
bl702	lighting-app	bl702	.debug_abbrev	`1506903`	1506897	-6	-0.0
		bl702+rpc	.debug_abbrev	1644448	1644442	-6	-0.0
			.text	1030328	`1030326`	-2	-0.0
cc13x2_26x2	lock-mtd	LP_CC2652R7	(read only)	659099	659091	-8	-0.0
			.text	555680	555672	-8	-0.0
	pump-controller-app	LP_CC2652R7	(read only)	671903	671895	-8	-0.0
			.text	585336	585328	-8	-0.0
esp32	all-clusters-app	m5stack	(read only)	1233103	`1233099`	-4	-0.0
			.flash.text	1227719	1227715	-4	-0.0
psoc6	all-clusters	cy8ckit_062s2_43012	.debug_abbrev	1229452	`1229446`	-6	-0.0
	all-clusters-minimal	cy8ckit_062s2_43012	.debug_abbrev	1221251	1221245	-6	-0.0
	light	cy8ckit_062s2_43012	.debug_abbrev	1055307	`1055301`	-6	-0.0
	lock	cy8ckit_062s2_43012	.debug_abbrev	1057410	1057404	-6	-0.0
telink	ota-requestor-app	tlsr9518adk80d	text	577180	577178	-2	-0.0

Full report (37 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)

platform	target	config	section	`5056270`	`1672f76`	change	% change
bl602	lighting-app	bl602	(read/write)	`1388826`	`1388826`	0	0.0
			.bss	90729	90729	0	0.0
			.data	9928	9928	0	0.0
			.text	`1068426`	1068428	2	0.0
		bl602+rpc	(read/write)	1434038	1434038	0	0.0
			.bss	98161	98161	0	0.0
			.data	10312	10312	0	0.0
			.text	1099774	1099776	2	0.0
bl702	lighting-app	bl702	(read only)	3262	3262	0	0.0
			(read/write)	1188027	1188027	0	0.0
			.bleromro	6296	6296	0	0.0
			.bleromrw	124	124	0	0.0
			.boot2	688	688	0	0.0
			.bss	67094	67094	0	0.0
			.bss_psram	29696	29696	0	0.0
			.comment	48	48	0	0.0
			.data	4272	4272	0	0.0
			.debug_abbrev	`1506903`	1506897	-6	-0.0
			.debug_aranges	133080	133088	8	0.0
			.debug_frame	486388	486444	56	0.0
			.debug_info	37900563	37902575	2012	0.0
			.debug_line	5252986	`5255713`	2727	0.1
			.debug_loc	3364667	3366207	1540	0.0
			.debug_ranges	359016	359136	120	0.0
			.debug_str	3456190	3456572	382	0.0
			.hbn	509	509	0	0.0
			.hbn_noinit	260	260	0	0.0
			.init	342	342	0	0.0
			.init_array	144	144	0	0.0
			.psram	0	0	0	0.0
			.riscv.attributes	47	47	0	0.0
			.rodata	116536	116536	0	0.0
			.rsvd	3188	3188	0	0.0
			.shstrtab	293	293	0	0.0
			.stack	2048	2048	0	0.0
			.strtab	564835	564835	0	0.0
			.symtab	171584	171584	0	0.0
			.tcm_data	36	36	0	0.0
			.tcmcode	3262	3262	0	0.0
			.text	0	0	0	0.0
				956624	956624	0	0.0
		bl702+rpc	(read only)	3262	3262	0	0.0
			(read/write)	1283963	1283963	0	0.0
			.bleromro	6296	6296	0	0.0
			.bleromrw	124	124	0	0.0
			.boot2	688	688	0	0.0
			.bss	75142	75142	0	0.0
			.bss_psram	29936	29936	0	0.0
			.comment	48	48	0	0.0
			.data	4800	4800	0	0.0
			.debug_abbrev	1644448	1644442	-6	-0.0
			.debug_aranges	140584	140592	8	0.0
			.debug_frame	511764	511820	56	0.0
			.debug_info	41807174	41809185	2011	0.0
			.debug_line	5627521	5630248	2727	0.0
			.debug_loc	3557370	3558839	1469	0.0
			.debug_ranges	381472	381592	120	0.0
			.debug_str	3852090	3852472	382	0.0
			.hbn	509	509	0	0.0
			.hbn_noinit	260	260	0	0.0
			.init	342	342	0	0.0
			.init_array	160	160	0	0.0
			.psram	0	0	0	0.0
			.riscv.attributes	47	47	0	0.0
			.rodata	129928	129928	0	0.0
			.rsvd	3188	3188	0	0.0
			.shstrtab	293	293	0	0.0
			.stack	2048	2048	0	0.0
			.strtab	624008	624008	0	0.0
			.symtab	189392	189392	0	0.0
			.tcm_data	36	36	0	0.0
			.tcmcode	3262	3262	0	0.0
			.text	0	0	0	0.0
				1030328	`1030326`	-2	-0.0
cc13x2_26x2	all-clusters-app	LP_CC2652R7	(read only)	676555	676555	0	0.0
			(read/write)	174980	174980	0	0.0
			.bss	81228	81228	0	0.0
			.data	3380	3380	0	0.0
			.rodata	89547	89547	0	0.0
			.text	586696	586696	0	0.0
	all-clusters-minimal-app	LP_CC2652R7	(read only)	640803	640803	0	0.0
			(read/write)	157996	157996	0	0.0
			.bss	80500	80500	0	0.0
			.data	3380	3380	0	0.0
			.rodata	78683	78683	0	0.0
			.text	561800	561800	0	0.0
	lock-ftd	LP_CC2652R7	(read only)	675239	675239	0	0.0
			(read/write)	173448	173448	0	0.0
			.bss	78468	78468	0	0.0
			.data	3304	3304	0	0.0
			.rodata	77071	77071	0	0.0
			.text	597688	597688	0	0.0
	lock-mtd	LP_CC2652R7	(read only)	659099	659091	-8	-0.0
			(read/write)	185276	185284	8	0.0
			.bss	74156	74156	0	0.0
			.data	3304	3304	0	0.0
			.rodata	102939	102939	0	0.0
			.text	555680	555672	-8	-0.0
	pump-app	LP_CC2652R7	(read only)	687403	687403	0	0.0
			(read/write)	162004	162004	0	0.0
			.bss	78420	78420	0	0.0
			.data	3296	3296	0	0.0
			.rodata	90531	90531	0	0.0
			.text	596388	596388	0	0.0
	pump-controller-app	LP_CC2652R7	(read only)	671903	671895	-8	-0.0
			(read/write)	177616	177624	8	0.0
			.bss	78532	78532	0	0.0
			.data	3292	3292	0	0.0
			.rodata	86087	86087	0	0.0
			.text	585336	585328	-8	-0.0
	shell	LP_CC2652R7	(read only)	667606	667606	0	0.0
			(read/write)	186240	186240	0	0.0
			.bss	83540	83540	0	0.0
			.data	3376	3376	0	0.0
			.rodata	86262	86262	0	0.0
			.text	581028	581028	0	0.0
cyw30739	light	cyw930739m2evb_01	(read/write)	587578	587578	0	0.0
			.app_xip_area	464204	464204	0	0.0
			.bss	65792	65792	0	0.0
			.data	760	760	0	0.0
			.rodata	0	0	0	0.0
			.text	112	112	0	0.0
	lock	cyw930739m2evb_01	(read/write)	591834	591834	0	0.0
			.app_xip_area	463140	463140	0	0.0
			.bss	71104	71104	0	0.0
			.data	768	768	0	0.0
			.rodata	0	0	0	0.0
			.text	112	112	0	0.0
	ota-requestor-no-progress-logging	cyw930739m2evb_01	(read/write)	543346	543346	0	0.0
			.app_xip_area	425028	425028	0	0.0
			.bss	60784	60784	0	0.0
			.data	716	716	0	0.0
			.rodata	0	0	0	0.0
			.text	112	112	0	0.0
efr32	lighting-app	BRD4161A	(read/write)	1110692	1110692	0	0.0
			.bss	137708	137708	0	0.0
			.data	2068	2068	0	0.0
			.text	970892	970892	0	0.0
		BRD4161A+rpc	(read/write)	973636	973636	0	0.0
			.bss	152236	152236	0	0.0
			.data	2248	2248	0	0.0
			.text	819132	819132	0	0.0
		BRD4161A+rs911x	(read/write)	1003840	1003840	0	0.0
			.bss	170544	170544	0	0.0
			.data	2060	2060	0	0.0
			.text	831216	831216	0	0.0
	lock-app	BRD4161A+wf200	(read/write)	1147760	1147760	0	0.0
			.bss	153424	153424	0	0.0
			.data	2068	2068	0	0.0
			.text	992244	992244	0	0.0
	window-app	BRD4161A	(read/write)	1102684	1102684	0	0.0
			.bss	139156	139156	0	0.0
			.data	2092	2092	0	0.0
			.text	961412	961412	0	0.0
esp32	all-clusters-app	c3devkit	(read only)	1223036	`1223038`	2	0.0
			(read/write)	1788126	1788126	0	0.0
			.dram0.bss	76944	76944	0	0.0
			.dram0.data	13840	13840	0	0.0
			.flash.rodata	257696	257696	0	0.0
			.flash.text	1223036	`1223038`	2	0.0
			.iram0.text	65204	65204	0	0.0
		m5stack	(read only)	1233103	`1233099`	-4	-0.0
			(read/write)	564004	564012	8	0.0
			.dram0.bss	82304	82304	0	0.0
			.dram0.data	34296	34296	0	0.0
			.flash.rodata	314736	314744	8	0.0
			.flash.text	1227719	1227715	-4	-0.0
			.iram0.text	123939	123939	0	0.0
k32w	light	k32w0+release	(read/write)	641472	641472	0	0.0
			.bss	74816	74816	0	0.0
			.data	2064	2064	0	0.0
			.text	561864	561864	0	0.0
	lock	k32w0+release	(read/write)	632768	632768	0	0.0
			.bss	75592	75592	0	0.0
			.data	2080	2080	0	0.0
			.text	552368	552368	0	0.0
linux	chip-tool-ipv6only	arm64	(read only)	10422652	10427436	4784	0.0
			(read/write)	706257	706401	144	0.0
			.bss	33953	33953	0	0.0
			.data	2864	2864	0	0.0
			.data.rel.ro	650560	650584	24	0.0
			.dynamic	560	560	0	0.0
			.got	13904	13904	0	0.0
			.init	24	24	0	0.0
			.init_array	208	208	0	0.0
			.rodata	517404	517436	32	0.0
			.text	8251268	8254692	3424	0.0
	thermostat-no-ble	arm64	(read only)	2387620	2388164	544	0.0
			(read/write)	143649	143681	32	0.0
			.bss	55361	55361	0	0.0
			.data	1912	1912	0	0.0
			.data.rel.ro	77208	77208	0	0.0
			.dynamic	560	560	0	0.0
			.got	5184	5184	0	0.0
			.init	24	24	0	0.0
			.init_array	440	440	0	0.0
			.rodata	143676	143676	0	0.0
			.text	2001648	2001840	192	0.0
mbed	lock-app	CY8CPROTO_062_4343W+release	(read only)	6224	6224	0	0.0
			(read/write)	2451792	2451792	0	0.0
			.bss	215028	215028	0	0.0
			.data	5872	5872	0	0.0
			.text	1414436	1414436	0	0.0
nrfconnect	all-clusters-app	nrf52840dk_nrf52840	(read/write)	`1182911`	`1182911`	0	0.0
			bss	144433	144433	0	0.0
			rodata	144208	144208	0	0.0
			text	815352	815352	0	0.0
	all-clusters-minimal-app	nrf52840dk_nrf52840	(read/write)	1161579	1161579	0	0.0
			bss	143660	143660	0	0.0
			rodata	135780	135780	0	0.0
			text	803248	803248	0	0.0
psoc6	all-clusters	cy8ckit_062s2_43012	(read only)	841968	841968	0	0.0
			(read/write)	`1744436`	`1744436`	0	0.0
			.ARM.attributes	46	46	0	0.0
			.ARM.exidx	8	8	0	0.0
			.bss	188712	188712	0	0.0
			.comment	204	204	0	0.0
			.copy.table	24	24	0	0.0
			.cy_m0p_image	6216	6216	0	0.0
			.cy_sharedmem	8	8	0	0.0
			.data	2664	2664	0	0.0
			.debug_abbrev	1229452	`1229446`	-6	-0.0
			.debug_aranges	111816	111824	8	0.0
			.debug_frame	373332	373372	40	0.0
			.debug_info	26823182	26826796	3614	0.0
			.debug_line	3668940	3670548	1608	0.0
			.debug_loc	`3583338`	3585589	2251	0.1
			.debug_ranges	338840	338936	96	0.0
			.debug_str	3439862	3440244	382	0.0
			.heap	841968	841968	0	0.0
			.noinit	148	148	0	0.0
			.ramVectors	736	736	0	0.0
			.shstrtab	288	288	0	0.0
			.stab	156	156	0	0.0
			.stabstr	335	335	0	0.0
			.stack_dummy	4096	4096	0	0.0
			.strtab	569480	569480	0	0.0
			.symtab	421056	421056	0	0.0
			.text	1544672	1544672	0	0.0
			.zero.table	8	8	0	0.0
			text	0	0	0	0.0
	all-clusters-minimal	cy8ckit_062s2_43012	(read only)	842704	842704	0	0.0
			(read/write)	1687044	1687044	0	0.0
			.ARM.attributes	46	46	0	0.0
			.ARM.exidx	8	8	0	0.0
			.bss	187976	187976	0	0.0
			.comment	204	204	0	0.0
			.copy.table	24	24	0	0.0
			.cy_m0p_image	6216	6216	0	0.0
			.cy_sharedmem	8	8	0	0.0
			.data	2664	2664	0	0.0
			.debug_abbrev	1221251	1221245	-6	-0.0
			.debug_aranges	111288	111296	8	0.0
			.debug_frame	376412	376452	40	0.0
			.debug_info	26559963	26563578	3615	0.0
			.debug_line	`3689656`	3691264	1608	0.0
			.debug_loc	3570975	3573226	2251	0.1
			.debug_ranges	337456	337552	96	0.0
			.debug_str	`3428875`	`3429257`	382	0.0
			.heap	842704	842704	0	0.0
			.noinit	148	148	0	0.0
			.ramVectors	736	736	0	0.0
			.shstrtab	288	288	0	0.0
			.stab	156	156	0	0.0
			.stabstr	335	335	0	0.0
			.stack_dummy	4096	4096	0	0.0
			.strtab	533569	533569	0	0.0
			.symtab	407488	407488	0	0.0
			.text	1488016	1488016	0	0.0
			.zero.table	0	0	0	0.0
				8	8	0	0.0
	light	cy8ckit_062s2_43012	(read only)	850896	850896	0	0.0
			(read/write)	1605572	1605572	0	0.0
			.ARM.attributes	46	46	0	0.0
			.ARM.exidx	8	8	0	0.0
			.bss	179992	179992	0	0.0
			.comment	204	204	0	0.0
			.copy.table	24	24	0	0.0
			.cy_m0p_image	6216	6216	0	0.0
			.cy_sharedmem	8	8	0	0.0
			.data	2456	2456	0	0.0
			.debug_abbrev	1055307	`1055301`	-6	-0.0
			.debug_aranges	103496	103504	8	0.0
			.debug_frame	346740	346780	40	0.0
			.debug_info	22023599	22027215	3616	0.0
			.debug_line	`3259572`	`3261180`	1608	0.0
			.debug_loc	`3268950`	`3271203`	2253	0.1
			.debug_ranges	302784	302880	96	0.0
			.debug_str	3234407	3234789	382	0.0
			.heap	850896	850896	0	0.0
			.noinit	148	148	0	0.0
			.ramVectors	736	736	0	0.0
			.shstrtab	288	288	0	0.0
			.stab	156	156	0	0.0
			.stabstr	335	335	0	0.0
			.stack_dummy	4096	4096	0	0.0
			.strtab	469946	469946	0	0.0
			.symtab	375936	375936	0	0.0
			.text	1414736	1414736	0	0.0
			.zero.table	0	0	0	0.0
				8	8	0	0.0
	lock	cy8ckit_062s2_43012	(read only)	845880	845880	0	0.0
			(read/write)	1639492	1639492	0	0.0
			.ARM.attributes	46	46	0	0.0
			.ARM.exidx	8	8	0	0.0
			.bss	184992	184992	0	0.0
			.comment	204	204	0	0.0
			.copy.table	24	24	0	0.0
			.cy_m0p_image	6216	6216	0	0.0
			.cy_sharedmem	8	8	0	0.0
			.data	2472	2472	0	0.0
			.debug_abbrev	1057410	1057404	-6	-0.0
			.debug_aranges	103896	103904	8	0.0
			.debug_frame	348632	348672	40	0.0
			.debug_info	22262269	22265884	3615	0.0
			.debug_line	`3257434`	`3259041`	1607	0.0
			.debug_loc	3297544	`3299781`	2237	0.1
			.debug_ranges	304728	304824	96	0.0
			.debug_str	`3253921`	3254303	382	0.0
			.heap	845880	845880	0	0.0
			.noinit	148	148	0	0.0
			.ramVectors	736	736	0	0.0
			.shstrtab	288	288	0	0.0
			.stab	156	156	0	0.0
			.stabstr	335	335	0	0.0
			.stack_dummy	4096	4096	0	0.0
			.strtab	472353	472353	0	0.0
			.symtab	377600	377600	0	0.0
			.text	1443640	1443640	0	0.0
			.zero.table	0	0	0	0.0
				8	8	0	0.0
qpg	lighting-app	qpg6105+debug	(read/write)	1148140	1148140	0	0.0
			.bss	110548	110548	0	0.0
			.data	832	832	0	0.0
			.text	595240	595240	0	0.0
	lock-app	qpg6105+debug	(read/write)	1113204	1113204	0	0.0
			.bss	106364	106364	0	0.0
			.data	836	836	0	0.0
			.text	560300	560300	0	0.0
telink	light-switch-app	tlsr9518adk80d	(read/write)	788804	788804	0	0.0
			bss	72472	72472	0	0.0
			noinit	43520	43520	0	0.0
			text	552776	552776	0	0.0
	lighting-app	tlsr9518adk80d	(read/write)	810908	810916	8	0.0
			bss	73320	73320	0	0.0
			noinit	43520	43520	0	0.0
			text	570996	570998	2	0.0
	ota-requestor-app	tlsr9518adk80d	(read/write)	818852	818852	0	0.0
			bss	74228	74228	0	0.0
			noinit	43520	43520	0	0.0
			text	577180	577178	-2	-0.0

src/app/util/attribute-storage.cpp

…o a uint16_t (project-chip#22998) Merge in WMN_TOOLS/matter from cherry_pick/unify_build_fix to silabs_1.0 Squashed commit of the following: commit 77f4fef74e58480c79bfb8dcfa67ffbdd57c61f3 Author: Sergei Lissianoi <[email protected]> Date: Mon Oct 3 16:51:28 2022 -0400 In emberAfEndpointConfigure() change endpoint ID to a uint16_t (project-chip#22998) * Test added march 8 (project-chip#15957) * Added new manual scripts * Added Auto generated File * [OTA] Fix OTARequestorDriverImpl inclusion (project-chip#15981) * Regen to fix CI failures (project-chip#15990) * [ota] Store Default OTA Providers in flash (project-chip#15970) * [ota] Store Default OTA Providers in flash Store Default OTA Providers in flash each time the attribute is modified and load it back on the application startup. * Restyled by clang-format * Fix build and reduce flash usage Co-authored-by: Restyled.io <[email protected]> * Remove merge artifacts * Cast boolean to uint8_t as setOnOffValue expects, fixes GCC 8.3 error * Make endpoint id a uint16_t as per spec * Restyled by clang-format * Update the signature of OnOffServer::setOnOffValue() to avoid casts * Fix printf argument and formatting Co-authored-by: kowsisoundhar12 <[email protected]> Co-authored-by: Carol Yang <[email protected]> Co-authored-by: Boris Zbarsky <[email protected]> Co-authored-by: Damian Królik <[email protected]> Co-authored-by: Restyled.io <[email protected]>

…ndpoint ID to a uint16_t (project-chip#22998) Merge in WMN_TOOLS/matter from cherry_pick/unify_build_fix to silabs_1.0 Squashed commit of the following: commit 77f4fef74e58480c79bfb8dcfa67ffbdd57c61f3 Author: Sergei Lissianoi <[email protected]> Date: Mon Oct 3 16:51:28 2022 -0400 In emberAfEndpointConfigure() change endpoint ID to a uint16_t (project-chip#22998) * Test added march 8 (project-chip#15957) * Added new manual scripts * Added Auto generated File * [OTA] Fix OTARequestorDriverImpl inclusion (project-chip#15981) * Regen to fix CI failures (project-chip#15990) * [ota] Store Default OTA Providers in flash (project-chip#15970) * [ota] Store Default OTA Providers in flash Store Default OTA Providers in flash each time the attribute is modified and load it back on the application startup. * Restyled by clang-format * Fix build and reduce flash usage Co-authored-by: Restyled.io <[email protected]> * Remove merge artifacts * Cast boolean to uint8_t as setOnOffValue expects, fixes GCC 8.3 error * Make endpoint id a uint16_t as per spec * Restyled by clang-format * Update the signature of OnOffServer::setOnOffValue() to avoid casts * Fix printf argument and formatting Co-authored-by: kowsisoundhar12 <[email protected]> Co-authored-by: Carol Yang <[email protected]> Co-authored-by: Boris Zbarsky <[email protected]> Co-authored-by: Damian Królik <[email protected]> Co-authored-by: Restyled.io <[email protected]>

…o a uint16_t (project-chip#22998) Merge in WMN_TOOLS/matter from cherry_pick/unify_build_fix to silabs_1.0 Squashed commit of the following: commit 77f4fef74e58480c79bfb8dcfa67ffbdd57c61f3 Author: Sergei Lissianoi <[email protected]> Date: Mon Oct 3 16:51:28 2022 -0400 In emberAfEndpointConfigure() change endpoint ID to a uint16_t (project-chip#22998) * Test added march 8 (project-chip#15957) * Added new manual scripts * Added Auto generated File * [OTA] Fix OTARequestorDriverImpl inclusion (project-chip#15981) * Regen to fix CI failures (project-chip#15990) * [ota] Store Default OTA Providers in flash (project-chip#15970) * [ota] Store Default OTA Providers in flash Store Default OTA Providers in flash each time the attribute is modified and load it back on the application startup. * Restyled by clang-format * Fix build and reduce flash usage Co-authored-by: Restyled.io <[email protected]> * Remove merge artifacts * Cast boolean to uint8_t as setOnOffValue expects, fixes GCC 8.3 error * Make endpoint id a uint16_t as per spec * Restyled by clang-format * Update the signature of OnOffServer::setOnOffValue() to avoid casts * Fix printf argument and formatting Co-authored-by: kowsisoundhar12 <[email protected]> Co-authored-by: Carol Yang <[email protected]> Co-authored-by: Boris Zbarsky <[email protected]> Co-authored-by: Damian Królik <[email protected]> Co-authored-by: Restyled.io <[email protected]>

kowsisoundhar12 and others added 30 commits March 8, 2022 22:28

Test added march 8 (project-chip#15957)

35fad7d

* Added new manual scripts * Added Auto generated File

[OTA] Fix OTARequestorDriverImpl inclusion (project-chip#15981)

d5f4b0b

Regen to fix CI failures (project-chip#15990)

9bee828

Merge branch 'master' of github.com:project-chip/connectedhomeip

c219807

Merge branch 'project-chip:master' into master

fa279bc

Merge branch 'project-chip:master' into master

9e18f9b

Merge branch 'project-chip:master' into master

b6ca5ca

Merge branch 'project-chip:master' into master

135dc25

Merge branch 'project-chip:master' into master

0446c67

Merge branch 'project-chip:master' into master

e69e63c

Merge branch 'project-chip:master' into master

a40f6df

Merge branch 'project-chip:master' into master

e5925ba

Merge branch 'project-chip:master' into master

0435362

Merge branch 'project-chip:master' into master

40b3412

Merge branch 'project-chip:master' into master

d197cb5

Merge branch 'project-chip:master' into master

e9b685d

Merge branch 'project-chip:master' into master

ef11df2

Merge branch 'project-chip:master' into master

b088f83

Merge branch 'project-chip:master' into master

8530810

Merge branch 'project-chip:master' into master

87a9581

Merge branch 'project-chip:master' into master

d9b496d

Merge branch 'project-chip:master' into master

2d283df

Merge branch 'project-chip:master' into master

2639213

Merge branch 'project-chip:master' into master

995a07e

Merge branch 'project-chip:master' into master

139f480

Merge branch 'project-chip:master' into master

f0beac0

Merge branch 'project-chip:master' into master

70059e4

Merge branch 'project-chip:master' into master

4faa576

Merge branch 'project-chip:master' into master

e22e2fc

pullapprove bot requested review from robszewczyk, saurabhst, tcarmelveilleux, tecimovic, turon, vijs, vivien-apple, wbschiller, woody-apple and xylophone21 October 3, 2022 14:56

pullapprove bot added the review - pending label Oct 3, 2022

github-actions bot added the app label Oct 3, 2022

jmartinez-silabs reviewed Oct 3, 2022

View reviewed changes

src/app/clusters/level-control/level-control.cpp Outdated Show resolved Hide resolved

selissia added 2 commits October 3, 2022 14:17

Update the signature of OnOffServer::setOnOffValue() to avoid casts

f95062f

Fix printf argument and formatting

1672f76

jmartinez-silabs approved these changes Oct 3, 2022

View reviewed changes

selissia requested a review from jmartinez-silabs October 3, 2022 18:39

jmartinez-silabs approved these changes Oct 3, 2022

View reviewed changes

mrjerryjohns reviewed Oct 3, 2022

View reviewed changes

src/app/util/attribute-storage.cpp Show resolved Hide resolved

mrjerryjohns approved these changes Oct 3, 2022

View reviewed changes

pullapprove bot added review - approved and removed review - pending labels Oct 3, 2022

andy31415 merged commit be8567b into project-chip:master Oct 3, 2022

selissia deleted the silabs_attribute_fix branch October 26, 2022 18:44

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

In emberAfEndpointConfigure() change endpoint ID to a uint16_t #22998

In emberAfEndpointConfigure() change endpoint ID to a uint16_t #22998

selissia commented Oct 3, 2022 •

edited

Loading

github-actions bot commented Oct 3, 2022

github-actions bot commented Oct 3, 2022 •

edited

Loading

In emberAfEndpointConfigure() change endpoint ID to a uint16_t #22998

In emberAfEndpointConfigure() change endpoint ID to a uint16_t #22998

Conversation

selissia commented Oct 3, 2022 • edited Loading

Problem

Change overview

Testing

github-actions bot commented Oct 3, 2022

github-actions bot commented Oct 3, 2022 • edited Loading

selissia commented Oct 3, 2022 •

edited

Loading

github-actions bot commented Oct 3, 2022 •

edited

Loading