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

RFC: Bring zcbor as CBOR decoder/encoder in replacement for TinyCBOR #43333

Closed
de-nordic opened this issue Mar 1, 2022 · 2 comments
Closed
Labels
RFC Request For Comments: want input from the community

Comments

@de-nordic
Copy link
Collaborator

de-nordic commented Mar 1, 2022

Origin

zcbor https://github.com/NordicSemiconductor/zcbor

Purpose

In replacement of the current heavily-modified TinyCBOR fork we use as a CBOR encoder/decoder.
zcbor is a light CBOR encoder/decoder with, optional, capability of generating decoding code from CDDL descriptions of data structures.
Our TinyCBOR fork will then be deprecated and removed in time.

Mode of integration

As a module: it is easier to maintain and update with patches from the origin.
Module path would be modules/lib/zcbor

Pull Request

** THIS IS WORK IN PROGRESS **
The zcbor module Kconfig and cmake recipe:
#43332

Description

Currently the only user of CBOR encoder/decoder within Zephyr is mcumgr. The encoder we use is custom version of TinyCBOR.
Complete discussion on TinyCBOR replacement, on what are possible alternatives and reasons is present here #40591

In general we are trying to replace hard to update non vanilla fork of TinyCBOR with lightweight, simpler and easier to maintain variant zcbor.

Dependencies

This is standalone project and depends only on C compiler.
Additional and optional feature of auto generating decoding source is dependent on python interpreter.

Revision

The newest commit on main:
NordicSemiconductor/zcbor@942ae07

License

As stated here: https://github.com/NordicSemiconductor/zcbor/blob/main/LICENSE
is Apache v2.

@de-nordic de-nordic added the RFC Request For Comments: want input from the community label Mar 1, 2022
@carlescufi carlescufi added the TSC Topics that need TSC discussion label Mar 2, 2022
@nashif nashif removed the TSC Topics that need TSC discussion label Mar 9, 2022
@carlescufi
Copy link
Member

Approved by the TSC

@carlescufi
Copy link
Member

This is now complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFC Request For Comments: want input from the community
Projects
None yet
Development

No branches or pull requests

3 participants