<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jul 13, 2023 at 4:11 PM Karel Gardas <karel@functional.vision> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
  Folks,<br>
<br>
please take this patch as an attempt to renew old discussion or rather <br>
start a new one focused solely on importing new ARM CMSIS code which is <br>
(i) under Apache 2 license but (ii) without any NOTICE file.<br></blockquote><div><br></div><div>The absence of a NOTICE file is OK. If they had one, we would just have</div><div>to pass it along.</div><div><br></div><div>I recall the issue is that "<span style="color:rgb(51,51,51);font-family:serif;font-size:14px;text-align:justify">You must cause any modified files to carry prominent </span></div><div><span style="color:rgb(51,51,51);font-family:serif;font-size:14px;text-align:justify">notices stating that You changed the files</span>"</div><div><br></div><div>If all changes are inside ifdef __rtems__ blocks like we do elsewhere would</div><div>this be sufficient? If we don't think that's "prominent", then adding a single like </div><div>at the top of each file with the rtems conditional should be enough. Something <br>like this:</div><div><br></div><div>/* Contains modifications for RTEMS */</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I hope technical advantages are huge and there is no hinder involved <br>
like with other possible Apache2 licensed projects with NOTICE file <br>
presented.<br></blockquote><div><br></div><div>My reading is that we would need to pass the NOTICE file along. I personally</div><div>do not see that as a burden unless it contains more than credit or permissively</div><div>license info. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
The patch itself is clear replace of existing files with a new files, <br>
plus required files added and old one not needed anymore removed.<br>
<br>
The patch was very lightly tested by building several ARM BSPs <br>
especially those affected by the change.<br></blockquote><div><br></div><div>Did you have any BSPs to actually test on? What ones can be tested</div><div>for this?</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Thanks for any comments!<br></blockquote><div><br></div><div>I'm happy with the update and think it is OK license wise. Technically, it you</div><div>trust it, I'm ok with it.  </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Karel<br>
<br>
On 7/13/23 23:05, Karel Gardas wrote:<br>
> CAVEAT: license change from BSD to Apache2 license!<br>
> <br>
> Explanation:<br>
> The imported files come from CMSIS v5 project available on:<br>
> <br>
> <a href="https://github.com/ARM-software/CMSIS_5/tree/develop" rel="noreferrer" target="_blank">https://github.com/ARM-software/CMSIS_5/tree/develop</a><br>
> <br>
> The files imported are located inside the CMSIS/Core/Include<br>
> project sub-directory.<br>
> <br>
> The project does not provide any NOTICE file in its root directory nor<br>
> in the directory of the imported files.<br>
> The NOTICE file and its usage in the Apache 2 license was/is<br>
> so far the only issue mentioned in discussion of RTEMS developers/users<br>
> when considering inclusion of the code under Apache 2 license<br>
> into the RTEMS project.<br>
> Since the CMSIS v5 project is free from this legal hinder, we may freely<br>
> use it and update files to the latest version.<br>
> <br>
> Technical: the patch replaces code from 2015 with the latest version<br>
> which brings quite a lot of bug fixes and most importantly opens<br>
> possibilities to support MCUs based on new ARM cores.<br>
> ---<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div>