All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-ti][scarthgap][PATCH v5] ti-librpmsg-dma: update ti-librpmsg-dma SRCREV & LICENSE
@ 2025-10-29  6:02 vishnu singh
  2025-10-29  6:05 ` PRC Automation
  2025-10-29 20:11 ` Denys Dmytriyenko
  0 siblings, 2 replies; 5+ messages in thread
From: vishnu singh @ 2025-10-29  6:02 UTC (permalink / raw)
  To: meta-ti, reatmon, denis

From: Vishnu Singh <v-singh1@ti.com>

Replace MIT license with TI-TFL to align with TI release policy
and product requirements. Legal approval: SRAS00013045
Update scarthgap srcrev and disable 2dfft example from lib build.

License-Update: Replace MIT to TI-TFL license

Signed-off-by: Vishnu Singh <v-singh1@ti.com>
---
Change log:
       v1->v2:
              - Update commit message with license-update
       v2->v3:
              - Update SRCREV (license updated in README) 
       v3->v4:
              - Add branch name in patch subject 
       v4->v5:
              - Update latest SRCREV 

 meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb b/meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb
index b624871d..7600f867 100644
--- a/meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb
+++ b/meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb
@@ -1,9 +1,9 @@
 SUMMARY = "RPMsg DMA shared library"
-LICENSE = "MIT"
-LIC_FILES_CHKSUM = "file://LICENSE;md5=744e63d2bb8c6151dcdd97f49aa02c53"
+LICENSE = "TI-TFL"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=91dc4ee6d125d0aaba4e5bd2fcc50ed3"
 
-SRC_URI = "git://github.com/TexasInstruments/rpmsg-dma.git;protocol=https;branch=main"
-SRCREV = "609fd72f458ba56f80d06810509ef88d010d2c03"
+SRC_URI = "git://github.com/TexasInstruments/rpmsg-dma.git;protocol=https;branch=scarthgap"
+SRCREV = "153951932925a72346b76d98b2774952c48a82fc"
 
 S = "${WORKDIR}/git"
 
@@ -11,6 +11,6 @@ inherit cmake
 
 DEPENDS = "ti-rpmsg-char"
 
-EXTRA_OECMAKE += "-DBUILD_LIB=ON -DBUILD_EXAMPLE=OFF"
+EXTRA_OECMAKE += "-DBUILD_LIB=ON -DBUILD_AUDIO_OFFLOAD_EXAMPLE=OFF -DBUILD_2DFFT_OFFLOAD_EXAMPLE=OFF"
 
 COMPATIBLE_MACHINE = "^(k3)$"
-- 
2.50.1



^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [meta-ti][scarthgap][PATCH v5] ti-librpmsg-dma: update ti-librpmsg-dma SRCREV & LICENSE
  2025-10-29  6:02 [meta-ti][scarthgap][PATCH v5] ti-librpmsg-dma: update ti-librpmsg-dma SRCREV & LICENSE vishnu singh
@ 2025-10-29  6:05 ` PRC Automation
  2025-10-29 20:11 ` Denys Dmytriyenko
  1 sibling, 0 replies; 5+ messages in thread
From: PRC Automation @ 2025-10-29  6:05 UTC (permalink / raw)
  To: Vishnu Singh; +Cc: meta-ti, reatmon, denis

meta-ti / na / 20251029060209.707510-1-v-singh1

PRC Results: PASS

=========================================================
  check-yocto-patches: PASS
=========================================================
Patches
----------------------------------------
All patches passed



=========================================================
  apply-yocto-patch: PASS
=========================================================
scarthgap
=====================
Summary:
- Patch Series: [meta-ti][scarthgap][PATCH v5] ti-librpmsg-dma: update ti-librpmsg-dma SRCREV & LICENSE
- Submitter: From: vishnu singh <v-singh1@ti.com>
From: Vishnu Singh <v-singh1@ti.com>
- Date: Date: Wed, 29 Oct 2025 11:32:09 +0530
- Num Patches: 1
- Mailing List (public inbox) Commit SHA: 0a29db882d61f3bb5afcd05ffcc3a6c0cb4e804e

Applied to:
- Repository: lcpd-prc-meta-ti
- Base Branch: scarthgap-wip
- Commit Author: Sabeeh Khan <sabeeh-khan@ti.com>
- Commit Subject: meta-ti-extras: cc33xx-fw: update cc33xx firmware to 1.7.0.323
- Commit SHA: 6245469825ca824c83b2796553bcb9163c6daf66

Patches
----------------------------------------
All patches applied



=========================================================
  check-yocto-repo: PASS
=========================================================
scarthgap
=====================
PASS





^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [meta-ti][scarthgap][PATCH v5] ti-librpmsg-dma: update ti-librpmsg-dma SRCREV & LICENSE
  2025-10-29  6:02 [meta-ti][scarthgap][PATCH v5] ti-librpmsg-dma: update ti-librpmsg-dma SRCREV & LICENSE vishnu singh
  2025-10-29  6:05 ` PRC Automation
@ 2025-10-29 20:11 ` Denys Dmytriyenko
  2025-10-30  3:39   ` Vishnu Singh
  1 sibling, 1 reply; 5+ messages in thread
From: Denys Dmytriyenko @ 2025-10-29 20:11 UTC (permalink / raw)
  To: v-singh1; +Cc: meta-ti, reatmon

On Wed, Oct 29, 2025 at 11:32:09AM +0530, Vishnu Singh via lists.yoctoproject.org wrote:
> From: Vishnu Singh <v-singh1@ti.com>
> 
> Replace MIT license with TI-TFL to align with TI release policy
> and product requirements. Legal approval: SRAS00013045
> Update scarthgap srcrev and disable 2dfft example from lib build.
> 
> License-Update: Replace MIT to TI-TFL license

Sorry to be pedantic and blunt, but that's not the reason, that's tautology.

License-Update tag (and the commit message in general) must answer the "Why?" 
question, not the "What?" question. We can all see the "What?" question being 
answered by the code change itself. There's no need to repeat that with the 
"changing X to Y". You need to explain why you are changing X to Y...


> Signed-off-by: Vishnu Singh <v-singh1@ti.com>
> ---
> Change log:
>        v1->v2:
>               - Update commit message with license-update
>        v2->v3:
>               - Update SRCREV (license updated in README) 
>        v3->v4:
>               - Add branch name in patch subject 
>        v4->v5:
>               - Update latest SRCREV 
> 
>  meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb b/meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb
> index b624871d..7600f867 100644
> --- a/meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb
> +++ b/meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb
> @@ -1,9 +1,9 @@
>  SUMMARY = "RPMsg DMA shared library"
> -LICENSE = "MIT"
> -LIC_FILES_CHKSUM = "file://LICENSE;md5=744e63d2bb8c6151dcdd97f49aa02c53"
> +LICENSE = "TI-TFL"
> +LIC_FILES_CHKSUM = "file://LICENSE;md5=91dc4ee6d125d0aaba4e5bd2fcc50ed3"
>  
> -SRC_URI = "git://github.com/TexasInstruments/rpmsg-dma.git;protocol=https;branch=main"
> -SRCREV = "609fd72f458ba56f80d06810509ef88d010d2c03"
> +SRC_URI = "git://github.com/TexasInstruments/rpmsg-dma.git;protocol=https;branch=scarthgap"
> +SRCREV = "153951932925a72346b76d98b2774952c48a82fc"
>  
>  S = "${WORKDIR}/git"
>  
> @@ -11,6 +11,6 @@ inherit cmake
>  
>  DEPENDS = "ti-rpmsg-char"
>  
> -EXTRA_OECMAKE += "-DBUILD_LIB=ON -DBUILD_EXAMPLE=OFF"
> +EXTRA_OECMAKE += "-DBUILD_LIB=ON -DBUILD_AUDIO_OFFLOAD_EXAMPLE=OFF -DBUILD_2DFFT_OFFLOAD_EXAMPLE=OFF"
>  
>  COMPATIBLE_MACHINE = "^(k3)$"
> -- 
> 2.50.1


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [meta-ti][scarthgap][PATCH v5] ti-librpmsg-dma: update ti-librpmsg-dma SRCREV & LICENSE
  2025-10-29 20:11 ` Denys Dmytriyenko
@ 2025-10-30  3:39   ` Vishnu Singh
  2025-10-30 12:57     ` Ryan Eatmon
  0 siblings, 1 reply; 5+ messages in thread
From: Vishnu Singh @ 2025-10-30  3:39 UTC (permalink / raw)
  To: Denys Dmytriyenko; +Cc: meta-ti, reatmon

[-- Attachment #1: Type: text/plain, Size: 2718 bytes --]

Hi Denys,

On 30/10/25 01:41, Denys Dmytriyenko wrote:
> On Wed, Oct 29, 2025 at 11:32:09AM +0530, Vishnu Singh via lists.yoctoproject.org wrote:
>> From: Vishnu Singh<v-singh1@ti.com>
>>
>> Replace MIT license with TI-TFL to align with TI release policy
>> and product requirements. Legal approval: SRAS00013045
>> Update scarthgap srcrev and disable 2dfft example from lib build.
>>
>> License-Update: Replace MIT to TI-TFL license
> Sorry to be pedantic and blunt, but that's not the reason, that's tautology.
>
> License-Update tag (and the commit message in general) must answer the "Why?"
> question, not the "What?" question. We can all see the "What?" question being
> answered by the code change itself. There's no need to repeat that with the
> "changing X to Y". You need to explain why you are changing X to Y...
>
>
Reason for changing the license is that SRAS for this is approved with 
TI-TFL license and C7x firmware used is licensed under TI-TFL.

I will update the commit msg and License-Update tag, if this is ok then 
i will send v6.

>> Signed-off-by: Vishnu Singh<v-singh1@ti.com>
>> ---
>> Change log:
>>         v1->v2:
>>                - Update commit message with license-update
>>         v2->v3:
>>                - Update SRCREV (license updated in README)
>>         v3->v4:
>>                - Add branch name in patch subject
>>         v4->v5:
>>                - Update latest SRCREV
>>
>>   meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb | 10 +++++-----
>>   1 file changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb b/meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb
>> index b624871d..7600f867 100644
>> --- a/meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb
>> +++ b/meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb
>> @@ -1,9 +1,9 @@
>>   SUMMARY = "RPMsg DMA shared library"
>> -LICENSE = "MIT"
>> -LIC_FILES_CHKSUM ="file://LICENSE;md5=744e63d2bb8c6151dcdd97f49aa02c53"
>> +LICENSE = "TI-TFL"
>> +LIC_FILES_CHKSUM ="file://LICENSE;md5=91dc4ee6d125d0aaba4e5bd2fcc50ed3"
>>   
>> -SRC_URI = "git://github.com/TexasInstruments/rpmsg-dma.git;protocol=https;branch=main"
>> -SRCREV = "609fd72f458ba56f80d06810509ef88d010d2c03"
>> +SRC_URI = "git://github.com/TexasInstruments/rpmsg-dma.git;protocol=https;branch=scarthgap"
>> +SRCREV = "153951932925a72346b76d98b2774952c48a82fc"
>>   
>>   S = "${WORKDIR}/git"
>>   
>> @@ -11,6 +11,6 @@ inherit cmake
>>   
>>   DEPENDS = "ti-rpmsg-char"
>>   
>> -EXTRA_OECMAKE += "-DBUILD_LIB=ON -DBUILD_EXAMPLE=OFF"
>> +EXTRA_OECMAKE += "-DBUILD_LIB=ON -DBUILD_AUDIO_OFFLOAD_EXAMPLE=OFF -DBUILD_2DFFT_OFFLOAD_EXAMPLE=OFF"
>>   
>>   COMPATIBLE_MACHINE = "^(k3)$"
>> -- 
>> 2.50.1

[-- Attachment #2: Type: text/html, Size: 3670 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [meta-ti][scarthgap][PATCH v5] ti-librpmsg-dma: update ti-librpmsg-dma SRCREV & LICENSE
  2025-10-30  3:39   ` Vishnu Singh
@ 2025-10-30 12:57     ` Ryan Eatmon
  0 siblings, 0 replies; 5+ messages in thread
From: Ryan Eatmon @ 2025-10-30 12:57 UTC (permalink / raw)
  To: Vishnu Singh, Denys Dmytriyenko; +Cc: meta-ti



On 10/29/2025 10:39 PM, Vishnu Singh wrote:
> Hi Denys,
> 
> On 30/10/25 01:41, Denys Dmytriyenko wrote:
>> On Wed, Oct 29, 2025 at 11:32:09AM +0530, Vishnu Singh via lists.yoctoproject.org wrote:
>>> From: Vishnu Singh<v-singh1@ti.com>
>>>
>>> Replace MIT license with TI-TFL to align with TI release policy
>>> and product requirements. Legal approval: SRAS00013045
>>> Update scarthgap srcrev and disable 2dfft example from lib build.
>>>
>>> License-Update: Replace MIT to TI-TFL license
>> Sorry to be pedantic and blunt, but that's not the reason, that's tautology.
>>
>> License-Update tag (and the commit message in general) must answer the "Why?"
>> question, not the "What?" question. We can all see the "What?" question being
>> answered by the code change itself. There's no need to repeat that with the
>> "changing X to Y". You need to explain why you are changing X to Y...
>>
>>
> Reason for changing the license is that SRAS for this is approved with 
> TI-TFL license and C7x firmware used is licensed under TI-TFL.
> 
> I will update the commit msg and License-Update tag, if this is ok then 
> i will send v6.


I think he is asking more for "Why did we decide to change the license 
from an open license to a more restrictive one?"  Just declaring that 
there is an SRAS is not an explanation of why we changed it.  What is 
the reasoning/history behind the relicense?


>>> Signed-off-by: Vishnu Singh<v-singh1@ti.com>
>>> ---
>>> Change log:
>>>         v1->v2:
>>>                - Update commit message with license-update
>>>         v2->v3:
>>>                - Update SRCREV (license updated in README)
>>>         v3->v4:
>>>                - Add branch name in patch subject
>>>         v4->v5:
>>>                - Update latest SRCREV
>>>
>>>   meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb | 10 +++++-----
>>>   1 file changed, 5 insertions(+), 5 deletions(-)
>>>
>>> diff --git a/meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb b/meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb
>>> index b624871d..7600f867 100644
>>> --- a/meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb
>>> +++ b/meta-ti-bsp/recipes-ti/ipc/ti-librpmsg-dma_1.0.bb
>>> @@ -1,9 +1,9 @@
>>>   SUMMARY = "RPMsg DMA shared library"
>>> -LICENSE = "MIT"
>>> -LIC_FILES_CHKSUM ="file://LICENSE;md5=744e63d2bb8c6151dcdd97f49aa02c53"
>>> +LICENSE = "TI-TFL"
>>> +LIC_FILES_CHKSUM ="file://LICENSE;md5=91dc4ee6d125d0aaba4e5bd2fcc50ed3"
>>>   
>>> -SRC_URI = "git://github.com/TexasInstruments/rpmsg-dma.git;protocol=https;branch=main"
>>> -SRCREV = "609fd72f458ba56f80d06810509ef88d010d2c03"
>>> +SRC_URI = "git://github.com/TexasInstruments/rpmsg-dma.git;protocol=https;branch=scarthgap"
>>> +SRCREV = "153951932925a72346b76d98b2774952c48a82fc"
>>>   
>>>   S = "${WORKDIR}/git"
>>>   
>>> @@ -11,6 +11,6 @@ inherit cmake
>>>   
>>>   DEPENDS = "ti-rpmsg-char"
>>>   
>>> -EXTRA_OECMAKE += "-DBUILD_LIB=ON -DBUILD_EXAMPLE=OFF"
>>> +EXTRA_OECMAKE += "-DBUILD_LIB=ON -DBUILD_AUDIO_OFFLOAD_EXAMPLE=OFF -DBUILD_2DFFT_OFFLOAD_EXAMPLE=OFF"
>>>   
>>>   COMPATIBLE_MACHINE = "^(k3)$"
>>> -- 
>>> 2.50.1

-- 
Ryan Eatmon                reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2025-10-30 12:58 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-10-29  6:02 [meta-ti][scarthgap][PATCH v5] ti-librpmsg-dma: update ti-librpmsg-dma SRCREV & LICENSE vishnu singh
2025-10-29  6:05 ` PRC Automation
2025-10-29 20:11 ` Denys Dmytriyenko
2025-10-30  3:39   ` Vishnu Singh
2025-10-30 12:57     ` Ryan Eatmon

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.