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

[DNM] TSC: documentation if binary blobs are forbidden #41044

Closed

Conversation

mbolivar-nordic
Copy link
Contributor

This pull request formally documents a potential decision by the technical steering committee to forbid hard dependencies on binary blobs within mainline zephyr.

For details on the decision and other options, please see #38570.

@mbolivar-nordic mbolivar-nordic added the TSC Topics that need TSC discussion label Dec 9, 2021
@mbolivar-nordic mbolivar-nordic added the DNM This PR should not be merged (Do Not Merge) label Dec 9, 2021
@zephyrbot zephyrbot requested review from gmarull and utzig December 9, 2021 19:04
@dleach02
Copy link
Member

dleach02 commented Dec 9, 2021

For those who need more context, please look at #38570 and the link to the https://docs.google.com/document/d/1heqcv7dzGvM5rA9xpTMW3kyKJLpZsl2Gjsmr5eqBje8/edit#heading=h.a4tmntn7vwmi for a vie for summary and options.

At some point, there will be a vote for yes/no on binary blobs.

And many thanks to @mbolivar-nordic for taking over this topic in the process working group.

west.yml Outdated Show resolved Hide resolved
build system.

You are of course free to develop your own means of distributing blobs and
integrating them into the zephyr build system.
Copy link
Collaborator

@marc-hb marc-hb Jan 5, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed and requested at https://docs.google.com/document/d/16QU8OT4pnKuhszZHIYfsj1WInN-LRXjPdt87hxG5IRU/edit?disco=AAAATP9jSq0, please clarify whether direct links to optional blobs are allowed in git repositories hosted at https://github.com/zephyrproject-rtos In other words, can optional blobs hosted outside zephyrproject-rtos be not hosted but "version-controlled" inside zephyrproject-rtos for convenience.

I heard the espressif extension does something like this but for clearer documentation purposes there are much simpler examples like git submodules or opt-in west imports.

This reflects a policy decision requiring source code for any
components distributed with the mainline zephyr repository and its
modules.

Describe how vendors may distribute their blobs via other means.

Signed-off-by: Martí Bolívar <[email protected]>
:zephyr_file:`zephyr/west.yml`. (This does not apply to situations such as open
source drivers communicating with proprietary firmware over well-defined
interfaces, e.g. a Bluetooth host implementation communicating over HCI with a
proprietary controller via UART.)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this still leaves room for interpretation, e.g.: what about a proprietary Bluetooth profile? Where Bluetooth is well-defined but the proprietary profile is not.

Also, how can there be a "hard" dependency when using only well-defined interfaces?

Would the "derivative work" concept help formalize this a bit?

@nashif
Copy link
Member

nashif commented Feb 9, 2022

@mbolivar-nordic close? :)

@marc-hb
Copy link
Collaborator

marc-hb commented Feb 9, 2022

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Documentation area: Modules DNM This PR should not be merged (Do Not Merge) TSC Topics that need TSC discussion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants