Skip to content

1wire: Move onewire_valid_rom to 1wire_crc.h#19216

Merged
xiaoxiang781216 merged 1 commit into
apache:masterfrom
ghnotgood:master
Jun 26, 2026
Merged

1wire: Move onewire_valid_rom to 1wire_crc.h#19216
xiaoxiang781216 merged 1 commit into
apache:masterfrom
ghnotgood:master

Conversation

@ghnotgood

Copy link
Copy Markdown
Contributor

To let developers use all procedures implemented in the corresponding .c file when 1wire_crc.h is included.

Summary

When nuttx/1wire/1wire_crc.h is included, all the procedures implemented in the drivers/1wire/1wire_crc.c all available except onewire_valid_rom.

Impact

onewire_valid_rom will be available after #include <nuttx/1wire/1wire_crc.h>.

Testing

I have compiled the NuttX with the custom implementation of 1wire protocol that uses onewire_valid_rom and sucessfuly used it.

I have not tested this on a board where 1wire is used. Could anyone, please, do it? It looks onewire_valid_rom is used only in drivers/1wire/1wire.c and

@@ -35,6 +35,7 @@
 #include <nuttx/kmalloc.h>
 #include <nuttx/1wire/1wire_master.h>
 #include <nuttx/1wire/1wire.h>
+#include <nuttx/1wire/1wire_crc.h>
 
 #include "1wire_internal.h"

is a part of this PR, but better save than sorry. Thanks!

@ghnotgood ghnotgood requested a review from jerpelea as a code owner June 25, 2026 12:52
@github-actions github-actions Bot added Area: Drivers Drivers issues Size: S The size of the change in this PR is small labels Jun 25, 2026
@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown

MemBrowse Memory Report

No memory changes detected for:

@simbit18

simbit18 commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

@michael-membrowse Please fix job [analyze (stm32-nucleo-f103rb, nucleo-f103rb:nsh, nuttx, boards/arm/stm32/nucleo-f103rb/scripts/ld.script]

ERROR: Linker script not found: sources/nuttx/boards/arm/stm32/nucleo-f103rb/scripts/ld.script

https://github.com/apache/nuttx/actions/runs/28171450180/job/83436526606?pr=19216#logs

new path
nuttx/boards/arm/stm32f1/nucleo-f103rb/scripts/ld.script

@michael-membrowse

Copy link
Copy Markdown
Contributor

@simbit18 please rebase, the fix was already pushed

@simbit18

Copy link
Copy Markdown
Contributor

@ghnotgood please rebase

To let developers use all procedures implemented in the corresponding
.c file when 1wire_crc.h is included.

Signed-off-by: Jiri Vlasak <jvlasak@elektroline.cz>
@linguini1

Copy link
Copy Markdown
Contributor

What's the point of exposing the header?

@ghnotgood

ghnotgood commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

What's the point of exposing the header?

The header is exposed already. This change fixes the inconsistency where the onewire_valid_rom function implemented in 1wire_crc.c file is not declared in 1wire_crc.h like other functions.

And, if that was a question, it helps myself to use onewire_valid_rom to validate received ROM in custom board code.

@linguini1

Copy link
Copy Markdown
Contributor

And, if that was a question, it helps myself to use onewire_valid_rom to validate received ROM in custom board code.

Does that mean you can test on a board?

@xiaoxiang781216 xiaoxiang781216 merged commit 9a4114a into apache:master Jun 26, 2026
53 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Drivers Drivers issues Size: S The size of the change in this PR is small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants