<div dir="ltr"><div class="gmail_default" style="font-size:small">ping.</div><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Aug 1, 2020 at 2:11 PM Niteesh G. S. <<a href="mailto:niteesh.gs@gmail.com">niteesh.gs@gmail.com</a>> 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"><div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Aug 1, 2020 at 1:37 AM Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank">joel@rtems.org</a>> 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"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jul 31, 2020 at 2:16 PM Niteesh G. S. <<a href="mailto:niteesh.gs@gmail.com" target="_blank">niteesh.gs@gmail.com</a>> 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"><div dir="ltr"><div style="font-size:small">Hello,<br></div><div style="font-size:small"><br></div><div style="font-size:small">In a recent review of these patches</div><div style="font-size:small"><a href="https://lists.rtems.org/pipermail/devel/2020-July/060653.html" target="_blank">https://lists.rtems.org/pipermail/devel/2020-July/060653.html</a></div><div style="font-size:small"><div>Gedare mentioned that we cannot use these patches with the</div><div>current license. More details regarding the conversation can be</div><div>found in the following archive.</div></div><div style="font-size:small"><a href="https://lists.rtems.org/pipermail/devel/2020-July/061000.html" target="_blank">https://lists.rtems.org/pipermail/devel/2020-July/061000.html</a></div><div style="font-size:small"><br></div><div style="font-size:small">The following files have been ported to RTEMS to implement</div><div style="font-size:small">the OFW API.</div><div style="font-size:small">1) openfirm.h  -- BSD-4 License</div><div style="font-size:small">2) openfirm.c  -- BSD-4 License</div><div style="font-size:small">3) ofw_fdt.c    -- BSD-2 License</div><div style="font-size:small"><br></div><div style="font-size:small">The files with BSD4 cannot be used and Gedare suggested to</div><div style="font-size:small">check if we can remove the entire 4-clause cluster or remove</div><div style="font-size:small">clauses #3 and #4. I checked this along with the help of Christian</div><div style="font-size:small">and it seems that we can't remove those. Christian suggested</div><div style="font-size:small">that we can use the header file with the BSD-4 license to some</div><div style="font-size:small">extent but the source files to pose a problem. We also checked</div><div style="font-size:small">OpenBSD it has the same licensing.</div></div></blockquote><div><br></div><div>NetBSD appears to be the origin of the code and although I believe </div><div>they did a largely blanket change from BSD-4, this code is old and</div><div>normally, I would doubt they found the original submitter.  Which </div><div>would be odd in this case because this is his website with email:</div><div><br><a href="https://solfrank.net/Wolfgang/" target="_blank">https://solfrank.net/Wolfgang/</a>  <br></div><div><br></div><div>I have privately emailed to politely ask him to relicense it to BSD-2 </div><div>for use in RTEMS. And try to do that in a way that gets it on a path </div><div>to get changed upstream.</div><div><br></div><div>Hopefully this will solve it.</div></div></div></blockquote><div><br></div><div class="gmail_default" style="font-size:small">Thanks for doing this Joel :).</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div style="font-size:small"><br></div><div style="font-size:small">So we have come up with the following suggestions</div><div style="font-size:small">1) Use the header files as it is.</div></div></blockquote><div><br></div><div>How close are you to being able to merge? Do we have time to let</div><div>him answer?</div></div></div></blockquote><div><br></div><div class="gmail_default" style="font-size:small">Yes, we do have a lot of time. All of my patches are based on the new build</div><div class="gmail_default" style="font-size:small">system so we won't be able to merge until the build system is merged. And</div><div class="gmail_default" style="font-size:small">also there are other things that have to be discussed regarding the patch.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div style="font-size:small">2) Most OF_* functions defined in openfirm.c have 1:1 mapping</div><div style="font-size:small">with the FDT implementation in ofw_fdt.c so there is a possibility</div><div style="font-size:small">to remove openfirm.c and only use openfirm.h and ofw_fdt.c.</div><div style="font-size:small">For those functions which don't have a 1:1 mapping, we can add</div><div style="font-size:small">an implementation in ofw_fdt.c. And remove the functions which</div><div style="font-size:small">don't have an FDT based implementation eg. OF_write, OF_open etc.</div><div style="font-size:small"><br></div><div style="font-size:small">Also please remember that these patches were created with a goal</div><div style="font-size:small">to import the OFW into RTEMS and remove them from libBSD so</div><div style="font-size:small">will using the above approach has a chance of breaking libBSD </div><div style="font-size:small">compatibility in the future?</div></div></blockquote><div><br></div><div>Yikes. That would mean having to create our own files that are</div><div>compatible but don't have the license issue. </div><div><br></div><div>And that our implementation is in a source transparent form that</div><div>allows updates easily from the upstream source.<br><br>If we can't get relicense permission, I think we have to rewrite the</div><div>BSD-4 code and provide compatible versions. :(</div></div></div></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">As of now, this seems to be the only option but let's hope for someone</div><div class="gmail_default" style="font-size:small">to come up with a better approach or get the license relaxed.</div></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Thanks,</div><div class="gmail_default" style="font-size:small">Niteesh</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"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div style="font-size:small"><br></div><div style="font-size:small">Thanks,</div><div style="font-size:small">Niteesh.</div></div>
_______________________________________________<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></blockquote></div></div>
</blockquote></div></div>
</blockquote></div>