You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by "jsanchez-2g (via GitHub)" <gi...@apache.org> on 2026/10/09 14:27:36 UTC
[PR] testing/crctest: Cover SMBus PEC and CRC8 compatibility. [nuttx-apps]
jsanchez-2g opened a new pull request, #3824:
URL: https://github.com/apache/nuttx-apps/pull/3824
## Summary
Extend testing/libc/crctest with regression coverage for the companion NuttX crc8smbus()/crc8smbuspart() API and preservation of the existing complemented crc8ccitt API. Test known vectors, all split points, empty buffers and all 256 seeds against an independent bitwise implementation. Existing CRC tests remain.
## Impact
Tests only, no production application behavior or configuration additions. Uses existing TESTING_CRC_TEST / TESTING_CMOCKA infrastructure. Depends on [Apache NuttX PR #20566](https://github.com/apache/nuttx/pull/20566), which supplies the SMBus helpers. Merge that API before this test change. Build against NuttX containing that API.
## Testing
Ubuntu 24.04 / WSL2 x86_64, native cc with -Wall -Wextra -Werror (unused test-state parameters suppressed), real CMocka 1.1.7. Compiled the actual crctest_main.c and NuttX CRC source files with host types and minimal FAR/config definitions. All nine cases pass. Full-file style, encoding and spelling checks pass. This is host execution, not a full NuttX target test build.
## PR verification Self-Check
- [x] One functional change: regression test coverage.
- [x] Impact and validation described.
- [x] Human Signed-off-by and post-commit message checks complete.
- [x] Ready for review; companion dependency noted.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] testing/crctest: Cover SMBus PEC and CRC8 compatibility. [nuttx-apps]
Posted by "xiaoxiang781216 (via GitHub)" <gi...@apache.org>.
xiaoxiang781216 merged PR #3824:
URL: https://github.com/apache/nuttx-apps/pull/3824
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org