Skip to content

Add integration with zephyr RTOS#166

Merged
j123b567 merged 1 commit into
j123b567:masterfrom
zaporozhets:feat/add_zephyr_integration
Dec 17, 2025
Merged

Add integration with zephyr RTOS#166
j123b567 merged 1 commit into
j123b567:masterfrom
zaporozhets:feat/add_zephyr_integration

Conversation

@zaporozhets
Copy link
Copy Markdown
Contributor

Hi Jan,

This MR adds infrastructure to use scpi-parser in Zephyr RTOS as external module. I added module description based on: https://docs.zephyrproject.org/latest/develop/modules.html#module-yaml-file-description. Here is an example of how to add scpi-parser module to Zephyr west manifest(west.yml):

manifest:
  self:
    west-commands: scripts/west-commands.yml

  remotes:
    - name: zephyrproject-rtos
      url-base: https://github.com/zephyrproject-rtos

  group-filter: [+optional]

  projects:
    - name: zephyr
      remote: zephyrproject-rtos
      revision: v4.3.0
      import:
        name-allowlist:
          - cmsis
          - cmsis_6
          - hal_atmel
    - name: scpi-parser
      url: https://github.com/zaporozhets/scpi-parser.git
      revision: 9d0e08af236533cd0ead6c9868e6711c16367211
      path: modules/lib/scpi-parser

Please let me know if you have any comments or suggestions.

Best,
Taras.

@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 92.602%. remained the same
when pulling 9d0e08a on zaporozhets:feat/add_zephyr_integration
into 698eabe on j123b567:master.

@j123b567
Copy link
Copy Markdown
Owner

j123b567 commented Dec 15, 2025

I'm not familiar with zephyr ecosystem. Is your example (west.yml) obvious to zephyr users or should there be also some mini documentation/example in zephyr folder?

@zaporozhets
Copy link
Copy Markdown
Contributor Author

Thank you for the approval.

The west manifest (west.yml) should be sufficient for Zephyr developers.

If you would still like an example demonstrating how to integrate/run scpi-parser with a Zephyr base application, I can prepare a separate MR and place it in the examples folder.

Best regards,
Taras

@j123b567
Copy link
Copy Markdown
Owner

thats fine for now

@j123b567 j123b567 merged commit befb37b into j123b567:master Dec 17, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants