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

sys/net/ipv4/addr: fix typos #19876

Merged
merged 1 commit into from
Aug 21, 2023
Merged

sys/net/ipv4/addr: fix typos #19876

merged 1 commit into from
Aug 21, 2023

Conversation

Enoch247
Copy link
Contributor

Contribution description

This patch fixes some typos in the doxygen doc.

Testing procedure

Nothing to test. No change to code.

Issues/PRs references

  • None known

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
This patch fixes some typos in the doxygen doc.
@github-actions github-actions bot added Area: network Area: Networking Area: sys Area: System labels Aug 11, 2023
@benpicco benpicco requested a review from miri64 August 11, 2023 13:13
@benpicco benpicco added Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Aug 11, 2023
@riot-ci
Copy link

riot-ci commented Aug 11, 2023

Murdock results

✔️ PASSED

c67fbd6 sys/net/ipv4/addr: fix typos

Success Failures Total Runtime
1 0 1 01m:08s

Artifacts

@benpicco
Copy link
Contributor

bors merge

bors bot added a commit that referenced this pull request Aug 21, 2023
19387: drivers/atwinc15x0: support dynamic scanning and connection to AP r=benpicco a=fabian18



19874: coap: add missing option numbers r=benpicco a=JKRhb



19875: coap: add missing Content-Format definitions r=benpicco a=JKRhb



19876: sys/net/ipv4/addr: fix typos r=benpicco a=Enoch247

### Contribution description

This patch fixes some typos in the doxygen doc.

### Testing procedure

Nothing to test. No change to code.

### Issues/PRs references

- None known


19878: makefiles/usb_board_reset.mk: declare term-delay target with test target r=benpicco a=aabadie



19886: cpu/efm32: fix DAC configuration r=benpicco a=gschorcht

### Contribution description

The EFM32 MCU allows the reference voltage to be configured per DAC device, not per DAC channel. Also, the DAC reference voltage was defined in the configuration but not used anywhere.

At the moment we have only defined one board (`stwstk6220a`) that uses the DAC, so changing the configuration interface shouldn't be critical.

### Testing procedure

`tests/periph/dac` should still work for the `stwstk6220a`
```
BOARD=slwstk6220a make -j8 -C tests/periph/dac flash
```
I don't have a `stwstk6220a` board (EFM32 Series 0) so that I can't test it. I could only test it for the `sltb009a` board (EFM32 Series 1) with the change for VDAC in PR #19887.

### Issues/PRs references


19888: boards/sltb009a: complete and fix documentation r=benpicco a=gschorcht

### Contribution description

This PR completes and fixes the documentation which was still in the state as generated automatically by `efm2riot`.

The PR also includes a fix of the configuration of the second UART device that was find out while completing the documentation.

### Testing procedure

Green CI

### Issues/PRs references


Co-authored-by: Fabian Hüßler <[email protected]@MLPA-NB119.(none)>
Co-authored-by: Fabian Hüßler <[email protected]>
Co-authored-by: Jan Romann <[email protected]>
Co-authored-by: Joshua DeWeese <[email protected]>
Co-authored-by: Alexandre Abadie <[email protected]>
Co-authored-by: Gunar Schorcht <[email protected]>
@bors
Copy link
Contributor

bors bot commented Aug 21, 2023

Build failed (retrying...):

@bors
Copy link
Contributor

bors bot commented Aug 21, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit f30f7b7 into RIOT-OS:master Aug 21, 2023
@Enoch247 Enoch247 deleted the fix-doc branch August 23, 2023 00:59
@MrKevinWeiss MrKevinWeiss added this to the Release 2023.10 milestone Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: network Area: Networking Area: sys Area: System CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants