1wire: Move onewire_valid_rom to 1wire_crc.h#19216
Conversation
|
@michael-membrowse Please fix job [analyze (stm32-nucleo-f103rb, nucleo-f103rb:nsh, nuttx, boards/arm/stm32/nucleo-f103rb/scripts/ld.script]
https://github.com/apache/nuttx/actions/runs/28171450180/job/83436526606?pr=19216#logs new path |
|
@simbit18 please rebase, the fix was already pushed |
|
@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>
|
What's the point of exposing the header? |
The header is exposed already. This change fixes the inconsistency where the And, if that was a question, it helps myself to use |
Does that mean you can test on a board? |
To let developers use all procedures implemented in the corresponding .c file when 1wire_crc.h is included.
Summary
When
nuttx/1wire/1wire_crc.his included, all the procedures implemented in thedrivers/1wire/1wire_crc.call available exceptonewire_valid_rom.Impact
onewire_valid_romwill 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_romand sucessfuly used it.I have not tested this on a board where 1wire is used. Could anyone, please, do it? It looks
onewire_valid_romis used only indrivers/1wire/1wire.candis a part of this PR, but better save than sorry. Thanks!