<div dir="ltr">If someone doesn't pipe up with complaints or questions in the next few days, go ahead andcommit this.<div><br></div><div>--joel</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jul 14, 2023 at 3:55 AM 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">On 7/14/23 01:24, Joel Sherrill wrote:<br>
> On Thu, Jul 13, 2023 at 4:11 PM Karel Gardas <karel@functional.vision> <br>
> wrote:<br>
> <br>
> <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>
> <br>
> <br>
> The absence of a NOTICE file is OK. If they had one, we would just have<br>
> to pass it along.<br>
> <br>
> I recall the issue is that "You must cause any modified files to carry <br>
> prominent<br>
> notices stating that You changed the files"<br>
> <br>
<br>
OK, but this is even easier since I have not modified files at all. They <br>
are just verbatim copy.<br>
<br>
> My reading is that we would need to pass the NOTICE file along. I personally<br>
> do not see that as a burden unless it contains more than credit or <br>
> permissively<br>
> license info.<br>
<br>
OK! I think decision process itself of "unless it contains more than..." <br>
may be seen as a burden. But, fortunately no NOTICE file there, hence no <br>
issue...<br>
<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>
> <br>
> <br>
> Did you have any BSPs to actually test on? What ones can be tested<br>
> for this?<br>
<br>
Yes, I can run testsuite on 2 boards I have sitting idle here: <br>
stm32h747i-disco and stm32h7b3i-dk where 747 would test both M7 and M4 <br>
variants. By comparing results from current tree and current tree + <br>
patch we may see any unexpected breakage...<br>
<br>
>     Thanks for any comments!<br>
> <br>
> <br>
> I'm happy with the update and think it is OK license wise. Technically, <br>
> it you<br>
> trust it, I'm ok with it.<br>
<br>
Cool! Thanks,<br>
<br>
Karel<br>
<br>
</blockquote></div>