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

[example/nrfconnect] Added support for sending UDP broadcast messages. #3110

Merged

Conversation

kkasperczyk-no
Copy link
Contributor

Problem

nrfconnect platform doesn't support sending UDP broadcast messages
periodically by PublishService as other platforms do (e.g. nrf5)

Summary of Changes

  • Added Service component with PublishService implementation
    to support sending UDP broadcast messages.

Fixes #3088

nrfconnect platform doesn't support sending UDP broadcast messages
periodically by PublishService as other platforms do (e.h. nrf5)

* Added Service component with PublishService implementation
to support sending UDP broadcast messages.
@github-actions
Copy link

github-actions bot commented Oct 7, 2020

Size increase report for "nrfconnect-example-build"

File Section File VM
chip-lighting.elf text 316 316
chip-lighting.elf rodata 36 36
chip-lighting.elf bss 0 16
chip-lighting.elf [LOAD #1 [RWX]] 4 4
chip-lighting.elf [LOAD #3 [RW]] 0 -16
chip-lock.elf text 316 316
chip-lock.elf rodata 40 40
chip-lock.elf [LOAD #3 [RW]] 0 16
chip-lock.elf bss 0 16
chip-lock.elf [LOAD #1 [RWX]] 4 4
Full report output
BLOAT REPORT

Files found only in the build output:
    report.csv

Comparing ./master_artifact/chip-shell.elf and ./pull_artifact/chip-shell.elf:

sections,vmsize,filesize

Comparing ./master_artifact/chip-lighting.elf and ./pull_artifact/chip-lighting.elf:

sections,vmsize,filesize
.debug_info,0,34877
.debug_line,0,4420
.debug_abbrev,0,2788
text,316,316
.debug_loc,0,315
.debug_str,0,313
.strtab,0,230
.symtab,0,176
.debug_ranges,0,120
.debug_frame,0,60
rodata,36,36
.debug_aranges,0,32
bss,16,0
[LOAD #1 [RWX]],4,4
[Unmapped],0,-7
[LOAD #3 [RW]],-16,0

Comparing ./master_artifact/chip-lock.elf and ./pull_artifact/chip-lock.elf:

sections,vmsize,filesize
.debug_info,0,34873
.debug_line,0,4410
.debug_abbrev,0,2788
.debug_loc,0,319
text,316,316
.debug_str,0,313
.strtab,0,230
.symtab,0,176
.debug_ranges,0,120
.debug_frame,0,60
rodata,40,40
.debug_aranges,0,32
[LOAD #3 [RW]],16,0
bss,16,0
[LOAD #1 [RWX]],4,4
[Unmapped],0,-5


@BroderickCarlin BroderickCarlin merged commit 2b85b54 into project-chip:master Oct 8, 2020
kpschoedel pushed a commit to kpschoedel/connectedhomeip that referenced this pull request Oct 9, 2020
project-chip#3110)

* [example/nrfconnect] Added support for sending UDP broadcast messages.

nrfconnect platform doesn't support sending UDP broadcast messages
periodically by PublishService as other platforms do (e.h. nrf5)

* Added Service component with PublishService implementation
to support sending UDP broadcast messages.

* Restyled by clang-format

Co-authored-by: Restyled.io <[email protected]>
@kkasperczyk-no kkasperczyk-no deleted the udp_broadcast_pr branch April 16, 2021 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for PublishService for nrfconnect platform
5 participants