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

license: Add Synopsys MIT license terms #262

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

benothmn-st
Copy link
Collaborator

Add Synopsys MIT license terms

Copy link
Member

@erwango erwango left a comment

Choose a reason for hiding this comment

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

Please update following line:
"Three licenses are in use in this repository: Apache-2.0, BSD-3-Clause and ST SLA0044."

LICENSE Outdated
@@ -6,6 +6,8 @@ They are applied according to the following scheme:
license: BSD-3-Clause. This covers following folders:
- stm32cube/stm32*
- lib/*
- All files in lib/stm32wba/STM32_WPAN/link_layer/ll_cmd_lib/ from Synopsys, Inc
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- All files in lib/stm32wba/STM32_WPAN/link_layer/ll_cmd_lib/ from Synopsys, Inc
- All files in lib/stm32wba/STM32_WPAN/link_layer/ll_cmd_lib/

LICENSE Outdated
@@ -6,6 +6,8 @@ They are applied according to the following scheme:
license: BSD-3-Clause. This covers following folders:
- stm32cube/stm32*
- lib/*
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- lib/*
- lib/* (except lib/stm32wba/STM32_WPAN/link_layer/ll_cmd_lib/ see below)

LICENSE Outdated
@@ -49,6 +51,26 @@ For these three licenses, full length texts are available hereafter:
POSSIBILITY OF SUCH DAMAGE.


Synopsys MIT License
Copy link
Member

Choose a reason for hiding this comment

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

Move below Apache License.
(Since we don't have rule for this, I'd propose to list them in order of frequency of use)

@erwango
Copy link
Member

erwango commented Feb 19, 2025

About

Please update following line:
"Three licenses are in use in this repository: Apache-2.0, BSD-3-Clause and ST SLA0044."

Let's use "Following" instead of "Three", this will avoid to modify further if other Licenses are added

Copy link
Collaborator

@asm5878 asm5878 left a comment

Choose a reason for hiding this comment

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

Agreed with @erwango comments, otherwise LGTM

Add Synopsys MIT license terms

Signed-off-by: Nidhal BEN OTHMEN <[email protected]>
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